Re: USB Host support for mx25

2014-03-06 Thread Fabio Estevam 
On Thu, Mar 6, 2014 at 1:36 AM, Peter Chen peter.c...@freescale.com wrote:

 Do you have usbmisc node? usbmisc is not probed or its probe has failed.

Thanks, Peter. usbmisc node was 'disabled' at imx25.dtsi. I removed
the 'disabled' and now the probe goes a bit further:

ehci_hcd: USB 2.0 'Enhanced' Host Controller (EHCI) Driver
usbcore: registered new interface driver usb-storage
53ff4400.usb supply vbus not found, using dummy regulator
ci_hdrc ci_hdrc.0: doesn't support gadget
ci_hdrc ci_hdrc.0: EHCI Host Controller
ci_hdrc ci_hdrc.0: new USB bus registered, assigned bus number 1
ci_hdrc ci_hdrc.0: USB 2.0 started, EHCI 1.00
hub 1-0:1.0: USB hub found
hub 1-0:1.0: 1 port detected

usb 1-1: new low-speed USB device number 2 using ci_hdrc
usb 1-1: device descriptor read/64, error -71
usb 1-1: device descriptor read/64, error -71
usb 1-1: new low-speed USB device number 3 using ci_hdrc
usb 1-1: device descriptor read/64, error -71
usb 1-1: device descriptor read/64, error -71
usb 1-1: new low-speed USB device number 4 using ci_hdrc
usb 1-1: device not accepting address 4, error -71
...
usb 1-1: new low-speed USB device number 5 using ci_hdrc
usb 1-1: device not accepting address 5, error -71
hub 1-0:1.0: unable to enumerate USB device on port 1

My changes now look like this:

diff --git a/arch/arm/boot/dts/imx25-pdk.dts b/arch/arm/boot/dts/imx25-pdk.dts
index 7a6d21f..6f17d11 100644
--- a/arch/arm/boot/dts/imx25-pdk.dts
+++ b/arch/arm/boot/dts/imx25-pdk.dts
@@ -107,3 +107,8 @@
 fsl,uart-has-rtscts;
 status = okay;
 };
+
+usbhost1 {
+phy_type = serial;
+status = okay;
+};
diff --git a/arch/arm/boot/dts/imx25.dtsi b/arch/arm/boot/dts/imx25.dtsi
index 32f760e..7fa123b 100644
--- a/arch/arm/boot/dts/imx25.dtsi
+++ b/arch/arm/boot/dts/imx25.dtsi
@@ -482,13 +482,17 @@
 };

 usbphy1: usbphy@1 {
-compatible = nop-usbphy;
-status = disabled;
+compatible = usb-nop-xceiv;
+clocks = clks 0;
+clock-names = main_clk;
+status = okay;
 };

 usbphy2: usbphy@2 {
-compatible = nop-usbphy;
-status = disabled;
+compatible = usb-nop-xceiv;
+clocks = clks 0;
+clock-names = main_clk;
+status = okay;
 };

 usbotg: usb@53ff4000 {
@@ -498,6 +502,7 @@
 clocks = clks 9, clks 70, clks 8;
 clock-names = ipg, ahb, per;
 fsl,usbmisc = usbmisc 0;
+fsl,usbphy = usbphy1;
 status = disabled;
 };

@@ -508,6 +513,7 @@
 clocks = clks 9, clks 70, clks 8;
 clock-names = ipg, ahb, per;
 fsl,usbmisc = usbmisc 1;
+fsl,usbphy = usbphy2;
 status = disabled;
 };

@@ -517,7 +523,6 @@
 clocks = clks 9, clks 70, clks 8;
 clock-names = ipg, ahb, per;
 reg = 0x53ff4600 0x00f;
