>-----Original Message----- >From: Sam Ravnborg <[email protected]> >Sent: Monday, June 5, 2023 4:23 PM >To: Ruhl, Michael J <[email protected]> >Cc: Thomas Zimmermann <[email protected]>; [email protected]; >[email protected]; [email protected]; [email protected]; >[email protected]; [email protected]; [email protected]; linux- >[email protected]; Rich Felker <[email protected]>; linux- >[email protected]; [email protected]; dri- >[email protected]; John Paul Adrian Glaubitz <[email protected] >berlin.de>; [email protected] >Subject: Re: [PATCH 02/30] backlight/gpio_backlight: Compare against struct >fb_info.device > >Hi Michael. > >> > >> >Fixes a bug in the backlight driver and prepares fbdev for making >> >struct fb_info.dev optional. >> >> I only see a rename from fbdev to dev... >> >> Is there missing code? >> >> Would a fixes: be useful? >> >> M >> >> >@@ -35,7 +35,7 @@ static int gpio_backlight_check_fb(struct >> >backlight_device *bl, >> > { >> > struct gpio_backlight *gbl = bl_get_data(bl); >> > >> >- return gbl->fbdev == NULL || gbl->fbdev == info->dev; >> >+ return !gbl->dev || gbl->dev == info->device; >> > } > >The real change is here where info->dev is replaced by info->device.
Yeah, after a few patches, I was getting the idea that the name was the bug. 😊 Thanks, M > Sam
