[U-Boot] FYI: Warning on tegra builds

Hi Tom,
I'm seeing the following on tegra builds:
Configuring for harmony board... text data bss dec hex filename 171562 4016 218244 393822 6025e /tmp/u-boot-build//harmony/u-boot ap20.c: In function 'init_pllx': ap20.c:81:18: warning: array subscript is above array bounds Configuring for seaboard board... text data bss dec hex filename 223058 4592 264716 492366 7834e /tmp/u-boot-build//seaboard/u-boot ap20.c: In function 'init_pllx': ap20.c:81:18: warning: array subscript is above array bounds /bin/sh: line 1: exit: too many arguments make[1]: *** [/tmp/u-boot-build/seaboard/dts/dt.dtb] Error 1 make: *** [/tmp/u-boot-build/seaboard/u-boot.dtb] Error 2 make: *** Waiting for unfinished jobs.... Configuring for ventana board... text data bss dec hex filename 175075 4020 218224 397319 61007 /tmp/u-boot-build//ventana/u-boot ap20.c: In function 'init_pllx': ap20.c:81:18: warning: array subscript is above array bounds /bin/sh: line 1: exit: too many arguments make[1]: *** [/tmp/u-boot-build/ventana/dts/dt.dtb] Error 1 make: *** [/tmp/u-boot-build/ventana/u-boot.dtb] Error 2
Cheers, -Joe

Joe,
What repo/branch? I assume u-boot-tegra/master?
What compiler/tools? I usually use either ELDK42 or GCC 4.4.1 and I don't see this.
Using 'git blame' and searching Patchwork, I do see that Simon posted a patch for this awhile back (tegra: Correct PLL access in ap20.c and clock.c) but I don't have it in the u-boot-tegra/master repo so I must have missed it/dropped it.
I'll apply it as part of the next pull request.
Thanks for catching this,
Tom
-----Original Message----- From: Joe Hershberger [mailto:joe.hershberger@gmail.com] Sent: Monday, May 21, 2012 4:11 PM To: u-boot; Tom Warren Subject: FYI: Warning on tegra builds
Hi Tom,
I'm seeing the following on tegra builds:
Configuring for harmony board... text data bss dec hex filename 171562 4016 218244 393822 6025e /tmp/u-boot-build//harmony/u-boot ap20.c: In function 'init_pllx': ap20.c:81:18: warning: array subscript is above array bounds Configuring for seaboard board... text data bss dec hex filename 223058 4592 264716 492366 7834e /tmp/u-boot-build//seaboard/u-boot ap20.c: In function 'init_pllx': ap20.c:81:18: warning: array subscript is above array bounds /bin/sh: line 1: exit: too many arguments make[1]: *** [/tmp/u-boot-build/seaboard/dts/dt.dtb] Error 1 make: *** [/tmp/u-boot-build/seaboard/u-boot.dtb] Error 2 make: *** Waiting for unfinished jobs.... Configuring for ventana board... text data bss dec hex filename 175075 4020 218224 397319 61007 /tmp/u-boot-build//ventana/u-boot ap20.c: In function 'init_pllx': ap20.c:81:18: warning: array subscript is above array bounds /bin/sh: line 1: exit: too many arguments make[1]: *** [/tmp/u-boot-build/ventana/dts/dt.dtb] Error 1 make: *** [/tmp/u-boot-build/ventana/u-boot.dtb] Error 2
Cheers, -Joe

Hi Tom,
On Mon, May 21, 2012 at 6:25 PM, Tom Warren TWarren@nvidia.com wrote:
Joe,
What repo/branch? I assume u-boot-tegra/master?
I was building u-boot/master when I noticed this.
What compiler/tools? I usually use either ELDK42 or GCC 4.4.1 and I don't see this.
arm-none-linux-gnueabi-gcc (Sourcery G++ Lite 2010.09-50) 4.5.1
Using 'git blame' and searching Patchwork, I do see that Simon posted a patch for this awhile back (tegra: Correct PLL access in ap20.c and clock.c) but I don't have it in the u-boot-tegra/master repo so I must have missed it/dropped it.
I'll apply it as part of the next pull request.
Sounds good.
Thanks for catching this,
No problem.
-Joe
participants (2)
-
Joe Hershberger
-
Tom Warren