From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 17A03CA0EFF for ; Wed, 27 Aug 2025 17:57:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=zl1ExOZJWFHCeLASWnSNUPBLGrLV75ri3vrfAPY51uw=; b=wks3Kp8UqvClJJ sAY7qJeUo/1/eNnATOQsq8DU8U/+MX3rb3OKXzWf0GxqtpjpjUsANqp/a6Dr7vWuEtqQlbmBGym6v vcrDAH7U433JdTVN0Vp9Q81uOECRkPHsl2XM+StGO7ADYUEFNxrDTAgxS6eMHyQ/Z3jkG1i1SWfq2 U2NvoOrqkuEJTGz/aYd9BYUwRMwMx7sBeSXoox1ysIrPde8KRlJfcefe3GbZfShIX4fXDavH6EF6n EvYurLHBY9YLDYZt134IO6qdJbYOxpM+4tNM+SoLY3I1Jz/nVliG9Rqg0JCoso2DovZtiq289KFch VVdSrbIZLuwDWK0ggMnA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1urKOt-0000000GOC0-3SRv; Wed, 27 Aug 2025 17:57:35 +0000 Received: from sea.source.kernel.org ([172.234.252.31]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1urIg3-0000000G84y-0NpP; Wed, 27 Aug 2025 16:07:12 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sea.source.kernel.org (Postfix) with ESMTP id 6AE8C43D3A; Wed, 27 Aug 2025 16:07:10 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 35D71C4CEEB; Wed, 27 Aug 2025 16:07:05 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1756310830; bh=8/nHBBY+4VBhT+YC6UcqQvlwOolIbFpQ9L4LggP+E0w=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=X4QVOu/5Y1nL6frM1iU4gWhATYdfU0A2DDYCR/HCU+tNO4/rv0GSkcWe1bav1YKZS qQ9atRMkj6ljLH8X6u7yydleAg+yG4d+QuB4R9PB+FsmKZHxZWYZT5clhh/lDkLG5O 6ypIT8ddQeiXDVgm1tRCXvHlGmz09/9B97PucyrKh+CY95TSCS3PyV+bs1LCKbtegx xCK8S/DKDtQ+GQvVGCdsQAmlNIqDcAHa9h3iYaW+VFn+19EiDRkU4WreLRRBGX3K78 5OQgM0pEeYdx+hlyoKl4p9avrBArAhHYmoJqIXBj61VIx1Uzmy2MLjog6VVU9OZnmF I2IwA8MCBqDvQ== Message-ID: Date: Wed, 27 Aug 2025 18:07:04 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH RFC 01/22] dt-bindings: usb: snps,dwc3: Allow multiple iommus To: Krzysztof Kozlowski Cc: Greg Kroah-Hartman , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Felipe Balbi , Janne Grunau , Alyssa Rosenzweig , Neal Gompa , Vinod Koul , Kishon Vijay Abraham I , Thinh Nguyen , Heikki Krogerus , Philipp Zabel , linux-usb@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, asahi@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-phy@lists.infradead.org References: <20250821-atcphy-6-17-v1-0-172beda182b8@kernel.org> <20250821-atcphy-6-17-v1-1-172beda182b8@kernel.org> <20250822-skinny-clay-harrier-64dc58@kuoka> Content-Language: en-US From: Sven Peter In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250827_090711_156698_5BFD2139 X-CRM114-Status: GOOD ( 15.69 ) X-BeenThere: linux-phy@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux Phy Mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-phy" Errors-To: linux-phy-bounces+linux-phy=archiver.kernel.org@lists.infradead.org On 24.08.25 10:31, Krzysztof Kozlowski wrote: > On 22/08/2025 09:22, Krzysztof Kozlowski wrote: >> On Thu, Aug 21, 2025 at 03:38:53PM +0000, Sven Peter wrote: >>> Apple's dwc3 variant requires two iommus. >>> >>> Signed-off-by: Sven Peter >>> --- >>> Documentation/devicetree/bindings/usb/snps,dwc3.yaml | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/Documentation/devicetree/bindings/usb/snps,dwc3.yaml b/Documentation/devicetree/bindings/usb/snps,dwc3.yaml >>> index 4380bb6fa2f08a475b557e80467abc2861f622e2..6d35dcc605c01977f4fc6fdb6f12976f1cef2b9e 100644 >>> --- a/Documentation/devicetree/bindings/usb/snps,dwc3.yaml >>> +++ b/Documentation/devicetree/bindings/usb/snps,dwc3.yaml >>> @@ -60,7 +60,7 @@ properties: >>> dma-coherent: true >>> >>> iommus: >>> - maxItems: 1 >>> + maxItems: 2 >> >> Never tested. You anyway need specific constraints. > I realized that's pretty vague, so clarifying: you changed all bindings > to have two iommus and that's unexpected. All other devices have only > one IOMMU mapping here, so they need to stay like that. If these IOMMUs > differ, you should also list them, but probably that's not the case here. Makes sense, thanks for the detailed explanation. I don't quite understand why this controller needs two iommus but they must essentially be programmed to the same mapping for xhci to work. All of device mode goes through one of them and xhci buffers are split between the two iommus (iirc I saw the command ring using the first one and the event ring the second one and then just decided to program them the same since anything else would result in a big mess). Thanks, Sven -- linux-phy mailing list linux-phy@lists.infradead.org https://lists.infradead.org/mailman/listinfo/linux-phy