From ccd92a19d41322d160298427600f1d1302a934a5 Mon Sep 17 00:00:00 2001 From: Stefan Tauner Date: Fri, 1 Jul 2011 00:39:01 +0000 Subject: ichspi.c: make REGWRITE* macros safer '+' does have a quite high precedence so "calling" those macros with a term including weaker operators in the off parameter may have unexpected consequences. Corresponding to flashrom svn r1359. Signed-off-by: Stefan Tauner Acked-by: Carl-Daniel Hailfinger --- ichspi.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'ichspi.c') diff --git a/ichspi.c b/ichspi.c index b4eb236..a78034a 100644 --- a/ichspi.c +++ b/ichspi.c @@ -216,9 +216,9 @@ static uint16_t REGREAD8(int X) return mmio_readb(ich_spibar + X); } -#define REGWRITE32(off,val) mmio_writel(val, ich_spibar+off) -#define REGWRITE16(off,val) mmio_writew(val, ich_spibar+off) -#define REGWRITE8(off,val) mmio_writeb(val, ich_spibar+off) +#define REGWRITE32(off, val) mmio_writel(val, ich_spibar+(off)) +#define REGWRITE16(off, val) mmio_writew(val, ich_spibar+(off)) +#define REGWRITE8(off, val) mmio_writeb(val, ich_spibar+(off)) /* Common SPI functions */ static int find_opcode(OPCODES *op, uint8_t opcode); -- cgit v1.1