From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Date: Tue, 03 Dec 2019 17:03:51 +0000 Subject: Re: [Intel-gfx] [PATCH v3 04/12] video: fbdev: uvesafb: modify the static fb_ops directly Message-Id: <20191203170351.GH624164@phenom.ffwll.local> List-Id: References: In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Jani Nikula Cc: linux-fbdev@vger.kernel.org, intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org 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@vger.kernel.org > Signed-off-by: Jani Nikula > --- > 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 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 as-is. Cheers, Daniel > > static void uvesafb_init_mtrr(struct fb_info *info) > -- > 2.20.1 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch