All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/9] Fix a few warnings that bug me
Date: Sat, 03 Mar 2012 15:31:25 +0100	[thread overview]
Message-ID: <20120303143125.17D8A126F3B0@gemini.denx.de> (raw)
In-Reply-To: <CAPnjgZ0Mb=LJBE3JSs0zOqivHY5pRfYNGpS3vnfs6O=SQXu9vQ@mail.gmail.com>

Dear Simon Glass,

In message <CAPnjgZ0Mb=LJBE3JSs0zOqivHY5pRfYNGpS3vnfs6O=SQXu9vQ@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: wd at denx.de
The optimum committee has no members.
                                                   - Norman Augustine

  parent reply	other threads:[~2012-03-03 14:31 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-06  3:54 [U-Boot] [PATCH 0/9] Fix a few warnings that bug me Simon Glass
2012-01-06  3:54 ` [U-Boot] [PATCH 1/9] usb: Fix strict-aliasing warning in host/ohci-hcd.c Simon Glass
2012-01-08  9:32   ` Mike Frysinger
2012-01-08 18:02     ` Simon Glass
2012-01-06  3:54 ` [U-Boot] [PATCH 2/9] Fix strict-aliasing warning in dlmalloc Simon Glass
2012-01-06  6:48   ` Wolfgang Denk
2012-01-06 20:39     ` Simon Glass
2012-01-06 23:37       ` Wolfgang Denk
2012-01-08  0:57         ` Simon Glass
2012-01-08  9:27       ` Mike Frysinger
2012-08-13 15:25   ` Andreas Bießmann
2012-09-04 21:25     ` Andreas Bießmann
2012-01-06  3:54 ` [U-Boot] [PATCH 3/9] mpc5xxx: Fix strict-aliasing warnings in usb_ohci.c Simon Glass
2012-01-06  3:54 ` [U-Boot] [PATCH 4/9] ppc4xx: " Simon Glass
2012-10-16  6:28   ` Marek Vasut
2012-10-16 13:35     ` Simon Glass
2012-01-06  3:54 ` [U-Boot] [PATCH 5/9] sandbox: sort header files in os.c Simon Glass
2012-01-08  8:49   ` Mike Frysinger
2012-01-09 22:17     ` Simon Glass
2012-01-06  3:54 ` [U-Boot] [PATCH 6/9] sandbox: Add required header to os.c Simon Glass
2012-01-08  8:49   ` Mike Frysinger
2012-01-09 22:18     ` Simon Glass
2012-01-06  3:54 ` [U-Boot] [PATCH 7/9] Remove CONFIG_SYS_EXTBDINFO from snapper9260.h Simon Glass
2012-02-22 11:07   ` Stefano Babic
2012-03-27 16:58   ` Anatolij Gustschin
2012-01-06  3:54 ` [U-Boot] [PATCH 8/9] m68k: Change memsz to a signed char to avoid warning Simon Glass
2012-01-08  9:29   ` Mike Frysinger
2012-02-22 12:01   ` Stefano Babic
2012-01-06  3:54 ` [U-Boot] [PATCH 9/9] ppc: Change memsz variable to signed char Simon Glass
2012-01-08  9:29   ` Mike Frysinger
2012-02-22 12:00   ` Stefano Babic
2012-01-06  6:20 ` [U-Boot] [PATCH 0/9] Fix a few warnings that bug me Wolfgang Denk
2012-01-06  6:28   ` Simon Glass
2012-02-20 18:49     ` Simon Glass
2012-02-21 10:20       ` Stefano Babic
2012-02-21 13:49         ` Stefano Babic
2012-02-21 17:06         ` Mike Frysinger
2012-02-21 17:15           ` Stefano Babic
2012-02-21 18:40             ` Simon Glass
2012-02-22  5:35               ` Mike Frysinger
2012-02-22 11:06               ` Stefano Babic
2012-03-03 14:31               ` Wolfgang Denk [this message]
2012-03-05 18:22 ` Marek Vasut
2012-03-05 18:33   ` Simon Glass
2012-03-05 18:42     ` Andy Pont
2012-03-05 19:17       ` Wolfgang Denk
2012-03-05 18:46     ` Marek Vasut

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120303143125.17D8A126F3B0@gemini.denx.de \
    --to=wd@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.