-status = disabled;
 };

 dryice@53ffc000 {

Let me know if you have some other suggestion.

Thanks,

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

Re: USB Host support for mx25

2014-03-06 Thread Eric Bénard 
Hi Fabio,

Le Thu, 6 Mar 2014 07:23:30 -0300,
Fabio Estevam feste...@gmail.com a écrit :
 Let me know if you have some other suggestion.

That's because usbmisc is not fully implemented for i.MX25  35 as
USB_CTRL must be configured (for both i.MX25 and i.MX35).
Denis is working on this.

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

Re: USB Host support for mx25

2014-03-06 Thread Alexander Shiyan 
Hello.

I am not familiar with mx25, but can say some hints.

Четверг,  6 марта 2014, 7:23 -03:00 от Fabio Estevam feste...@gmail.com:
 On Thu, Mar 6, 2014 at 1:36 AM, Peter Chen peter.c...@freescale.com wrote:
 
  Do you have usbmisc node? usbmisc is not probed or its probe has failed.
 
 Thanks, Peter. usbmisc node was 'disabled' at imx25.dtsi. I removed
 the 'disabled' and now the probe goes a bit further:
 
 ehci_hcd: USB 2.0 'Enhanced' Host Controller (EHCI) Driver
 usbcore: registered new interface driver usb-storage
 53ff4400.usb supply vbus not found, using dummy regulator
 ci_hdrc ci_hdrc.0: doesn't support gadget
 ci_hdrc ci_hdrc.0: EHCI Host Controller
 ci_hdrc ci_hdrc.0: new USB bus registered, assigned bus number 1
 ci_hdrc ci_hdrc.0: USB 2.0 started, EHCI 1.00
 hub 1-0:1.0: USB hub found
 hub 1-0:1.0: 1 port detected
 
 usb 1-1: new low-speed USB device number 2 using ci_hdrc
 usb 1-1: device descriptor read/64, error -71
 usb 1-1: device descriptor read/64, error -71
 usb 1-1: new low-speed USB device number 3 using ci_hdrc
 usb 1-1: device descriptor read/64, error -71
 usb 1-1: device descriptor read/64, error -71
 usb 1-1: new low-speed USB device number 4 using ci_hdrc
 usb 1-1: device not accepting address 4, error -71
 ...
 usb 1-1: new low-speed USB device number 5 using ci_hdrc
 usb 1-1: device not accepting address 5, error -71
 hub 1-0:1.0: unable to enumerate USB device on port 1
 
 My changes now look like this:
 
 diff --git a/arch/arm/boot/dts/imx25-pdk.dts b/arch/arm/boot/dts/imx25-pdk.dts
 index 7a6d21f..6f17d11 100644
 --- a/arch/arm/boot/dts/imx25-pdk.dts
 +++ b/arch/arm/boot/dts/imx25-pdk.dts
 @@ -107,3 +107,8 @@
  fsl,uart-has-rtscts;
  status = okay;
  };
 +
 +usbhost1 {
 +phy_type = serial;
 +status = okay;
 +};
 diff --git a/arch/arm/boot/dts/imx25.dtsi b/arch/arm/boot/dts/imx25.dtsi
 index 32f760e..7fa123b 100644
 --- a/arch/arm/boot/dts/imx25.dtsi
 +++ b/arch/arm/boot/dts/imx25.dtsi
 @@ -482,13 +482,17 @@
  };
 
  usbphy1: usbphy@1 {
 -compatible = nop-usbphy;
 -status = disabled;
 +compatible = usb-nop-xceiv;
 +clocks = clks 0;

Dummy clk?

 +clock-names = main_clk;
 +status = okay;
  };
 
  usbphy2: usbphy@2 {
 -compatible = nop-usbphy;
 -status = disabled;
 +compatible = usb-nop-xceiv;
 +clocks = clks 0;
 +clock-names = main_clk;
 +status = okay;
  };
 
  usbotg: usb@53ff4000 {
 @@ -498,6 +502,7 @@
  clocks = clks 9, clks 70, clks 8;
  clock-names = ipg, ahb, per;

Why here three clk? Chipidea driver requires only one.

  fsl,usbmisc = usbmisc 0;
 +fsl,usbphy = usbphy1;
  status = disabled;
  };
 
 @@ -508,6 +513,7 @@
  clocks = clks 9, clks 70, clks 8;
  clock-names = ipg, ahb, per;
  fsl,usbmisc = usbmisc 1;
 +fsl,usbphy = usbphy2;
  status = disabled;
  };
 
 @@ -517,7 +523,6 @@
  clocks = clks 9, clks 70, clks 8;
  clock-names = ipg, ahb, per;
  reg = 0x53ff4600 0x00f;
 -status = disabled;
  };
 
  dryice@53ffc000 {
 
 Let me know if you have some other suggestion.

I have added support for i.MX27 recently. You can use this as an example
for i.MX25.
https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/?id=a2e502c214ae65a3e0871e6216f1bc105524d9ad
https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/?id=9089ce520f6fdcb3c921a12d24834197251bd56f

---

Re: USB Host support for mx25

2014-03-06 Thread Fabio Estevam 
On Thu, Mar 6, 2014 at 2:54 PM, Alexander Shiyan shc_w...@mail.ru wrote:
 Hello.

 I am not familiar with mx25, but can say some hints.

Thank you all for the comments.

I will resume working on mx25 usb host support next week and will let
you know of any progress.

Thanks,

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

Re: USB Host support for mx25

2014-03-06 Thread Denis Carikli 

Hi,

On 03/06/2014 11:23 AM, Fabio Estevam wrote:
 
 usb 1-1: new low-speed USB device number 2 using ci_hdrc
 usb 1-1: device descriptor read/64, error -71
Is that USB peripheral really low-speed?

I'm also trying to make the USB host work on the cpuimx25 and the 
cpuimx35, so I got the same error.


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

USB Host support for mx25

2014-03-05 Thread Fabio Estevam 
Hi,

Has anyone succeeded on getting USB host support on mx25 with the
chipidea driver?

I am trying to add USB host1 support for mx25pdk and I am starting to
debug it, but


Looking at imx25.dtsi we have:

usbphy1: usbphy@1 {
compatible = nop-usbphy;
status = disabled;
};

usbphy2: usbphy@2 {
compatible = nop-usbphy;
status = disabled;
};

,but nop-usbphy is not defined anywhere.

Any help is appreciated.

Regards,

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

RE: USB Host support for mx25

2014-03-05 Thread Peter Chen 

 
 Hi,
 
 Has anyone succeeded on getting USB host support on mx25 with the
 chipidea driver?
 
 I am trying to add USB host1 support for mx25pdk and I am starting to
 debug it, but
 
 
 Looking at imx25.dtsi we have:
 
 usbphy1: usbphy@1 {
 compatible = nop-usbphy;
 status = disabled;
 };
 
 usbphy2: usbphy@2 {
 compatible = nop-usbphy;
 status = disabled;
 };
 
 ,but nop-usbphy is not defined anywhere.
 

The nop phy driver is at: drivers/usb/phy/phy-generic.c
The compatible string is: usb-nop-xceiv

 Any help is appreciated.
 
 Regards,
 
 Fabio Estevam
 

N�r��yb�X��ǧv�^�)޺{.n�+{��^n�r���z���h����G���h�(�階�ݢj���m��z�ޖ���f���h���~�m�

Re: USB Host support for mx25

2014-03-05 Thread Fabio Estevam 
Hi Peter,

On Wed, Mar 5, 2014 at 10:33 PM, Peter Chen peter.c...@freescale.com wrote:

 The nop phy driver is at: drivers/usb/phy/phy-generic.c
 The compatible string is: usb-nop-xceiv

This is not enough.

Here are my changes:

diff --git a/arch/arm/boot/dts/imx25-pdk.dts b/arch/arm/boot/dts/imx25-pdk.dts
index 7a6d21f..fc91592 100644
--- a/arch/arm/boot/dts/imx25-pdk.dts
+++ b/arch/arm/boot/dts/imx25-pdk.dts
@@ -107,3 +107,8 @@
fsl,uart-has-rtscts;
status = okay;
 };
+
+usbhost1 {
+   external-vbus-divider;
+   status = okay;
+};
diff --git a/arch/arm/boot/dts/imx25.dtsi b/arch/arm/boot/dts/imx25.dtsi
index 32f760e..11198af 100644
--- a/arch/arm/boot/dts/imx25.dtsi
+++ b/arch/arm/boot/dts/imx25.dtsi
@@ -482,13 +482,17 @@
};

usbphy1: usbphy@1 {
-   compatible = nop-usbphy;
-   status = disabled;
+   compatible = usb-nop-xceiv;
+   clocks = clks 0;
+   clock-names = main_clk;
+   status = okay;
};

usbphy2: usbphy@2 {
-   compatible = nop-usbphy;
-   status = disabled;
usbphy2: usbphy@2 {
-   compatible = nop-usbphy;
-   status = disabled;
+   compatible = usb-nop-xceiv;
+   clocks = clks 0;
+   clock-names = main_clk;
+   status = okay;
};

usbotg: usb@53ff4000 {
@@ -498,6 +502,7 @@
clocks = clks 9, clks 70, clks 8;
clock-names = ipg, ahb, per;
fsl,usbmisc = usbmisc 0;
+   fsl,usbphy = usbphy1;
status = disabled;
};

@@ -508,6 +513,7 @@
clocks = clks 9, clks 70, clks 8;
clock-names = ipg, ahb, per;
fsl,usbmisc = usbmisc 1;
+   fsl,usbphy = usbphy2;
status = disabled;
};


,and then it fails at:

imx_usb 53ff4400.usb: usbmisc init failed, ret=-517
platform 53ff4400.usb: Driver imx_usb requests probe deferral

Any other idea?

Regards,

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

RE: USB Host support for mx25

2014-03-05 Thread Peter Chen 
 
 Here are my changes:
 
 diff --git a/arch/arm/boot/dts/imx25-pdk.dts b/arch/arm/boot/dts/imx25-
 pdk.dts index 7a6d21f..fc91592 100644
 --- a/arch/arm/boot/dts/imx25-pdk.dts
 +++ b/arch/arm/boot/dts/imx25-pdk.dts
 @@ -107,3 +107,8 @@
 fsl,uart-has-rtscts;
 status = okay;
  };
 +
 +usbhost1 {
 +   external-vbus-divider;
 +   status = okay;
 +};
 diff --git a/arch/arm/boot/dts/imx25.dtsi b/arch/arm/boot/dts/imx25.dtsi
 index 32f760e..11198af 100644
 --- a/arch/arm/boot/dts/imx25.dtsi
 +++ b/arch/arm/boot/dts/imx25.dtsi
 @@ -482,13 +482,17 @@
 };
 
 usbphy1: usbphy@1 {
 -   compatible = nop-usbphy;
 -   status = disabled;
 +   compatible = usb-nop-xceiv;
 +   clocks = clks 0;
 +   clock-names = main_clk;
 +   status = okay;
 };
 
 usbphy2: usbphy@2 {
 -   compatible = nop-usbphy;
 -   status = disabled;
 usbphy2: usbphy@2 {
 -   compatible = nop-usbphy;
 -   status = disabled;
 +   compatible = usb-nop-xceiv;
 +   clocks = clks 0;
 +   clock-names = main_clk;
 +   status = okay;
 };
 
 usbotg: usb@53ff4000 { @@ -498,6 +502,7 @@
 clocks = clks 9, clks 70, clks 8;
 clock-names = ipg, ahb, per;
 fsl,usbmisc = usbmisc 0;
 +   fsl,usbphy = usbphy1;
 status = disabled;
 };
 
 @@ -508,6 +513,7 @@
 clocks = clks 9, clks 70, clks 8;
 clock-names = ipg, ahb, per;
 fsl,usbmisc = usbmisc 1;
 +   fsl,usbphy = usbphy2;
 status = disabled;
 };
 
 
 ,and then it fails at:
 
 imx_usb 53ff4400.usb: usbmisc init failed, ret=-517 platform 53ff4400.usb:
 Driver imx_usb requests probe deferral
 

Do you have usbmisc node? usbmisc is not probed or its probe has failed.

Peter

N�r��yb�X��ǧv�^�)޺{.n�+{��^n�r���z���h����G���h�(�階�ݢj���m��z�ޖ���f���h���~�m