From: Daniel Mack <zonque@gmail.com>
To: Ajay Gupta <ajayg@nvidia.com>
Cc: "B, Ravi" <ravibabu@ti.com>,
"linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
"Balbi, Felipe" <balbi@ti.com>,
"grant.likely@secretlab.ca" <grant.likely@secretlab.ca>,
"devicetree-discuss@lists.ozlabs.org"
<devicetree-discuss@lists.ozlabs.org>,
"tony@atomide.com" <tony@atomide.com>,
"ajayguptaj@gmail.com" <ajayguptaj@gmail.com>,
"Hiremath, Vaibhav" <hvaibhav@ti.com>
Subject: Re: [PATCH v7 00/11] usb: musb: adding multi instance support
Date: Fri, 03 Aug 2012 11:02:35 +0200 [thread overview]
Message-ID: <501B93AB.7050501@gmail.com> (raw)
In-Reply-To: <F20D4CDD972CFA4D89D3A083D2D561967FF1F67DE3@BGMAIL01.nvidia.com>
On 03.08.2012 10:43, Ajay Gupta wrote:
>> On 03.08.2012 02:54, B, Ravi wrote:
>>>> On 02.08.2012 14:12, Ravi Babu wrote:
>> However, I needed to add a phandle "usb0-phy = <&usb0_phy>" to the
>> usb_otg_hs DTSI block, otherwise devm_usb_get_phy_by_phandle() in
>> drivers/usb/musb/musb_dsps.c would fail. Is that correct? I can't seem to
>> find that in your patches.
>
> It's getting done in patch 11/11.
Ah, ok.
>> With this addition, I see the following:
>>
>> [ 1.782180] musb-hdrc: version 6.0, ?dma?, otg (peripheral+host)
>> [ 1.809966] musb-hdrc musb-hdrc.0: MUSB HDRC host driver
>> [ 1.819068] musb-hdrc musb-hdrc.0: new USB bus registered, assigned
>> bus number 1
>> [ 1.827970] usb usb1: New USB device found, idVendor=1d6b,
>> idProduct=0002
>> [ 1.835184] usb usb1: New USB device strings: Mfr=3, Product=2,
>> SerialNumber=1
>> [ 1.842818] usb usb1: Product: MUSB HDRC host driver
>> [ 1.848031] usb usb1: Manufacturer: Linux
>> 3.6.0-rc1-00038-g8a1ec8f-dirty musb-hcd
>> [ 1.855933] usb usb1: SerialNumber: musb-hdrc.0
>> [ 1.866913] hub 1-0:1.0: USB hub found
>> [ 1.871192] hub 1-0:1.0: 1 port detected
>> [ 1.878106] musb-hdrc musb-hdrc.0: USB Host mode controller at
>> d08c0000 using PIO, IRQ 18
>>
>> ... but no USB functions. Also, every two seconds, the following message is
>> printed:
>>
>> [ 11.036608] musb_bus_suspend 2308: trying to suspend as a_wait_vrise
>> while active
>> [ 13.044811] musb_bus_suspend 2308: trying to suspend as a_wait_vrise
>> while active
>> [ 15.052196] musb_bus_suspend 2308: trying to suspend as a_wait_vrise
>> while active
>
> Do you see them even when you connect a device to port?
Yes. Will investigate more later today. Just wanted to know whether this
is a typical config problem.
Daniel
next prev parent reply other threads:[~2012-08-03 9:02 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-02 12:12 [PATCH v7 00/11] usb: musb: adding multi instance support Ravi Babu
2012-08-02 12:12 ` [PATCH v7 03/11] usb: musb: am335x: add support for dual instance Ravi Babu
2012-08-02 12:12 ` [PATCH v7 04/11] usb: otg: nop: add support for multiple tranceiver Ravi Babu
2012-08-02 12:12 ` [PATCH v7 07/11] usb: otg: nop: add dt support Ravi Babu
[not found] ` <1343909574-15633-1-git-send-email-ravibabu-l0cyMroinI0@public.gmane.org>
2012-08-02 12:12 ` [PATCH v7 01/11] usb: musb: add musb_ida for multi instance support Ravi Babu
2012-08-02 12:12 ` [PATCH v7 02/11] usb: musb: kill global and static for multi instance Ravi Babu
2012-08-02 12:12 ` [PATCH v7 05/11] usb: musb: dsps: add dt support Ravi Babu
2012-08-02 12:12 ` [PATCH v7 06/11] arm/dts: am33xx: Add dt data for usbss Ravi Babu
2012-08-02 12:12 ` [PATCH v7 08/11] arm/dts: am33xx: add dt data for usb nop phy Ravi Babu
2012-08-02 12:12 ` [PATCH v7 09/11] usb: musb: dsps: remove explicit NOP device creation Ravi Babu
2012-08-02 12:12 ` [PATCH v7 10/11] usb: musb: dsps: get the PHY using phandle api Ravi Babu
2012-08-02 22:08 ` [PATCH v7 00/11] usb: musb: adding multi instance support Daniel Mack
[not found] ` <501AFA6B.4010005-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2012-08-03 0:54 ` B, Ravi
[not found] ` <6C6B28D4DC342643927BEAFCE8707BF63E9DB6AE-Er742YJ7I/eIQmiDNMet8wC/G2K4zDHf@public.gmane.org>
2012-08-03 8:19 ` Daniel Mack
2012-08-03 8:43 ` Ajay Gupta
[not found] ` <F20D4CDD972CFA4D89D3A083D2D561967FF1F67DE3-kdsAE/FnitNDw2glCA4ptUEOCMrvLtNR@public.gmane.org>
2012-08-03 8:53 ` B, Ravi
2012-08-03 9:02 ` Daniel Mack [this message]
[not found] ` <501B93AB.7050501-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2012-08-03 9:07 ` Hiremath, Vaibhav
[not found] ` <79CD15C6BA57404B839C016229A409A83EA7F4A3-Er742YJ7I/eIQmiDNMet8wC/G2K4zDHf@public.gmane.org>
2012-08-03 11:41 ` Daniel Mack
2012-08-03 11:48 ` B, Ravi
2012-08-03 12:35 ` Koen Kooi
[not found] ` <501BB8F2.8030909-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2012-08-03 15:48 ` Hiremath, Vaibhav
2012-08-03 16:01 ` Koen Kooi
2012-08-05 19:52 ` Daniel Mack
2012-08-06 6:40 ` Hiremath, Vaibhav
2012-08-15 14:39 ` Christopher Harvey
2012-08-02 12:12 ` [PATCH v7 11/11] arm/dts: am33xx: add phy phandle to usbss Ravi Babu
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=501B93AB.7050501@gmail.com \
--to=zonque@gmail.com \
--cc=ajayg@nvidia.com \
--cc=ajayguptaj@gmail.com \
--cc=balbi@ti.com \
--cc=devicetree-discuss@lists.ozlabs.org \
--cc=grant.likely@secretlab.ca \
--cc=hvaibhav@ti.com \
--cc=linux-omap@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=ravibabu@ti.com \
--cc=tony@atomide.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;
as well as URLs for NNTP newsgroup(s).