On Mon, 18 May 2020 at 10:22, Jani Nikula <[email protected]> wrote: > > On Sun, 17 May 2020, Emil Velikov <[email protected]> wrote: > > On Thu, 17 Aug 2017 at 12:34, Jani Nikula <[email protected]> > > wrote: > >> > >> On Thu, 17 Aug 2017, Michael Tretter <[email protected]> wrote: > >> > Using plain echo to set the "force" connector attribute fails with > >> > -EINVAL, because echo appends a newline to the output. > >> > > >> > Replace strcmp with sysfs_streq to also accept strings that end with a > >> > newline. > >> > > >> > v2: use sysfs_streq instead of stripping trailing whitespace > >> > > >> > Signed-off-by: Michael Tretter <[email protected]> > >> > >> Reviewed-by: Jani Nikula <[email protected]> > >> > > Seems like this fell through the cracks. Pushed to drm-misc-next. > > From 2017? How'd you find it? :o > > My bad, thanks. > A colleague mentioned about their experiences with \n, so I remembered seeing something. >From a quick grep, we have have handful of other cases mostly in amdgpu and radeon. Will see it I can prep a patch later on today.
-Emil _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
