Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.34 as permitted sender) client-ip=23.128.96.34;
From:   Laurentiu Tudor <laurentiu.tudor@nxp.com>
To:     shawnguo@kernel.org, leoyang.li@nxp.com, robh+dt@kernel.org,
        krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org,
        linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org
Cc:     Laurentiu Tudor <laurentiu.tudor@nxp.com>
Subject: [PATCH] arm64: dts: ls208xa: use a pseudo-bus to constrain usb dma size
Date:   Thu, 14 Sep 2023 17:56:52 +0300
Message-Id: <20230914145652.25728-1-laurentiu.tudor@nxp.com>
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?JpCZHgcz9YKGlImyBOhkl/nhXIwIYGYWrDTm670BdrfbErKK8DEkdv2ffgsm?=
 =?us-ascii?Q?rCW5ZEJvdKBBa8/Eq5DdyS6EXMS9t6oB13+UNVQr3ghUuY6S1E8P7X/8fWjb?=
 =?us-ascii?Q?1K8Hvdg4fV+6EXqtEHtW88wnPpB6IRxPziQZVXzy1EEni2nIrCjgHkd5hwK9?=
 =?us-ascii?Q?6+BN7W396gkfCmMhi9lIDp4jE+ZghctgvySg/LIwlDua9CUo2lpq30GWbrKx?=
 =?us-ascii?Q?RAuXUugRYxSZvieZXfnJ7huTJTZ8RxlAgktvKmlOmxXh3tizhrydYkK7PUAT?=
 =?us-ascii?Q?dNmlt4nTQtfF7w97vDARW6mPzTOwHbqFju9qpXF7Hn73/AeaHe8IzIoy1VE/?=
 =?us-ascii?Q?VRrq8KRgdRdYmc72nxH2c6FsSzjLSPKhVqY8GP0WZmhQ+hTJUZo3tmOro8jC?=
 =?us-ascii?Q?WGZ0e3rkGnq/qbkXP5cjhocpzw8Mu/S35Id8LzF8fCCsrfivDA+dB8EOMVRo?=
 =?us-ascii?Q?TbW5xaUhqKCJY9S4omt3biglTYt7hqE/Dfy1dw/kYnQemWZFv6eGM0enDXc1?=
 =?us-ascii?Q?hfOOAUhWvpeHVnyHspO9NGQ52+omRiNbb6YQoNjGbjEBINhLLfdVvaG4OVQn?=
 =?us-ascii?Q?PBvQxxxMh+1mtgO/j7vl0UrvH1jQbC2Ug6xFaESutuTszfSx5nCUkE3PvsYK?=
 =?us-ascii?Q?r7NYCcjGSTA7TSNIC8KrH7VgK2NJKJ3qD7kWyWTbr1gpxoBJHvetamPlh80a?=
 =?us-ascii?Q?AzFI3Nfom1HMUDuVbTxC6wK2suxVWiwr8zX2LeCIwhdhx/oG8vg1o9Au9mhv?=
 =?us-ascii?Q?/QG0v3Fm8EF/D5P93rkWBK8k3C598+fjiJycgYdURwAF5oX8Fw0+Eocm7UEZ?=
 =?us-ascii?Q?f5QmuiwXbht0O9f9vQTnb4n3A1hnY2YdGX1aGXb6V2nzG1B7ZUbhvolpDsk8?=
 =?us-ascii?Q?nosgmSuJPA4wnXFoZodspMr04BSjXXxtiolyybzEQ/QqQ1zgEFLCHYiNubzm?=
 =?us-ascii?Q?g5QACUpUGnJAF+YNZmV2yQPdZVNSUpdaqQlkNS4P16RYBks7dCygoX8ge64d?=
 =?us-ascii?Q?jN0qEk5S+VGx4crrdIMQ9dMea1dtVSV7mduKyi4YNYtcDzZOA/VOz09RZcjJ?=
 =?us-ascii?Q?t0SXE5zp8phixFvcKcGHcoX96I0gV1FDu4BQl3pHSHYFTrbsYqEpEfTK+sMY?=
 =?us-ascii?Q?sx0I19fSxTpSHvWEpCzORCTcUe7eWH/mS0HFUe9jr/y3sbGT2GEokM7WsxMy?=
 =?us-ascii?Q?5XmdnSDUE7Chh+2B0hSh5wY0hym60ygXuqfCHi1OHgUTfZ3bsaZjfz4AjGPy?=
 =?us-ascii?Q?4ijCCxumbhJQESGjdpg6+u11T9jDNppAsvsJbRFKiihUIzjVq+is8wHznjlR?=
 =?us-ascii?Q?SFLsra3aFfe5+LTxqKa+IQ4QmpN6PN0MF55gx99u4BqCJ1VPezfuMw0/7uNF?=
 =?us-ascii?Q?VpdKWJ7pXetn9KcUUFfMMBGogkY4NBYCbFZ4WrSiB3YCAf3/+V+dmio/16CV?=
 =?us-ascii?Q?DwO3Ppr0u2FBwCZUKDwKa5dLbFV0vab66rsrrXRBtbLemTO5/NP5xgKFWeKW?=
 =?us-ascii?Q?KSy5IHS12MOTAEjdG8TW6uuWDYFyUHvJcqirzOqSWPHx/nQPekkH7YPXIcm3?=
 =?us-ascii?Q?Zy7yykkwvmDKpMUMhraGKcAqfxWd5nO8s4wYGrx/?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 418cdd3f-9edd-4533-7771-08dbb532dbcf
X-MS-Exchange-CrossTenant-AuthSource: DB9PR04MB9380.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Sep 2023 14:57:04.5516
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 1epRoF4AC8HkqkRcZr5/nexm2s7eYbdF6gVIhhPer6frVbjbfwhpWuyPfaNQjWQ57LXsx5DEi5B+4c7Hxick5w==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXPR04MB8142
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (howler.vger.email [0.0.0.0]); Thu, 14 Sep 2023 07:57:10 -0700 (PDT)

Wrap the usb controllers in an intermediate simple-bus and use it to
constrain the dma address size of these usb controllers to the 40b
that they generate toward the interconnect. This is required because
the SoC uses 48b address sizes and this mismatch would lead to smmu
context faults [1] because the usb generates 40b addresses while the
smmu page tables are populated with 48b wide addresses.

[1]
xhci-hcd xhci-hcd.0.auto: xHCI Host Controller
xhci-hcd xhci-hcd.0.auto: new USB bus registered, assigned bus number 1
xhci-hcd xhci-hcd.0.auto: hcc params 0x0220f66d hci version 0x100 quirks 0x0000000002000010
xhci-hcd xhci-hcd.0.auto: irq 108, io mem 0x03100000
xhci-hcd xhci-hcd.0.auto: xHCI Host Controller
xhci-hcd xhci-hcd.0.auto: new USB bus registered, assigned bus number 2
xhci-hcd xhci-hcd.0.auto: Host supports USB 3.0 SuperSpeed
arm-smmu 5000000.iommu: Unhandled context fault: fsr=0x402, iova=0xffffffb000, fsynr=0x0, cbfrsynra=0xc01, cb=3

Signed-off-by: Laurentiu Tudor <laurentiu.tudor@nxp.com>
---
 .../arm64/boot/dts/freescale/fsl-ls208xa.dtsi | 46 +++++++++++--------
 1 file changed, 27 insertions(+), 19 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
index d2f5345d0560..47cc7699ca16 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
@@ -1186,26 +1186,34 @@
 			dma-coherent;
 		};
 
-		usb0: usb@3100000 {
-			status = "disabled";
-			compatible = "snps,dwc3";
-			reg = <0x0 0x3100000 0x0 0x10000>;
-			interrupts = <0 80 0x4>; /* Level high type */
-			dr_mode = "host";
-			snps,quirk-frame-length-adjustment = <0x20>;
-			snps,dis_rxdet_inp3_quirk;
-			snps,incr-burst-type-adjustment = <1>, <4>, <8>, <16>;
-		};
+		aux_bus: aux_bus {
+			#address-cells = <2>;
+			#size-cells = <2>;
+			compatible = "simple-bus";
+			ranges;
+			dma-ranges = <0x0 0x0 0x0 0x0 0x100 0x00000000>;
+
+			usb0: usb@3100000 {
+				status = "disabled";
+				compatible = "snps,dwc3";
+				reg = <0x0 0x3100000 0x0 0x10000>;
+				interrupts = <0 80 0x4>; /* Level high type */
+				dr_mode = "host";
+				snps,quirk-frame-length-adjustment = <0x20>;
+				snps,dis_rxdet_inp3_quirk;
+				snps,incr-burst-type-adjustment = <1>, <4>, <8>, <16>;
+			};
 
-		usb1: usb@3110000 {
-			status = "disabled";
-			compatible = "snps,dwc3";
-			reg = <0x0 0x3110000 0x0 0x10000>;
-			interrupts = <0 81 0x4>; /* Level high type */
-			dr_mode = "host";
-			snps,quirk-frame-length-adjustment = <0x20>;
-			snps,dis_rxdet_inp3_quirk;
-			snps,incr-burst-type-adjustment = <1>, <4>, <8>, <16>;
+			usb1: usb@3110000 {
+				status = "disabled";
+				compatible = "snps,dwc3";
+				reg = <0x0 0x3110000 0x0 0x10000>;
+				interrupts = <0 81 0x4>; /* Level high type */
+				dr_mode = "host";
+				snps,quirk-frame-length-adjustment = <0x20>;
+				snps,dis_rxdet_inp3_quirk;
+				snps,incr-burst-type-adjustment = <1>, <4>, <8>, <16>;
+			};
 		};
 
 		ccn@4000000 {
-- 
2.17.1