From: Tony Lindgren <tony@atomide.com>
To: "Gadiyar, Anand" <gadiyar@ti.com>
Cc: Andreas Hartmetz <ahartmetz@gmail.com>,
Thomas Weber <weber@corscience.de>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
Felipe Balbi <felipe.balbi@nokia.com>,
"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
"linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>
Subject: Re: [PATCH 10/11] omap: musb: get rid of dyn_fifo
Date: Tue, 23 Feb 2010 10:58:07 -0800 [thread overview]
Message-ID: <20100223185807.GI28173@atomide.com> (raw)
In-Reply-To: <5A47E75E594F054BAF48C5E4FC4B92AB0321A20527@dbde02.ent.ti.com>
* Gadiyar, Anand <gadiyar@ti.com> [100223 07:18]:
> > > I think this patch breaks musb_hdrc.
> > >
> > > In musb_core_init is musb->config->dyn_fifo checked but is never set.
> > >
> > > So I get the following dmesg output:
> > >
> > > musb_hdrc: ConfigData=0xde (UTMI-8, dyn FIFOs, bulk combine, bulk split,
> > > HB-ISO Rx, HB-ISO Tx, SoftConn)
> > > musb_hdrc: MHDRC RTL version 1.400
> > > musb_core_init 1405: reconfigure software for Dynamic FIFOs
> > > musb_hdrc musb_hdrc: musb_init_controller failed with status -19
> > >
> > > Thomas
> >
> > I think I have the same problem.
> > There are other problems right now (I'm trying several different branch/defconfig
> > combinations) so I can't be 100% sure.
> >
>
> Ajay reported earlier today [1] that there is a dependency patch [2]
> that needs to be taken in. Looks like it was missed when the MUSB
> patchset was added to linux-omap for extra testing.
>
> - Anand
>
> [1] http://marc.info/?l=linux-omap&m=126691578831301&w=2
> [2] http://marc.info/?l=linux-omap&m=126088393130177&w=2
Thanks, I'll drop this patch.
Sounds like Felipe is back from vacation and can then merge it via
linux-usb along with the other patch.
Regards,
Tony
WARNING: multiple messages have this Message-ID (diff)
From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 10/11] omap: musb: get rid of dyn_fifo
Date: Tue, 23 Feb 2010 10:58:07 -0800 [thread overview]
Message-ID: <20100223185807.GI28173@atomide.com> (raw)
In-Reply-To: <5A47E75E594F054BAF48C5E4FC4B92AB0321A20527@dbde02.ent.ti.com>
* Gadiyar, Anand <gadiyar@ti.com> [100223 07:18]:
> > > I think this patch breaks musb_hdrc.
> > >
> > > In musb_core_init is musb->config->dyn_fifo checked but is never set.
> > >
> > > So I get the following dmesg output:
> > >
> > > musb_hdrc: ConfigData=0xde (UTMI-8, dyn FIFOs, bulk combine, bulk split,
> > > HB-ISO Rx, HB-ISO Tx, SoftConn)
> > > musb_hdrc: MHDRC RTL version 1.400
> > > musb_core_init 1405: reconfigure software for Dynamic FIFOs
> > > musb_hdrc musb_hdrc: musb_init_controller failed with status -19
> > >
> > > Thomas
> >
> > I think I have the same problem.
> > There are other problems right now (I'm trying several different branch/defconfig
> > combinations) so I can't be 100% sure.
> >
>
> Ajay reported earlier today [1] that there is a dependency patch [2]
> that needs to be taken in. Looks like it was missed when the MUSB
> patchset was added to linux-omap for extra testing.
>
> - Anand
>
> [1] http://marc.info/?l=linux-omap&m=126691578831301&w=2
> [2] http://marc.info/?l=linux-omap&m=126088393130177&w=2
Thanks, I'll drop this patch.
Sounds like Felipe is back from vacation and can then merge it via
linux-usb along with the other patch.
Regards,
Tony
next prev parent reply other threads:[~2010-02-23 18:57 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-02-19 23:29 [PATCH 00/11] omap updates for 2.6.34, part 4 Tony Lindgren
2010-02-19 23:29 ` Tony Lindgren
2010-02-19 23:30 ` [PATCH 01/11] USB: Add empty functions in otg.h Tony Lindgren
2010-02-19 23:30 ` Tony Lindgren
2010-02-19 23:30 ` [PATCH 02/11] omap: musb: Remove #ifdef from board-omap3evm.c Tony Lindgren
2010-02-19 23:30 ` Tony Lindgren
[not found] ` <20100219232648.10559.380.stgit-XB442sTQ5VIxQTnqJstS8Q@public.gmane.org>
2010-02-19 23:30 ` [PATCH 03/11] arm: omap: musb: ioremap only what's ours Tony Lindgren
2010-02-19 23:30 ` Tony Lindgren
2010-02-19 23:37 ` Aguirre, Sergio
2010-02-19 23:37 ` Aguirre, Sergio
2010-02-19 23:44 ` Tony Lindgren
2010-02-19 23:44 ` Tony Lindgren
2010-02-19 23:55 ` Aguirre, Sergio
2010-02-19 23:55 ` Aguirre, Sergio
2010-02-20 0:10 ` Tony Lindgren
2010-02-20 0:10 ` Tony Lindgren
2010-02-19 23:30 ` [PATCH 07/11] omap: musb: remove unused soft_con field Tony Lindgren
2010-02-19 23:30 ` Tony Lindgren
2010-02-19 23:30 ` [PATCH 08/11] omap: musb: remove unused dma data Tony Lindgren
2010-02-19 23:30 ` Tony Lindgren
2010-02-19 23:30 ` [PATCH 04/11] omap: musb: Pass board specific data from board file Tony Lindgren
2010-02-19 23:30 ` Tony Lindgren
2010-02-19 23:30 ` [PATCH 05/11] omap: musb: remove unused data Tony Lindgren
2010-02-19 23:30 ` Tony Lindgren
2010-02-19 23:30 ` [PATCH 06/11] arm: omap: musb: we can use clk framework Tony Lindgren
2010-02-19 23:30 ` Tony Lindgren
2010-02-19 23:30 ` [PATCH 09/11] omap: musb: remove unnecessary return Tony Lindgren
2010-02-19 23:30 ` Tony Lindgren
2010-02-19 23:30 ` [PATCH 10/11] omap: musb: get rid of dyn_fifo Tony Lindgren
2010-02-19 23:30 ` Tony Lindgren
2010-02-23 9:02 ` Gupta, Ajay Kumar
2010-02-23 14:55 ` Thomas Weber
2010-02-23 14:55 ` Thomas Weber
2010-02-23 15:09 ` Andreas Hartmetz
2010-02-23 15:09 ` Andreas Hartmetz
2010-02-23 15:20 ` Gadiyar, Anand
2010-02-23 15:20 ` Gadiyar, Anand
2010-02-23 18:58 ` Tony Lindgren [this message]
2010-02-23 18:58 ` Tony Lindgren
2010-02-19 23:30 ` [PATCH 11/11] omap: musb: Add USB support to 4430 SDP board file Tony Lindgren
2010-02-19 23:30 ` Tony Lindgren
2010-02-20 8:30 ` [PATCH 00/11] omap updates for 2.6.34, part 4 Shilimkar, Santosh
2010-02-20 8:30 ` Shilimkar, Santosh
2010-02-22 21:37 ` Tony Lindgren
2010-02-22 21:37 ` Tony Lindgren
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=20100223185807.GI28173@atomide.com \
--to=tony@atomide.com \
--cc=ahartmetz@gmail.com \
--cc=felipe.balbi@nokia.com \
--cc=gadiyar@ti.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=weber@corscience.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.