All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/9] Fix a few warnings that bug me
Date: Mon, 5 Mar 2012 19:46:23 +0100	[thread overview]
Message-ID: <201203051946.23945.marex@denx.de> (raw)
In-Reply-To: <CAPnjgZ3_1pzq5tW2QBd5ExPakhYfyBEDhaoLzn7oTaEcGYi3Gg@mail.gmail.com>

Dear Simon Glass,

> Hi Marek,
> 
> On Mon, Mar 5, 2012 at 10:22 AM, Marek Vasut <marex@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

      parent reply	other threads:[~2012-03-05 18:46 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
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 [this message]

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=201203051946.23945.marex@denx.de \
    --to=marex@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.