All of lore.kernel.org
 help / color / mirror / Atom feed
From: kishon <kishon@ti.com>
To: Koen Kooi <koen@dominion.thruhere.net>
Cc: <tony@atomide.com>, <balbi@ti.com>, <linux@arm.linux.org.uk>,
	<eballetbo@gmail.com>, <javier@dowhile0.org>,
	<gregkh@linuxfoundation.org>, <linux-usb@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	"linux-omap@vger.kernel.org List" <linux-omap@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>,
	<devicetree-discuss@lists.ozlabs.org>,
	Jason Kridner <jkridner@beagleboard.org>
Subject: Re: [PATCH v1 0/6] USB: Add support for multiple PHYs of same type
Date: Tue, 22 Jan 2013 21:15:01 +0530	[thread overview]
Message-ID: <50FEB3FD.3050001@ti.com> (raw)
In-Reply-To: <2D906ABA-71F2-4843-9DBE-F82527A1240A@dominion.thruhere.net>

On Tuesday 22 January 2013 09:11 PM, Koen Kooi wrote:
>
> Op 22 jan. 2013, om 10:58 heeft Kishon Vijay Abraham I <kishon@ti.com> het volgende geschreven:
>
>> This patch series adds support for adding multiple PHY's (of same type).
>> The binding information has to be present in the PHY library (otg.c) in
>> order for it to return the appropriate PHY whenever the USB controller
>> request for the PHY. So added a new API usb_bind_phy() to pass the binding
>> information. This API should be called by platform specific initialization
>> code.
>>
>> So the binding should be done something like
>> usb_bind_phy("musb-hdrc.0.auto", 0, "omap-usb2.1.auto"); specifying the USB
>> controller device name, index, and the PHY device name.
>> I have done this binding for OMAP platforms, but it should be done for
>> all the platforms.
>>
>> After this design, the phy can be got by passing the USB controller device
>> pointer and the index.
>>
>> Developed this patch series on
>> git://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git xceiv
>> after applying "usb: musb: add driver for control module" patch series
>> and "ARM: dts: omap: add dt data for MUSB"
>>
>> Did basic enumeration testing in omap4 panda and omap3 beagle.
>
> With this patchset USB completely breaks on am33xx beaglebone, is that intended?
Not really.
Does am33xx makes use of omap2430.c? Which PHY does am33xx uses?

Thanks
Kishon



WARNING: multiple messages have this Message-ID (diff)
From: kishon <kishon@ti.com>
To: Koen Kooi <koen@dominion.thruhere.net>
Cc: tony@atomide.com, balbi@ti.com, linux@arm.linux.org.uk,
	eballetbo@gmail.com, javier@dowhile0.org,
	gregkh@linuxfoundation.org, linux-usb@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	"linux-omap@vger.kernel.org List" <linux-omap@vger.kernel.org>,
	linux-kernel@vger.kernel.org,
	devicetree-discuss@lists.ozlabs.org,
	Jason Kridner <jkridner@beagleboard.org>
Subject: Re: [PATCH v1 0/6] USB: Add support for multiple PHYs of same type
Date: Tue, 22 Jan 2013 21:15:01 +0530	[thread overview]
Message-ID: <50FEB3FD.3050001@ti.com> (raw)
In-Reply-To: <2D906ABA-71F2-4843-9DBE-F82527A1240A@dominion.thruhere.net>

On Tuesday 22 January 2013 09:11 PM, Koen Kooi wrote:
>
> Op 22 jan. 2013, om 10:58 heeft Kishon Vijay Abraham I <kishon@ti.com> het volgende geschreven:
>
>> This patch series adds support for adding multiple PHY's (of same type).
>> The binding information has to be present in the PHY library (otg.c) in
>> order for it to return the appropriate PHY whenever the USB controller
>> request for the PHY. So added a new API usb_bind_phy() to pass the binding
>> information. This API should be called by platform specific initialization
>> code.
>>
>> So the binding should be done something like
>> usb_bind_phy("musb-hdrc.0.auto", 0, "omap-usb2.1.auto"); specifying the USB
>> controller device name, index, and the PHY device name.
>> I have done this binding for OMAP platforms, but it should be done for
>> all the platforms.
>>
>> After this design, the phy can be got by passing the USB controller device
>> pointer and the index.
>>
>> Developed this patch series on
>> git://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git xceiv
>> after applying "usb: musb: add driver for control module" patch series
>> and "ARM: dts: omap: add dt data for MUSB"
>>
>> Did basic enumeration testing in omap4 panda and omap3 beagle.
>
> With this patchset USB completely breaks on am33xx beaglebone, is that intended?
Not really.
Does am33xx makes use of omap2430.c? Which PHY does am33xx uses?

Thanks
Kishon

WARNING: multiple messages have this Message-ID (diff)
From: kishon@ti.com (kishon)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v1 0/6] USB: Add support for multiple PHYs of same type
Date: Tue, 22 Jan 2013 21:15:01 +0530	[thread overview]
Message-ID: <50FEB3FD.3050001@ti.com> (raw)
In-Reply-To: <2D906ABA-71F2-4843-9DBE-F82527A1240A@dominion.thruhere.net>

On Tuesday 22 January 2013 09:11 PM, Koen Kooi wrote:
>
> Op 22 jan. 2013, om 10:58 heeft Kishon Vijay Abraham I <kishon@ti.com> het volgende geschreven:
>
>> This patch series adds support for adding multiple PHY's (of same type).
>> The binding information has to be present in the PHY library (otg.c) in
>> order for it to return the appropriate PHY whenever the USB controller
>> request for the PHY. So added a new API usb_bind_phy() to pass the binding
>> information. This API should be called by platform specific initialization
>> code.
>>
>> So the binding should be done something like
>> usb_bind_phy("musb-hdrc.0.auto", 0, "omap-usb2.1.auto"); specifying the USB
>> controller device name, index, and the PHY device name.
>> I have done this binding for OMAP platforms, but it should be done for
>> all the platforms.
>>
>> After this design, the phy can be got by passing the USB controller device
>> pointer and the index.
>>
>> Developed this patch series on
>> git://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git xceiv
>> after applying "usb: musb: add driver for control module" patch series
>> and "ARM: dts: omap: add dt data for MUSB"
>>
>> Did basic enumeration testing in omap4 panda and omap3 beagle.
>
> With this patchset USB completely breaks on am33xx beaglebone, is that intended?
Not really.
Does am33xx makes use of omap2430.c? Which PHY does am33xx uses?

Thanks
Kishon

  reply	other threads:[~2013-01-22 15:45 UTC|newest]

Thread overview: 84+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-22  9:58 [PATCH v1 0/6] USB: Add support for multiple PHYs of same type Kishon Vijay Abraham I
2013-01-22  9:58 ` Kishon Vijay Abraham I
2013-01-22  9:58 ` Kishon Vijay Abraham I
2013-01-22  9:58 ` [PATCH v1 1/6] usb: otg: Add an API to bind the USB controller and PHY Kishon Vijay Abraham I
2013-01-22  9:58   ` Kishon Vijay Abraham I
2013-01-22  9:58   ` Kishon Vijay Abraham I
2013-01-22  9:58 ` [PATCH v1 2/6] usb: otg: utils: add facilities in phy lib to support multiple PHYs of same type Kishon Vijay Abraham I
2013-01-22  9:58   ` Kishon Vijay Abraham I
2013-01-22  9:58   ` Kishon Vijay Abraham I
2013-01-22 14:07   ` Roger Quadros
2013-01-22 14:07     ` Roger Quadros
2013-01-22 14:07     ` Roger Quadros
2013-01-22 14:13     ` kishon
2013-01-22 14:13       ` kishon
2013-01-22 14:13       ` kishon
2013-01-22  9:58 ` [PATCH v1 3/6] ARM: OMAP: USB: Add phy binding information Kishon Vijay Abraham I
2013-01-22  9:58   ` Kishon Vijay Abraham I
2013-01-22  9:58   ` Kishon Vijay Abraham I
2013-01-22  9:58 ` [PATCH v1 4/6] drivers: usb: musb: omap: make use of the new PHY lib APIs Kishon Vijay Abraham I
2013-01-22  9:58   ` Kishon Vijay Abraham I
2013-01-22  9:58   ` Kishon Vijay Abraham I
2013-01-22 14:09   ` Roger Quadros
2013-01-22 14:09     ` Roger Quadros
2013-01-22 14:09     ` Roger Quadros
2013-01-24 18:58     ` kishon
2013-01-24 18:58       ` kishon
2013-01-24 18:58       ` kishon
2013-01-22  9:58 ` [PATCH v1 5/6] usb: otg: add device tree support to otg library Kishon Vijay Abraham I
2013-01-22  9:58   ` Kishon Vijay Abraham I
2013-01-22  9:58   ` Kishon Vijay Abraham I
2013-01-22 10:36   ` Mark Rutland
2013-01-22 10:36     ` Mark Rutland
2013-01-22 10:36     ` Mark Rutland
2013-01-22 15:37     ` kishon
2013-01-22 15:37       ` kishon
2013-01-22 15:37       ` kishon
2013-01-22  9:58 ` [PATCH v1 6/6] USB: MUSB: OMAP: get PHY by phandle for dt boot Kishon Vijay Abraham I
2013-01-22  9:58   ` Kishon Vijay Abraham I
2013-01-22  9:58   ` Kishon Vijay Abraham I
2013-01-22 14:17   ` Roger Quadros
2013-01-22 14:17     ` Roger Quadros
2013-01-22 14:17     ` Roger Quadros
2013-01-22 14:37     ` kishon
2013-01-22 14:37       ` kishon
2013-01-22 14:37       ` kishon
2013-01-22 14:47       ` Roger Quadros
2013-01-22 14:47         ` Roger Quadros
2013-01-22 14:47         ` Roger Quadros
2013-01-22 15:21         ` kishon
2013-01-22 15:21           ` kishon
2013-01-22 15:21           ` kishon
2013-01-22 15:38           ` Peter Ujfalusi
2013-01-22 15:38             ` Peter Ujfalusi
2013-01-22 15:38             ` Peter Ujfalusi
2013-01-22 15:40             ` kishon
2013-01-22 15:40               ` kishon
2013-01-22 15:40               ` kishon
2013-01-22 15:41 ` [PATCH v1 0/6] USB: Add support for multiple PHYs of same type Koen Kooi
2013-01-22 15:41   ` Koen Kooi
2013-01-22 15:41   ` Koen Kooi
2013-01-22 15:45   ` kishon [this message]
2013-01-22 15:45     ` kishon
2013-01-22 15:45     ` kishon
2013-01-22 16:16     ` kishon
2013-01-22 16:16       ` kishon
2013-01-22 16:16       ` kishon
2013-01-22 17:02       ` Koen Kooi
2013-01-22 17:02         ` Koen Kooi
2013-01-22 17:02         ` Koen Kooi
2013-01-23  5:19         ` kishon
2013-01-23  5:19           ` kishon
2013-01-23  5:19           ` kishon
2013-01-23 13:58         ` Mohammed, Afzal
2013-01-23 13:58           ` Mohammed, Afzal
2013-01-23 13:58           ` Mohammed, Afzal
2013-01-23 14:26           ` kishon
2013-01-23 14:26             ` kishon
2013-01-23 14:26             ` kishon
2013-01-24 11:51             ` Mohammed, Afzal
2013-01-24 11:51               ` Mohammed, Afzal
2013-01-24 11:51               ` Mohammed, Afzal
2013-01-25 14:46               ` Mohammed, Afzal
2013-01-25 14:46                 ` Mohammed, Afzal
2013-01-25 14:46                 ` Mohammed, Afzal

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=50FEB3FD.3050001@ti.com \
    --to=kishon@ti.com \
    --cc=balbi@ti.com \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=eballetbo@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=javier@dowhile0.org \
    --cc=jkridner@beagleboard.org \
    --cc=koen@dominion.thruhere.net \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --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 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.