# \[PATCH\] video: fbdev: atmel: fix warning for const return value - Subsystems: FRAMEBUFFER LAYER, MICROCHIP LCDFB DRIVER, THE REST - Maintainers: Helge Deller, Nicolas Ferre, Linus Torvalds - List: linux-fbdev - 4 messages, 4 authors - Page: https://ratatoskr.run/linux-fbdev/2015/07/4543897/t ## \[PATCH\] video: fbdev: atmel: fix warning for const return value - From: Rob Herring \ - Date: 2015-07-27 15:41:50 - Message: https://ratatoskr.run/linux-fbdev/2015/07/4543897.md ```text A const on a return value is meaningless and generates a warning on some versions of gcc: drivers/video/fbdev/atmel_lcdfb.c:1003: warning: type qualifiers ignored on function return type This was found on 0-day with avr32 builds with CONFIG_OF enabled, but it doesn't seem to generate warnings for other arches. Reported-by: Fengguang Wu Signed-off-by: Rob Herring Cc: Nicolas Ferre Cc: Jean-Christophe Plagniol-Villard Cc: Tomi Valkeinen --- drivers/video/fbdev/atmel_lcdfb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/video/fbdev/atmel_lcdfb.c b/drivers/video/fbdev/atmel_lcdfb.c index abadc49..66f4b70 100644 --- a/drivers/video/fbdev/atmel_lcdfb.c +++ b/drivers/video/fbdev/atmel_lcdfb.c @@ -999,7 +999,7 @@ static const char *atmel_lcdfb_wiring_modes[] = { [ATMEL_LCDC_WIRING_RGB] = "RGB", }; -const int atmel_lcdfb_get_of_wiring_modes(struct device_node *np) +int atmel_lcdfb_get_of_wiring_modes(struct device_node *np) { const char *mode; int err, i; -- 2.1.0 ``` ## Re: \[PATCH\] video: fbdev: atmel: fix warning for const return value - From: Nicolas Ferre \ - Date: 2015-07-27 15:50:55 - Message: https://ratatoskr.run/linux-fbdev/2015/07/4543905.md ```text Le 27/07/2015 17:41, Rob Herring a écrit : > A const on a return value is meaningless and generates a warning on some > versions of gcc: > > drivers/video/fbdev/atmel_lcdfb.c:1003: warning: type qualifiers ignored on function return type > > This was found on 0-day with avr32 builds with CONFIG_OF enabled, but > it doesn't seem to generate warnings for other arches. > > Reported-by: Fengguang Wu > Signed-off-by: Rob Herring > Cc: Nicolas Ferre Sure: Acked-by: Nicolas Ferre Thanks Rob! > Cc: Jean-Christophe Plagniol-Villard > Cc: Tomi Valkeinen > --- > drivers/video/fbdev/atmel_lcdfb.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/video/fbdev/atmel_lcdfb.c b/drivers/video/fbdev/atmel_lcdfb.c > index abadc49..66f4b70 100644 > --- a/drivers/video/fbdev/atmel_lcdfb.c > +++ b/drivers/video/fbdev/atmel_lcdfb.c > @@ -999,7 +999,7 @@ static const char *atmel_lcdfb_wiring_modes[] = { > [ATMEL_LCDC_WIRING_RGB] = "RGB", > }; > > -const int atmel_lcdfb_get_of_wiring_modes(struct device_node *np) > +int atmel_lcdfb_get_of_wiring_modes(struct device_node *np) > { > const char *mode; > int err, i; > -- Nicolas Ferre ``` ## Re: \[PATCH\] video: fbdev: atmel: fix warning for const return value - From: Sudip Mukherjee \ - Date: 2015-08-07 05:22:30 - Message: https://ratatoskr.run/linux-fbdev/2015/08/4552924.md ```text On Mon, Jul 27, 2015 at 10:41:41AM -0500, Rob Herring wrote: > A const on a return value is meaningless and generates a warning on some > versions of gcc: > > drivers/video/fbdev/atmel_lcdfb.c:1003: warning: type qualifiers ignored on function return type maybe this one should have been static instead of const. regards sudip ``` ## Re: \[PATCH\] video: fbdev: atmel: fix warning for const return value - From: Tomi Valkeinen \ - Date: 2015-08-20 10:51:28 - Message: https://ratatoskr.run/linux-fbdev/2015/08/4561044.md ```text On 07/08/15 08:22, Sudip Mukherjee wrote: > On Mon, Jul 27, 2015 at 10:41:41AM -0500, Rob Herring wrote: >> A const on a return value is meaningless and generates a warning on some >> versions of gcc: >> >> drivers/video/fbdev/atmel_lcdfb.c:1003: warning: type qualifiers ignored on function return type > maybe this one should have been static instead of const. I think that makes sense. I will apply the following patch instead of Rob's, if no one complains. Author: Tomi Valkeinen [ref] Date: Thu Aug 20 13:46:10 2015 +0300 video: fbdev: atmel: fix warning for const return value A const on a return value is meaningless and generates a warning on some versions of gcc: drivers/video/fbdev/atmel_lcdfb.c:1003: warning: type qualifiers ignored on function return type The function in question is only used inside the .c file, so the author of the code most likely means "static" instead of "const". Change the const to static. Reported-by: Fengguang Wu [ref] Cc: Nicolas Ferre [ref] Signed-off-by: Tomi Valkeinen [ref] diff --git a/drivers/video/fbdev/atmel_lcdfb.c b/drivers/video/fbdev/atmel_lcdfb.c index abadc490fa1f..016cae1425a4 100644 --- a/drivers/video/fbdev/atmel_lcdfb.c +++ b/drivers/video/fbdev/atmel_lcdfb.c @@ -999,7 +999,7 @@ static const char *atmel_lcdfb_wiring_modes[] = { [ATMEL_LCDC_WIRING_RGB] = "RGB", }; -const int atmel_lcdfb_get_of_wiring_modes(struct device_node *np) +static int atmel_lcdfb_get_of_wiring_modes(struct device_node *np) { const char *mode; int err, i; ```