From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.65, mailfrom: chasel.chiu@intel.com) Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by groups.io with SMTP; Thu, 22 Aug 2019 11:08:27 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 22 Aug 2019 11:08:27 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,417,1559545200"; d="scan'208";a="354368234" Received: from kmsmsx156.gar.corp.intel.com ([172.21.138.133]) by orsmga005.jf.intel.com with ESMTP; 22 Aug 2019 11:08:25 -0700 Received: from pgsmsx109.gar.corp.intel.com (10.221.44.109) by KMSMSX156.gar.corp.intel.com (172.21.138.133) with Microsoft SMTP Server (TLS) id 14.3.439.0; Fri, 23 Aug 2019 02:08:24 +0800 Received: from pgsmsx111.gar.corp.intel.com ([169.254.2.22]) by PGSMSX109.gar.corp.intel.com ([169.254.14.101]) with mapi id 14.03.0439.000; Fri, 23 Aug 2019 02:08:24 +0800 From: "Chiu, Chasel" To: "devel@edk2.groups.io" , "Kubacki, Michael A" CC: "Desimone, Nathaniel L" , "Gao, Liming" Subject: Re: [edk2-devel] [edk2-platforms][PATCH V1 1/2] KabylakeOpenBoardPkg: Fix GCC Build Failures Thread-Topic: [edk2-devel] [edk2-platforms][PATCH V1 1/2] KabylakeOpenBoardPkg: Fix GCC Build Failures Thread-Index: AQHVWRCw4pCV2/xWV0yoTLmIaiymx6cHdsMw Date: Thu, 22 Aug 2019 18:08:24 +0000 Message-ID: <3C3EFB470A303B4AB093197B6777CCEC50468788@PGSMSX111.gar.corp.intel.com> References: <20190822174008.20624-1-michael.a.kubacki@intel.com> <20190822174008.20624-2-michael.a.kubacki@intel.com> In-Reply-To: <20190822174008.20624-2-michael.a.kubacki@intel.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.2.0.6 dlp-reaction: no-action x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiNTk0YWI3MTYtNjFiMy00YmQ5LTkwZWQtYTEyZjZmNjlkOWRhIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoidTh2Q1daazFsWE5QQmpWNHh4V0tDc1FydDJ1YnFrVVwvaTA4K25YYUsrcXdwZ2JlSHpNTnFGQmNNSmhnSDRBS3gifQ== x-ctpclassification: CTP_NT x-originating-ip: [172.30.20.206] MIME-Version: 1.0 Return-Path: chasel.chiu@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable BaseGpioExpanderLib.c still having 2017 in copyright, please help to exten= d it. With above change Reviewed-by: Chasel Chiu > -----Original Message----- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > Kubacki, Michael A > Sent: Friday, August 23, 2019 1:40 AM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Gao, Liming > Subject: [edk2-devel] [edk2-platforms][PATCH V1 1/2] KabylakeOpenBoardPk= g: > Fix GCC Build Failures >=20 > REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D2109 >=20 > Fixes build failures on GCC7.3.0. Tested on Ubunutu 18.04.1 LTS. >=20 > Cc: Chasel Chiu > Cc: Nate DeSimone > Cc: Liming Gao > Signed-off-by: Michael Kubacki > --- >=20 > Platform/Intel/KabylakeOpenBoardPkg/FspWrapper/Library/SecFspWrapper > PlatformSecLib/FspWrapperPlatformSecLib.c | 2 +- >=20 > Platform/Intel/KabylakeOpenBoardPkg/Library/BaseGpioExpanderLib/BaseG > pioExpanderLib.c | 6 ++---- > 2 files changed, 3 insertions(+), 5 deletions(-) >=20 > diff --git > a/Platform/Intel/KabylakeOpenBoardPkg/FspWrapper/Library/SecFspWrapp > erPlatformSecLib/FspWrapperPlatformSecLib.c > b/Platform/Intel/KabylakeOpenBoardPkg/FspWrapper/Library/SecFspWrapp > erPlatformSecLib/FspWrapperPlatformSecLib.c > index d73fc77f69..d40eecae95 100644 > --- > a/Platform/Intel/KabylakeOpenBoardPkg/FspWrapper/Library/SecFspWrapp > erPlatformSecLib/FspWrapperPlatformSecLib.c > +++ > b/Platform/Intel/KabylakeOpenBoardPkg/FspWrapper/Library/SecFspWrapp > erPlatformSecLib/FspWrapperPlatformSecLib.c > @@ -174,7 +174,7 @@ SecPlatformMain ( > // > CopyMem (CopyDestinationPointer, mPeiCoreFvLocationPpiList, sizeof > (mPeiCoreFvLocationPpiList)); > TopOfTemporaryRamPpiIndex =3D 1; > - (UINT8 *) CopyDestinationPointer +=3D sizeof(mPeiCoreFvLocationPpiL= ist); > + CopyDestinationPointer +=3D sizeof (mPeiCoreFvLocationPpiList); > } > CopyMem (CopyDestinationPointer, mPeiSecPlatformPpi, > sizeof(mPeiSecPlatformPpi)); > // > diff --git > a/Platform/Intel/KabylakeOpenBoardPkg/Library/BaseGpioExpanderLib/Bas > eGpioExpanderLib.c > b/Platform/Intel/KabylakeOpenBoardPkg/Library/BaseGpioExpanderLib/Bas > eGpioExpanderLib.c > index ead1e6df19..acab1326ff 100644 > --- > a/Platform/Intel/KabylakeOpenBoardPkg/Library/BaseGpioExpanderLib/Bas > eGpioExpanderLib.c > +++ > b/Platform/Intel/KabylakeOpenBoardPkg/Library/BaseGpioExpanderLib/Bas > eGpioExpanderLib.c > @@ -70,12 +70,11 @@ GpioExpGetRegister ( > IN UINT8 Register > ) > { > - EFI_STATUS Status; > UINT8 WriBuf[1]; > UINT8 ReBuf[1] =3D {0}; >=20 > WriBuf[0] =3D Register; > - Status =3D I2cWriteRead( Bar0, TCA6424_I2C_ADDRESS+Address, 1, WriBuf= , 1, > ReBuf, WAIT_1_SECOND); > + I2cWriteRead (Bar0, TCA6424_I2C_ADDRESS + Address, 1, WriBuf, 1, ReBu= f, > WAIT_1_SECOND); >=20 > return ReBuf[0]; > } > @@ -99,13 +98,12 @@ GpioExpSetRegister ( > IN UINT8 Value > ) > { > - EFI_STATUS Status; > UINT8 WriBuf[2]; >=20 > WriBuf[0] =3D Register; > WriBuf[1] =3D Value; > - Status =3D I2cWriteRead( Bar0, TCA6424_I2C_ADDRESS+Address, 2, WriBuf= , 0, > NULL, WAIT_1_SECOND); >=20 > + I2cWriteRead (Bar0, TCA6424_I2C_ADDRESS + Address, 2, WriBuf, 0, NULL= , > WAIT_1_SECOND); > } > /** > Set the input register to a give value mentioned in the function. > -- > 2.16.2.windows.1 >=20 >=20 >=20