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 has come back.
 
 
 Simon Glass (9):
   usb: Fix strict-aliasing warning in host/ohci-hcd.c
   Fix strict-aliasing warning in dlmalloc
   mpc5xxx: Fix strict-aliasing warnings in usb_ohci.c
   ppc4xx: Fix strict-aliasing warnings in usb_ohci.c
   sandbox: sort header files in os.c
   sandbox: Add required header to os.c
   Remove CONFIG_SYS_EXTBDINFO from snapper9260.h
   m68k: Change memsz to a signed char to avoid warning
   ppc: Change memsz variable to signed char
 
  arch/m68k/lib/board.c   |2 +-
  arch/powerpc/cpu/mpc5xxx/usb_ohci.c |   79
 -- arch/powerpc/cpu/ppc4xx/usb_ohci.c  |  
 81 -- arch/powerpc/lib/board.c   
 |7 +--
  arch/sandbox/cpu/os.c   |   11 +++--
  common/dlmalloc.c   |8 ++--
  drivers/usb/host/ohci-hcd.c |   26 ++--
  include/configs/snapper9260.h   |1 -
  8 files changed, 122 insertions(+), 93 deletions(-)

Can you please split out the USB part and send it to me (I became new usb 
maint.) please?

Thanks in advance!

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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 problem in USB is a bit odd, since I thought it was already
 fixed. But perhaps it has come back.


 Simon Glass (9):
   usb: Fix strict-aliasing warning in host/ohci-hcd.c
   Fix strict-aliasing warning in dlmalloc
   mpc5xxx: Fix strict-aliasing warnings in usb_ohci.c
   ppc4xx: Fix strict-aliasing warnings in usb_ohci.c
   sandbox: sort header files in os.c
   sandbox: Add required header to os.c
   Remove CONFIG_SYS_EXTBDINFO from snapper9260.h
   m68k: Change memsz to a signed char to avoid warning
   ppc: Change memsz variable to signed char

  arch/m68k/lib/board.c               |    2 +-
  arch/powerpc/cpu/mpc5xxx/usb_ohci.c |   79
 -- arch/powerpc/cpu/ppc4xx/usb_ohci.c  |
 81 -- arch/powerpc/lib/board.c
 |    7 +--
  arch/sandbox/cpu/os.c               |   11 +++--
  common/dlmalloc.c                   |    8 ++--
  drivers/usb/host/ohci-hcd.c         |   26 ++--
  include/configs/snapper9260.h       |    1 -
  8 files changed, 122 insertions(+), 93 deletions(-)

 Can you please split out the USB part and send it to me (I became new usb
 maint.) please?

 Thanks in advance!

Of course - I think you want to look at this one:

http://patchwork.ozlabs.org/patch/134600/

The two PowerPC ones apparently only happen with the 4.4 toolchain, so
I think we are going to drop those and I am going to upgrade my
toolchain instead :-)

Regards,
Simon


 Best regards,
 Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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 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 Glass (9):
usb: Fix strict-aliasing warning in host/ohci-hcd.c
Fix strict-aliasing warning in dlmalloc
mpc5xxx: Fix strict-aliasing warnings in usb_ohci.c
ppc4xx: Fix strict-aliasing warnings in usb_ohci.c
sandbox: sort header files in os.c
sandbox: Add required header to os.c
Remove CONFIG_SYS_EXTBDINFO from snapper9260.h
m68k: Change memsz to a signed char to avoid warning
ppc: Change memsz variable to signed char
  
   arch/m68k/lib/board.c   |2 +-
   arch/powerpc/cpu/mpc5xxx/usb_ohci.c |   79
  -- arch/powerpc/cpu/ppc4xx/usb_ohci.c  |
  81 -- arch/powerpc/lib/board.c
  
  |7 +--
  
   arch/sandbox/cpu/os.c   |   11 +++--
   common/dlmalloc.c   |8 ++--
   drivers/usb/host/ohci-hcd.c |   26 ++--
   include/configs/snapper9260.h   |1 -
   8 files changed, 122 insertions(+), 93 deletions(-)
  
  Can you please split out the USB part and send it to me (I became new usb
  maint.) please?
  
  Thanks in advance!
 
 Of course - I think you want to look at this one:
 
 http://patchwork.ozlabs.org/patch/134600/
 
 The two PowerPC ones apparently only happen with the 4.4 toolchain, so
 I think we are going to drop those and I am going to upgrade my
 toolchain instead :-)
 

Can you actually paste the MAKEALL output for those boards where you observe 
the 
issue please?

I took a look at that patch and I'm not yet completely convinced by the memcpy 
business there. Also, I don't want to pick patches from the middle of a 
patchset, so can you please resend what you consider eligible?

Thanks!

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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 going to drop those and I am going to upgrade my
 toolchain instead :-)

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 there a generic
issue with 4.4.x versions or just a rogue board or two?

Regards,

Andy.


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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 there a generic
 issue with 4.4.x versions or just a rogue board or two?

The problem appears to pop up only for some boards, but that means I
would not trust this tool chain at all.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Documentation is like sex: when it is good, it is  very,  very  good;
and when it is bad, it is better than nothing. - Dick Brandon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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 these and re-submit them, clearly indicating which
errors / warnings these are supposed to fix.

 PPC only:
  arch/powerpc/cpu/mpc5xxx/usb_ohci.c |   79 ---=
 ---
  arch/powerpc/cpu/ppc4xx/usb_ohci.c  |   81 ---=
 ---
 - These two seem not to happen with a later toolchain. Please skip
 these if you like.

They also seem not to happen with older tool chains (like GCC 4.2.2).

  common/dlmalloc.c   |8 ++--
 - These happen with MIPS, AVR, 68k for me (with Mike's toolchains).
 Does not happen on 4.6 ARM.

Not seen with GCC 4.2.2 nor with GCC 4.6.

 ARM and PPC:
  drivers/usb/host/ohci-hcd.c |   26 ++--
 - These happen with my 4.4.4 compiler, apparently not on 4.6. Drop
 them if you like.

Not seen with GCC 4.2.2 nor with GCC 4.6.


Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
The optimum committee has no members.
   - Norman Augustine
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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 for the reasons described in the
 patch, regardless of what the toolchain does.

Agree - I will send my ACK and merge it into my u-boot-staging

  arch/powerpc/lib/board.c|7 +--
 - Similar to the68k patch, again I think it makes sense to apply this.
 The variable is unsigned char for no good reason.

Agree, there is no reason. I will merge also this one into u-boot-staging.

 
 Sandbox:
  arch/sandbox/cpu/os.c   |   11 +++--
 - This should be applied as it is not toolchain dependent (Mike may
 have picked it up already)

Ok, the patch has already found its way to mainline

 
 ARM and PPC:
  drivers/usb/host/ohci-hcd.c |   26 ++--
 - These happen with my 4.4.4 compiler, apparently not on 4.6. Drop
 them if you like.
 
 ARM:
  include/configs/snapper9260.h   |1 -
 - This fixes a real problem regardless of toolchain.

Right. I will send my ACK with CC to Albert. He can pick-up it directly
into u-boot-arm.

Best regards,
Stefano

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: off...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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
 --prefix=/opt/cross/gcc-4.4.4-nolibc/powerpc-linux/
 --enable-languages=c --with-newlib --without-headers
 --enable-sjlj-exceptions --with-system-libunwind --disable-nls
 --disable-threads --disable-shared --disable-libmudflap
 --disable-libssp --disable-libgomp --disable-decimal-float
 --enable-checking=release --with-mpfr=/home/tony/buildall/src/sys
 --with-gmp=/home/tony/buildall/src/sys
 Thread model: single
 gcc version 4.4.4 (GCC)

 I got this from Mike F (from the Gentoo site).

 
 What should we do about this series? I still see these warnings with
 the toolchains I use [1]. Does anyone else?

I read the whole thread, and it seems that it is not clear if the
warnings are real or must be related to the used toolchain.

I will try myself with mips, as this is one of the architectures
reporting warnings.

However, current toolchains based on more restrictive compilers
(I mean, GCC = 4.6) seem to be happy with the code. At least a couple
of patches reassembles code that seems correct...

IMHO if the code cannot be built with an older toolchain we can try to
fix, and also we must drop warnings with newer and current toolchains (a
lot of patches fixed gcc 4.6 warnings). I am aware that we cannot
support every toolchain, even if we try to do it. And maybe (only my
opinion) it is acceptable to have some warnings with older toolchains.

So tendentially my vote is not to take these patches, but Wolfgang has
the last word on them.

Stefano

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: off...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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 more related to the selected toolchain.

Stefano

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: off...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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
Description: This is a digitally signed message part.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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 these warnings won't help

In the thread I read that Simon got warnings with both mips and
powerpc, as well as with avr32 and m68k. But mips and powerpc are free
of warnings when I build with gcc 4.6.

Stefano

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: off...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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 say that mips was throwing these warnings,
 so testing that arch and looking for these warnings won't help

Part of the problem is that I mislabeled the commits, claiming that I
was using a 4.6 compiler when in fact mostly it was 4.4.4.

 In the thread I read that Simon got warnings with both mips and
 powerpc, as well as with avr32 and m68k. But mips and powerpc are free
 of warnings when I build with gcc 4.6.

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 for the reasons described in the
patch, regardless of what the toolchain does.

PPC only:
 arch/powerpc/cpu/mpc5xxx/usb_ohci.c |   79 --
 arch/powerpc/cpu/ppc4xx/usb_ohci.c  |   81 --
- These two seem not to happen with a later toolchain. Please skip
these if you like.

 arch/powerpc/lib/board.c|7 +--
- Similar to the68k patch, again I think it makes sense to apply this.
The variable is unsigned char for no good reason.

Sandbox:
 arch/sandbox/cpu/os.c   |   11 +++--
- This should be applied as it is not toolchain dependent (Mike may
have picked it up already)

 common/dlmalloc.c   |8 ++--
- These happen with MIPS, AVR, 68k for me (with Mike's toolchains).
Does not happen on 4.6 ARM.

ARM and PPC:
 drivers/usb/host/ohci-hcd.c |   26 ++--
- These happen with my 4.4.4 compiler, apparently not on 4.6. Drop
them if you like.

ARM:
 include/configs/snapper9260.h   |1 -
- This fixes a real problem regardless of toolchain.

Regards,
Simon


 Stefano

 --
 =
 DENX Software Engineering GmbH,     MD: Wolfgang Denk  Detlev Zundel
 HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
 Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: off...@denx.de
 =
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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 the68k patch, again I think it makes sense to apply this.
 The variable is unsigned char for no good reason.

i agree

 Sandbox:
  arch/sandbox/cpu/os.c   |   11 +++--
 - This should be applied as it is not toolchain dependent (Mike may
 have picked it up already)

i have

  common/dlmalloc.c   |8 ++--
 - These happen with MIPS, AVR, 68k for me (with Mike's toolchains).

what boards ?  i tried a bunch of m68k ones and aren't seeing it ...
-mike


signature.asc
Description: This is a digitally signed message part.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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 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.

 Can you please let me know which tool chain is throwing these errors?
 I built master yesterday, and before adding the commit that
 introduced the ``_LINUX_CONFIG_H redefined'' issue the PPC tree was
 built clean - with ELDK 4.2 (gcc 4.2.2), ELDK 5.1 (gcc 4.6.1) and with
 ELDK 5.2 (gcc 4.6.3).

 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
 --prefix=/opt/cross/gcc-4.4.4-nolibc/powerpc-linux/
 --enable-languages=c --with-newlib --without-headers
 --enable-sjlj-exceptions --with-system-libunwind --disable-nls
 --disable-threads --disable-shared --disable-libmudflap
 --disable-libssp --disable-libgomp --disable-decimal-float
 --enable-checking=release --with-mpfr=/home/tony/buildall/src/sys
 --with-gmp=/home/tony/buildall/src/sys
 Thread model: single
 gcc version 4.4.4 (GCC)

 I got this from Mike F (from the Gentoo site).


What should we do about this series? I still see these warnings with
the toolchains I use [1]. Does anyone else?

Regards,
Simon

[1] http://dev.gentoo.org/~vapier/u-boot/

 Regards,
 Simon


 Best regards,

 Wolfgang Denk

 --
 DENX Software Engineering GmbH,     MD: Wolfgang Denk  Detlev Zundel
 HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
 Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
 Documentation is like sex: when it is good, it is  very,  very  good;
 and when it is bad, it is better than nothing.         - Dick Brandon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[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 Glass (9):
  usb: Fix strict-aliasing warning in host/ohci-hcd.c
  Fix strict-aliasing warning in dlmalloc
  mpc5xxx: Fix strict-aliasing warnings in usb_ohci.c
  ppc4xx: Fix strict-aliasing warnings in usb_ohci.c
  sandbox: sort header files in os.c
  sandbox: Add required header to os.c
  Remove CONFIG_SYS_EXTBDINFO from snapper9260.h
  m68k: Change memsz to a signed char to avoid warning
  ppc: Change memsz variable to signed char

 arch/m68k/lib/board.c   |2 +-
 arch/powerpc/cpu/mpc5xxx/usb_ohci.c |   79 --
 arch/powerpc/cpu/ppc4xx/usb_ohci.c  |   81 --
 arch/powerpc/lib/board.c|7 +--
 arch/sandbox/cpu/os.c   |   11 +++--
 common/dlmalloc.c   |8 ++--
 drivers/usb/host/ohci-hcd.c |   26 ++--
 include/configs/snapper9260.h   |1 -
 8 files changed, 122 insertions(+), 93 deletions(-)

-- 
1.7.3.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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 USB is a bit odd, since I thought it was already
 fixed. But perhaps it has come back.

Can you please let me know which tool chain is throwing these errors?
I built master yesterday, and before adding the commit that
introduced the ``_LINUX_CONFIG_H redefined'' issue the PPC tree was
built clean - with ELDK 4.2 (gcc 4.2.2), ELDK 5.1 (gcc 4.6.1) and with
ELDK 5.2 (gcc 4.6.3).

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Documentation is like sex: when it is good, it is  very,  very  good;
and when it is bad, it is better than nothing. - Dick Brandon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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 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.

 Can you please let me know which tool chain is throwing these errors?
 I built master yesterday, and before adding the commit that
 introduced the ``_LINUX_CONFIG_H redefined'' issue the PPC tree was
 built clean - with ELDK 4.2 (gcc 4.2.2), ELDK 5.1 (gcc 4.6.1) and with
 ELDK 5.2 (gcc 4.6.3).

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
--prefix=/opt/cross/gcc-4.4.4-nolibc/powerpc-linux/
--enable-languages=c --with-newlib --without-headers
--enable-sjlj-exceptions --with-system-libunwind --disable-nls
--disable-threads --disable-shared --disable-libmudflap
--disable-libssp --disable-libgomp --disable-decimal-float
--enable-checking=release --with-mpfr=/home/tony/buildall/src/sys
--with-gmp=/home/tony/buildall/src/sys
Thread model: single
gcc version 4.4.4 (GCC)

I got this from Mike F (from the Gentoo site).

Regards,
Simon


 Best regards,

 Wolfgang Denk

 --
 DENX Software Engineering GmbH,     MD: Wolfgang Denk  Detlev Zundel
 HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
 Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
 Documentation is like sex: when it is good, it is  very,  very  good;
 and when it is bad, it is better than nothing.         - Dick Brandon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot