From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=40.107.5.62; helo=eur03-ve1-obe.outbound.protection.outlook.com; envelope-from=vabhav.sharma@nxp.com; receiver=edk2-devel@lists.01.org Received: from EUR03-VE1-obe.outbound.protection.outlook.com (mail-eopbgr50062.outbound.protection.outlook.com [40.107.5.62]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id BD30721E08183 for ; Tue, 24 Apr 2018 05:48:13 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=Y8jf9sTbNga49/S5UfENEH3tfusMCOvVH/NveGV05c8=; b=oJOWpVDZeN7wxxpqLKXhOcZ6Y7QCXZTAhELc2FcirZeIa3zmNO20BcdFTpgHU+EdP9BNUKYG429c+zqRpEW16XnCjxtQiT+yliC0fkL5zo4P3AddH2JG94Qbeq78lQsD3JvMj5qMFoBf/yZESjoow47Fa87WXJ5M4MD2gADRiE8= Received: from DB4PR04MB299.eurprd04.prod.outlook.com (10.141.239.21) by DB4PR04MB508.eurprd04.prod.outlook.com (10.141.239.156) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.696.15; Tue, 24 Apr 2018 12:48:10 +0000 Received: from DB4PR04MB299.eurprd04.prod.outlook.com ([fe80::2dcd:1bc7:8d4e:5dc2]) by DB4PR04MB299.eurprd04.prod.outlook.com ([fe80::2dcd:1bc7:8d4e:5dc2%15]) with mapi id 15.20.0696.017; Tue, 24 Apr 2018 12:48:10 +0000 From: Vabhav Sharma To: Leif Lindholm , Meenakshi Aggarwal CC: "ard.biesheuvel@linaro.org" , "edk2-devel@lists.01.org" , Udit Kumar , Varun Sethi Thread-Topic: [PATCH edk2-platforms 38/39] Platform/NXP:PCIe enablement for LS1046A RDB Thread-Index: AQHTpwPtCEE4Vi4pdU2CxA86CF3lTKQKK8MAgAYbDlA= Date: Tue, 24 Apr 2018 12:48:10 +0000 Message-ID: References: <1518771035-6733-1-git-send-email-meenakshi.aggarwal@nxp.com> <1518771035-6733-39-git-send-email-meenakshi.aggarwal@nxp.com> <20180420153314.a2bpppacu5bjxdya@bivouac.eciton.net> In-Reply-To: <20180420153314.a2bpppacu5bjxdya@bivouac.eciton.net> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: spf=none (sender IP is ) smtp.mailfrom=vabhav.sharma@nxp.com; x-originating-ip: [14.142.187.166] x-ms-publictraffictype: Email x-microsoft-exchange-diagnostics: 1; DB4PR04MB508; 7:97am5kjhj08YWnjx3yhYIiL/HDpMd9zFHQTqlXl+2TOJ7cXwN2n0jBnQnOUgLMS/JWkeKE0ahzSiO9NVdgaRWBdNPJ1hd15mXwd1aa30IDGamLjQrN42r9f/+xjVHqgiqVgT5Ps1K5HE5QRIFztMmQxEKY8lz2PQdX7X2jZDZlSZcjgGhxzy08Iwr6OC3n6LDwZRQn+Ula2nLz3YdX+AysyPw6jef+1YKoX78iSQSvoVTQ4jX+J2/opcs0Tjh6CG x-ms-exchange-antispam-srfa-diagnostics: SOS; x-ms-office365-filtering-ht: Tenant x-microsoft-antispam: UriScan:; BCL:0; PCL:0; RULEID:(7020095)(4652020)(5600026)(4534165)(7168020)(4627221)(201703031133081)(201702281549075)(48565401081)(2017052603328)(7153060)(7193020); SRVR:DB4PR04MB508; x-ms-traffictypediagnostic: DB4PR04MB508: x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:(185117386973197)(162533806227266); x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(5005006)(8121501046)(3002001)(3231232)(944501410)(52105095)(93006095)(93001095)(10201501046)(6055026)(6041310)(20161123564045)(20161123562045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123558120)(20161123560045)(6072148)(201708071742011); SRVR:DB4PR04MB508; BCL:0; PCL:0; RULEID:; SRVR:DB4PR04MB508; x-forefront-prvs: 0652EA5565 x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(39860400002)(39380400002)(346002)(366004)(376002)(396003)(13464003)(199004)(189003)(476003)(446003)(11346002)(3660700001)(486006)(478600001)(4326008)(6246003)(25786009)(74316002)(2900100001)(14454004)(7736002)(6436002)(316002)(5250100002)(8676002)(54906003)(81166006)(81156014)(86362001)(110136005)(2906002)(3846002)(6116002)(55016002)(53936002)(9686003)(44832011)(26005)(59450400001)(186003)(5660300001)(76176011)(229853002)(99286004)(106356001)(7696005)(33656002)(8936002)(105586002)(102836004)(68736007)(66066001)(6636002)(6506007)(55236004)(305945005)(97736004)(3280700002)(19627235001); DIR:OUT; SFP:1101; SCL:1; SRVR:DB4PR04MB508; H:DB4PR04MB299.eurprd04.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1; received-spf: None (protection.outlook.com: nxp.com does not designate permitted sender hosts) x-microsoft-antispam-message-info: kpmllxQH7Pc9eV3SxXCiNba4JjtaIDVplLOPBlcmk2yYEBns05c2kY1NGy/s1b8HVd7BTEeiiEVcQPQkWir6JlciCaUTa8PL5JOVEA49VIh9v99X0g8R+6JpGtKKFneUEnjJU8TyhmlQx2Wp9Xfq48C3DBv4zQoMbsVV5/wOx7T51eAx3v8/WScuixvxQs3V spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM MIME-Version: 1.0 X-MS-Office365-Filtering-Correlation-Id: 73c7702b-6994-4f2c-a396-08d5a9e1a2d1 X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 73c7702b-6994-4f2c-a396-08d5a9e1a2d1 X-MS-Exchange-CrossTenant-originalarrivaltime: 24 Apr 2018 12:48:10.5926 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB4PR04MB508 Subject: Re: [PATCH edk2-platforms 38/39] Platform/NXP:PCIe enablement for LS1046A RDB X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 24 Apr 2018 12:48:14 -0000 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable >-----Original Message----- >From: Leif Lindholm [mailto:leif.lindholm@linaro.org] >Sent: Friday, April 20, 2018 9:03 PM >To: Meenakshi Aggarwal >Cc: ard.biesheuvel@linaro.org; edk2-devel@lists.01.org; Udit Kumar >; Varun Sethi ; Vabhav Sharma > >Subject: Re: [PATCH edk2-platforms 38/39] Platform/NXP:PCIe enablement for >LS1046A RDB > >On Fri, Feb 16, 2018 at 02:20:34PM +0530, Meenakshi wrote: >> From: Vabhav >> >> Compilation: Update the fdf, dsc and dec files. >> >> Contributed-under: TianoCore Contribution Agreement 1.1 >> Signed-off-by: Vabhav >> --- >> Platform/NXP/LS1046aRdbPkg/LS1046aRdbPkg.dsc | 15 >+++++++++++++++ >> Platform/NXP/LS1046aRdbPkg/LS1046aRdbPkg.fdf | 9 ++++++++= + >> .../LS1046aRdbPkg/Library/PlatformLib/ArmPlatformLib.inf | 2 ++ >> .../NXP/LS1046aRdbPkg/Library/PlatformLib/NxpQoriqLsMem.c | 6 ++++++ >> Silicon/NXP/LS1046A/LS1046A.dsc | 3 +++ >> 5 files changed, 35 insertions(+) >> >> diff --git a/Platform/NXP/LS1046aRdbPkg/LS1046aRdbPkg.dsc >> b/Platform/NXP/LS1046aRdbPkg/LS1046aRdbPkg.dsc >> index 36002d5..231207d 100644 >> --- a/Platform/NXP/LS1046aRdbPkg/LS1046aRdbPkg.dsc >> +++ b/Platform/NXP/LS1046aRdbPkg/LS1046aRdbPkg.dsc >> @@ -41,6 +41,8 @@ >> IfcLib|Silicon/NXP/Library/IfcLib/IfcLib.inf >> BoardLib|Platform/NXP/LS1046aRdbPkg/Library/BoardLib/BoardLib.inf >> FpgaLib|Platform/NXP/LS1046aRdbPkg/Library/FpgaLib/FpgaLib.inf >> + PciSegmentLib|Silicon/NXP/Library/PciSegmentLib/PciSegmentLib.inf >> + >> + PciHostBridgeLib|Silicon/NXP/Library/PciHostBridgeLib/PciHostBridgeL >> + ib.inf >> >> [PcdsFixedAtBuild.common] >> >> @@ -65,6 +67,7 @@ >> gNxpQoriqLsTokenSpaceGuid.PcdGurBigEndian|TRUE >> gNxpQoriqLsTokenSpaceGuid.PcdWdogBigEndian|TRUE >> gNxpQoriqLsTokenSpaceGuid.PcdIfcBigEndian|TRUE >> + gNxpQoriqLsTokenSpaceGuid.PcdPciLutBigEndian|TRUE >> >> # >> # I2C controller Pcds >> @@ -77,6 +80,12 @@ >> gNxpQoriqLsTokenSpaceGuid.PcdI2cSlaveAddress|0x51 >> gNxpQoriqLsTokenSpaceGuid.PcdI2cSpeed|100000 >> >> + # >> + # PCI PCDs. >> + # >> + gNxpQoriqLsTokenSpaceGuid.PcdPciDebug|FALSE >> + gNxpQoriqLsTokenSpaceGuid.PcdPcieLutBase|0x80000 >> + gNxpQoriqLsTokenSpaceGuid.PcdPcieLutDbg|0x407FC >> >> >################################################################## >#### >> ########## >> # >> # Components Section - list of all EDK II Modules needed by this >> Platform @@ -90,5 +99,11 @@ >> >> Silicon/NXP/Drivers/WatchDog/WatchDogDxe.inf >> Silicon/NXP/Drivers/I2cDxe/I2cDxe.inf >> + Silicon/NXP/Drivers/PciCpuIo2Dxe/PciCpuIo2Dxe.inf >> + MdeModulePkg/Bus/Pci/PciHostBridgeDxe/PciHostBridgeDxe.inf { >> + >> + gEfiMdePkgTokenSpaceGuid.PcdDebugPrintErrorLevel|0x8010004F >> + } >> + MdeModulePkg/Bus/Pci/PciBusDxe/PciBusDxe.inf >> >> ## >> diff --git a/Platform/NXP/LS1046aRdbPkg/LS1046aRdbPkg.fdf >> b/Platform/NXP/LS1046aRdbPkg/LS1046aRdbPkg.fdf >> index 834e3a4..3351a06 100644 >> --- a/Platform/NXP/LS1046aRdbPkg/LS1046aRdbPkg.fdf >> +++ b/Platform/NXP/LS1046aRdbPkg/LS1046aRdbPkg.fdf >> @@ -123,6 +123,13 @@ READ_LOCK_STATUS =3D TRUE >> INF EmbeddedPkg/SimpleTextInOutSerial/SimpleTextInOutSerial.inf >> >> # >> + # PCI >> + # >> + INF Silicon/NXP/Drivers/PciCpuIo2Dxe/PciCpuIo2Dxe.inf >> + INF MdeModulePkg/Bus/Pci/PciHostBridgeDxe/PciHostBridgeDxe.inf >> + INF MdeModulePkg/Bus/Pci/PciBusDxe/PciBusDxe.inf >> + >> + # >> # Network modules >> # >> INF MdeModulePkg/Universal/Network/SnpDxe/SnpDxe.inf >> @@ -147,6 +154,8 @@ READ_LOCK_STATUS =3D TRUE >> INF MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Dxe.inf >> !endif >> >> + INF >> + >ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand.inf >> + > >Same comment as for previous platform(s): please conditionalise and mentio= n in >commit message. > >/ > Leif Ok, I will update it. > >> # >> # FAT filesystem + GPT/MBR partitioning >> # >> diff --git >> a/Platform/NXP/LS1046aRdbPkg/Library/PlatformLib/ArmPlatformLib.inf >> b/Platform/NXP/LS1046aRdbPkg/Library/PlatformLib/ArmPlatformLib.inf >> index 49b57fc..5e09757 100644 >> --- >> a/Platform/NXP/LS1046aRdbPkg/Library/PlatformLib/ArmPlatformLib.inf >> +++ b/Platform/NXP/LS1046aRdbPkg/Library/PlatformLib/ArmPlatformLib.in >> +++ f >> @@ -42,6 +42,8 @@ >> gArmTokenSpaceGuid.PcdArmPrimaryCore >> gNxpQoriqLsTokenSpaceGuid.PcdCcsrBaseAddr >> gNxpQoriqLsTokenSpaceGuid.PcdCcsrSize >> + gNxpQoriqLsTokenSpaceGuid.PcdRomBaseAddr >> + gNxpQoriqLsTokenSpaceGuid.PcdRomSize >> gNxpQoriqLsTokenSpaceGuid.PcdIfcRegion1BaseAddr >> gNxpQoriqLsTokenSpaceGuid.PcdIfcRegion1Size >> gNxpQoriqLsTokenSpaceGuid.PcdIfcRegion2BaseAddr >> diff --git >> a/Platform/NXP/LS1046aRdbPkg/Library/PlatformLib/NxpQoriqLsMem.c >> b/Platform/NXP/LS1046aRdbPkg/Library/PlatformLib/NxpQoriqLsMem.c >> index 64c5612..1ef3292 100644 >> --- a/Platform/NXP/LS1046aRdbPkg/Library/PlatformLib/NxpQoriqLsMem.c >> +++ b/Platform/NXP/LS1046aRdbPkg/Library/PlatformLib/NxpQoriqLsMem.c >> @@ -67,6 +67,12 @@ ArmPlatformGetVirtualMemoryMap ( >> VirtualMemoryTable[Index].Length =3D FixedPcdGet64 (PcdCcsrSize= ); >> VirtualMemoryTable[Index].Attributes =3D >ARM_MEMORY_REGION_ATTRIBUTE_DEVICE; >> >> + // ROM Space >> + VirtualMemoryTable[++Index].PhysicalBase =3D FixedPcdGet64 >> + (PcdRomBaseAddr); VirtualMemoryTable[Index].VirtualBase =3D >FixedPcdGet64 (PcdRomBaseAddr); >> + VirtualMemoryTable[Index].Length =3D FixedPcdGet64 (PcdRomSize)= ; >> + VirtualMemoryTable[Index].Attributes =3D >ARM_MEMORY_REGION_ATTRIBUTE_DEVICE; >> + >> // IFC region 1 >> // >> // A-009241 : Unaligned write transactions to IFC may result in cor= ruption of >data >> diff --git a/Silicon/NXP/LS1046A/LS1046A.dsc >> b/Silicon/NXP/LS1046A/LS1046A.dsc index 9f87028..59a6150 100644 >> --- a/Silicon/NXP/LS1046A/LS1046A.dsc >> +++ b/Silicon/NXP/LS1046A/LS1046A.dsc >> @@ -64,5 +64,8 @@ >> gNxpQoriqLsTokenSpaceGuid.PcdQspiRegionBaseAddr|0x40000000 >> gNxpQoriqLsTokenSpaceGuid.PcdQspiRegionSize|0x20000000 >> gNxpQoriqLsTokenSpaceGuid.PcdIfcBaseAddr|0x1530000 >> + gNxpQoriqLsTokenSpaceGuid.PcdNumPciController|3 >> + gNxpQoriqLsTokenSpaceGuid.PcdRomBaseAddr|0x00000000 >> + gNxpQoriqLsTokenSpaceGuid.PcdRomSize|0x00100000 >> >> ## >> -- >> 1.9.1 >>