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.vet...@intel.com>
> > Cc: Dave Airlie <airl...@redhat.com>
> > Cc: Junwei Zhang <jerry.zh...@amd.com>
> > Cc: Alex Deucher <alexander.deuc...@amd.com>
> > Cc: "Christian König" <christian.koe...@amd.com>
> > Cc: Daniel Vetter <daniel.vet...@ffwll.ch>
> > Cc: Sean Paul <s...@poorly.run>
> > Cc: YueHaibing <yuehaib...@huawei.com>
> > Cc: Sam Bobroff <sbobr...@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
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
