Re: [PATCH] fbdev: Use helper to get fb_info in all file operations

From: Javier Martinez Canillas
Date: Tue May 03 2022 - 16:31:59 EST


Hello Sam,

On 5/3/22 19:14, Sam Ravnborg wrote:
> Hi Javier,
>

[snip]

>>
>> int fb_deferred_io_fsync(struct file *file, loff_t start, loff_t end, int datasync)
>> {
>> - struct fb_info *info = file->private_data;
>> + struct fb_info *info = fb_file_fb_info(file->private_data);
> This looks wrong. I assume you wanted to write:
>> + struct fb_info *info = fb_file_fb_info(file);

Right I did. Thanks for pointing out.

--
Best regards,

Javier Martinez Canillas
Linux Engineering
Red Hat