On Sun, Mar 19, 2023 at 02:44:08PM +0100, Stephen Kitt wrote: > Hi, > > On Mon, 09 Jan 2023 11:12:02 +0100, Robin van der Gracht <robin@xxxxxxxxxxx> > wrote: > > On 2023-01-08 10:29, Sam Ravnborg wrote: > > > On Sat, Jan 07, 2023 at 10:02:38PM +0100, Miguel Ojeda wrote: > > >> On Sat, Jan 7, 2023 at 7:26 PM Sam Ravnborg via B4 Submission Endpoint > > >> <devnull+sam.ravnborg.org@xxxxxxxxxx> wrote: > > >> > > > >> > Introduce backlight_get_brightness() to simplify logic > > >> > and avoid direct access to backlight properties. > > >> > > >> Note: Stephen sent this one too a while ago (with some more details in > > >> the commit message, which is always nice); and then he sent yesterday > > >> v2 [1] (to mention the functional change with `BL_CORE_SUSPENDED` > > >> [2]). > > > Thanks for the pointers. I will try to move forward with Stephen's > > > patches. > > >> > > >> Anyway, if it goes via drm-misc, feel free to have my: > > >> > > >> Acked-by: Miguel Ojeda <ojeda@xxxxxxxxxx> > > >> > > >> Though it would be nice to have Robin test the change. > > > > > > Robin - can I get your ack to apply Stephen's original v2 patch to > > > drm-misc? > > > > done! see: > > https://lore.kernel.org/lkml/0b16391f997e6ed005a326e4e48f2033@xxxxxxxxxxx/ > > As far as I can tell, this never got applied to drm-misc, and I don’t see it > anywhere else. I guess it slipped through the cracks ;-) Yes, I have been busy with a lot of other stuff lately, and cannot promise when I get back to do Linux work. So if someone else could pick it up that would be nice. Sam