summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAngel Pons <th3fanbus@gmail.com>2020-07-07 18:53:17 +0200
committerAngel Pons <th3fanbus@gmail.com>2020-07-25 10:23:16 +0000
commitea07702f62c9266e38f3e024cee7ce21b78aba8e (patch)
tree7ca2074c6a5493f792a1124cc43b2dbf6ba4e15a
parentbaebe2afc153172850333968601e33d2686de47b (diff)
downloadcoreboot-ea07702f62c9266e38f3e024cee7ce21b78aba8e.tar.gz
coreboot-ea07702f62c9266e38f3e024cee7ce21b78aba8e.tar.bz2
coreboot-ea07702f62c9266e38f3e024cee7ce21b78aba8e.zip
soc/intel/baytrail/include/soc/irq.h: Add braces
This reduces the differences between Bay Trail and Braswell, and avoids unlikely but potential bugs regarding missing braces in macros. Tested with BUILD_TIMELESS=1, Google Ninja remains identical. Change-Id: Ic341fe70e7d6fb4751f2fefbdedbee5c90dd8d1f Signed-off-by: Angel Pons <th3fanbus@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/43201 Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
-rw-r--r--src/soc/intel/baytrail/include/soc/irq.h4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/soc/intel/baytrail/include/soc/irq.h b/src/soc/intel/baytrail/include/soc/irq.h
index 967bc3534129..3a1d7c79e426 100644
--- a/src/soc/intel/baytrail/include/soc/irq.h
+++ b/src/soc/intel/baytrail/include/soc/irq.h
@@ -140,8 +140,8 @@ extern const struct baytrail_irq_route global_baytrail_irq_route;
/* The following macros are used for ACPI by the ASL compiler */
#define PCI_DEV_PIRQ_ROUTE(dev_, a_, b_, c_, d_) \
- [dev_] = ((PIRQ ## d_) << 12) | ((PIRQ ## c_) << 8) | \
- ((PIRQ ## b_) << 4) | ((PIRQ ## a_) << 0)
+ [dev_] = (((PIRQ ## d_) << 12) | ((PIRQ ## c_) << 8) | \
+ ((PIRQ ## b_) << 4) | ((PIRQ ## a_) << 0))
#define PIRQ_PIC(pirq_, pic_irq_) \
[PIRQ ## pirq_] = PIRQ_PIC_IRQ ## pic_irq_