From: Daniel Mack <zonque-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: "B, Ravi" <ravibabu-l0cyMroinI0@public.gmane.org>
Cc: "ajayguptaj-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org"
<ajayguptaj-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
"devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org"
<devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org>,
"linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"Balbi, Felipe" <balbi-l0cyMroinI0@public.gmane.org>,
"linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Subject: Re: [PATCH v7 00/11] usb: musb: adding multi instance support
Date: Fri, 03 Aug 2012 10:19:35 +0200 [thread overview]
Message-ID: <501B8997.2070307@gmail.com> (raw)
In-Reply-To: <6C6B28D4DC342643927BEAFCE8707BF63E9DB6AE-Er742YJ7I/eIQmiDNMet8wC/G2K4zDHf@public.gmane.org>
On 03.08.2012 02:54, B, Ravi wrote:
>> On 02.08.2012 14:12, Ravi Babu wrote:
>>> This series of patches adds,
>>> a) Multi instances support in musb driver
>>> b) DT support for musb_dsps glue layer
>>> c) DT support for NOP transceiver
>>>
>>> AM33xx and TI81xx has dual musb controller and has two usb
>> PHY of same type.
>>> This patch series uses 'phandle' based API
>>> devm_usb_get_phy_by_phandle() to get the PHY of same type. This API
>>> support is being added by Kishon's patch discussed at [1]
>>>
>>> The series applies to linux-omap (master branch)
>>> + Vaibhav baseport patches on his tree at [3]
>>> + Kishon's multi phy patches on Felipe's branch 'xceiv'
>>> + Kishon's patch on phandle at [1]
>>> + AM33xx musb glue compile and bugfix patches at [4],
>> [5], [6] and [7]
>>> + Damodar's recent patch at [2]
>>>
>>> and have been tested on Beaglebone board.
>
> Have you applied the above patches before applying these patches.
Somehow, I was missing some of Ajay's patches. I resolved that, and now
the series applied.
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.
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
Anything obvious that I'm missing?
Thanks,
Daniel
next prev parent reply other threads:[~2012-08-03 8:19 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 [this message]
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
[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=501B8997.2070307@gmail.com \
--to=zonque-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
--cc=ajayguptaj-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=balbi-l0cyMroinI0@public.gmane.org \
--cc=devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org \
--cc=linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=ravibabu-l0cyMroinI0@public.gmane.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 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).