diff options
author | David S. Miller <davem@davemloft.net> | 2008-08-25 19:44:58 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-08-25 19:44:58 -0400 |
commit | 51e0f004a9ab9104acbe323c0b20e0279bf9be85 (patch) | |
tree | 477ccb9efa52eff7531cd04dc605bbf9e90152a4 | |
parent | 783c98b911fce8d47aa2906468ca39d44d46d7ce (diff) |
sparc64: Fix irq_of_parse_and_map() and irq_dispose_mapping().
Stephen Rothwell noticed that I committed an earlier version
of the patch that didn't have two things fixed:
1) irq_of_parse_and_map() should return "unsigned int" not "int"
and it should return zero for "no irq"
2) irq_dispose_mapping() should be an inline function, not a macro,
for type checking
With feedback and suggestions from Anton Vorontsov.
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | arch/sparc/include/asm/prom.h | 6 | ||||
-rw-r--r-- | arch/sparc/kernel/of_device.c | 4 | ||||
-rw-r--r-- | arch/sparc64/kernel/of_device.c | 4 |
3 files changed, 8 insertions, 6 deletions
diff --git a/arch/sparc/include/asm/prom.h b/arch/sparc/include/asm/prom.h index 58b85fa56066..900d44714f8d 100644 --- a/arch/sparc/include/asm/prom.h +++ b/arch/sparc/include/asm/prom.h | |||
@@ -101,8 +101,10 @@ static inline void of_node_put(struct device_node *node) | |||
101 | * register them in the of_device objects, whereas powerpc computes them | 101 | * register them in the of_device objects, whereas powerpc computes them |
102 | * on request. | 102 | * on request. |
103 | */ | 103 | */ |
104 | extern int irq_of_parse_and_map(struct device_node *node, int index); | 104 | extern unsigned int irq_of_parse_and_map(struct device_node *node, int index); |
105 | #define irq_dispose_mapping(irq) do { } while (0) | 105 | static inline void irq_dispose_mapping(unsigned int virq) |
106 | { | ||
107 | } | ||
106 | 108 | ||
107 | /* | 109 | /* |
108 | * NB: This is here while we transition from using asm/prom.h | 110 | * NB: This is here while we transition from using asm/prom.h |
diff --git a/arch/sparc/kernel/of_device.c b/arch/sparc/kernel/of_device.c index 56e9a718ef8f..aace71f7d6e5 100644 --- a/arch/sparc/kernel/of_device.c +++ b/arch/sparc/kernel/of_device.c | |||
@@ -29,12 +29,12 @@ struct of_device *of_find_device_by_node(struct device_node *dp) | |||
29 | } | 29 | } |
30 | EXPORT_SYMBOL(of_find_device_by_node); | 30 | EXPORT_SYMBOL(of_find_device_by_node); |
31 | 31 | ||
32 | int irq_of_parse_and_map(struct device_node *node, int index) | 32 | unsigned int irq_of_parse_and_map(struct device_node *node, int index) |
33 | { | 33 | { |
34 | struct of_device *op = of_find_device_by_node(node); | 34 | struct of_device *op = of_find_device_by_node(node); |
35 | 35 | ||
36 | if (!op || index >= op->num_irqs) | 36 | if (!op || index >= op->num_irqs) |
37 | return 0xffffffff; | 37 | return 0; |
38 | 38 | ||
39 | return op->irqs[index]; | 39 | return op->irqs[index]; |
40 | } | 40 | } |
diff --git a/arch/sparc64/kernel/of_device.c b/arch/sparc64/kernel/of_device.c index 8a0d82343a25..a30f2af0bf2c 100644 --- a/arch/sparc64/kernel/of_device.c +++ b/arch/sparc64/kernel/of_device.c | |||
@@ -55,12 +55,12 @@ struct of_device *of_find_device_by_node(struct device_node *dp) | |||
55 | } | 55 | } |
56 | EXPORT_SYMBOL(of_find_device_by_node); | 56 | EXPORT_SYMBOL(of_find_device_by_node); |
57 | 57 | ||
58 | int irq_of_parse_and_map(struct device_node *node, int index) | 58 | unsigned int irq_of_parse_and_map(struct device_node *node, int index) |
59 | { | 59 | { |
60 | struct of_device *op = of_find_device_by_node(node); | 60 | struct of_device *op = of_find_device_by_node(node); |
61 | 61 | ||
62 | if (!op || index >= op->num_irqs) | 62 | if (!op || index >= op->num_irqs) |
63 | return 0xffffffff; | 63 | return 0; |
64 | 64 | ||
65 | return op->irqs[index]; | 65 | return op->irqs[index]; |
66 | } | 66 | } |