All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dan Carpenter <error27@gmail.com>
To: kernel-janitors@vger.kernel.org
Subject: Re: [patch] USB: xhci: unsigned char never equals -1
Date: Thu, 17 Mar 2011 21:07:41 +0000	[thread overview]
Message-ID: <20110317210741.GQ2008@bicker> (raw)
In-Reply-To: <20110317193949.GN2008@bicker>

On Thu, Mar 17, 2011 at 01:52:23PM -0700, Sarah Sharp wrote:
> On Thu, Mar 17, 2011 at 10:47:46PM +0300, Dan Carpenter wrote:
> > Btw, xhci-ring.c causes some valid looking gcc warnings:
> 
> Which git commit are you working off of?
> 

This is linux-next for 20110316.

commit b3694afad1864eaa7b59a1a5bcf576ed00d4089a
Merge: 37f454a 500132a
Author: Stephen Rothwell <sfr@canb.auug.org.au>
Date:   Wed Mar 16 16:53:13 2011 +1100

    Merge remote-tracking branch 'usb/usb-next'
    
    Conflicts:
        arch/arm/mach-omap2/board-omap3evm.c
        arch/arm/mach-omap2/board-overo.c
        arch/arm/mach-omap2/clock3xxx_data.c
        arch/arm/mach-omap2/usb-musb.c
        arch/arm/plat-omap/include/plat/usb.h
        drivers/usb/gadget/Kconfig
        drivers/usb/gadget/fsl_mxc_udc.c
        drivers/usb/musb/musb_core.h

commit ba0a4d9aaae789a6a632968b27c21d49b858b13a
Author: Sarah Sharp <sarah.a.sharp@linux.intel.com>
Date:   Wed Feb 23 18:13:43 2011 -0800

    xhci: Clean up cycle bit math used during stalls.


> > C [M]  drivers/usb/host/xhci-ring.o
> > drivers/usb/host/xhci-ring.c: In function ‘handle_port_status’:
> > drivers/usb/host/xhci-ring.c:1229: warning: ‘hcd’ may be used uninitialized in this function
> > drivers/usb/host/xhci-ring.c: In function ‘handle_tx_event’:
> > drivers/usb/host/xhci-ring.c:1895: warning: ‘event_trb’ may be used uninitialized in this function
> 
> I think the second warning is just the compiler getting confused, but
> I'll look into the first warning.  What gcc version are you using?  I
> had compile-tested the patches I recently sent to Greg, but on an older
> (un-updateable, thanks Ubuntu) box with an older gcc.
> 

I'm using gcc 4.4.3-4ubuntu5.

regards,
dan carpenter

      parent reply	other threads:[~2011-03-17 21:07 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-17 19:39 [patch] USB: xhci: unsigned char never equals -1 Dan Carpenter
2011-03-17 19:47 ` Dan Carpenter
2011-03-17 20:52 ` Sarah Sharp
2011-03-17 21:07 ` Dan Carpenter [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=20110317210741.GQ2008@bicker \
    --to=error27@gmail.com \
    --cc=kernel-janitors@vger.kernel.org \
    /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.