On Fri, Jul 03, 2020 at 08:45:42PM +0200, Sam Ravnborg wrote: > Use the backlight_is_blank() helper to simplify the code a bit. > This add support for fb_blank as a side-effect. > > Signed-off-by: Sam Ravnborg <s...@ravnborg.org> > Cc: Lee Jones <lee.jo...@linaro.org> > Cc: Daniel Thompson <daniel.thomp...@linaro.org> > Cc: Jingoo Han <jingooh...@gmail.com>
This driver exhibits the "how does userspace know what writing brightness zero to sysfs actually does" problem so can't use the backlight_get_brightness() helper. If you respin it would be good to see that in the patch description but that's such an extreme nitpick It think, either way, this can be: Reviewed-by: Daniel Thompson <daniel.thomp...@linaro.org> Daniel. _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel