RE: [PATCH 02/30] backlight/gpio_backlight: Compare against struct fb_info.device

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



>-----Original Message-----
>From: Sam Ravnborg <sam@xxxxxxxxxxxx>
>Sent: Monday, June 5, 2023 4:23 PM
>To: Ruhl, Michael J <michael.j.ruhl@xxxxxxxxx>
>Cc: Thomas Zimmermann <tzimmermann@xxxxxxx>; daniel@xxxxxxxx;
>javierm@xxxxxxxxxx; deller@xxxxxx; geert+renesas@xxxxxxxxx;
>lee@xxxxxxxxxx; daniel.thompson@xxxxxxxxxx; jingoohan1@xxxxxxxxx; linux-
>fbdev@xxxxxxxxxxxxxxx; Rich Felker <dalias@xxxxxxxx>; linux-
>sh@xxxxxxxxxxxxxxx; linux-staging@xxxxxxxxxxxxxxx; dri-
>devel@xxxxxxxxxxxxxxxxxxxxx; John Paul Adrian Glaubitz <glaubitz@xxxxxxxxx-
>berlin.de>; linux-omap@xxxxxxxxxxxxxxx
>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




[Index of Archives]     [Linux Driver Development]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux