>-----Original Message-----
>From: Sam Ravnborg <s...@ravnborg.org>
>Sent: Monday, June 5, 2023 4:23 PM
>To: Ruhl, Michael J <michael.j.r...@intel.com>
>Cc: Thomas Zimmermann <tzimmerm...@suse.de>; dan...@ffwll.ch;
>javi...@redhat.com; del...@gmx.de; geert+rene...@glider.be;
>l...@kernel.org; daniel.thomp...@linaro.org; jingooh...@gmail.com; linux-
>fb...@vger.kernel.org; Rich Felker <dal...@libc.org>; linux-
>s...@vger.kernel.org; linux-stag...@lists.linux.dev; dri-
>de...@lists.freedesktop.org; John Paul Adrian Glaubitz <glaub...@physik.fu-
>berlin.de>; linux-o...@vger.kernel.org
>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
