diff options
author | Randy Dunlap <randy.dunlap@oracle.com> | 2010-04-16 20:51:42 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2010-04-17 04:37:20 -0400 |
commit | a289cc7c70da784a2d370b91885cab4f966dcb0f (patch) | |
tree | 7e06be85ce3754061f20ccd3f77439c2d83f4549 /arch | |
parent | b8f7fb13d2d7ff14818fd1d3edd8b834d38b0217 (diff) |
x86, UV: uv_irq.c: Fix all sparse warnings
Fix all sparse warnings in building uv_irq.c.
arch/x86/kernel/uv_irq.c:46:17: warning: symbol 'uv_irq_chip' was not declared. Should it be static?
arch/x86/kernel/uv_irq.c:143:50: error: no identifier for function argument
arch/x86/kernel/uv_irq.c:162:13: error: typename in expression
arch/x86/kernel/uv_irq.c:162:13: error: undefined identifier 'restrict'
arch/x86/kernel/uv_irq.c:250:44: error: no identifier for function argument
arch/x86/kernel/uv_irq.c:260:17: error: typename in expression
arch/x86/kernel/uv_irq.c:260:17: error: undefined identifier 'restrict'
arch/x86/kernel/uv_irq.c:233:50: warning: incorrect type in argument 3 (different signedness)
arch/x86/kernel/uv_irq.c:233:50: expected int *pnode
arch/x86/kernel/uv_irq.c:233:50: got unsigned int *<noident>
arch/x86/include/asm/uv/uv_hub.h:318:44: warning: incorrect type in argument 2 (different address spaces)
arch/x86/include/asm/uv/uv_hub.h:318:44: expected void volatile [noderef] <asn:2>*addr
arch/x86/include/asm/uv/uv_hub.h:318:44: got unsigned long *
Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
Cc: Dimitri Sivanich <sivanich@sgi.com>
Cc: Russ Anderson <rja@sgi.com>
Cc: Robin Holt <holt@sgi.com>
Cc: Mike Travis <travis@sgi.com>
Cc: Cliff Wickman <cpw@sgi.com>
Cc: Jack Steiner <steiner@sgi.com>
LKML-Reference: <20100416175142.f4b59683.randy.dunlap@oracle.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'arch')
-rw-r--r-- | arch/x86/include/asm/uv/uv_hub.h | 2 | ||||
-rw-r--r-- | arch/x86/kernel/uv_irq.c | 12 |
2 files changed, 7 insertions, 7 deletions
diff --git a/arch/x86/include/asm/uv/uv_hub.h b/arch/x86/include/asm/uv/uv_hub.h index 14cc74ba5d23..bf6b88ef8eeb 100644 --- a/arch/x86/include/asm/uv/uv_hub.h +++ b/arch/x86/include/asm/uv/uv_hub.h | |||
@@ -307,7 +307,7 @@ static inline unsigned long uv_read_global_mmr32(int pnode, unsigned long offset | |||
307 | * Access Global MMR space using the MMR space located at the top of physical | 307 | * Access Global MMR space using the MMR space located at the top of physical |
308 | * memory. | 308 | * memory. |
309 | */ | 309 | */ |
310 | static inline unsigned long *uv_global_mmr64_address(int pnode, unsigned long offset) | 310 | static inline volatile void __iomem *uv_global_mmr64_address(int pnode, unsigned long offset) |
311 | { | 311 | { |
312 | return __va(UV_GLOBAL_MMR64_BASE | | 312 | return __va(UV_GLOBAL_MMR64_BASE | |
313 | UV_GLOBAL_MMR64_PNODE_BITS(pnode) | offset); | 313 | UV_GLOBAL_MMR64_PNODE_BITS(pnode) | offset); |
diff --git a/arch/x86/kernel/uv_irq.c b/arch/x86/kernel/uv_irq.c index ece73d8e3240..1a9f55a33489 100644 --- a/arch/x86/kernel/uv_irq.c +++ b/arch/x86/kernel/uv_irq.c | |||
@@ -43,7 +43,7 @@ static void uv_ack_apic(unsigned int irq) | |||
43 | ack_APIC_irq(); | 43 | ack_APIC_irq(); |
44 | } | 44 | } |
45 | 45 | ||
46 | struct irq_chip uv_irq_chip = { | 46 | static struct irq_chip uv_irq_chip = { |
47 | .name = "UV-CORE", | 47 | .name = "UV-CORE", |
48 | .startup = uv_noop_ret, | 48 | .startup = uv_noop_ret, |
49 | .shutdown = uv_noop, | 49 | .shutdown = uv_noop, |
@@ -140,7 +140,7 @@ int uv_irq_2_mmr_info(int irq, unsigned long *offset, int *pnode) | |||
140 | */ | 140 | */ |
141 | static int | 141 | static int |
142 | arch_enable_uv_irq(char *irq_name, unsigned int irq, int cpu, int mmr_blade, | 142 | arch_enable_uv_irq(char *irq_name, unsigned int irq, int cpu, int mmr_blade, |
143 | unsigned long mmr_offset, int restrict) | 143 | unsigned long mmr_offset, int limit) |
144 | { | 144 | { |
145 | const struct cpumask *eligible_cpu = cpumask_of(cpu); | 145 | const struct cpumask *eligible_cpu = cpumask_of(cpu); |
146 | struct irq_desc *desc = irq_to_desc(irq); | 146 | struct irq_desc *desc = irq_to_desc(irq); |
@@ -159,7 +159,7 @@ arch_enable_uv_irq(char *irq_name, unsigned int irq, int cpu, int mmr_blade, | |||
159 | if (err != 0) | 159 | if (err != 0) |
160 | return err; | 160 | return err; |
161 | 161 | ||
162 | if (restrict == UV_AFFINITY_CPU) | 162 | if (limit == UV_AFFINITY_CPU) |
163 | desc->status |= IRQ_NO_BALANCING; | 163 | desc->status |= IRQ_NO_BALANCING; |
164 | else | 164 | else |
165 | desc->status |= IRQ_MOVE_PCNTXT; | 165 | desc->status |= IRQ_MOVE_PCNTXT; |
@@ -213,7 +213,7 @@ static int uv_set_irq_affinity(unsigned int irq, const struct cpumask *mask) | |||
213 | unsigned long mmr_value; | 213 | unsigned long mmr_value; |
214 | struct uv_IO_APIC_route_entry *entry; | 214 | struct uv_IO_APIC_route_entry *entry; |
215 | unsigned long mmr_offset; | 215 | unsigned long mmr_offset; |
216 | unsigned mmr_pnode; | 216 | int mmr_pnode; |
217 | 217 | ||
218 | if (set_desc_affinity(desc, mask, &dest)) | 218 | if (set_desc_affinity(desc, mask, &dest)) |
219 | return -1; | 219 | return -1; |
@@ -247,7 +247,7 @@ static int uv_set_irq_affinity(unsigned int irq, const struct cpumask *mask) | |||
247 | * interrupt is raised. | 247 | * interrupt is raised. |
248 | */ | 248 | */ |
249 | int uv_setup_irq(char *irq_name, int cpu, int mmr_blade, | 249 | int uv_setup_irq(char *irq_name, int cpu, int mmr_blade, |
250 | unsigned long mmr_offset, int restrict) | 250 | unsigned long mmr_offset, int limit) |
251 | { | 251 | { |
252 | int irq, ret; | 252 | int irq, ret; |
253 | 253 | ||
@@ -257,7 +257,7 @@ int uv_setup_irq(char *irq_name, int cpu, int mmr_blade, | |||
257 | return -EBUSY; | 257 | return -EBUSY; |
258 | 258 | ||
259 | ret = arch_enable_uv_irq(irq_name, irq, cpu, mmr_blade, mmr_offset, | 259 | ret = arch_enable_uv_irq(irq_name, irq, cpu, mmr_blade, mmr_offset, |
260 | restrict); | 260 | limit); |
261 | if (ret == irq) | 261 | if (ret == irq) |
262 | uv_set_irq_2_mmr_info(irq, mmr_offset, mmr_blade); | 262 | uv_set_irq_2_mmr_info(irq, mmr_offset, mmr_blade); |
263 | else | 263 | else |