Re: [PATCH v2 01/15] platform/x86: dell-uart-backlight: Use blacklight power constant

["Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>]

[-- Attachment #1: Type: text/plain, Size: 1742 bytes --]

On Mon, 21 Jul 2025, Thomas Zimmermann wrote:

> Hi
> 
> Am 21.07.25 um 13:43 schrieb Ilpo Järvinen:
> > On Tue, 15 Jul 2025, Thomas Zimmermann wrote:
> > 
> > > The backlight subsystem has gotten its own power constants. Replace
> > > FB_BLANK_UNBLANK with BACKLIGHT_POWER_ON.
> > > 
> > > Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
> > > Reviewed-by: Hans de Goede <hansg@kernel.org>
> > > Acked-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
> > > ---
> > >   drivers/platform/x86/dell/dell-uart-backlight.c | 2 +-
> > >   1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > diff --git a/drivers/platform/x86/dell/dell-uart-backlight.c
> > > b/drivers/platform/x86/dell/dell-uart-backlight.c
> > > index 8f868f845350..f323a667dc2d 100644
> > > --- a/drivers/platform/x86/dell/dell-uart-backlight.c
> > > +++ b/drivers/platform/x86/dell/dell-uart-backlight.c
> > > @@ -305,7 +305,7 @@ static int dell_uart_bl_serdev_probe(struct
> > > serdev_device *serdev)
> > >   	dev_dbg(dev, "Firmware version: %.*s\n", resp[RESP_LEN] - 3, resp +
> > > RESP_DATA);
> > >     	/* Initialize bl_power to a known value */
> > > -	ret = dell_uart_set_bl_power(dell_bl, FB_BLANK_UNBLANK);
> > > +	ret = dell_uart_set_bl_power(dell_bl, BACKLIGHT_POWER_ON);
> > >   	if (ret)
> > >   		return ret;
> > Hi Thomas,
> > 
> > Do you expect this entire series to go in this cycle through some other
> > tree than pdx86? If not, I'll take this through pdx86 tree in this cycle.
> 
> I don't know when the series will get merged, but it might still take a bit.
> Please take this patch through your tree. Good to have it off the list.

Thanks.

Applied it to the review-ilpo-next branch.

-- 
 i.