From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from EUR01-VE1-obe.outbound.protection.outlook.com (EUR01-VE1-obe.outbound.protection.outlook.com [40.107.14.55]) by mx.groups.io with SMTP id smtpd.web11.8317.1574097583001473852 for ; Mon, 18 Nov 2019 09:19:44 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@armh.onmicrosoft.com header.s=selector2-armh-onmicrosoft-com header.b=Hqz2spbV; spf=pass (domain: arm.com, ip: 40.107.14.55, mailfrom: samer.el-haj-mahmoud@arm.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=armh.onmicrosoft.com; s=selector2-armh-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=vyQSDXYpgtSwxi+Jn4fVUjEUVNQbfdVZ/rdbqLP8rBA=; b=Hqz2spbVRukkfaCJGmV9dCm+ePBCgsKJhzfEA3OXEZqfeNheIXrVs8Zyj4S+in21FFon55yOplvSk9pGhgM76RfoZunfKvESVcGXcWyayH4YqGT3A1/Puup20+l3Y0Dw0+qfZWxNSTi7rk/ua4xqwGhbG67Ko1744B7ISEN0VjI= Received: from DB6PR0801CA0046.eurprd08.prod.outlook.com (2603:10a6:4:2b::14) by AM4PR0802MB2196.eurprd08.prod.outlook.com (2603:10a6:200:5a::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2451.23; Mon, 18 Nov 2019 17:19:40 +0000 Received: from VE1EUR03FT037.eop-EUR03.prod.protection.outlook.com (2a01:111:f400:7e09::207) by DB6PR0801CA0046.outlook.office365.com (2603:10a6:4:2b::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.20.2451.23 via Frontend Transport; Mon, 18 Nov 2019 17:19:40 +0000 Authentication-Results: spf=fail (sender IP is 63.35.35.123) smtp.mailfrom=arm.com; edk2.groups.io; dkim=pass (signature was verified) header.d=armh.onmicrosoft.com;edk2.groups.io; dmarc=none action=none header.from=arm.com; Received-SPF: Fail (protection.outlook.com: domain of arm.com does not designate 63.35.35.123 as permitted sender) receiver=protection.outlook.com; client-ip=63.35.35.123; helo=64aa7808-outbound-1.mta.getcheckrecipient.com; Received: from 64aa7808-outbound-1.mta.getcheckrecipient.com (63.35.35.123) by VE1EUR03FT037.mail.protection.outlook.com (10.152.19.70) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.20.2451.23 via Frontend Transport; Mon, 18 Nov 2019 17:19:39 +0000 Received: ("Tessian outbound e4042aced47b:v33"); Mon, 18 Nov 2019 17:19:39 +0000 X-CR-MTA-TID: 64aa7808 Received: from f6608c769d05.1 (cr-mta-lb-1.cr-mta-net [104.47.1.58]) by 64aa7808-outbound-1.mta.getcheckrecipient.com id 4C5934BC-0C89-4F0C-B232-92DD20ABD4A3.1; Mon, 18 Nov 2019 17:19:34 +0000 Received: from EUR01-VE1-obe.outbound.protection.outlook.com (mail-ve1eur01lp2058.outbound.protection.outlook.com [104.47.1.58]) by 64aa7808-outbound-1.mta.getcheckrecipient.com with ESMTPS id f6608c769d05.1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384); Mon, 18 Nov 2019 17:19:34 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Qd+el+aD8ov9aAQkq8uBEXOFxDn6H+jy26XS1X3CJwjwRfH9+29byTMsZRw9q28oh48Her+hHrtTYtyVwMwVPmag78UZ6auy+RX2A1vbihGcw5j75Chf+QVbpLbxJokVqbKZEzFsd4XqTA/1ywNmEsu7JrvTHdakpeOHUPulSaLOJb9NYDB90gMYoJV7/QZHhF5mmPTRD6kAqDbeXXMwDqmsjoQIXQFzQxlOixFFLAFgkU7hRvvtPaAG28xsxwCcBNImUgCQ8x9sqBQZrDP1VwKtxrEOVoOhvzhUp63ypDGPFYZ5TuP/TdP0L4LmZcGaeyIwG3EsMBBrtXBl/LMO/g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=hmOvTxoJHIuKFg/obo2DVOwHoEwkKyn/xSDEiFJBq8w=; b=C+yJSGSZhOF1LZN/3UAhbqscncVfKJIvfeRHuE30LHSTXruGZZQLe9m3wCnobG6kqsw6Q9aO6C89InNOVyAG0ewvd9zLjKpRPXtqoYTwE0/L2FCkhs9Iv24lU7CtpcAsU1zJudp8/u4iQRWlhUmOw4AA/8NfWECDE/3/vw+/rQ+M8dln1Gjf+WpkMST39P+ZlFSR04Q/7hrutfQrK1jboLe0TCk9BWP6FqhQX0g5UOgolpZljzvZgJpruR2JYycJUWYnrYHOvbQIKcD9sfcI5d3ALuIkI2lxxn384Ak3iZkNXo91JCFQBkcSvH4bTDxnwTmpfNcKWCrj6k47O36KvA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=armh.onmicrosoft.com; s=selector2-armh-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=hmOvTxoJHIuKFg/obo2DVOwHoEwkKyn/xSDEiFJBq8w=; b=rJ+UEMrH8zK5yndALFJMZR5l0dnDAIehuFFW2HI6mTCBA4+BtlBACtVzoD6+PcjRJ1mHVCwRd17NzHGDUtW/msRK0EM3EXa/TzhVqSxpM/NJwvJwaq70GWW4X7jZKZiaiv82KUuE6mSQp0/hAnzoKxWNzW5LQ2PnnjSjzRbZfow= Received: from VE1PR08MB4830.eurprd08.prod.outlook.com (10.255.113.19) by VE1PR08MB5072.eurprd08.prod.outlook.com (20.179.30.223) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2451.30; Mon, 18 Nov 2019 17:19:32 +0000 Received: from VE1PR08MB4830.eurprd08.prod.outlook.com ([fe80::ed9b:5d7e:2b1b:22de]) by VE1PR08MB4830.eurprd08.prod.outlook.com ([fe80::ed9b:5d7e:2b1b:22de%4]) with mapi id 15.20.2451.029; Mon, 18 Nov 2019 17:19:32 +0000 From: samer.el-haj-mahmoud@arm.com To: "devel@edk2.groups.io" , "leif.lindholm@linaro.org" , Pete Batard CC: "ard.biesheuvel@linaro.org" , "philmd@redhat.com" , Samer El-Haj-Mahmoud Subject: Re: [edk2-devel] [edk2-platforms][PATCH 2/8] Platform/RPi: Replace Bcm283x SoC base register address with a PCD Thread-Topic: [edk2-devel] [edk2-platforms][PATCH 2/8] Platform/RPi: Replace Bcm283x SoC base register address with a PCD Thread-Index: AQHVmwW2Q/v/rauNF0uPs4i+a1zleaeRKhOAgAAGXfA= Date: Mon, 18 Nov 2019 17:19:32 +0000 Message-ID: References: <20191114160740.10072-1-pete@akeo.ie> <20191114160740.10072-3-pete@akeo.ie> <20191118164853.GQ7323@bivouac.eciton.net> In-Reply-To: <20191118164853.GQ7323@bivouac.eciton.net> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ts-tracking-id: f607b875-0561-466e-b53f-52a5ee34d332.0 x-checkrecipientchecked: true Authentication-Results-Original: spf=none (sender IP is ) smtp.mailfrom=Samer.El-Haj-Mahmoud@arm.com; x-originating-ip: [217.140.111.135] x-ms-publictraffictype: Email X-MS-Office365-Filtering-HT: Tenant X-MS-Office365-Filtering-Correlation-Id: 62bb80ca-55bf-48e5-c688-08d76c4b7e7a X-MS-TrafficTypeDiagnostic: VE1PR08MB5072:|AM4PR0802MB2196: X-Microsoft-Antispam-PRVS: x-checkrecipientrouted: true x-ms-oob-tlc-oobclassifiers: OLM:9508;OLM:9508; x-forefront-prvs: 0225B0D5BC X-Forefront-Antispam-Report-Untrusted: SFV:NSPM;SFS:(10009020)(4636009)(39860400002)(136003)(366004)(396003)(346002)(376002)(199004)(189003)(13464003)(4326008)(71200400001)(71190400001)(8676002)(8936002)(81156014)(33656002)(81166006)(9686003)(64756008)(52536014)(76176011)(102836004)(66446008)(66556008)(30864003)(66946007)(76116006)(2906002)(6506007)(53546011)(316002)(6116002)(186003)(3846002)(54906003)(110136005)(256004)(7696005)(19627235002)(5660300002)(26005)(66476007)(2501003)(99286004)(966005)(478600001)(86362001)(45080400002)(229853002)(476003)(11346002)(6306002)(6436002)(14454004)(74316002)(25786009)(7736002)(66066001)(305945005)(446003)(6246003)(55016002)(486006);DIR:OUT;SFP:1101;SCL:1;SRVR:VE1PR08MB5072;H:VE1PR08MB4830.eurprd08.prod.outlook.com;FPR:;SPF:None;LANG:en;PTR:InfoNoRecords;A:1;MX:1; received-spf: None (protection.outlook.com: arm.com does not designate permitted sender hosts) X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam-Untrusted: BCL:0; X-Microsoft-Antispam-Message-Info-Original: zxZss8GBMdOmcJ+CPKkP7qmsb2P4W7VjaTNEvoHsgWneI2WsghuWt8UpnMm4OsY/b37lOvMdYdrSg2qNHcC+jiN/osdYSIkJ00lidRhT6NNLSanGx1YAOBiW32Pu7KAFvzwdowXFgkkJNdMJjprdvSxVM+3E8ej7p89UJJXR6J6DS9RHky/roBWuX6ykVQnWS8jGADUd7VJ7bG0zJtuR3GpB0MxOm0ySUVV6StVK9B+ZvoXUOijJ49vti7sXmrg9/VqLN9Q6VCOvyBCyV10Y5PRfrgpx0x3+bvj+MpfJjsG7qvmwVyPVbMAoM8y02IJLY3TckLDIz8ot44+XfY/TXQT5IML5piZFFUGe4pwtBcGikxoCGSO/9nradZKWIF9mbGt3+TB1eJITWBTs7ZOOtdHIgRhwyEd2JxW72cRwQX3sCJq7DQajFPYHYlBbZobA x-ms-exchange-transport-forked: True MIME-Version: 1.0 X-MS-Exchange-Transport-CrossTenantHeadersStamped: VE1PR08MB5072 Original-Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Samer.El-Haj-Mahmoud@arm.com; Return-Path: Samer.El-Haj-Mahmoud@arm.com X-EOPAttributedMessage: 0 X-MS-Exchange-Transport-CrossTenantHeadersStripped: VE1EUR03FT037.eop-EUR03.prod.protection.outlook.com X-Forefront-Antispam-Report: =?us-ascii?Q?CIP:63.35.35.123;IPV:CAL;SCL:-1;CTRY:IE;EFV:NLI;SFV:NSPM;SFS?= =?us-ascii?Q?:(10009020)(4636009)(136003)(346002)(396003)(376002)(3986040?= =?us-ascii?Q?0002)(1110001)(339900001)(13464003)(189003)(199004)(40434004?= =?us-ascii?Q?)(81166006)(97756001)(102836004)(50466002)(6306002)(26005)(5?= =?us-ascii?Q?024004)(14444005)(14454004)(70206006)(2906002)(23726003)(486?= =?us-ascii?Q?006)(7736002)(8676002)(336012)(70586007)(229853002)(19627235?= =?us-ascii?Q?002)(36906005)(305945005)(30864003)(74316002)(99286004)(1101?= =?us-ascii?Q?36005)(33656002)(356004)(105606002)(76130400001)(22756006)(5?= =?us-ascii?Q?4906003)(52536014)(5660300002)(6246003)(66066001)(8746002)(8?= =?us-ascii?Q?936002)(186003)(6506007)(53546011)(47776003)(46406003)(31600?= =?us-ascii?Q?2)(81156014)(11346002)(9686003)(476003)(446003)(478600001)(7?= =?us-ascii?Q?6176011)(26826003)(3846002)(6116002)(4326008)(126002)(550160?= =?us-ascii?Q?02)(107886003)(25786009)(966005)(45080400002)(2501003)(76960?= =?us-ascii?Q?05)(86362001);DIR:OUT;SFP:1101;SCL:1;SRVR:AM4PR0802MB2196;H:?= =?us-ascii?Q?64aa7808-outbound-1.mta.getcheckrecipient.com;FPR:;SPF:Fail;?= =?us-ascii?Q?LANG:en;PTR:ec2-63-35-35-123.eu-west-1.compute.amazonaws.com?= =?us-ascii?Q?;MX:1;A:1;?= X-MS-Office365-Filtering-Correlation-Id-Prvs: 50e3c67b-e8c0-4699-ea23-08d76c4b7a6d X-MS-Exchange-PUrlCount: 3 X-Forefront-PRVS: 0225B0D5BC X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: Xl+Mk2ZOKX52HL0uZ7uBgpwXpN/cTGe9t3xHKB6nKGxw/rgoNW35/xhSco8AjaoYvsTPZz1gIXYXDv4ctzvQ7mKcoW31XKWbOkryGduwWMTucnwbD56rCanwpTmZJs5Ozyh84W0xSUtfEE2VEVYgc+Ys4u/pWWMet1nyEDhaNRIFigkMr2vXEFDwhNe9vetnKF7cAbcvd3Hpe70OVRnb8ij8vcHbyKgLZ8+3bSPWDyQaQ2S/cmHUwKs8Rbr9p4XBUYHMRJamlRj3YWNZFXI6Lmyb04yqDB1yrzC8RwztpQ+nLnslgW+K3LYJ14DVJ3Fy0TyjPjWD0ZiazY9N6Uy8MJmCWmhNdNUixT/TYE1XslpWq2eRvnHnVWTw7Mnt9vpe+yTtGTAPU5fzNxbYY3pDV534/oqtBag3ObKbrLp2aLTvQePyoM7T8suk7hD0kjvowAPMqRpS/mXj9Uac7OnBpGYG4mvRD7nRcUEXBG8eXfU= X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Nov 2019 17:19:39.5065 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 62bb80ca-55bf-48e5-c688-08d76c4b7e7a X-MS-Exchange-CrossTenant-Id: f34e5979-57d9-4aaa-ad4d-b122a662184d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=f34e5979-57d9-4aaa-ad4d-b122a662184d;Ip=[63.35.35.123];Helo=[64aa7808-outbound-1.mta.getcheckrecipient.com] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM4PR0802MB2196 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable > I don't actually have a strong opinion here, but would appreciate an expl= icit statement that the code is being submitted > as copyright ARM with author @elhajmahmoud.com and not @arm.com? Leif, I use @elhajmahmoud.com for Github. But the code is Arm copyrighted = (since I am doing this for work). Please let me know if you have any concer= ns . Thanks for your review! --Samer -----Original Message----- From: devel@edk2.groups.io On Behalf Of Leif Lindho= lm via Groups.Io Sent: Monday, November 18, 2019 10:49 AM To: Pete Batard Cc: devel@edk2.groups.io; ard.biesheuvel@linaro.org; philmd@redhat.com; Sa= mer El-Haj-Mahmoud Subject: Re: [edk2-devel] [edk2-platforms][PATCH 2/8] Platform/RPi: Replac= e Bcm283x SoC base register address with a PCD On Thu, Nov 14, 2019 at 04:07:34PM +0000, Pete Batard wrote: > From: Samer El-Haj-Mahmoud > > Define BCM2836_SOC_REGISTERS from PcdBcm283xRegistersAddress. This is > needed in preparation for adding Raspberry Pi 4 support, since the two > Pi's have a different base addresses for the Bcm283x specific registers. Minor style comments below, would be nice if you could fold into any v2 coming based on Phil's comments. (Although from my reading the discussi= on, most of these intances may in fact fall out in a v2.) Also adding Samer to cc on this reply so he sees the feedback. > Signed-off-by: Pete Batard > --- > Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.inf | 2 = ++ > Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.inf | 2 = ++ > Platform/RaspberryPi/Drivers/DwUsbHostDxe/DwUsbHostDxe.inf | 4 = ++++ > Platform/RaspberryPi/Drivers/RpiFirmwareDxe/RpiFirmwareDxe.inf | 5 = ++++- > Platform/RaspberryPi/Drivers/SdHostDxe/SdHostDxe.inf | 2 = ++ > Platform/RaspberryPi/Library/PlatformLib/PlatformLib.inf | 3 = ++- > Platform/RaspberryPi/RPi3/RPi3.dsc | 6 = +++++- > Silicon/Broadcom/Bcm283x/Bcm283x.dec | 7 = +++++++ > Silicon/Broadcom/Bcm283x/Drivers/InterruptDxe/InterruptDxe.inf | 4 = +++- > Silicon/Broadcom/Bcm283x/Drivers/RngDxe/RngDxe.inf | 5 = +++++ > Silicon/Broadcom/Bcm283x/Include/IndustryStandard/Bcm2836.h | 3 = ++- > Silicon/Broadcom/Bcm283x/Library/GpioLib/GpioLib.inf | 4 = ++++ > 12 files changed, 42 insertions(+), 5 deletions(-) > > diff --git > a/Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.inf > b/Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.inf > index 3f0d7b6b9e9d..034c8c449f00 100644 > --- > a/Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.inf > +++ b/Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.i > +++ nf > @@ -1,5 +1,6 @@ > #/** @file > # > +# Copyright (c) 2019, ARM Limited. All rights reserved. I don't actually have a strong opinion here, but would appreciate an expli= cit statement that the code is being submitted as copyright ARM with author= @elhajmahmoud.com and not @arm.com? > # Copyright (c) 2017, Andrei Warkentin > # Copyright (c) Microsoft Corporation. All rights reserved. > # > @@ -42,6 +43,7 @@ [Protocols] > > [Pcd] > gRaspberryPiTokenSpaceGuid.PcdSdIsArasan > + gBcm283xTokenSpaceGuid.PcdBcm283xRegistersAddress When inserting new items like this, please do so alphabetically. > > [Depex] > gRaspberryPiFirmwareProtocolGuid AND > gRaspberryPiConfigAppliedProtocolGuid > diff --git a/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.inf > b/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.inf > index 28fc2682b585..4f4fdef4e003 100644 > --- a/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.inf > +++ b/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.inf > @@ -1,5 +1,6 @@ > #/** @file > # > +# Copyright (c) 2019, ARM Limited. All rights reserved. > # Copyright (c) 2018, Andrei Warkentin > # # SPDX-License-Identifier: BSD-2-Clause-Patent @@ -66,6 +67,7 @@ > [Pcd] > gRaspberryPiTokenSpaceGuid.PcdDebugShowUEFIExit > gRaspberryPiTokenSpaceGuid.PcdDisplayEnableScaledVModes > gRaspberryPiTokenSpaceGuid.PcdDisplayEnableSShot > + gBcm283xTokenSpaceGuid.PcdBcm283xRegistersAddress Also here. > > [FeaturePcd] > > diff --git > a/Platform/RaspberryPi/Drivers/DwUsbHostDxe/DwUsbHostDxe.inf > b/Platform/RaspberryPi/Drivers/DwUsbHostDxe/DwUsbHostDxe.inf > index e880c2fb0261..8817f20622d6 100644 > --- a/Platform/RaspberryPi/Drivers/DwUsbHostDxe/DwUsbHostDxe.inf > +++ b/Platform/RaspberryPi/Drivers/DwUsbHostDxe/DwUsbHostDxe.inf > @@ -1,5 +1,6 @@ > #/** @file > # > +# Copyright (c) 2019, ARM Limited. All rights reserved. > # Copyright (c) 2017-2018, Andrei Warkentin > # Copyright (c) 2015-2016, Linaro Limited= . All rights reserved. > # > @@ -51,5 +52,8 @@ [Protocols] > gEfiUsb2HcProtocolGuid > gRaspberryPiFirmwareProtocolGuid > > +[FixedPcd] > + gBcm283xTokenSpaceGuid.PcdBcm283xRegistersAddress > + > [Depex] > gRaspberryPiFirmwareProtocolGuid > diff --git > a/Platform/RaspberryPi/Drivers/RpiFirmwareDxe/RpiFirmwareDxe.inf > b/Platform/RaspberryPi/Drivers/RpiFirmwareDxe/RpiFirmwareDxe.inf > index 87bca98fec28..a3fc0fa49a3c 100644 > --- a/Platform/RaspberryPi/Drivers/RpiFirmwareDxe/RpiFirmwareDxe.inf > +++ b/Platform/RaspberryPi/Drivers/RpiFirmwareDxe/RpiFirmwareDxe.inf > @@ -1,5 +1,5 @@ > #/** @file > -# > +# Copyright (c) 2019, ARM Limited. All rights reserved. > # Copyright (c) 2017-2018, Andrei Warkentin > # Copyright (c) 2016, Linaro, Ltd. All ri= ghts reserved. > # > @@ -40,5 +40,8 @@ [LibraryClasses] > [Protocols] > gRaspberryPiFirmwareProtocolGuid ## PRODUCES > > +[FixedPcd] > + gBcm283xTokenSpaceGuid.PcdBcm283xRegistersAddress > + > [Depex] > TRUE > diff --git a/Platform/RaspberryPi/Drivers/SdHostDxe/SdHostDxe.inf > b/Platform/RaspberryPi/Drivers/SdHostDxe/SdHostDxe.inf > index 7386ff251864..b99f197bb007 100644 > --- a/Platform/RaspberryPi/Drivers/SdHostDxe/SdHostDxe.inf > +++ b/Platform/RaspberryPi/Drivers/SdHostDxe/SdHostDxe.inf > @@ -1,5 +1,6 @@ > #/** @file > # > +# Copyright (c) 2019, ARM Limited. All rights reserved. > # Copyright (c) 2017, Andrei Warkentin > # Copyright (c) Microsoft Corporation. All rights reserved. > # > @@ -44,6 +45,7 @@ [Protocols] > > [Pcd] > gRaspberryPiTokenSpaceGuid.PcdSdIsArasan > + gBcm283xTokenSpaceGuid.PcdBcm283xRegistersAddress Also here. > > [Depex] > gRaspberryPiFirmwareProtocolGuid AND > gRaspberryPiConfigAppliedProtocolGuid > diff --git a/Platform/RaspberryPi/Library/PlatformLib/PlatformLib.inf > b/Platform/RaspberryPi/Library/PlatformLib/PlatformLib.inf > index ed986034b957..85462febdd8d 100644 > --- a/Platform/RaspberryPi/Library/PlatformLib/PlatformLib.inf > +++ b/Platform/RaspberryPi/Library/PlatformLib/PlatformLib.inf > @@ -2,7 +2,7 @@ > # > # Copyright (c) 2017-2018, Andrei Warkentin > # Copyright (c) 2014-2016, Linaro Limited= . All rights reserved. > -# Copyright (c) 2011-2014, ARM Limited. All rights reserved. > +# Copyright (c) 2011-2019, ARM Limited. All rights reserved. > # > # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -54,6 +54,7 @@ > [FixedPcd] > gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageVariableSize > gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwWorkingSize > gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwSpareSize > + gBcm283xTokenSpaceGuid.PcdBcm283xRegistersAddress Also here. > > [Ppis] > gArmMpCoreInfoPpiGuid > diff --git a/Platform/RaspberryPi/RPi3/RPi3.dsc b/Platform/RaspberryPi/R= Pi3/RPi3.dsc > index a0365c5cf606..4e5a9f0b05e6 100644 > --- a/Platform/RaspberryPi/RPi3/RPi3.dsc > +++ b/Platform/RaspberryPi/RPi3/RPi3.dsc > @@ -1,6 +1,6 @@ > # @file > # > -# Copyright (c) 2011-2015, ARM Limited. All rights reserved. > +# Copyright (c) 2011 - 2019, ARM Limited. All rights reserved. > # Copyright (c) 2014, Linaro Limited. All rights reserved. > # Copyright (c) 2015 - 2016, Intel Corporation. All rights reserved. > # Copyright (c) 2017 - 2018, Andrei Warkentin > @@ -372,6 +372,10 @@ [PcdsFixedAtBuild.common] > gArmTokenSpaceGuid.PcdSystemMemoryBase|0x00400000 > gArmTokenSpaceGuid.PcdSystemMemorySize|0x3FC00000 > > + # > + # Device specific addresses > + # > + gBcm283xTokenSpaceGuid.PcdBcm283xRegistersAddress|0x3f000000 > ## NS16550 compatible UART > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialRegisterBase|0x3f215040 > gEfiMdeModulePkgTokenSpaceGuid.PcdSerialUseMmio|TRUE > diff --git a/Silicon/Broadcom/Bcm283x/Bcm283x.dec b/Silicon/Broadcom/Bcm= 283x/Bcm283x.dec > index ec62ff27fbb3..5b839b00d286 100644 > --- a/Silicon/Broadcom/Bcm283x/Bcm283x.dec > +++ b/Silicon/Broadcom/Bcm283x/Bcm283x.dec > @@ -1,5 +1,6 @@ > ## @file > # > +# Copyright (c) 2019, ARM Limited. All rights reserved. > # Copyright (c) 2019, Pete Batard > # > # SPDX-License-Identifier: BSD-2-Clause-Patent > @@ -14,3 +15,9 @@ [Defines] > > [Includes] > Include > + > +[Guids] > + gBcm283xTokenSpaceGuid =3D {0x82f36a92, 0xfb7e, 0x43a1, {0xb9, 0x9e, = 0x49, 0x13, 0x3f, 0xc7, 0xa4, 0x2e}} > + > +[PcdsFixedAtBuild.common] > + gBcm283xTokenSpaceGuid.PcdBcm283xRegistersAddress|0x0|UINT32|0x000000= 01 > diff --git a/Silicon/Broadcom/Bcm283x/Drivers/InterruptDxe/InterruptDxe.= inf b/Silicon/Broadcom/Bcm283x/Drivers/InterruptDxe/InterruptDxe.inf > index cdce11a51e14..6c58df5c3285 100644 > --- a/Silicon/Broadcom/Bcm283x/Drivers/InterruptDxe/InterruptDxe.inf > +++ b/Silicon/Broadcom/Bcm283x/Drivers/InterruptDxe/InterruptDxe.inf > @@ -1,5 +1,5 @@ > #/** @file > -# > +# Copyright (c) 2019, ARM Limited. All rights reserved. > # Copyright (c) 2017, Andrei Warkentin > # Copyright (c) 2016 Linaro, Ltd. All rights reserved. > # > @@ -30,6 +30,7 @@ [LibraryClasses] > UefiBootServicesTableLib > UefiLib > UefiDriverEntryPoint > + PcdLib Also here. > > [Protocols] > gHardwareInterruptProtocolGuid ## PRODUCES > @@ -37,6 +38,7 @@ [Protocols] > > [FixedPcd] > gEmbeddedTokenSpaceGuid.PcdInterruptBaseAddress > + gBcm283xTokenSpaceGuid.PcdBcm283xRegistersAddress Also here. > > [Depex] > TRUE > diff --git a/Silicon/Broadcom/Bcm283x/Drivers/RngDxe/RngDxe.inf b/Silico= n/Broadcom/Bcm283x/Drivers/RngDxe/RngDxe.inf > index cb1695bd2dfc..4481d71aaff0 100644 > --- a/Silicon/Broadcom/Bcm283x/Drivers/RngDxe/RngDxe.inf > +++ b/Silicon/Broadcom/Bcm283x/Drivers/RngDxe/RngDxe.inf > @@ -1,5 +1,6 @@ > #/** @file > # > +# Copyright (c) 2019, ARM Limited. All rights reserved. > # Copyright (c) 2019 Linaro, Ltd. All rights reserved. > # > # SPDX-License-Identifier: BSD-2-Clause-Patent > @@ -28,6 +29,7 @@ [LibraryClasses] > IoLib > UefiBootServicesTableLib > UefiDriverEntryPoint > + PcdLib Also here. / Leif > > [Protocols] > gEfiRngProtocolGuid ## PRODUCES > @@ -35,5 +37,8 @@ [Protocols] > [Guids] > gEfiRngAlgorithmRaw > > +[FixedPcd] > + gBcm283xTokenSpaceGuid.PcdBcm283xRegistersAddress > + > [Depex] > TRUE > diff --git a/Silicon/Broadcom/Bcm283x/Include/IndustryStandard/Bcm2836.h= b/Silicon/Broadcom/Bcm283x/Include/IndustryStandard/Bcm2836.h > index 4007301228be..8bd68c234bfd 100644 > --- a/Silicon/Broadcom/Bcm283x/Include/IndustryStandard/Bcm2836.h > +++ b/Silicon/Broadcom/Bcm283x/Include/IndustryStandard/Bcm2836.h > @@ -1,5 +1,6 @@ > /** @file > * > + * Copyright (c) 2019, ARM Limited. All rights reserved. > * Copyright (c) 2017, Andrei Warkentin > * Copyright (c) 2016, Linaro Limited. All rights reserved. > * > @@ -13,7 +14,7 @@ > /* > * Both "core" and SoC perpherals (1M each). > */ > -#define BCM2836_SOC_REGISTERS 0x3f000000 > +#define BCM2836_SOC_REGISTERS (FixedPcdGe= t64 (PcdBcm283xRegistersAddress)) > #define BCM2836_SOC_REGISTER_LENGTH 0x02000000 > > /* > diff --git a/Silicon/Broadcom/Bcm283x/Library/GpioLib/GpioLib.inf b/Sili= con/Broadcom/Bcm283x/Library/GpioLib/GpioLib.inf > index 50da4eb771f3..ff1b5af6db6e 100644 > --- a/Silicon/Broadcom/Bcm283x/Library/GpioLib/GpioLib.inf > +++ b/Silicon/Broadcom/Bcm283x/Library/GpioLib/GpioLib.inf > @@ -2,6 +2,7 @@ > # > # Manipulate GPIOs. > # > +# Copyright (c) 2019, ARM Limited. All rights reserved. > # Copyright (c) 2018, Andrei Warkentin > # > # SPDX-License-Identifier: BSD-2-Clause-Patent > @@ -30,4 +31,7 @@ [LibraryClasses] > DebugLib > IoLib > > +[FixedPcd] > + gBcm283xTokenSpaceGuid.PcdBcm283xRegistersAddress > + > [Guids] > -- > 2.21.0.windows.1 > IMPORTANT NOTICE: The contents of this email and any attachments are confi= dential and may also be privileged. If you are not the intended recipient, = please notify the sender immediately and do not disclose the contents to an= y other person, use it for any purpose, or store or copy the information in= any medium. Thank you.