On Sat, 07 Jan 2023 19:26:24 +0100, Sam Ravnborg via B4 Submission Endpoint <devnull+sam.ravnborg....@kernel.org> wrote:
> From: Sam Ravnborg <s...@ravnborg.org> > > Avoiding direct access to backlight_properties.props. > > Access to the deprecated props.fb_blank replaced by backlight_is_blank(). > Access to props.power is dropped - it was only used for debug. > > Signed-off-by: Sam Ravnborg <s...@ravnborg.org> > Cc: Thomas Zimmermann <tzimmerm...@suse.de> > Cc: Andy Shevchenko <andriy.shevche...@linux.intel.com> > Cc: Javier Martinez Canillas <javi...@redhat.com> > Cc: Greg Kroah-Hartman <gre...@linuxfoundation.org> > Cc: Sam Ravnborg <s...@ravnborg.org> > Cc: Stephen Kitt <st...@sk2.org> > Cc: Peter Suti <peter.s...@streamunlimited.com> > Cc: linux-fb...@vger.kernel.org > --- > drivers/staging/fbtft/fbtft-core.c | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/drivers/staging/fbtft/fbtft-core.c > b/drivers/staging/fbtft/fbtft-core.c index afaba94d1d1c..1746327e1939 100644 > --- a/drivers/staging/fbtft/fbtft-core.c > +++ b/drivers/staging/fbtft/fbtft-core.c > @@ -132,15 +132,15 @@ static int fbtft_backlight_update_status(struct > backlight_device *bd) { > struct fbtft_par *par = bl_get_data(bd); > bool polarity = par->polarity; > + bool blank = backlight_is_blank(bd); > > - fbtft_par_dbg(DEBUG_BACKLIGHT, par, > - "%s: polarity=%d, power=%d, fb_blank=%d\n", > - __func__, polarity, bd->props.power, > bd->props.fb_blank); > + fbtft_par_dbg(DEBUG_BACKLIGHT, par, "%s: polarity=%d, blank=%d\n", > + __func__, polarity, blank); > > - if (!backlight_is_blank(bd)) > - gpiod_set_value(par->gpio.led[0], polarity); > - else > + if (blank) > gpiod_set_value(par->gpio.led[0], !polarity); > + else > + gpiod_set_value(par->gpio.led[0], polarity); > > return 0; > } > > -- > 2.34.1 Reviewed-by: Stephen Kitt <st...@sk2.org>
pgpiJ7cx7VMKJ.pgp
Description: OpenPGP digital signature