[BUG] phycard a l1 on lowlevel

[BUG] phycard a l1 on lowlevel

[Jean-Christophe PLAGNIOL-VILLARD]

Hi Sasha,

	I get this when compiling this board

Building arm phycard_a_l1_defconfig
Configure: OK
/opt/work/barebox/arch/arm/boards/phycard-a-l1/lowlevel.c: In function 'barebox_arm_reset_vector':
/opt/work/barebox/arch/arm/boards/phycard-a-l1/lowlevel.c:122:3: warning: array subscript is above array bounds

	Can you take a look?

Best Regards,
J.

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

Re: [BUG] phycard a l1 on lowlevel

[Sascha Hauer]

Hi J,

On Wed, Oct 23, 2013 at 06:48:00PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> Hi Sasha,
> 
> 	I get this when compiling this board
> 
> Building arm phycard_a_l1_defconfig
> Configure: OK
> /opt/work/barebox/arch/arm/boards/phycard-a-l1/lowlevel.c: In function 'barebox_arm_reset_vector':
> /opt/work/barebox/arch/arm/boards/phycard-a-l1/lowlevel.c:122:3: warning: array subscript is above array bounds

I don't see this here. You probably have a different gcc version.

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

Re: [BUG] phycard a l1 on lowlevel

[Uwe Kleine-König]

On Thu, Oct 24, 2013 at 09:30:02AM +0200, Sascha Hauer wrote:
> Hi J,
> 
> On Wed, Oct 23, 2013 at 06:48:00PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > Hi Sasha,
> > 
> > 	I get this when compiling this board
> > 
> > Building arm phycard_a_l1_defconfig
> > Configure: OK
> > /opt/work/barebox/arch/arm/boards/phycard-a-l1/lowlevel.c: In function 'barebox_arm_reset_vector':
> > /opt/work/barebox/arch/arm/boards/phycard-a-l1/lowlevel.c:122:3: warning: array subscript is above array bounds
> 
> I don't see this here. You probably have a different gcc version.
This is from

	sdrc_config[(uchar) cfg]

where cfg is set to -1 a few lines above.

(BTW, the function hint is wrong, this is in pcaal1_sdrc_init().)

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

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