Current Barebox/Next and GCC-4.6.2

Current Barebox/Next and GCC-4.6.2

[Juergen Beisert]

Hi,

compiling the current Barebox Next branch with a 4.6.2 GCC gives me the
following list of warnings.

common/console.c: In function 'fprintf':
common/console.c:333:7: warning: variable 'i' set but not used [-Wunused-but-set-variable]

drivers/mtd/nand/nand_bbt.c: In function 'search_bbt':
drivers/mtd/nand/nand_bbt.c:466:6: warning: variable 'bits' set but not used [-Wunused-but-set-variable]

drivers/usb/core/usb.c: In function 'usb_parse_config':
drivers/usb/core/usb.c:183:17: warning: variable 'ch' set but not used [-Wunused-but-set-variable]
drivers/usb/core/usb.c: In function 'usb_hub_port_connect_change':
drivers/usb/core/usb.c:1011:29: warning: variable 'portchange' set but not used [-Wunused-but-set-variable]

drivers/usb/host/ohci-hcd.c: In function 'dl_transfer_length':
drivers/usb/host/ohci-hcd.c:961:8: warning: variable 'tdINFO' set but not used [-Wunused-but-set-variable]

This one is a little bit strange:

lib/vsprintf.c: In function 'vasprintf':
lib/vsprintf.c:595:2: warning: generating trampoline in object (requires executable stack) [-Wtrampolines]

Should we care?

jbe

-- 
Pengutronix e.K.                              | Juergen Beisert             |
Linux Solutions for Science and Industry      | http://www.pengutronix.de/  |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: Current Barebox/Next and GCC-4.6.2

[Sascha Hauer]

On Sat, Nov 26, 2011 at 04:03:15PM +0100, Juergen Beisert wrote:
> Hi,
> 
> compiling the current Barebox Next branch with a 4.6.2 GCC gives me the
> following list of warnings.
> 
> common/console.c: In function 'fprintf':
> common/console.c:333:7: warning: variable 'i' set but not used [-Wunused-but-set-variable]

This one should be fixed in master.

> 
> drivers/mtd/nand/nand_bbt.c: In function 'search_bbt':
> drivers/mtd/nand/nand_bbt.c:466:6: warning: variable 'bits' set but not used [-Wunused-but-set-variable]
> 
> drivers/usb/core/usb.c: In function 'usb_parse_config':
> drivers/usb/core/usb.c:183:17: warning: variable 'ch' set but not used [-Wunused-but-set-variable]
> drivers/usb/core/usb.c: In function 'usb_hub_port_connect_change':
> drivers/usb/core/usb.c:1011:29: warning: variable 'portchange' set but not used [-Wunused-but-set-variable]
> 
> drivers/usb/host/ohci-hcd.c: In function 'dl_transfer_length':
> drivers/usb/host/ohci-hcd.c:961:8: warning: variable 'tdINFO' set but not used [-Wunused-but-set-variable]
> 
> This one is a little bit strange:
> 
> lib/vsprintf.c: In function 'vasprintf':
> lib/vsprintf.c:595:2: warning: generating trampoline in object (requires executable stack) [-Wtrampolines]

We should probably just silence the compiler. Our stack is executable.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox