Re: [PATCH v6 1/3] arm64: dts: imx95: add USB2.0 nodes

Xu Yang
2025-04-23T02:55:06Z

Hi Shawn,

On Thu, Apr 10, 2025 at 02:49:05PM +0800, Xu Yang wrote:
> Add USB2.0 controller and phy nodes.
>
> Tested-by: Alexander Stein <alexander.stein@ew.tq-group.com> # TQMa95xxSA
> Reviewed-by: Frank Li <Frank.Li@nxp.com>
> Signed-off-by: Xu Yang <xu.yang_2@nxp.com>

Could you pick up these dts patches?

It was originally going to be sent together with the dt-binding patches [1].
With Greg's request, I split dt-binding and dts patch after v5. So Rob's
bot report DTB warnings due to his tree lack dt-binding patch. As now Greg has
already picked up patchset [1], you can ignore DTB warnings.

[1] https://lore.kernel.org/imx/20250318150908.1583652-1-xu.yang_2@nxp.com/

Thanks,
Xu Yang

>
> ---
> Changes in v6:
> - no changes
> Changes in v5:
> - no changes
> Changes in v4:
> - add Tb tag
> Changes in v3:
> - no changes
> Changes in v2:
> - add Rb tag
> ---
> arch/arm64/boot/dts/freescale/imx95.dtsi | 30 ++++++++++++++++++++++++
> 1 file changed, 30 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx95.dtsi b/arch/arm64/boot/dts/freescale/imx95.dtsi
> index 9bb26b466a06..8dd859d8d319 100644
> --- a/arch/arm64/boot/dts/freescale/imx95.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx95.dtsi
> @@ -470,6 +470,13 @@ its: msi-controller@48040000 {
> };
> };
>
> + usbphynop: usbphynop {
> + compatible = "usb-nop-xceiv";
> + clocks = <&scmi_clk IMX95_CLK_HSIO>;
> + clock-names = "main_clk";
> + #phy-cells = <0>;
> + };
> +
> soc {
> compatible = "simple-bus";
> #address-cells = <2>;
> @@ -1621,6 +1628,29 @@ usb3_phy: phy@4c1f0040 {
> status = "disabled";
> };
>
> + usb2: usb@4c200000 {
> + compatible = "fsl,imx95-usb", "fsl,imx7d-usb", "fsl,imx27-usb";
> + reg = <0x0 0x4c200000 0x0 0x200>;
> + interrupts = <GIC_SPI 176 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 174 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&scmi_clk IMX95_CLK_HSIO>,
> + <&scmi_clk IMX95_CLK_32K>;
> + clock-names = "usb_ctrl_root", "usb_wakeup";
> + iommus = <&smmu 0xf>;
> + phys = <&usbphynop>;
> + power-domains = <&scmi_devpd IMX95_PD_HSIO_TOP>;
> + fsl,usbmisc = <&usbmisc 0>;
> + status = "disabled";
> + };
> +
> + usbmisc: usbmisc@4c200200 {
> + compatible = "fsl,imx95-usbmisc", "fsl,imx7d-usbmisc",
> + "fsl,imx6q-usbmisc";
> + reg = <0x0 0x4c200200 0x0 0x200>,
> + <0x0 0x4c010014 0x0 0x04>;
> + #index-cells = <1>;
> + };
> +
> pcie0: pcie@4c300000 {
> compatible = "fsl,imx95-pcie";
> reg = <0 0x4c300000 0 0x10000>,
> --
> 2.34.1
>