From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.88, mailfrom: nathaniel.l.desimone@intel.com) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by groups.io with SMTP; Fri, 23 Aug 2019 15:00:46 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga004.jf.intel.com ([10.7.209.38]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 23 Aug 2019 15:00:46 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,422,1559545200"; d="scan'208";a="330849217" Received: from orsmsx107.amr.corp.intel.com ([10.22.240.5]) by orsmga004.jf.intel.com with ESMTP; 23 Aug 2019 15:00:46 -0700 Received: from orsmsx157.amr.corp.intel.com (10.22.240.23) by ORSMSX107.amr.corp.intel.com (10.22.240.5) with Microsoft SMTP Server (TLS) id 14.3.439.0; Fri, 23 Aug 2019 15:00:45 -0700 Received: from orsmsx114.amr.corp.intel.com ([169.254.8.225]) by ORSMSX157.amr.corp.intel.com ([169.254.9.22]) with mapi id 14.03.0439.000; Fri, 23 Aug 2019 15:00:45 -0700 From: "Nate DeSimone" To: "devel@edk2.groups.io" , "Kubacki, Michael A" CC: "Gao, Liming" , "Sinha, Ankit" Subject: Re: [edk2-devel] [edk2-platforms][PATCH V1 2/2] ClevoOpenBoardPkg: Fix GCC Build Failures Thread-Topic: [edk2-devel] [edk2-platforms][PATCH V1 2/2] ClevoOpenBoardPkg: Fix GCC Build Failures Thread-Index: AQHVWRC08BLwdguyok+Cx+xiGS0jF6cJSjWw Date: Fri, 23 Aug 2019 22:00:45 +0000 Message-ID: <02A34F284D1DA44BB705E61F7180EF0AAEE47F56@ORSMSX114.amr.corp.intel.com> References: <20190822174008.20624-1-michael.a.kubacki@intel.com> <20190822174008.20624-3-michael.a.kubacki@intel.com> In-Reply-To: <20190822174008.20624-3-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: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiNGQ1M2IyZjYtOGNmMS00N2Q5LTljZDYtZDRmNDJlZWI4ZjljIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiMlBNQlRcL3BKVjRXYWdlNm50eHUwa0VUOGFsXC81SUVHTVFscFd1U3Jyd3g5U2ZOU0NVUDZGT1EwWWlxb0VKUlhlIn0= x-ctpclassification: CTP_NT x-originating-ip: [10.22.254.139] MIME-Version: 1.0 Return-Path: nathaniel.l.desimone@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Reviewed-by: Nate DeSimone -----Original Message----- From: devel@edk2.groups.io On Behalf Of Kubacki, Mi= chael A Sent: Thursday, August 22, 2019 10:40 AM To: devel@edk2.groups.io Cc: Desimone, Nathaniel L ; Gao, Liming ; Sinha, Ankit Subject: [edk2-devel] [edk2-platforms][PATCH V1 2/2] ClevoOpenBoardPkg: Fi= x GCC Build Failures Fixes build failures on GCC7.3.0. Tested on Ubunutu 18.04.1 LTS. Cc: Nate DeSimone Cc: Liming Gao Cc: Ankit Sinha Signed-off-by: Michael Kubacki --- Platform/Intel/ClevoOpenBoardPkg/Library/BaseGpioExpanderLib/BaseGpioExpa= nderLib.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/Platform/Intel/ClevoOpenBoardPkg/Library/BaseGpioExpanderLib/= BaseGpioExpanderLib.c b/Platform/Intel/ClevoOpenBoardPkg/Library/BaseGpioEx= panderLib/BaseGpioExpanderLib.c index 260c5bb6a2..cc70f15c24 100644 --- a/Platform/Intel/ClevoOpenBoardPkg/Library/BaseGpioExpanderLib/BaseGpi= oExpanderLib.c +++ b/Platform/Intel/ClevoOpenBoardPkg/Library/BaseGpioExpanderLib/BaseGpi= oExpanderLib.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, ReBuf,= WAIT_1_SECOND); =20 return ReBuf[0]; } @@ -99,13 +98,11 @@ 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); - + 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. --=20 2.16.2.windows.1