Messages in this thread | | | Subject | Re: [PATCH] drm: ast: fix double __iomem sparse warning | From | Thomas Zimmermann <> | Date | Wed, 19 Aug 2020 09:17:33 +0200 |
| |
Hi
Am 18.08.20 um 22:28 schrieb Randy Dunlap: > From: Randy Dunlap <rdunlap@infradead.org> > > sparse complains about having 2 "__iomem" attributes on the same line > where only one is needed since the first one applies to everything > up to the ending ';'. > However, to make it clear(er) that both of these pointers are > "__iomem", use separate lines for them. > > ../drivers/gpu/drm/ast/ast_cursor.c:256:26: CK: warning: duplicate [noderef] > ../drivers/gpu/drm/ast/ast_cursor.c:256:26: CK: error: multiple address space given: __iomem & __iomem > > Signed-off-by: Randy Dunlap <rdunlap@infradead.org> > Cc: Dave Airlie <airlied@redhat.com> > Cc: David Airlie <airlied@linux.ie> > Cc: Daniel Vetter <daniel@ffwll.ch> > Cc: dri-devel@lists.freedesktop.org > --- > drivers/gpu/drm/ast/ast_cursor.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-)
Thanks, added to drm-misc-next
Best regards Thomas
> > --- lnx-59-rc1.orig/drivers/gpu/drm/ast/ast_cursor.c > +++ lnx-59-rc1/drivers/gpu/drm/ast/ast_cursor.c > @@ -253,7 +253,8 @@ void ast_cursor_show(struct ast_private > unsigned int offset_x, unsigned int offset_y) > { > u8 x_offset, y_offset; > - u8 __iomem *dst, __iomem *sig; > + u8 __iomem *dst; > + u8 __iomem *sig; > u8 jreg; > > dst = ast->cursor.vaddr[ast->cursor.next_index]; > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel >
-- Thomas Zimmermann Graphics Driver Developer SUSE Software Solutions Germany GmbH Maxfeldstr. 5, 90409 Nürnberg, Germany (HRB 36809, AG Nürnberg) Geschäftsführer: Felix Imendörffer
[unhandled content-type:application/pgp-signature] | |