[coreboot-gerrit] New patch to review for coreboot: soc/intel/common: Remove parenthesis

Lee Leahy (leroy.p.leahy@intel.com) gerrit at coreboot.org
Fri Mar 10 22:51:42 CET 2017


Lee Leahy (leroy.p.leahy at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18757

-gerrit

commit 47160304bcdcfc4b914aabd5a984e32c7e9ac52f
Author: Lee Leahy <leroy.p.leahy at intel.com>
Date:   Fri Mar 10 13:48:35 2017 -0800

    soc/intel/common: Remove parenthesis
    
    Fix the following error detected by checkpatch.pl:
    
    ERROR: return is not a function, parentheses are not required
    
    TEST=Build and run on Galileo Gen2
    
    Change-Id: Idf7723d4fd48124a26bbb626afc310820f859f66
    Signed-off-by: Lee Leahy <Leroy.P.Leahy at intel.com>
---
 src/soc/intel/common/acpi/acpi.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/soc/intel/common/acpi/acpi.c b/src/soc/intel/common/acpi/acpi.c
index f969158..458b95f 100644
--- a/src/soc/intel/common/acpi/acpi.c
+++ b/src/soc/intel/common/acpi/acpi.c
@@ -109,7 +109,7 @@ __attribute__((weak)) acpi_tstate_t *soc_get_tss_table(int *entries)
 __attribute__((weak)) uint16_t soc_get_acpi_base_address(void)
 {
 #define ACPI_BASE_ADDR 0x400
-	return (ACPI_BASE_ADDR);
+	return ACPI_BASE_ADDR;
 }
 
 static int calculate_power(int tdp, int p1_ratio, int ratio)



More information about the coreboot-gerrit mailing list