Message ID | 51C26B56.20802@infradead.org (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On 19:39 Wed 19 Jun , Randy Dunlap wrote: > From: Randy Dunlap <rdunlap@infradead.org> > > Fix compiler warnings of data defined but not used. They are only used > with certain kconfig settings. > > drivers/video/aty/atyfb_base.c:534:13: warning: 'ram_dram' defined but not used [-Wunused-variable] > drivers/video/aty/atyfb_base.c:535:13: warning: 'ram_resv' defined but not used [-Wunused-variable] > > Signed-off-by: Randy Dunlap <rdunlap@infradead.org> > Cc: Paul Mackerras <paulus@samba.org> > Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org> > Cc: linux-fbdev@vger.kernel.org > Cc: Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com> > Cc: Tomi Valkeinen <tomi.valkeinen@ti.com> > --- > drivers/video/aty/atyfb_base.c | 2 ++ > 1 file changed, 2 insertions(+) > > --- linux-next-20130619.orig/drivers/video/aty/atyfb_base.c > +++ linux-next-20130619/drivers/video/aty/atyfb_base.c > @@ -531,8 +531,10 @@ static int correct_chipset(struct atyfb_ > return 0; > } > > +#if defined(CONFIG_FB_ATY_GX) || defined(CONFIG_FB_ATY_CT) > static char ram_dram[] = "DRAM"; > static char ram_resv[] = "RESV"; use __maybe_unused macro instead of the ifdef > +#endif > #ifdef CONFIG_FB_ATY_GX > static char ram_vram[] = "VRAM"; > #endif /* CONFIG_FB_ATY_GX */ > -- > To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
--- linux-next-20130619.orig/drivers/video/aty/atyfb_base.c +++ linux-next-20130619/drivers/video/aty/atyfb_base.c @@ -531,8 +531,10 @@ static int correct_chipset(struct atyfb_ return 0; } +#if defined(CONFIG_FB_ATY_GX) || defined(CONFIG_FB_ATY_CT) static char ram_dram[] = "DRAM"; static char ram_resv[] = "RESV"; +#endif #ifdef CONFIG_FB_ATY_GX static char ram_vram[] = "VRAM"; #endif /* CONFIG_FB_ATY_GX */