From: Tony Lindgren <tony@atomide.com>
To: Felipe Balbi <me@felipebalbi.com>
Cc: Andrzej Zaborowski <balrogg@gmail.com>, linux-omap@vger.kernel.org
Subject: Re: [PATCH] MUSB: misc clean-up.
Date: Tue, 6 May 2008 15:14:43 -0700 [thread overview]
Message-ID: <20080506221442.GI18682@atomide.com> (raw)
In-Reply-To: <d1368e13f1a909c545bc51824c754176@felipebalbi.com>
* Felipe Balbi <me@felipebalbi.com> [080505 00:55]:
>
>
> On Sat, 3 May 2008 15:19:50 +0200, Andrzej Zaborowski <balrogg@gmail.com>
> wrote:
> > Correct the interrupt mask in TUSB6010 and other nit-picking from my
> tree.
> >
> > Signed-off-by: Andrzej Zaborowski <balrog@zabor.org>
>
> Acked-by: Felipe Balbi <felipe.balbi@nokia.com>
Pushing.
Tony
>
> > ---
> > drivers/usb/musb/musb_host.c | 2 +-
> > drivers/usb/musb/musb_procfs.c | 2 +-
> > drivers/usb/musb/tusb6010.c | 6 +++---
> > drivers/usb/musb/tusb6010_omap.c | 2 +-
> > 4 files changed, 6 insertions(+), 6 deletions(-)
> >
> > diff --git a/drivers/usb/musb/musb_host.c b/drivers/usb/musb/musb_host.c
> > index 64673b7..e9c766f 100644
> > --- a/drivers/usb/musb/musb_host.c
> > +++ b/drivers/usb/musb/musb_host.c
> > @@ -1203,7 +1203,7 @@ void musb_host_tx(struct musb *musb, u8 epnum)
> > * we have a candidate... NAKing is *NOT* an error
> > */
> > musb_ep_select(mbase, epnum);
> > - musb_writew(epio, MUSB_CSR0,
> > + musb_writew(epio, MUSB_TXCSR,
> > MUSB_TXCSR_H_WZC_BITS
> > | MUSB_TXCSR_TXPKTRDY);
> > goto finish;
> > diff --git a/drivers/usb/musb/musb_procfs.c
> > b/drivers/usb/musb/musb_procfs.c
> > index e6e050c..da683eb 100644
> > --- a/drivers/usb/musb/musb_procfs.c
> > +++ b/drivers/usb/musb/musb_procfs.c
> > @@ -453,7 +453,7 @@ dump_end_info(struct musb *musb, u8 epnum, char
> > *aBuffer, unsigned max)
> >
> > /* Dump the current status and compile options.
> > * @param musb the device driver instance
> > - * @param buffer where to dump the status; it must be big enough hold
> the
> > + * @param buffer where to dump the status; it must be big enough to hold
> > the
> > * result otherwise "BAD THINGS HAPPENS(TM)".
> > */
> > static int dump_header_stats(struct musb *musb, char *buffer)
> > diff --git a/drivers/usb/musb/tusb6010.c b/drivers/usb/musb/tusb6010.c
> > index bed2e18..1f3643b 100644
> > --- a/drivers/usb/musb/tusb6010.c
> > +++ b/drivers/usb/musb/tusb6010.c
> > @@ -740,7 +740,7 @@ tusb_otg_ints(struct musb *musb, u32 int_src, void
> > __iomem *tbase)
> > break;
> > case OTG_STATE_A_WAIT_VFALL:
> > /* REVISIT this irq triggers during short
> > - * spikes causet by enumeration ...
> > + * spikes caused by enumeration ...
> > */
> > if (musb->vbuserr_retry) {
> > musb->vbuserr_retry--;
> > @@ -888,7 +888,7 @@ static irqreturn_t tusb_interrupt(int irq, void
> > *__hci)
> > musb_writel(tbase, TUSB_DMA_INT_CLEAR, dma_src);
> > }
> >
> > - /* EP interrupts. In OCP mode tusb6010 mirrors the MUSB * interrupts */
> > + /* EP interrupts. In OCP mode tusb6010 mirrors the MUSB interrupts */
> > if (int_src & (TUSB_INT_SRC_USB_IP_TX | TUSB_INT_SRC_USB_IP_RX)) {
> > u32 musb_src = musb_readl(tbase, TUSB_USBIP_INT_SRC);
> >
> > @@ -972,7 +972,7 @@ void musb_platform_disable(struct musb *musb)
> >
> > /* disable all IRQs */
> > musb_writel(tbase, TUSB_INT_MASK, ~TUSB_INT_MASK_RESERVED_BITS);
> > - musb_writel(tbase, TUSB_USBIP_INT_MASK, 0);
> > + musb_writel(tbase, TUSB_USBIP_INT_MASK, 0x7fffffff);
> > musb_writel(tbase, TUSB_DMA_INT_MASK, 0x7fffffff);
> > musb_writel(tbase, TUSB_GPIO_INT_MASK, 0x1ff);
> >
> > diff --git a/drivers/usb/musb/tusb6010_omap.c
> > b/drivers/usb/musb/tusb6010_omap.c
> > index 66e146e..f7a3ffe 100644
> > --- a/drivers/usb/musb/tusb6010_omap.c
> > +++ b/drivers/usb/musb/tusb6010_omap.c
> > @@ -188,7 +188,7 @@ static void tusb_omap_dma_cb(int lch, u16 ch_status,
> > void *data)
> >
> > channel->status = MUSB_DMA_STATUS_FREE;
> >
> > - /* Handle only RX callbacks here. TX callbacks musb be handled based
> > + /* Handle only RX callbacks here. TX callbacks must be handled based
> > * on the TUSB DMA status interrupt.
> > * REVISIT: Use both TUSB DMA status interrupt and OMAP DMA callback
> > * interrupt for RX and TX.
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> --
> Best Regards,
>
> Felipe Balbi
> http://felipebalbi.com
> me@felipebalbi.com
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
prev parent reply other threads:[~2008-05-06 22:14 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-03 13:19 [PATCH] MUSB: misc clean-up Andrzej Zaborowski
2008-05-05 7:55 ` Felipe Balbi
2008-05-06 22:14 ` Tony Lindgren [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=20080506221442.GI18682@atomide.com \
--to=tony@atomide.com \
--cc=balrogg@gmail.com \
--cc=linux-omap@vger.kernel.org \
--cc=me@felipebalbi.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox