[PATCH/RESEND 0/2] usb: musb: am335x support part-2

[PATCH/RESEND 0/2] usb: musb: am335x support part-2

[Afzal Mohammed]

Hi Balbi,

This is a resend of remaining changes to get am335x usb working. These
were sent on 31 Oct with subject, "usb: musb: dsps: fixes for -rc.

First one restores capability to support at least one instance of musb.
Without it, even a single instance can't be supported as change which
is reverted by it was made along with multi phy changes and nop
transciever dt support, both other changes didn't make it to mainline. 

Second one corrects binding document; changes were made in driver as
per review comments, but documentation was not updated.

This is made on top of your "musb" branch. To get USB0 functional on
am335x based boards like beagle bone, first one is required.

Regards
Afzal

Afzal Mohammed (2):
  Revert "usb: musb: dsps: remove explicit NOP device creation"
  usb: musb: dsps: document dt bindings properly

 Documentation/devicetree/bindings/usb/am33xx-usb.txt | 8 ++++----
 drivers/usb/musb/musb_dsps.c                         | 3 ++-
 2 files changed, 6 insertions(+), 5 deletions(-)

-- 
1.7.12

[PATCH/RESEND 1/2] Revert "usb: musb: dsps: remove explicit NOP device creation"

[Afzal Mohammed]

This reverts commit d8c3ef256f88b7c6ecd673d03073b5645be9c5e4.

Above mentioned change was made along with multi usb phy change and
adding DT support for nop transceiver. But other two changes did not
make it to mainline. This in effect makes dsps musb wrapper unusable
even for single instance.

Hence revert it so that at least single instance can be supported.

Signed-off-by: Afzal Mohammed <afzal-l0cyMroinI0@public.gmane.org>
---
 drivers/usb/musb/musb_dsps.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/usb/musb/musb_dsps.c b/drivers/usb/musb/musb_dsps.c
index 6053af1..e770f79 100644
--- a/drivers/usb/musb/musb_dsps.c
+++ b/drivers/usb/musb/musb_dsps.c
@@ -411,7 +411,8 @@ static int dsps_musb_init(struct musb *musb)
 	/* mentor core register starts at offset of 0x400 from musb base */
 	musb->mregs += wrp->musb_core_offset;
 
-	/* Get the NOP PHY */
+	/* NOP driver needs change if supporting dual instance */
+	usb_nop_xceiv_register();
 	musb->xceiv = usb_get_phy(USB_PHY_TYPE_USB2);
 	if (IS_ERR_OR_NULL(musb->xceiv))
 		return -ENODEV;
-- 
1.7.12

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH/RESEND 2/2] usb: musb: dsps: document dt bindings properly

[Afzal Mohammed]

DT bindings normally use '-' (hyphens) instead of '_' (underscore),
driver has it the proper way, but binding documentation does not
reflect it, fix it.

Signed-off-by: Afzal Mohammed <afzal@ti.com>
---
 Documentation/devicetree/bindings/usb/am33xx-usb.txt | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/Documentation/devicetree/bindings/usb/am33xx-usb.txt b/Documentation/devicetree/bindings/usb/am33xx-usb.txt
index ca8fa56..a922505 100644
--- a/Documentation/devicetree/bindings/usb/am33xx-usb.txt
+++ b/Documentation/devicetree/bindings/usb/am33xx-usb.txt
@@ -3,12 +3,12 @@ AM33XX MUSB GLUE
  - ti,hwmods : must be "usb_otg_hs"
  - multipoint : Should be "1" indicating the musb controller supports
    multipoint. This is a MUSB configuration-specific setting.
- - num_eps : Specifies the number of endpoints. This is also a
+ - num-eps : Specifies the number of endpoints. This is also a
    MUSB configuration-specific setting. Should be set to "16"
- - ram_bits : Specifies the ram address size. Should be set to "12"
- - port0_mode : Should be "3" to represent OTG. "1" signifies HOST and "2"
+ - ram-bits : Specifies the ram address size. Should be set to "12"
+ - port0-mode : Should be "3" to represent OTG. "1" signifies HOST and "2"
    represents PERIPHERAL.
- - port1_mode : Should be "1" to represent HOST. "3" signifies OTG and "2"
+ - port1-mode : Should be "1" to represent HOST. "3" signifies OTG and "2"
    represents PERIPHERAL.
  - power : Should be "250". This signifies the controller can supply upto
    500mA when operating in host mode.
-- 
1.7.12

Re: [PATCH/RESEND 0/2] usb: musb: am335x support part-2

[Felipe Balbi]

[-- Attachment #1: Type: text/plain, Size: 799 bytes --]

Hi,

On Tue, Nov 06, 2012 at 08:44:59PM +0530, Afzal Mohammed wrote:
> Hi Balbi,
> 
> This is a resend of remaining changes to get am335x usb working. These
> were sent on 31 Oct with subject, "usb: musb: dsps: fixes for -rc.

there is no more time for this to reach -rc. Do we need it for v3.8
merge window ?

> First one restores capability to support at least one instance of musb.
> Without it, even a single instance can't be supported as change which
> is reverted by it was made along with multi phy changes and nop
> transciever dt support, both other changes didn't make it to mainline. 
> 
> Second one corrects binding document; changes were made in driver as
> per review comments, but documentation was not updated.

second one is clearly not for -rc.

-- 
balbi

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

RE: [PATCH/RESEND 0/2] usb: musb: am335x support part-2

[Mohammed, Afzal]

Hi Balbi,

On Thu, Nov 08, 2012 at 23:47:08, Balbi, Felipe wrote:
> On Tue, Nov 06, 2012 at 08:44:59PM +0530, Afzal Mohammed wrote:

> > This is a resend of remaining changes to get am335x usb working. These
> > were sent on 31 Oct with subject, "usb: musb: dsps: fixes for -rc.
> 
> there is no more time for this to reach -rc. Do we need it for v3.8
> merge window ?

Yes, we need it for 3.8, please consider it for inclusion in
3.8 merge window.

> > First one restores capability to support at least one instance of musb.
> > Without it, even a single instance can't be supported as change which
> > is reverted by it was made along with multi phy changes and nop
> > transciever dt support, both other changes didn't make it to mainline. 
> > 
> > Second one corrects binding document; changes were made in driver as
> > per review comments, but documentation was not updated.
> 
> second one is clearly not for -rc.

Ok, please consider both for merge window, without first one,
am335x (hence beagle bone) usb will not work.

Regards
Afzal