Re: [U-Boot] [PATCH v5 1/3] ARM: Tegra: FDT: Add USB EHCI function for T30/T114

2013-11-13 Thread Andreas Müller
On Fri, Jun 21, 2013 at 6:20 PM, Stephen Warren swar...@wwwdotorg.org wrote:
 On 06/21/2013 05:05 AM, Jim Lin wrote:
 Add DT node for USB EHCI function.
 Add support for T30-Cardhu, T30-Beaver, T114-Dalmore boards.

 Changes in v5:
  - Move changes on fdtdec.h and fdtdec.c to patch 2/3
  - Modify PHY type to hsic for USB2 port

 HSIC is an odd choice; ULPI is much more common. Still, this isn't a big
 deal; this is simply a default value, so any board that enables USB2 can
 simply set the property to ulpi if needed.

Long time ago but now I am working on a board support for tegra30
which has ASIX-eth on USB2-HSIC and don't get it to work. Checking the
code leads me to the question:

Is it possible that phy_type = hsic is not handled at all?

Andreas
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v5 1/3] ARM: Tegra: FDT: Add USB EHCI function for T30/T114

2013-11-13 Thread Stephen Warren
On 11/13/2013 02:28 AM, Andreas Müller wrote:
 On Fri, Jun 21, 2013 at 6:20 PM, Stephen Warren swar...@wwwdotorg.org wrote:
 On 06/21/2013 05:05 AM, Jim Lin wrote:
 Add DT node for USB EHCI function.
 Add support for T30-Cardhu, T30-Beaver, T114-Dalmore boards.

 Changes in v5:
  - Move changes on fdtdec.h and fdtdec.c to patch 2/3
  - Modify PHY type to hsic for USB2 port

 HSIC is an odd choice; ULPI is much more common. Still, this isn't a big
 deal; this is simply a default value, so any board that enables USB2 can
 simply set the property to ulpi if needed.

 Long time ago but now I am working on a board support for tegra30
 which has ASIX-eth on USB2-HSIC and don't get it to work. Checking the
 code leads me to the question:
 
 Is it possible that phy_type = hsic is not handled at all?

Almost certainly. I don't believe any of the boards we currently support
in upstream U-Boot or the Linux kernel actually use/support HSIC, so
it's quite unlikely that the driver support for HSIC is present.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v5 1/3] ARM: Tegra: FDT: Add USB EHCI function for T30/T114

2013-06-26 Thread Marek Vasut
Dear Tom Warren,

 This is a Tegra change, Marek. I believe it's assigned to me in Patchwork.
 
 I'll apply it to u-boot-tegra/next today or tomorrow.
 
 Tom

This is also a USB change, but it doesn't touch anything vital, so be my guest. 
Thanks

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v5 1/3] ARM: Tegra: FDT: Add USB EHCI function for T30/T114

2013-06-25 Thread Marek Vasut
Dear Stephen Warren,

 On 06/21/2013 05:05 AM, Jim Lin wrote:
  Add DT node for USB EHCI function.
  Add support for T30-Cardhu, T30-Beaver, T114-Dalmore boards.
 
 The series,
 
 Reviewed-by: Stephen Warren swar...@nvidia.com
 
 Also, on Beaver, Cardhu-A04, and Dalmore, I tested downloading/booting a
 kernel over a USB Ethernet dongle, so,

I will apply this v5, OK?

Best regards,
Marek Vasut
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v5 1/3] ARM: Tegra: FDT: Add USB EHCI function for T30/T114

2013-06-25 Thread Tom Warren
This is a Tegra change, Marek. I believe it's assigned to me in Patchwork.

I'll apply it to u-boot-tegra/next today or tomorrow.

Tom

 -Original Message-
 From: Marek Vasut [mailto:ma...@denx.de]
 Sent: Tuesday, June 25, 2013 11:49 AM
 To: u-boot@lists.denx.de
 Cc: Stephen Warren; Jim Lin; Tom Warren
 Subject: Re: [U-Boot] [PATCH v5 1/3] ARM: Tegra: FDT: Add USB EHCI
 function for T30/T114
 
 Dear Stephen Warren,
 
  On 06/21/2013 05:05 AM, Jim Lin wrote:
   Add DT node for USB EHCI function.
   Add support for T30-Cardhu, T30-Beaver, T114-Dalmore boards.
 
  The series,
 
  Reviewed-by: Stephen Warren swar...@nvidia.com
 
  Also, on Beaver, Cardhu-A04, and Dalmore, I tested downloading/booting
  a kernel over a USB Ethernet dongle, so,
 
 I will apply this v5, OK?
 
 Best regards,
 Marek Vasut
--
nvpublic
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v5 1/3] ARM: Tegra: FDT: Add USB EHCI function for T30/T114

2013-06-24 Thread Stephen Warren
On 06/21/2013 05:05 AM, Jim Lin wrote:
 Add DT node for USB EHCI function.
 Add support for T30-Cardhu, T30-Beaver, T114-Dalmore boards.

The series,

Reviewed-by: Stephen Warren swar...@nvidia.com

Also, on Beaver, Cardhu-A04, and Dalmore, I tested downloading/booting a
kernel over a USB Ethernet dongle, so,

Tested-by: Stephen Warren swar...@nvidia.com

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v5 1/3] ARM: Tegra: FDT: Add USB EHCI function for T30/T114

2013-06-21 Thread Stephen Warren
On 06/21/2013 05:05 AM, Jim Lin wrote:
 Add DT node for USB EHCI function.
 Add support for T30-Cardhu, T30-Beaver, T114-Dalmore boards.

 Changes in v5:
  - Move changes on fdtdec.h and fdtdec.c to patch 2/3
  - Modify PHY type to hsic for USB2 port

HSIC is an odd choice; ULPI is much more common. Still, this isn't a big
deal; this is simply a default value, so any board that enables USB2 can
simply set the property to ulpi if needed.

Missing from the changelog is the fact you fixed the VBUS GPIO values
for Cardhu and Beaver to be correct.

I'll try to test and do a final review of this soon.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot