Re: [PATCH 12/15] auxdisplay: ht16k33: Introduce backlight_get_brightness()

From: Sam Ravnborg
Date: Sun Mar 19 2023 - 14:44:32 EST


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