On Mon, Jan 30, 2023 at 10:00 PM Hans de Goede <hdegoede@xxxxxxxxxx> wrote: > On 1/30/23 11:17, Sakari Ailus wrote: > > Hi Hans, > > > > On Fri, Jan 27, 2023 at 09:37:25PM +0100, Hans de Goede wrote: > >> Make v4l2_async_register_subdev_sensor() try to get a privacy LED > >> associated with the sensor and extend the call_s_stream() wrapper to > >> enable/disable the privacy LED if found. > >> > >> This makes the core handle privacy LED control, rather then having to > >> duplicate this code in all the sensor drivers. > >> > >> Suggested-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > >> Acked-by: Linus Walleij <linus.walleij@xxxxxxxxxx> > >> Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx> > > > > Please wrap the lines over 80, unless there are tangible reasons to keep > > them as-is. > > > > For this patch: > > > > Acked-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > > > > On my behalf it can be merged via another tree, I don't expect conflicts. > > Also cc Hans Verkuil. > > Thanks. > > I've merged the entire series into my pdx86/review-hans branch now: > https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/log/?h=review-hans > (with the lines in this patch shortened to 80 chars). > > Once the builders have had some time to play with this branch > (and once I've run some tests to make sure the patches still > work as expected) I will push to platform-drivers-x86/for-next . Excellent progress with this Hans, thanks for investing so heavily in getting this right after my initial complaints, the result is extremely appealing an useful. Yours, Linus Walleij
