diff options
author | Greg Ungerer <gerg@uclinux.org> | 2014-08-28 00:39:45 -0400 |
---|---|---|
committer | Greg Ungerer <gerg@uclinux.org> | 2014-09-28 19:18:36 -0400 |
commit | ecb6bdcde2474c8c954179b74fd3df3ad84fba75 (patch) | |
tree | ebd3ec082d8d319b95c2eb2a230b52a52cc08e4c | |
parent | 6f4a8856e084fbeae226ef52de1fc4f8d2c68765 (diff) |
m68knommu: fix size of address field for 5272 interrupt controller
Picked up by the 0-day buidler:
All warnings:
>> arch/m68k/coldfire/intc-5272.c:46:20: warning: large integer implicitly truncated to unsigned type [-Woverflow]
/*MCF_IRQ_EINT1*/ { .icr = MCFSIM_ICR1, .index = 28, .ack = 1, },
...
The problem stems from the changes to make all ColdFire register addresses
absolute, in commit d72a5abb ("make remaining ColdFire 5272 register
definitions absolute"). That change did not take into account that the
addresses were stored as offsets in the irqmap of the intc-5272.c code.
Make the field that now stores register addresses big enough to hold
addresses.
Signed-off-by: Greg Ungerer <gerg@uclinux.org>
-rw-r--r-- | arch/m68k/coldfire/intc-5272.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/m68k/coldfire/intc-5272.c b/arch/m68k/coldfire/intc-5272.c index d7b695629a7e..d1e2fbad327c 100644 --- a/arch/m68k/coldfire/intc-5272.c +++ b/arch/m68k/coldfire/intc-5272.c | |||
@@ -36,7 +36,7 @@ | |||
36 | * they also need acknowledging via acknowledge bits. | 36 | * they also need acknowledging via acknowledge bits. |
37 | */ | 37 | */ |
38 | struct irqmap { | 38 | struct irqmap { |
39 | unsigned char icr; | 39 | unsigned int icr; |
40 | unsigned char index; | 40 | unsigned char index; |
41 | unsigned char ack; | 41 | unsigned char ack; |
42 | }; | 42 | }; |