From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from loongson.cn (loongson.cn [114.242.206.163]) by mx.groups.io with SMTP id smtpd.web10.7708.1666257906552794091 for ; Thu, 20 Oct 2022 02:25:07 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: loongson.cn, ip: 114.242.206.163, mailfrom: lichao@loongson.cn) Received: from loongson.cn (unknown [10.2.9.245]) by gateway (Coremail) with SMTP id _____8CxjdrvE1FjuwABAA--.5090S3; Thu, 20 Oct 2022 17:25:03 +0800 (CST) Received: from code-server.gen (unknown [10.2.9.245]) by localhost.localdomain (Coremail) with SMTP id AQAAf8AxPuDvE1FjF9gBAA--.7513S2; Thu, 20 Oct 2022 17:25:03 +0800 (CST) From: "Chao Li" To: devel@edk2.groups.io Cc: Bob Feng , Liming Gao , Yuwei Chen Subject: [PATCH v2] BaseTools: Fixed the multiple pairs brackets issue in GenFv Date: Thu, 20 Oct 2022 17:25:01 +0800 Message-Id: <20221020092501.3783878-1-lichao@loongson.cn> X-Mailer: git-send-email 2.27.0 MIME-Version: 1.0 X-CM-TRANSID: AQAAf8AxPuDvE1FjF9gBAA--.7513S2 X-CM-SenderInfo: xolfxt3r6o00pqjv00gofq/1tbiAQAFCGNP6OEYAgACsx X-Coremail-Antispam: 1Uk129KBjvJXoW7Cw4DZry3Zry7uw1xur45ZFb_yoW8Wr1rpF 10kF48Kry0gr1rC3ZYy34q9ryUAFs3uFy8Wa1vyw4qv3s3uwnrW34Skr4agFWUXa1qy3WS vrn5CF9FvF1vyrJanT9S1TB71UUUUUUqnTZGkaVYY2UrUUUUj1kv1TuYvTs0mT0YCTnIWj qI5I8CrVACY4xI64kE6c02F40Ex7xfYxn0WfASr-VFAUDa7-sFnT9fnUUIcSsGvfJTRUUU bn8Fc2x0x2IEx4CE42xK8VAvwI8IcIk0rVWrJVCq3wA2ocxC64kIII0Yj41l84x0c7CEw4 AK67xGY2AK021l84ACjcxK6xIIjxv20xvE14v26r4j6ryUM28EF7xvwVC0I7IYx2IY6xkF 7I0E14v26r4j6F4UM28EF7xvwVC2z280aVAFwI0_Gr1j6F4UJwA2z4x0Y4vEx4A2jsIEc7 CjxVAFwI0_Cr1j6rxdM2AIxVAIcxkEcVAq07x20xvEncxIr21l57IF6xkI12xvs2x26I8E 6xACxx1l5I8CrVACY4xI64kE6c02F40Ex7xfMcIj6x8ErcxFaVAv8VWrMcvjeVCFs4IE7x kEbVWUJVW8JwACjcxG0xvY0x0EwIxGrwCF04k20xvY0x0EwIxGrwCF04k20xvE74AGY7Cv 6cx26rWl4I8I3I0E4IkC6x0Yz7v_Jr0_Gr1lx2IqxVAqx4xG67AKxVWUJVWUGwC20s026x 8GjcxK67AKxVWUGVWUWwC2zVAF1VAY17CE14v26r126r1DMIIYrxkI7VAKI48JMIIF0xvE 2Ix0cI8IcVAFwI0_Jr0_JF4lIxAIcVC0I7IYx2IY6xkF7I0E14v26r1j6r4UMIIF0xvE42 xK8VAvwI8IcIk0rVWUJVWUCwCI42IY6I8E87Iv67AKxVW8JVWxJwCI42IY6I8E87Iv6xkF 7I0E14v26r4j6r4UJbIYCTnIWIevJa73UjIFyTuYvj4RC_MaUUUUU Content-Transfer-Encoding: quoted-printable If operation Werro is turned on when compiling BaseTools, the multi-brackets warning will be reported. This issue is comes from on of the LoongArch enabled patche. Removed extra pairs brackets to fix it. REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D4111 Cc: Bob Feng Cc: Liming Gao Cc: Yuwei Chen Signed-off-by: Chao Li --- BaseTools/Source/C/GenFv/GenFvInternalLib.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/BaseTools/Source/C/GenFv/GenFvInternalLib.c b/BaseTools/Source= /C/GenFv/GenFvInternalLib.c index 5c3d54f5f7..b5b9425003 100644 --- a/BaseTools/Source/C/GenFv/GenFvInternalLib.c +++ b/BaseTools/Source/C/GenFv/GenFvInternalLib.c @@ -3559,7 +3559,7 @@ Returns: }=0D =0D // Machine type is LOONGARCH64, set a flag so LoongArch64 reset vect= or processed.=0D - if ((MachineType =3D=3D EFI_IMAGE_MACHINE_LOONGARCH64)) {=0D + if (MachineType =3D=3D EFI_IMAGE_MACHINE_LOONGARCH64) {=0D VerboseMsg("Located LoongArch64 SEC core in child FV");=0D mLoongArch =3D TRUE;=0D }=0D @@ -3721,7 +3721,7 @@ Returns: mRiscV =3D TRUE;=0D }=0D =0D - if ( (ImageContext.Machine =3D=3D EFI_IMAGE_MACHINE_LOONGARCH64) ) {=0D + if (ImageContext.Machine =3D=3D EFI_IMAGE_MACHINE_LOONGARCH64) {=0D mLoongArch =3D TRUE;=0D }=0D =0D @@ -4002,7 +4002,7 @@ Returns: mArm =3D TRUE;=0D }=0D =0D - if ( (ImageContext.Machine =3D=3D EFI_IMAGE_MACHINE_LOONGARCH64) ) {=0D + if (ImageContext.Machine =3D=3D EFI_IMAGE_MACHINE_LOONGARCH64) {=0D mLoongArch =3D TRUE;=0D }=0D =0D --=20 2.27.0