* [U-Boot] [PATCH v6 10/18] tegra: usb: fdt: Add additional device tree definitions for USB ports
@ 2012-02-03 17:08 Simon Glass
2012-02-03 17:08 ` [U-Boot] [PATCH v6 11/18] tegra: usb: fdt: Add USB definitions for Tegra2 Seaboard Simon Glass
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Simon Glass @ 2012-02-03 17:08 UTC (permalink / raw)
To: u-boot
This adds clock references to the USB part of the device tree for U-Boot,
and marks USB1 as supporting legacy mode (which we disable in the driver).
The USB timing information may vary between boards sometimes, but for
now we hard-code it in C. This is because all current T2x boards use
the same values, we will deal with T3x later and we first need to agree
on the format for this timing information in the fdt and may in fact
decide that it has no place there.
Signed-off-by: Simon Glass <sjg@chromium.org>
---
Changes in v5:
- Add dr_mode property to control host/device/otg mode
- Add nvidia,has-legacy-mode property per review comments
- Change device tree comment style from // to /* */
Changes in v6:
- Remove dr_mode properties from SOC .dtsi file and move to boards
arch/arm/dts/tegra20.dtsi | 4 ++++
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/arch/arm/dts/tegra20.dtsi b/arch/arm/dts/tegra20.dtsi
index ec75747..df1eda4 100644
--- a/arch/arm/dts/tegra20.dtsi
+++ b/arch/arm/dts/tegra20.dtsi
@@ -176,6 +176,8 @@
reg = <0xc5000000 0x4000>;
interrupts = < 52 >;
phy_type = "utmi";
+ clocks = <&periph_clk 22>; /* PERIPH_ID_USBD */
+ nvidia,has-legacy-mode;
};
usb at c5004000 {
@@ -183,6 +185,7 @@
reg = <0xc5004000 0x4000>;
interrupts = < 53 >;
phy_type = "ulpi";
+ clocks = <&periph_clk 58>; /* PERIPH_ID_USB2 */
};
usb at c5008000 {
@@ -190,6 +193,7 @@
reg = <0xc5008000 0x4000>;
interrupts = < 129 >;
phy_type = "utmi";
+ clocks = <&periph_clk 59>; /* PERIPH_ID_USB3 */
};
};
--
1.7.7.3
^ permalink raw reply related [flat|nested] 6+ messages in thread* [U-Boot] [PATCH v6 11/18] tegra: usb: fdt: Add USB definitions for Tegra2 Seaboard
2012-02-03 17:08 [U-Boot] [PATCH v6 10/18] tegra: usb: fdt: Add additional device tree definitions for USB ports Simon Glass
@ 2012-02-03 17:08 ` Simon Glass
2012-02-03 20:02 ` Stephen Warren
2012-02-03 20:03 ` [U-Boot] [PATCH v6 10/18] tegra: usb: fdt: Add additional device tree definitions for USB ports Stephen Warren
2012-02-26 23:14 ` Marek Vasut
2 siblings, 1 reply; 6+ messages in thread
From: Simon Glass @ 2012-02-03 17:08 UTC (permalink / raw)
To: u-boot
We set up two USB ports, one of which can be host or device.
For some reason the kernel version does enable both ports.
Signed-off-by: Simon Glass <sjg@chromium.org>
---
Changes in v2:
- Remove 0x from fdt aliases
- Use "okay" instead of "ok" for fdt node status
Changes in v3:
- Disable USB2 which is not used on Seaboard
- Fix device tree indenting with tabs instead of spaces
- Remove "okay" from nodes since this is the default anyway
Changes in v5:
- Remove support-host-mode property
Changes in v6:
- Remove dr_mode properties from SOC .dtsi file and move to boards
board/nvidia/dts/tegra2-seaboard.dts | 11 +++++++++++
1 files changed, 11 insertions(+), 0 deletions(-)
diff --git a/board/nvidia/dts/tegra2-seaboard.dts b/board/nvidia/dts/tegra2-seaboard.dts
index dde5d03..f00341f 100644
--- a/board/nvidia/dts/tegra2-seaboard.dts
+++ b/board/nvidia/dts/tegra2-seaboard.dts
@@ -11,6 +11,12 @@
bootargs = "vmalloc=192M video=tegrafb console=ttyS0,115200n8 root=/dev/mmcblk1p3 rw rootwait";
};
+ aliases {
+ /* This defines the order of our USB ports */
+ usb0 = "/usb at c5008000";
+ usb1 = "/usb at c5000000";
+ };
+
memory {
device_type = "memory";
reg = < 0x00000000 0x40000000 >;
@@ -32,5 +38,10 @@
usb at c5000000 {
nvidia,vbus-gpio = <&gpio 24 0>; /* PD0 */
+ dr_mode = "otg";
+ };
+
+ usb at c5004000 {
+ status = "disabled";
};
};
--
1.7.7.3
^ permalink raw reply related [flat|nested] 6+ messages in thread* [U-Boot] [PATCH v6 10/18] tegra: usb: fdt: Add additional device tree definitions for USB ports
2012-02-03 17:08 [U-Boot] [PATCH v6 10/18] tegra: usb: fdt: Add additional device tree definitions for USB ports Simon Glass
2012-02-03 17:08 ` [U-Boot] [PATCH v6 11/18] tegra: usb: fdt: Add USB definitions for Tegra2 Seaboard Simon Glass
@ 2012-02-03 20:03 ` Stephen Warren
2012-02-03 21:44 ` Simon Glass
2012-02-26 23:14 ` Marek Vasut
2 siblings, 1 reply; 6+ messages in thread
From: Stephen Warren @ 2012-02-03 20:03 UTC (permalink / raw)
To: u-boot
Simon Glass wrote at Friday, February 03, 2012 10:08 AM:
> This adds clock references to the USB part of the device tree for U-Boot,
> and marks USB1 as supporting legacy mode (which we disable in the driver).
>
> The USB timing information may vary between boards sometimes, but for
> now we hard-code it in C. This is because all current T2x boards use
> the same values, we will deal with T3x later and we first need to agree
> on the format for this timing information in the fdt and may in fact
> decide that it has no place there.
>
> Signed-off-by: Simon Glass <sjg@chromium.org>
The has-legacy-mode property looks good to me.
The clock bindings need updating.
--
nvpublic
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH v6 10/18] tegra: usb: fdt: Add additional device tree definitions for USB ports
2012-02-03 20:03 ` [U-Boot] [PATCH v6 10/18] tegra: usb: fdt: Add additional device tree definitions for USB ports Stephen Warren
@ 2012-02-03 21:44 ` Simon Glass
0 siblings, 0 replies; 6+ messages in thread
From: Simon Glass @ 2012-02-03 21:44 UTC (permalink / raw)
To: u-boot
Hi Stephen,
On Fri, Feb 3, 2012 at 12:03 PM, Stephen Warren <swarren@nvidia.com> wrote:
> Simon Glass wrote at Friday, February 03, 2012 10:08 AM:
>> This adds clock references to the USB part of the device tree for U-Boot,
>> and marks USB1 as supporting legacy mode (which we disable in the driver).
>>
>> The USB timing information may vary between boards sometimes, but for
>> now we hard-code it in C. This is because all current T2x boards use
>> the same values, we will deal with T3x later and we first need to agree
>> on the format for this timing information in the fdt and may in fact
>> decide that it has no place there.
>>
>> Signed-off-by: Simon Glass <sjg@chromium.org>
>
> The has-legacy-mode property looks good to me.
>
> The clock bindings need updating.
Are you referring to your new arbitrary peripheral number assignment
or the peripheral clock binding that you sent?
Regards,
Simon
>
> --
> nvpublic
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH v6 10/18] tegra: usb: fdt: Add additional device tree definitions for USB ports
2012-02-03 17:08 [U-Boot] [PATCH v6 10/18] tegra: usb: fdt: Add additional device tree definitions for USB ports Simon Glass
2012-02-03 17:08 ` [U-Boot] [PATCH v6 11/18] tegra: usb: fdt: Add USB definitions for Tegra2 Seaboard Simon Glass
2012-02-03 20:03 ` [U-Boot] [PATCH v6 10/18] tegra: usb: fdt: Add additional device tree definitions for USB ports Stephen Warren
@ 2012-02-26 23:14 ` Marek Vasut
2 siblings, 0 replies; 6+ messages in thread
From: Marek Vasut @ 2012-02-26 23:14 UTC (permalink / raw)
To: u-boot
> This adds clock references to the USB part of the device tree for U-Boot,
> and marks USB1 as supporting legacy mode (which we disable in the driver).
>
> The USB timing information may vary between boards sometimes, but for
> now we hard-code it in C. This is because all current T2x boards use
> the same values, we will deal with T3x later and we first need to agree
> on the format for this timing information in the fdt and may in fact
> decide that it has no place there.
>
Hi,
what's the status of this patch/patchset?
Thanks
M
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-02-26 23:14 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-02-03 17:08 [U-Boot] [PATCH v6 10/18] tegra: usb: fdt: Add additional device tree definitions for USB ports Simon Glass
2012-02-03 17:08 ` [U-Boot] [PATCH v6 11/18] tegra: usb: fdt: Add USB definitions for Tegra2 Seaboard Simon Glass
2012-02-03 20:02 ` Stephen Warren
2012-02-03 20:03 ` [U-Boot] [PATCH v6 10/18] tegra: usb: fdt: Add additional device tree definitions for USB ports Stephen Warren
2012-02-03 21:44 ` Simon Glass
2012-02-26 23:14 ` Marek Vasut
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox