diff mbox

spi/pl022: Explicitly truncate large bitmask

Message ID 1406911735-17239-1-git-send-email-broonie@kernel.org (mailing list archive)
State Accepted
Commit d555ea05f9d8ebf567eaa6b4e4cb5776aacf2940
Headers show

Commit Message

Mark Brown Aug. 1, 2014, 4:48 p.m. UTC 
From: Mark Brown <broonie@linaro.org>

When building on 64 bit architectures the use of bitwise negation generates
constants larger than 32 bits which won't fit in u32s used to represent
32 bit register values on the device. Explicitly cast to let the compiler
know that the higher bits are not significant and can be discarded.

Signed-off-by: Mark Brown <broonie@linaro.org>
---
 drivers/spi/spi-pl022.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Linus Walleij Aug. 8, 2014, 9:02 a.m. UTC | #1 
On Fri, Aug 1, 2014 at 6:48 PM, Mark Brown <broonie@kernel.org> wrote:

> From: Mark Brown <broonie@linaro.org>
>
> When building on 64 bit architectures the use of bitwise negation generates
> constants larger than 32 bits which won't fit in u32s used to represent
> 32 bit register values on the device. Explicitly cast to let the compiler
> know that the higher bits are not significant and can be discarded.
>
> Signed-off-by: Mark Brown <broonie@linaro.org>

Reviewed-by: Linus Walleij <linus.walleij@linaro.org>

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/spi/spi-pl022.c b/drivers/spi/spi-pl022.c
index 66d2ae2..1189cfd 100644
--- a/drivers/spi/spi-pl022.c
+++ b/drivers/spi/spi-pl022.c
@@ -1417,7 +1417,7 @@  static void do_interrupt_dma_transfer(struct pl022 *pl022)
 	 * Default is to enable all interrupts except RX -
 	 * this will be enabled once TX is complete
 	 */
-	u32 irqflags = ENABLE_ALL_INTERRUPTS & ~SSP_IMSC_MASK_RXIM;
+	u32 irqflags = (u32)(ENABLE_ALL_INTERRUPTS & ~SSP_IMSC_MASK_RXIM);
 
 	/* Enable target chip, if not already active */
 	if (!pl022->next_msg_cs_active)