[Intel-gfx] [PATCH v3 04/12] video: fbdev: uvesafb: modify the static fb_ops directly
Daniel Vetter
daniel at ffwll.ch
Tue Dec 3 17:03:51 UTC 2019
On Tue, Dec 03, 2019 at 06:38:46PM +0200, Jani Nikula wrote:
> Avoid modifying the fb_ops via info->fbops to let us make the pointer
> const in the future.
>
> Cc: linux-fbdev at vger.kernel.org
> Signed-off-by: Jani Nikula <jani.nikula at intel.com>
> ---
> drivers/video/fbdev/uvesafb.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/video/fbdev/uvesafb.c b/drivers/video/fbdev/uvesafb.c
> index 439565cae7ab..53d08d1b56f5 100644
> --- a/drivers/video/fbdev/uvesafb.c
> +++ b/drivers/video/fbdev/uvesafb.c
> @@ -1440,7 +1440,7 @@ static void uvesafb_init_info(struct fb_info *info, struct vbe_mode_ib *mode)
>
> /* Disable blanking if the user requested so. */
> if (!blank)
> - info->fbops->fb_blank = NULL;
> + uvesafb_ops.fb_blank = NULL;
>
> /*
> * Find out how much IO memory is required for the mode with
> @@ -1510,7 +1510,7 @@ static void uvesafb_init_info(struct fb_info *info, struct vbe_mode_ib *mode)
> (par->ypan ? FBINFO_HWACCEL_YPAN : 0);
>
> if (!par->ypan)
> - info->fbops->fb_pan_display = NULL;
> + uvesafb_ops.fb_pan_display = NULL;
> }
I read code.
I cried.
Reviewed-by: Daniel Vetter <daniel.vetter at ffwll.ch>
Also, with more reading and more bad dreams I'll get about this, I'm
revising my take on patches 1&2 in this series. On those also
Reviewed-by: Daniel Vetter <daniel.vetter at ffwll.ch>
as-is.
Cheers, Daniel
>
> static void uvesafb_init_mtrr(struct fb_info *info)
> --
> 2.20.1
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
More information about the Intel-gfx
mailing list