| Message ID | 20191002080051.11142-6-jckuo@nvidia.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Add Tegra194 XUSB host and pad controller support | expand |
On Wed, Oct 02, 2019 at 04:00:50PM +0800, JC Kuo wrote: > Adds the XUSB pad and XUSB controllers on Tegra194. > > Signed-off-by: JC Kuo <jckuo@nvidia.com> > --- > arch/arm64/boot/dts/nvidia/tegra194.dtsi | 130 +++++++++++++++++++++++ > 1 file changed, 130 insertions(+) > > diff --git a/arch/arm64/boot/dts/nvidia/tegra194.dtsi b/arch/arm64/boot/dts/nvidia/tegra194.dtsi > index 3c0cf54f0aab..4d3371d3a407 100644 > --- a/arch/arm64/boot/dts/nvidia/tegra194.dtsi > +++ b/arch/arm64/boot/dts/nvidia/tegra194.dtsi > @@ -1599,4 +1599,134 @@ > interrupt-parent = <&gic>; > always-on; > }; > + > + xusb_padctl: padctl@3520000 { > + compatible = "nvidia,tegra194-xusb-padctl"; > + reg = <0x0 0x03520000 0x0 0x1000>, > + <0x0 0x03540000 0x0 0x1000>; These should generally be aligned. Use tabs first and then spaces to make the opening < on subsequent lines align with the opening < on the first line. There are a couple more like this below. > + reg-names = "padctl", "ao"; > + > + resets = <&bpmp TEGRA194_RESET_XUSB_PADCTL>; > + reset-names = "padctl"; > + > + status = "disabled"; > + > + pads { > + usb2 { > + clocks = <&bpmp TEGRA194_CLK_USB2_TRK>; > + clock-names = "trk"; > + > + lanes { > + usb2-0 { > + nvidia,function = "xusb"; > + status = "disabled"; > + #phy-cells = <0>; > + }; > + usb2-1 { I prefer blank lines to visually separate blocks here and below. > + nvidia,function = "xusb"; > + status = "disabled"; > + #phy-cells = <0>; > + }; > + usb2-2 { > + nvidia,function = "xusb"; > + status = "disabled"; > + #phy-cells = <0>; > + }; > + usb2-3 { > + nvidia,function = "xusb"; > + status = "disabled"; > + #phy-cells = <0>; > + }; > + }; > + }; > + usb3 { > + lanes { > + usb3-0 { > + nvidia,function = "xusb"; > + status = "disabled"; > + #phy-cells = <0>; > + }; > + usb3-1 { > + nvidia,function = "xusb"; > + status = "disabled"; > + #phy-cells = <0>; > + }; > + usb3-2 { > + nvidia,function = "xusb"; > + status = "disabled"; > + #phy-cells = <0>; > + }; > + usb3-3 { > + nvidia,function = "xusb"; > + status = "disabled"; > + #phy-cells = <0>; > + }; > + }; > + }; > + }; > + > + ports { > + usb2-0 { > + status = "disabled"; > + }; > + usb2-1 { > + status = "disabled"; > + }; > + usb2-2 { > + status = "disabled"; > + }; > + usb2-3 { > + status = "disabled"; > + }; > + usb3-0 { > + status = "disabled"; > + }; > + usb3-1 { > + status = "disabled"; > + }; > + usb3-2 { > + status = "disabled"; > + }; > + usb3-3 { > + status = "disabled"; > + }; > + }; > + }; > + > + tegra_xhci: xhci@3610000 { The tegra_xhci is unused, so I don't think we need to add it. Also, the name of this node should be usb@3610000 since it's the compatible string that defines (together with the bindings) that this is XHCI capable. But it is fundamentally a USB controller, so the name should reflect that. > + compatible = "nvidia,tegra194-xusb"; > + reg = <0x0 0x03610000 0x0 0x40000>, > + <0x0 0x03600000 0x0 0x10000>; > + reg-names = "hcd", "fpci"; > + > + interrupts = <GIC_SPI 163 IRQ_TYPE_LEVEL_HIGH>, > + <GIC_SPI 164 IRQ_TYPE_LEVEL_HIGH>, > + <GIC_SPI 167 IRQ_TYPE_LEVEL_HIGH>; > + > + clocks = <&bpmp TEGRA194_CLK_XUSB_CORE_MUX>, > + <&bpmp TEGRA194_CLK_XUSB_CORE_HOST>, > + <&bpmp TEGRA194_CLK_XUSB_CORE_SS>, > + <&bpmp TEGRA194_CLK_XUSB_FALCON>, > + <&bpmp TEGRA194_CLK_XUSB_FALCON_HOST>, > + <&bpmp TEGRA194_CLK_XUSB_FALCON_SS>, > + <&bpmp TEGRA194_CLK_XUSB_FS>, > + <&bpmp TEGRA194_CLK_XUSB_FS_HOST>, > + <&bpmp TEGRA194_CLK_XUSB_SS>, > + <&bpmp TEGRA194_CLK_XUSB_SS_SUPERSPEED>, > + <&bpmp TEGRA194_CLK_UTMIPLL>, > + <&bpmp TEGRA194_CLK_CLK_M>, > + <&bpmp TEGRA194_CLK_PLLE>; > + clock-names = "xusb_hs_src", "xusb_host", > + "xusb_core_superspeed_clk", "xusb_falcon_src", > + "xusb_falcon_host_clk", "xusb_falcon_superspeed_clk", > + "xusb_fs_src", "xusb_fs_host_clk", "xusb_ss_src", > + "xusb_ss", "pll_u_480m", "clk_m", "pll_e"; Some of these clocks are not defined in the bindings. Many of them are also not used by the driver. Are all of these really needed? If they are, please add the required ones to the bindings. Also, for new ones, drop the _clk suffix. The fact that these are clocks is already conveyed by the property name. Thierry > + > + power-domains = <&bpmp TEGRA194_POWER_DOMAIN_XUSBC>, > + <&bpmp TEGRA194_POWER_DOMAIN_XUSBA>; > + power-domain-names = "xusb_host", "xusb_ss"; > + > + nvidia,xusb-padctl = <&xusb_padctl>; > + status = "disabled"; > + }; > }; > -- > 2.17.1 >
On Wed, Oct 02, 2019 at 04:00:50PM +0800, JC Kuo wrote: > Adds the XUSB pad and XUSB controllers on Tegra194. > > Signed-off-by: JC Kuo <jckuo@nvidia.com> > --- > arch/arm64/boot/dts/nvidia/tegra194.dtsi | 130 +++++++++++++++++++++++ > 1 file changed, 130 insertions(+) > > diff --git a/arch/arm64/boot/dts/nvidia/tegra194.dtsi b/arch/arm64/boot/dts/nvidia/tegra194.dtsi > index 3c0cf54f0aab..4d3371d3a407 100644 > --- a/arch/arm64/boot/dts/nvidia/tegra194.dtsi > +++ b/arch/arm64/boot/dts/nvidia/tegra194.dtsi > @@ -1599,4 +1599,134 @@ > interrupt-parent = <&gic>; > always-on; > }; > + > + xusb_padctl: padctl@3520000 { I also noticed that this is outside of the /cbb bus node. It really belongs inside /cbb. Same for the XHCI controller node. Thierry > + compatible = "nvidia,tegra194-xusb-padctl"; > + reg = <0x0 0x03520000 0x0 0x1000>, > + <0x0 0x03540000 0x0 0x1000>; > + reg-names = "padctl", "ao"; > + > + resets = <&bpmp TEGRA194_RESET_XUSB_PADCTL>; > + reset-names = "padctl"; > + > + status = "disabled"; > + > + pads { > + usb2 { > + clocks = <&bpmp TEGRA194_CLK_USB2_TRK>; > + clock-names = "trk"; > + > + lanes { > + usb2-0 { > + nvidia,function = "xusb"; > + status = "disabled"; > + #phy-cells = <0>; > + }; > + usb2-1 { > + nvidia,function = "xusb"; > + status = "disabled"; > + #phy-cells = <0>; > + }; > + usb2-2 { > + nvidia,function = "xusb"; > + status = "disabled"; > + #phy-cells = <0>; > + }; > + usb2-3 { > + nvidia,function = "xusb"; > + status = "disabled"; > + #phy-cells = <0>; > + }; > + }; > + }; > + usb3 { > + lanes { > + usb3-0 { > + nvidia,function = "xusb"; > + status = "disabled"; > + #phy-cells = <0>; > + }; > + usb3-1 { > + nvidia,function = "xusb"; > + status = "disabled"; > + #phy-cells = <0>; > + }; > + usb3-2 { > + nvidia,function = "xusb"; > + status = "disabled"; > + #phy-cells = <0>; > + }; > + usb3-3 { > + nvidia,function = "xusb"; > + status = "disabled"; > + #phy-cells = <0>; > + }; > + }; > + }; > + }; > + > + ports { > + usb2-0 { > + status = "disabled"; > + }; > + usb2-1 { > + status = "disabled"; > + }; > + usb2-2 { > + status = "disabled"; > + }; > + usb2-3 { > + status = "disabled"; > + }; > + usb3-0 { > + status = "disabled"; > + }; > + usb3-1 { > + status = "disabled"; > + }; > + usb3-2 { > + status = "disabled"; > + }; > + usb3-3 { > + status = "disabled"; > + }; > + }; > + }; > + > + tegra_xhci: xhci@3610000 { > + compatible = "nvidia,tegra194-xusb"; > + reg = <0x0 0x03610000 0x0 0x40000>, > + <0x0 0x03600000 0x0 0x10000>; > + reg-names = "hcd", "fpci"; > + > + interrupts = <GIC_SPI 163 IRQ_TYPE_LEVEL_HIGH>, > + <GIC_SPI 164 IRQ_TYPE_LEVEL_HIGH>, > + <GIC_SPI 167 IRQ_TYPE_LEVEL_HIGH>; > + > + clocks = <&bpmp TEGRA194_CLK_XUSB_CORE_MUX>, > + <&bpmp TEGRA194_CLK_XUSB_CORE_HOST>, > + <&bpmp TEGRA194_CLK_XUSB_CORE_SS>, > + <&bpmp TEGRA194_CLK_XUSB_FALCON>, > + <&bpmp TEGRA194_CLK_XUSB_FALCON_HOST>, > + <&bpmp TEGRA194_CLK_XUSB_FALCON_SS>, > + <&bpmp TEGRA194_CLK_XUSB_FS>, > + <&bpmp TEGRA194_CLK_XUSB_FS_HOST>, > + <&bpmp TEGRA194_CLK_XUSB_SS>, > + <&bpmp TEGRA194_CLK_XUSB_SS_SUPERSPEED>, > + <&bpmp TEGRA194_CLK_UTMIPLL>, > + <&bpmp TEGRA194_CLK_CLK_M>, > + <&bpmp TEGRA194_CLK_PLLE>; > + clock-names = "xusb_hs_src", "xusb_host", > + "xusb_core_superspeed_clk", "xusb_falcon_src", > + "xusb_falcon_host_clk", "xusb_falcon_superspeed_clk", > + "xusb_fs_src", "xusb_fs_host_clk", "xusb_ss_src", > + "xusb_ss", "pll_u_480m", "clk_m", "pll_e"; > + > + power-domains = <&bpmp TEGRA194_POWER_DOMAIN_XUSBC>, > + <&bpmp TEGRA194_POWER_DOMAIN_XUSBA>; > + power-domain-names = "xusb_host", "xusb_ss"; > + > + nvidia,xusb-padctl = <&xusb_padctl>; > + status = "disabled"; > + }; > }; > -- > 2.17.1 >
On 10/2/19 6:10 PM, Thierry Reding wrote: > On Wed, Oct 02, 2019 at 04:00:50PM +0800, JC Kuo wrote: >> Adds the XUSB pad and XUSB controllers on Tegra194. >> >> Signed-off-by: JC Kuo <jckuo@nvidia.com> >> --- >> arch/arm64/boot/dts/nvidia/tegra194.dtsi | 130 +++++++++++++++++++++++ >> 1 file changed, 130 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/nvidia/tegra194.dtsi b/arch/arm64/boot/dts/nvidia/tegra194.dtsi >> index 3c0cf54f0aab..4d3371d3a407 100644 >> --- a/arch/arm64/boot/dts/nvidia/tegra194.dtsi >> +++ b/arch/arm64/boot/dts/nvidia/tegra194.dtsi >> @@ -1599,4 +1599,134 @@ >> interrupt-parent = <&gic>; >> always-on; >> }; >> + >> + xusb_padctl: padctl@3520000 { >> + compatible = "nvidia,tegra194-xusb-padctl"; >> + reg = <0x0 0x03520000 0x0 0x1000>, >> + <0x0 0x03540000 0x0 0x1000>; > > These should generally be aligned. Use tabs first and then spaces to > make the opening < on subsequent lines align with the opening < on the > first line. There are a couple more like this below. Thanks. I will make those aligned. > >> + reg-names = "padctl", "ao"; >> + >> + resets = <&bpmp TEGRA194_RESET_XUSB_PADCTL>; >> + reset-names = "padctl"; >> + >> + status = "disabled"; >> + >> + pads { >> + usb2 { >> + clocks = <&bpmp TEGRA194_CLK_USB2_TRK>; >> + clock-names = "trk"; >> + >> + lanes { >> + usb2-0 { >> + nvidia,function = "xusb"; >> + status = "disabled"; >> + #phy-cells = <0>; >> + }; >> + usb2-1 { > > I prefer blank lines to visually separate blocks here and below. Sure, will do. > >> + nvidia,function = "xusb"; >> + status = "disabled"; >> + #phy-cells = <0>; >> + }; >> + usb2-2 { >> + nvidia,function = "xusb"; >> + status = "disabled"; >> + #phy-cells = <0>; >> + }; >> + usb2-3 { >> + nvidia,function = "xusb"; >> + status = "disabled"; >> + #phy-cells = <0>; >> + }; >> + }; >> + }; >> + usb3 { >> + lanes { >> + usb3-0 { >> + nvidia,function = "xusb"; >> + status = "disabled"; >> + #phy-cells = <0>; >> + }; >> + usb3-1 { >> + nvidia,function = "xusb"; >> + status = "disabled"; >> + #phy-cells = <0>; >> + }; >> + usb3-2 { >> + nvidia,function = "xusb"; >> + status = "disabled"; >> + #phy-cells = <0>; >> + }; >> + usb3-3 { >> + nvidia,function = "xusb"; >> + status = "disabled"; >> + #phy-cells = <0>; >> + }; >> + }; >> + }; >> + }; >> + >> + ports { >> + usb2-0 { >> + status = "disabled"; >> + }; >> + usb2-1 { >> + status = "disabled"; >> + }; >> + usb2-2 { >> + status = "disabled"; >> + }; >> + usb2-3 { >> + status = "disabled"; >> + }; >> + usb3-0 { >> + status = "disabled"; >> + }; >> + usb3-1 { >> + status = "disabled"; >> + }; >> + usb3-2 { >> + status = "disabled"; >> + }; >> + usb3-3 { >> + status = "disabled"; >> + }; >> + }; >> + }; >> + >> + tegra_xhci: xhci@3610000 { > > The tegra_xhci is unused, so I don't think we need to add it. Also, the > name of this node should be usb@3610000 since it's the compatible string > that defines (together with the bindings) that this is XHCI capable. But > it is fundamentally a USB controller, so the name should reflect that. > Understood. I will fix in the next revision. >> + compatible = "nvidia,tegra194-xusb"; >> + reg = <0x0 0x03610000 0x0 0x40000>, >> + <0x0 0x03600000 0x0 0x10000>; >> + reg-names = "hcd", "fpci"; >> + >> + interrupts = <GIC_SPI 163 IRQ_TYPE_LEVEL_HIGH>, >> + <GIC_SPI 164 IRQ_TYPE_LEVEL_HIGH>, >> + <GIC_SPI 167 IRQ_TYPE_LEVEL_HIGH>; >> + >> + clocks = <&bpmp TEGRA194_CLK_XUSB_CORE_MUX>, >> + <&bpmp TEGRA194_CLK_XUSB_CORE_HOST>, >> + <&bpmp TEGRA194_CLK_XUSB_CORE_SS>, >> + <&bpmp TEGRA194_CLK_XUSB_FALCON>, >> + <&bpmp TEGRA194_CLK_XUSB_FALCON_HOST>, >> + <&bpmp TEGRA194_CLK_XUSB_FALCON_SS>, >> + <&bpmp TEGRA194_CLK_XUSB_FS>, >> + <&bpmp TEGRA194_CLK_XUSB_FS_HOST>, >> + <&bpmp TEGRA194_CLK_XUSB_SS>, >> + <&bpmp TEGRA194_CLK_XUSB_SS_SUPERSPEED>, >> + <&bpmp TEGRA194_CLK_UTMIPLL>, >> + <&bpmp TEGRA194_CLK_CLK_M>, >> + <&bpmp TEGRA194_CLK_PLLE>; >> + clock-names = "xusb_hs_src", "xusb_host", >> + "xusb_core_superspeed_clk", "xusb_falcon_src", >> + "xusb_falcon_host_clk", "xusb_falcon_superspeed_clk", >> + "xusb_fs_src", "xusb_fs_host_clk", "xusb_ss_src", >> + "xusb_ss", "pll_u_480m", "clk_m", "pll_e"; > > Some of these clocks are not defined in the bindings. Many of them are > also not used by the driver. Are all of these really needed? If they > are, please add the required ones to the bindings. > > Also, for new ones, drop the _clk suffix. The fact that these are clocks > is already conveyed by the property name. > > Thierry Thanks. I will clean this up. > >> + >> + power-domains = <&bpmp TEGRA194_POWER_DOMAIN_XUSBC>, >> + <&bpmp TEGRA194_POWER_DOMAIN_XUSBA>; >> + power-domain-names = "xusb_host", "xusb_ss"; >> + >> + nvidia,xusb-padctl = <&xusb_padctl>; >> + status = "disabled"; >> + }; >> }; >> -- >> 2.17.1 >>
On 10/2/19 6:11 PM, Thierry Reding wrote: > On Wed, Oct 02, 2019 at 04:00:50PM +0800, JC Kuo wrote: >> Adds the XUSB pad and XUSB controllers on Tegra194. >> >> Signed-off-by: JC Kuo <jckuo@nvidia.com> >> --- >> arch/arm64/boot/dts/nvidia/tegra194.dtsi | 130 +++++++++++++++++++++++ >> 1 file changed, 130 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/nvidia/tegra194.dtsi b/arch/arm64/boot/dts/nvidia/tegra194.dtsi >> index 3c0cf54f0aab..4d3371d3a407 100644 >> --- a/arch/arm64/boot/dts/nvidia/tegra194.dtsi >> +++ b/arch/arm64/boot/dts/nvidia/tegra194.dtsi >> @@ -1599,4 +1599,134 @@ >> interrupt-parent = <&gic>; >> always-on; >> }; >> + >> + xusb_padctl: padctl@3520000 { > > I also noticed that this is outside of the /cbb bus node. It really > belongs inside /cbb. Same for the XHCI controller node. > > Thierry Thanks. I will move both inside /cbb. > >> + compatible = "nvidia,tegra194-xusb-padctl"; >> + reg = <0x0 0x03520000 0x0 0x1000>, >> + <0x0 0x03540000 0x0 0x1000>; >> + reg-names = "padctl", "ao"; >> + >> + resets = <&bpmp TEGRA194_RESET_XUSB_PADCTL>; >> + reset-names = "padctl"; >> + >> + status = "disabled"; >> + >> + pads { >> + usb2 { >> + clocks = <&bpmp TEGRA194_CLK_USB2_TRK>; >> + clock-names = "trk"; >> + >> + lanes { >> + usb2-0 { >> + nvidia,function = "xusb"; >> + status = "disabled"; >> + #phy-cells = <0>; >> + }; >> + usb2-1 { >> + nvidia,function = "xusb"; >> + status = "disabled"; >> + #phy-cells = <0>; >> + }; >> + usb2-2 { >> + nvidia,function = "xusb"; >> + status = "disabled"; >> + #phy-cells = <0>; >> + }; >> + usb2-3 { >> + nvidia,function = "xusb"; >> + status = "disabled"; >> + #phy-cells = <0>; >> + }; >> + }; >> + }; >> + usb3 { >> + lanes { >> + usb3-0 { >> + nvidia,function = "xusb"; >> + status = "disabled"; >> + #phy-cells = <0>; >> + }; >> + usb3-1 { >> + nvidia,function = "xusb"; >> + status = "disabled"; >> + #phy-cells = <0>; >> + }; >> + usb3-2 { >> + nvidia,function = "xusb"; >> + status = "disabled"; >> + #phy-cells = <0>; >> + }; >> + usb3-3 { >> + nvidia,function = "xusb"; >> + status = "disabled"; >> + #phy-cells = <0>; >> + }; >> + }; >> + }; >> + }; >> + >> + ports { >> + usb2-0 { >> + status = "disabled"; >> + }; >> + usb2-1 { >> + status = "disabled"; >> + }; >> + usb2-2 { >> + status = "disabled"; >> + }; >> + usb2-3 { >> + status = "disabled"; >> + }; >> + usb3-0 { >> + status = "disabled"; >> + }; >> + usb3-1 { >> + status = "disabled"; >> + }; >> + usb3-2 { >> + status = "disabled"; >> + }; >> + usb3-3 { >> + status = "disabled"; >> + }; >> + }; >> + }; >> + >> + tegra_xhci: xhci@3610000 { >> + compatible = "nvidia,tegra194-xusb"; >> + reg = <0x0 0x03610000 0x0 0x40000>, >> + <0x0 0x03600000 0x0 0x10000>; >> + reg-names = "hcd", "fpci"; >> + >> + interrupts = <GIC_SPI 163 IRQ_TYPE_LEVEL_HIGH>, >> + <GIC_SPI 164 IRQ_TYPE_LEVEL_HIGH>, >> + <GIC_SPI 167 IRQ_TYPE_LEVEL_HIGH>; >> + >> + clocks = <&bpmp TEGRA194_CLK_XUSB_CORE_MUX>, >> + <&bpmp TEGRA194_CLK_XUSB_CORE_HOST>, >> + <&bpmp TEGRA194_CLK_XUSB_CORE_SS>, >> + <&bpmp TEGRA194_CLK_XUSB_FALCON>, >> + <&bpmp TEGRA194_CLK_XUSB_FALCON_HOST>, >> + <&bpmp TEGRA194_CLK_XUSB_FALCON_SS>, >> + <&bpmp TEGRA194_CLK_XUSB_FS>, >> + <&bpmp TEGRA194_CLK_XUSB_FS_HOST>, >> + <&bpmp TEGRA194_CLK_XUSB_SS>, >> + <&bpmp TEGRA194_CLK_XUSB_SS_SUPERSPEED>, >> + <&bpmp TEGRA194_CLK_UTMIPLL>, >> + <&bpmp TEGRA194_CLK_CLK_M>, >> + <&bpmp TEGRA194_CLK_PLLE>; >> + clock-names = "xusb_hs_src", "xusb_host", >> + "xusb_core_superspeed_clk", "xusb_falcon_src", >> + "xusb_falcon_host_clk", "xusb_falcon_superspeed_clk", >> + "xusb_fs_src", "xusb_fs_host_clk", "xusb_ss_src", >> + "xusb_ss", "pll_u_480m", "clk_m", "pll_e"; >> + >> + power-domains = <&bpmp TEGRA194_POWER_DOMAIN_XUSBC>, >> + <&bpmp TEGRA194_POWER_DOMAIN_XUSBA>; >> + power-domain-names = "xusb_host", "xusb_ss"; >> + >> + nvidia,xusb-padctl = <&xusb_padctl>; >> + status = "disabled"; >> + }; >> }; >> -- >> 2.17.1 >>
diff --git a/arch/arm64/boot/dts/nvidia/tegra194.dtsi b/arch/arm64/boot/dts/nvidia/tegra194.dtsi index 3c0cf54f0aab..4d3371d3a407 100644 --- a/arch/arm64/boot/dts/nvidia/tegra194.dtsi +++ b/arch/arm64/boot/dts/nvidia/tegra194.dtsi @@ -1599,4 +1599,134 @@ interrupt-parent = <&gic>; always-on; }; + + xusb_padctl: padctl@3520000 { + compatible = "nvidia,tegra194-xusb-padctl"; + reg = <0x0 0x03520000 0x0 0x1000>, + <0x0 0x03540000 0x0 0x1000>; + reg-names = "padctl", "ao"; + + resets = <&bpmp TEGRA194_RESET_XUSB_PADCTL>; + reset-names = "padctl"; + + status = "disabled"; + + pads { + usb2 { + clocks = <&bpmp TEGRA194_CLK_USB2_TRK>; + clock-names = "trk"; + + lanes { + usb2-0 { + nvidia,function = "xusb"; + status = "disabled"; + #phy-cells = <0>; + }; + usb2-1 { + nvidia,function = "xusb"; + status = "disabled"; + #phy-cells = <0>; + }; + usb2-2 { + nvidia,function = "xusb"; + status = "disabled"; + #phy-cells = <0>; + }; + usb2-3 { + nvidia,function = "xusb"; + status = "disabled"; + #phy-cells = <0>; + }; + }; + }; + usb3 { + lanes { + usb3-0 { + nvidia,function = "xusb"; + status = "disabled"; + #phy-cells = <0>; + }; + usb3-1 { + nvidia,function = "xusb"; + status = "disabled"; + #phy-cells = <0>; + }; + usb3-2 { + nvidia,function = "xusb"; + status = "disabled"; + #phy-cells = <0>; + }; + usb3-3 { + nvidia,function = "xusb"; + status = "disabled"; + #phy-cells = <0>; + }; + }; + }; + }; + + ports { + usb2-0 { + status = "disabled"; + }; + usb2-1 { + status = "disabled"; + }; + usb2-2 { + status = "disabled"; + }; + usb2-3 { + status = "disabled"; + }; + usb3-0 { + status = "disabled"; + }; + usb3-1 { + status = "disabled"; + }; + usb3-2 { + status = "disabled"; + }; + usb3-3 { + status = "disabled"; + }; + }; + }; + + tegra_xhci: xhci@3610000 { + compatible = "nvidia,tegra194-xusb"; + reg = <0x0 0x03610000 0x0 0x40000>, + <0x0 0x03600000 0x0 0x10000>; + reg-names = "hcd", "fpci"; + + interrupts = <GIC_SPI 163 IRQ_TYPE_LEVEL_HIGH>, + <GIC_SPI 164 IRQ_TYPE_LEVEL_HIGH>, + <GIC_SPI 167 IRQ_TYPE_LEVEL_HIGH>; + + clocks = <&bpmp TEGRA194_CLK_XUSB_CORE_MUX>, + <&bpmp TEGRA194_CLK_XUSB_CORE_HOST>, + <&bpmp TEGRA194_CLK_XUSB_CORE_SS>, + <&bpmp TEGRA194_CLK_XUSB_FALCON>, + <&bpmp TEGRA194_CLK_XUSB_FALCON_HOST>, + <&bpmp TEGRA194_CLK_XUSB_FALCON_SS>, + <&bpmp TEGRA194_CLK_XUSB_FS>, + <&bpmp TEGRA194_CLK_XUSB_FS_HOST>, + <&bpmp TEGRA194_CLK_XUSB_SS>, + <&bpmp TEGRA194_CLK_XUSB_SS_SUPERSPEED>, + <&bpmp TEGRA194_CLK_UTMIPLL>, + <&bpmp TEGRA194_CLK_CLK_M>, + <&bpmp TEGRA194_CLK_PLLE>; + clock-names = "xusb_hs_src", "xusb_host", + "xusb_core_superspeed_clk", "xusb_falcon_src", + "xusb_falcon_host_clk", "xusb_falcon_superspeed_clk", + "xusb_fs_src", "xusb_fs_host_clk", "xusb_ss_src", + "xusb_ss", "pll_u_480m", "clk_m", "pll_e"; + + power-domains = <&bpmp TEGRA194_POWER_DOMAIN_XUSBC>, + <&bpmp TEGRA194_POWER_DOMAIN_XUSBA>; + power-domain-names = "xusb_host", "xusb_ss"; + + nvidia,xusb-padctl = <&xusb_padctl>; + status = "disabled"; + }; };
Adds the XUSB pad and XUSB controllers on Tegra194. Signed-off-by: JC Kuo <jckuo@nvidia.com> --- arch/arm64/boot/dts/nvidia/tegra194.dtsi | 130 +++++++++++++++++++++++ 1 file changed, 130 insertions(+)