From nobody Tue Feb 10 00:59:24 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) client-ip=66.175.222.12; envelope-from=bounce+27952+51250+1787277+3901457@groups.io; helo=web01.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+51250+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=linaro.org ARC-Seal: i=1; a=rsa-sha256; t=1574705245; cv=none; d=zohomail.com; s=zohoarc; b=Pb2u5XmxAB1toztoP2bTUlGBHz0vsKv+glw1xVn7++F8Uzg+Penv6y57jD4h+sIHGF+3V5IJ2oqTPKKfjUSKPCjX36EKDALTGOpkyWmfJBdfCb31IZY/171mHqLXdlHl2jKrVMzde9r929dKYX5vOzwIXA7UsTKuCWhd1mmoi7c= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1574705245; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Id:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:References:Sender:Subject:To; bh=Bd086EvHmkFbq4BI/cEkhGlk+KnqMn4NbbyCy9O5nUg=; b=e/IILCq0pzV4wKZfx20JjGwmT2PPYnyisluPUoQKfk0WJ6dXV6SajdjdZFOV7/3vPsThxrm/bcylsopu29TfvWKj9R1S5D6uxMRYTmN/niQoc94ouFF0ogpAPT3eNP9Jhgm0dRedSpdpSuwkU2MzaoXK/KWKN1Qim+YctfBYq2Q= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+51250+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) header.from= Received: from web01.groups.io (web01.groups.io [66.175.222.12]) by mx.zohomail.com with SMTPS id 1574705245658982.2773445688133; Mon, 25 Nov 2019 10:07:25 -0800 (PST) Return-Path: X-Received: by 127.0.0.2 with SMTP id J95QYY1788612xt41SyXBG3M; Mon, 25 Nov 2019 10:07:25 -0800 X-Received: from mail-wm1-f67.google.com (mail-wm1-f67.google.com [209.85.128.67]) by mx.groups.io with SMTP id smtpd.web12.395.1574705244490236309 for ; Mon, 25 Nov 2019 10:07:24 -0800 X-Received: by mail-wm1-f67.google.com with SMTP id u18so294658wmc.3 for ; Mon, 25 Nov 2019 10:07:24 -0800 (PST) X-Gm-Message-State: YUdHLy7bqgqgQSJzm60wC0iDx1787277AA= X-Google-Smtp-Source: APXvYqwagZ2k8v2NtnqfcHTOGHd5MmDTDc690GcSECBhhVzJGYh8lVNVFGzGN+y/qmyyffgAIomLZw== X-Received: by 2002:a1c:2846:: with SMTP id o67mr140830wmo.7.1574705242802; Mon, 25 Nov 2019 10:07:22 -0800 (PST) X-Received: from localhost.localdomain (91-167-84-221.subs.proxad.net. [91.167.84.221]) by smtp.gmail.com with ESMTPSA id b3sm50478wmj.44.2019.11.25.10.07.21 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 25 Nov 2019 10:07:22 -0800 (PST) From: "Ard Biesheuvel" To: devel@edk2.groups.io Cc: leif.lindholm@linaro.org, Ard Biesheuvel Subject: [edk2-devel] [PATCH edk2-platforms 3/3] Platform/Overdrive: clean up stream ID descriptions in DT Date: Mon, 25 Nov 2019 19:07:10 +0100 Message-Id: <20191125180710.62600-4-ard.biesheuvel@linaro.org> In-Reply-To: <20191125180710.62600-1-ard.biesheuvel@linaro.org> References: <20191125180710.62600-1-ard.biesheuvel@linaro.org> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,ard.biesheuvel@linaro.org Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1574705245; bh=pGVZQ1/MYc2oZ+rKq1Yk76mtUbEuBH+Xqw2LGVJzDkQ=; h=Cc:Date:From:Reply-To:Subject:To; b=uGF1megCH4pP63P9WvdV279hy9FH2nJqSZ4tLkr4SGgk90Iuk1+63qR57DvLO0NmCbb IWdeOo2Hz7pbWCTGZGRRcaGI6KY6p8vyLdsXPJ+8W8qWq84D7RmD/coGd/dtO+67B2Wbq RPrI9GM3dX3gBoze2GgSAkpAgHlHReRbriU= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" Align the DT description of the SMMU topology and stream IDs with the actual routing of the SoC. As with the preceding IORT change, this is mostly a cleanup exercise, but it does actually fix an issue with the CCP crypto accelerator on B1 silicon. This time, let's retain the CCP description for B0 silicon, and just remove the 'iommus' DT property in this case. Since the SMMU is shared, it is no longer an error if the SMMU node is missing when attempting to delete it, so remove the warning that we issue in that case. Signed-off-by: Ard Biesheuvel --- Platform/AMD/OverdriveBoard/DeviceTree/OverdriveBoard.dts | 19 ++++++++= +++++------ Silicon/AMD/Styx/Library/StyxDtbLoaderLib/StyxDtbLoaderLib.c | 5 ++--- 2 files changed, 15 insertions(+), 9 deletions(-) diff --git a/Platform/AMD/OverdriveBoard/DeviceTree/OverdriveBoard.dts b/Pl= atform/AMD/OverdriveBoard/DeviceTree/OverdriveBoard.dts index 1ba0d403eaf0..ba31240f2e91 100644 --- a/Platform/AMD/OverdriveBoard/DeviceTree/OverdriveBoard.dts +++ b/Platform/AMD/OverdriveBoard/DeviceTree/OverdriveBoard.dts @@ -86,7 +86,7 @@ */ <0 332 4>, <0 332 4>; - #iommu-cells =3D <2>; + #iommu-cells =3D <1>; dma-coherent; }; =20 @@ -99,7 +99,7 @@ */ <0 331 4>, <0 331 4>; - #iommu-cells =3D <2>; + #iommu-cells =3D <1>; dma-coherent; }; =20 @@ -109,7 +109,9 @@ interrupts =3D <0x0 0x163 0x4>; clocks =3D <&sata_clk>; dma-coherent; - iommus =3D <&sata0_smmu 0x00 0x1f>; /* 0-31 */ + iommus =3D <&sata0_smmu 0x0e>, + <&sata0_smmu 0x0f>, + <&sata0_smmu 0x1e>; }; =20 sata@e0d00000 { @@ -119,7 +121,9 @@ interrupts =3D <0x0 0x162 0x4>; clocks =3D <&sata_clk>; dma-coherent; - iommus =3D <&sata1_smmu 0x00 0x1f>; /* 0-31 */ + iommus =3D <&sata1_smmu 0x0e>, + <&sata1_smmu 0x0f>, + <&sata1_smmu 0x1e>; }; =20 i2c@e1000000 { @@ -233,6 +237,9 @@ interrupts =3D <0x0 0x3 0x4>; dma-coherent; amd,zlib-support =3D <0x1>; + iommus =3D <&sata1_smmu 0x00>, + <&sata1_smmu 0x02>, + <&sata1_smmu 0x40>; }; =20 pcie: pcie@f0000000 { @@ -409,7 +416,7 @@ phy-handle =3D <&xgmac0_phy>; phy-mode =3D "xgmii"; dma-coherent; - iommus =3D <&xgmac0_smmu 0x00 0x1f>; /* 0-31 */ + iommus =3D <&xgmac0_smmu 0x00 0x17>; /* 0-7, 16-23 */ }; =20 xgmac@e0900000 { @@ -428,7 +435,7 @@ phy-handle =3D <&xgmac1_phy>; phy-mode =3D "xgmii"; dma-coherent; - iommus =3D <&xgmac1_smmu 0x00 0x1f>; /* 0-31 */ + iommus =3D <&xgmac1_smmu 0x00 0x17>; /* 0-7, 16-23 */ }; }; =20 diff --git a/Silicon/AMD/Styx/Library/StyxDtbLoaderLib/StyxDtbLoaderLib.c b= /Silicon/AMD/Styx/Library/StyxDtbLoaderLib/StyxDtbLoaderLib.c index c84c1a81c3ec..5608946a5396 100644 --- a/Silicon/AMD/Styx/Library/StyxDtbLoaderLib/StyxDtbLoaderLib.c +++ b/Silicon/AMD/Styx/Library/StyxDtbLoaderLib/StyxDtbLoaderLib.c @@ -212,8 +212,6 @@ DisableSmmu ( =20 Node =3D fdt_path_offset (Fdt, SmmuNodeName); if (Node <=3D 0) { - DEBUG ((DEBUG_WARN, "%a: Failed to find path %s: %a\n", - __FUNCTION__, SmmuNodeName, fdt_strerror (Node))); return; } =20 @@ -253,8 +251,9 @@ SetSocIdStatus ( DisableSmmu (Fdt, "iommus", "/smb/smmu@e0200000", "/smb/sata@e0300000"= ); } =20 - if (!PcdGetBool (PcdEnableSmmus) || !IsRevB1 || FixedPcdGet8 (PcdSata1Po= rtCount) =3D=3D 0) { + if (!PcdGetBool (PcdEnableSmmus) || !IsRevB1) { DisableSmmu (Fdt, "iommus", "/smb/smmu@e0c00000", "/smb/sata@e0d00000"= ); + DisableSmmu (Fdt, "iommus", "/smb/smmu@e0c00000", "/smb/ccp@e0100000"); } =20 if (!FixedPcdGetBool (PcdXgbeEnable) || !PcdGetBool (PcdEnableSmmus)) { --=20 2.20.1 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#51250): https://edk2.groups.io/g/devel/message/51250 Mute This Topic: https://groups.io/mt/61945305/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-