ipic_set_priority() has been unused since 2006 when the last usage was
removed in commit b9f0f1bb2bca ("[POWERPC] Adapt ipic driver to new
host_ops interface, add set_irq_type to set IRQ sense").

Reported-by: Dan Carpenter <dan.carpen...@oracle.com>
Signed-off-by: Michael Ellerman <m...@ellerman.id.au>
---
 arch/powerpc/include/asm/ipic.h |  1 -
 arch/powerpc/sysdev/ipic.c      | 28 ----------------------------
 2 files changed, 29 deletions(-)

v2: Also remove the declaration, as noticed by Thiago.

diff --git a/arch/powerpc/include/asm/ipic.h b/arch/powerpc/include/asm/ipic.h
index fb59829983b8..3dbd47f2bffe 100644
--- a/arch/powerpc/include/asm/ipic.h
+++ b/arch/powerpc/include/asm/ipic.h
@@ -69,7 +69,6 @@ enum ipic_mcp_irq {
        IPIC_MCP_MU   = 7,
 };
 
-extern int ipic_set_priority(unsigned int irq, unsigned int priority);
 extern void ipic_set_highest_priority(unsigned int irq);
 extern void ipic_set_default_priority(void);
 extern void ipic_enable_mcp(enum ipic_mcp_irq mcp_irq);
diff --git a/arch/powerpc/sysdev/ipic.c b/arch/powerpc/sysdev/ipic.c
index 6300123ce965..8030a0f55e96 100644
--- a/arch/powerpc/sysdev/ipic.c
+++ b/arch/powerpc/sysdev/ipic.c
@@ -771,34 +771,6 @@ struct ipic * __init ipic_init(struct device_node *node, 
unsigned int flags)
        return ipic;
 }
 
-int ipic_set_priority(unsigned int virq, unsigned int priority)
-{
-       struct ipic *ipic = ipic_from_irq(virq);
-       unsigned int src = virq_to_hw(virq);
-       u32 temp;
-
-       if (priority > 7)
-               return -EINVAL;
-       if (src > 127)
-               return -EINVAL;
-       if (ipic_info[src].prio == 0)
-               return -EINVAL;
-
-       temp = ipic_read(ipic->regs, ipic_info[src].prio);
-
-       if (priority < 4) {
-               temp &= ~(0x7 << (20 + (3 - priority) * 3));
-               temp |= ipic_info[src].prio_mask << (20 + (3 - priority) * 3);
-       } else {
-               temp &= ~(0x7 << (4 + (7 - priority) * 3));
-               temp |= ipic_info[src].prio_mask << (4 + (7 - priority) * 3);
-       }
-
-       ipic_write(ipic->regs, ipic_info[src].prio, temp);
-
-       return 0;
-}
-
 void ipic_set_highest_priority(unsigned int virq)
 {
        struct ipic *ipic = ipic_from_irq(virq);
-- 
2.17.2

Reply via email to