[Intel-gfx] [PATCH 09/26] drm/ast: Use drm_fb_helper_fill_info
Daniel Vetter
daniel at ffwll.ch
Fri Jan 25 08:51:36 UTC 2019
On Thu, Jan 24, 2019 at 07:00:11PM +0100, Sam Ravnborg wrote:
> Hi Daniel.
>
> On Thu, Jan 24, 2019 at 05:58:14PM +0100, Daniel Vetter wrote:
> > Should not result in any changes.
> >
> > Signed-off-by: Daniel Vetter <daniel.vetter at intel.com>
> > Cc: Dave Airlie <airlied at redhat.com>
> > Cc: Junwei Zhang <Jerry.Zhang at amd.com>
> > Cc: Alex Deucher <alexander.deucher at amd.com>
> > Cc: "Christian König" <christian.koenig at amd.com>
> > Cc: Daniel Vetter <daniel.vetter at ffwll.ch>
> > Cc: Sean Paul <sean at poorly.run>
> > Cc: YueHaibing <yuehaibing at huawei.com>
> > Cc: Sam Bobroff <sbobroff at linux.ibm.com>
> > ---
> > drivers/gpu/drm/ast/ast_drv.h | 2 +-
> > drivers/gpu/drm/ast/ast_fb.c | 7 +------
> > 2 files changed, 2 insertions(+), 7 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/ast/ast_drv.h b/drivers/gpu/drm/ast/ast_drv.h
> > index bfc65040dfcb..ffce4608e0c5 100644
> > --- a/drivers/gpu/drm/ast/ast_drv.h
> > +++ b/drivers/gpu/drm/ast/ast_drv.h
> > @@ -259,7 +259,7 @@ struct ast_framebuffer {
> > };
> >
> > struct ast_fbdev {
> > - struct drm_fb_helper helper;
> > + struct drm_fb_helper helper; /* must be first */
>
> Again in the category "I do not know the code",
> but I failed to see whay "struct drm_fb_helper" have to be first.
>
> I saw no ugly cast or similar when browsign the code, but I
> must have missed something?
It's casting the void *info->par to either struct ast_fbdev *(in ast code)
or struct drm_fb_helper * (in fb helper code).
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
More information about the Intel-gfx
mailing list