Re: [PATCH 1/2] arm64: dts: imx8qm-ss-conn: add usbotg3 iommu information

[Peng Fan <peng.fan@oss.nxp.com>]

On Fri, Oct 30, 2026 at 12:09:35PM -0500, Frank Li wrote:
>On Thu, Oct 29, 2026 at 07:55:41AM +0100, Emanuele Ghidoli wrote:
>>
>>
>> On 10/28/26 22:05, Frank Li wrote:
>> > Does it only impact superspeed? High speed work?
>>
>> No, it's not even enumerated, on the Dev board.
>>
>> >
>> > Thank you report this problem. Suppose below patch should fix this problem.
>> >
>> > I have not time to test it today.
>> >
>> >
>> > Frank
>>
>> Tested, it doesn't help.
>>
>> xhci-hcd.1.auto has no fwnode, so platform_dma_configure() never calls
>> of_dma_configure() and the new code in of_iommu_configure() is never
>> reached.
>
>Finially, it is hack at uboot.
>
>592: static int config_smmu_fdt_device_sid(void *blob, int device_offset, int sid)
>601:     prop = fdt_getprop(blob, device_offset, "fsl,sc_rsrc_id", &proplen);
>618:     ret = fdtdec_parse_phandle_with_args(blob, device_offset,
>620:                                          "power-domains", ...
>578:     err = sc_rm_set_master_sid(-1, rsrc, sid);
>681:     prop = fdt_getprop(blob, offset, "iommus", &proplen);
>848:     if (is_imx8qm()) { ret = config_smmu_fdt(blob);
>
>
>Need combine power-domains with iommus.

StreamID is not default programmed, so we did that in U-Boot.
If not do that in U-Boot, we may need to find a place to configure streamID
in kernel.

Regards
Peng

>
>diff --git a/arch/arm64/boot/dts/freescale/imx8-ss-conn.dtsi b/arch/arm64/boot/dts/freescale/imx8-ss-conn.dtsi
>index 176e2e332f87c..cc2b9a212780f 100644
>--- a/arch/arm64/boot/dts/freescale/imx8-ss-conn.dtsi
>+++ b/arch/arm64/boot/dts/freescale/imx8-ss-conn.dtsi
>@@ -189,6 +189,7 @@ usbotg3_cdns3: usb@5b120000 {
>                        phys = <&usb3_phy>;
>                        phy-names = "cdns3,usb3-phy";
>                        cdns,on-chip-buff-size = /bits/ 16 <18>;
>+                       power-domains = <&pd IMX_SC_R_USB_2>;
>                        status = "disabled";
>                };
>        };
>
>Frank
>
>>
>> Emanuele