Re: [U-Boot] [PATCH 0/9] Fix a few warnings that bug me

2012-03-05 Thread Marek Vasut
Dear Simon Glass, This series fixes some warnings which seem to still be left over from all the warning-squashing efforts for 2011.12. If patches exist for some of these already then please ignore. The alias problem in USB is a bit odd, since I thought it was already fixed. But perhaps it

Re: [U-Boot] [PATCH 0/9] Fix a few warnings that bug me

2012-03-05 Thread Simon Glass
Hi Marek, On Mon, Mar 5, 2012 at 10:22 AM, Marek Vasut ma...@denx.de wrote: Dear Simon Glass, This series fixes some warnings which seem to still be left over from all the warning-squashing efforts for 2011.12. If patches exist for some of these already then please ignore. The alias

Re: [U-Boot] [PATCH 0/9] Fix a few warnings that bug me

2012-03-05 Thread Marek Vasut
Dear Simon Glass, Hi Marek, On Mon, Mar 5, 2012 at 10:22 AM, Marek Vasut ma...@denx.de wrote: Dear Simon Glass, This series fixes some warnings which seem to still be left over from all the warning-squashing efforts for 2011.12. If patches exist for some of these already then

Re: [U-Boot] [PATCH 0/9] Fix a few warnings that bug me

2012-03-05 Thread Andy Pont
Hi Simon, snip  arch/powerpc/cpu/mpc5xxx/usb_ohci.c |   79 -- arch/powerpc/cpu/ppc4xx/usb_ohci.c  | 81 -- arch/powerpc/lib/board.c snip The two PowerPC ones apparently only happen with the 4.4 toolchain, so I think we are

Re: [U-Boot] [PATCH 0/9] Fix a few warnings that bug me

2012-03-05 Thread Wolfgang Denk
Dear Andy Pont, In message 00e201ccfaff$bbc8b840$335a28c0$@p...@sdcsystems.com you wrote: This has just grabbed my attention with the comment regarding the 4.4 toolchain as I know that MontaVista ship a version of the 4.4.0 toolchain for PowerPC with one of their embedded Linux products. Is

Re: [U-Boot] [PATCH 0/9] Fix a few warnings that bug me

2012-03-03 Thread Wolfgang Denk
Dear Simon Glass, In message CAPnjgZ0Mb=LJBE3JSs0zOqivHY5pRfYNGpS3vnfs6O=sqxu...@mail.gmail.com you wrote: Yes but not just those. There are detailed notes with each patch, but here is a summary. I think 6 out of 9 patches should be applied regardless of toolchain. Then please re-test

Re: [U-Boot] [PATCH 0/9] Fix a few warnings that bug me

2012-02-22 Thread Stefano Babic
On 21/02/2012 19:40, Simon Glass wrote: Yes but not just those. There are detailed notes with each patch, but here is a summary. I think 6 out of 9 patches should be applied regardless of toolchain. 68k only: arch/m68k/lib/board.c |2 +- - I believe this fix is sensible

Re: [U-Boot] [PATCH 0/9] Fix a few warnings that bug me

2012-02-21 Thread Stefano Babic
On 20/02/2012 19:49, Simon Glass wrote: Hi Wolfgang / Stefano, Hi Simon, I just check that - actually my PowerPC one is v4.4.4: Using built-in specs. Target: powerpc-linux Configured with: /home/tony/buildall/src/gcc/configure --target=powerpc-linux --enable-targets=all

Re: [U-Boot] [PATCH 0/9] Fix a few warnings that bug me

2012-02-21 Thread Stefano Babic
On 21/02/2012 11:20, Stefano Babic wrote: I will try myself with mips, as this is one of the architectures reporting warnings. Just to report this: a MAKEALL mips reports no warning with a ggc 4.6 compiler : gcc version 4.6.1 20110627 (prerelease) (GCC) It seems to me that the warnings are

Re: [U-Boot] [PATCH 0/9] Fix a few warnings that bug me

2012-02-21 Thread Mike Frysinger
On Tuesday 21 February 2012 05:20:47 Stefano Babic wrote: I will try myself with mips, as this is one of the architectures reporting warnings. i don't think Simon did say that mips was throwing these warnings, so testing that arch and looking for these warnings won't help -mike signature.asc

Re: [U-Boot] [PATCH 0/9] Fix a few warnings that bug me

2012-02-21 Thread Stefano Babic
On 21/02/2012 18:06, Mike Frysinger wrote: On Tuesday 21 February 2012 05:20:47 Stefano Babic wrote: I will try myself with mips, as this is one of the architectures reporting warnings. i don't think Simon did say that mips was throwing these warnings, so testing that arch and looking for

Re: [U-Boot] [PATCH 0/9] Fix a few warnings that bug me

2012-02-21 Thread Simon Glass
Hi Stefano, On Tue, Feb 21, 2012 at 9:15 AM, Stefano Babic sba...@denx.de wrote: On 21/02/2012 18:06, Mike Frysinger wrote: On Tuesday 21 February 2012 05:20:47 Stefano Babic wrote: I will try myself with mips, as this is one of the architectures reporting warnings. i don't think Simon did

Re: [U-Boot] [PATCH 0/9] Fix a few warnings that bug me

2012-02-21 Thread Mike Frysinger
On Tuesday 21 February 2012 13:40:36 Simon Glass wrote: 68k only: arch/m68k/lib/board.c |2 +- - I believe this fix is sensible for the reasons described in the patch, regardless of what the toolchain does. arch/powerpc/lib/board.c|7 +-- - Similar to

Re: [U-Boot] [PATCH 0/9] Fix a few warnings that bug me

2012-02-20 Thread Simon Glass
Hi Wolfgang / Stefano, On Thu, Jan 5, 2012 at 10:28 PM, Simon Glass s...@chromium.org wrote: Hi Wolfgang, On Thu, Jan 5, 2012 at 10:20 PM, Wolfgang Denk w...@denx.de wrote: Dear Simon Glass, In message 1325822097-15227-1-git-send-email-...@chromium.org you wrote: This series fixes some

[U-Boot] [PATCH 0/9] Fix a few warnings that bug me

2012-01-05 Thread Simon Glass
This series fixes some warnings which seem to still be left over from all the warning-squashing efforts for 2011.12. If patches exist for some of these already then please ignore. The alias problem in USB is a bit odd, since I thought it was already fixed. But perhaps it has come back. Simon

Re: [U-Boot] [PATCH 0/9] Fix a few warnings that bug me

2012-01-05 Thread Wolfgang Denk
Dear Simon Glass, In message 1325822097-15227-1-git-send-email-...@chromium.org you wrote: This series fixes some warnings which seem to still be left over from all the warning-squashing efforts for 2011.12. If patches exist for some of these already then please ignore. The alias problem in

Re: [U-Boot] [PATCH 0/9] Fix a few warnings that bug me

2012-01-05 Thread Simon Glass
Hi Wolfgang, On Thu, Jan 5, 2012 at 10:20 PM, Wolfgang Denk w...@denx.de wrote: Dear Simon Glass, In message 1325822097-15227-1-git-send-email-...@chromium.org you wrote: This series fixes some warnings which seem to still be left over from all the warning-squashing efforts for 2011.12. If