Skip to content

Commit

Permalink
sparc32: fix sparse warnings in sun4d_irq.c
Browse files Browse the repository at this point in the history
Fix following warnings:
sun4d_irq.c:146:6: warning: symbol 'sun4d_handler_irq' was not declared. Should it be static?
sun4d_irq.c:239:17: warning: symbol 'sun4d_irq' was not declared. Should it be static?
sun4d_irq.c:288:14: warning: symbol '_sun4d_build_device_irq' was not declared. Should it be static?
sun4d_irq.c:323:14: warning: symbol 'sun4d_build_device_irq' was not declared. Should it be static?
sun4d_irq.c:386:14: warning: symbol 'sun4d_build_timer_irq' was not declared. Should it be static?
sun4d_irq.c:482:13: warning: symbol 'sun4d_init_sbi_irq' was not declared. Should it be static?

Apply static when applicable, otherwise add prototype

Signed-off-by: Sam Ravnborg <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
  • Loading branch information
sravnborg authored and davem330 committed Apr 29, 2014
1 parent 2b39917 commit 5ac7568
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 8 deletions.
1 change: 1 addition & 0 deletions arch/sparc/include/asm/irq_32.h
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
#define irq_canonicalize(irq) (irq)

extern void __init init_IRQ(void);
void __init sun4d_init_sbi_irq(void);

#define NO_IRQ 0xffffffff

Expand Down
3 changes: 3 additions & 0 deletions arch/sparc/kernel/irq.h
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,9 @@ unsigned long leon_get_irqmask(unsigned int irq);
/* sun4m_irq.c */
void sun4m_nmi(struct pt_regs *regs);

/* sun4d_irq.c */
void sun4d_handler_irq(unsigned int pil, struct pt_regs *regs);

#ifdef CONFIG_SMP

/* All SUN4D IPIs are sent on this IRQ, may be shared with hard IRQs */
Expand Down
17 changes: 9 additions & 8 deletions arch/sparc/kernel/sun4d_irq.c
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ static void sun4d_sbus_handler_irq(int sbusl)
}
}

void sun4d_handler_irq(int pil, struct pt_regs *regs)
void sun4d_handler_irq(unsigned int pil, struct pt_regs *regs)
{
struct pt_regs *old_regs;
/* SBUS IRQ level (1 - 7) */
Expand Down Expand Up @@ -236,7 +236,7 @@ static void sun4d_shutdown_irq(struct irq_data *data)
irq_unlink(data->irq);
}

struct irq_chip sun4d_irq = {
static struct irq_chip sun4d_irq = {
.name = "sun4d",
.irq_startup = sun4d_startup_irq,
.irq_shutdown = sun4d_shutdown_irq,
Expand Down Expand Up @@ -285,9 +285,9 @@ static void __init sun4d_load_profile_irqs(void)
}
}

unsigned int _sun4d_build_device_irq(unsigned int real_irq,
unsigned int pil,
unsigned int board)
static unsigned int _sun4d_build_device_irq(unsigned int real_irq,
unsigned int pil,
unsigned int board)
{
struct sun4d_handler_data *handler_data;
unsigned int irq;
Expand Down Expand Up @@ -320,8 +320,8 @@ unsigned int _sun4d_build_device_irq(unsigned int real_irq,



unsigned int sun4d_build_device_irq(struct platform_device *op,
unsigned int real_irq)
static unsigned int sun4d_build_device_irq(struct platform_device *op,
unsigned int real_irq)
{
struct device_node *dp = op->dev.of_node;
struct device_node *board_parent, *bus = dp->parent;
Expand Down Expand Up @@ -383,7 +383,8 @@ unsigned int sun4d_build_device_irq(struct platform_device *op,
return irq;
}

unsigned int sun4d_build_timer_irq(unsigned int board, unsigned int real_irq)
static unsigned int sun4d_build_timer_irq(unsigned int board,
unsigned int real_irq)
{
return _sun4d_build_device_irq(real_irq, real_irq, board);
}
Expand Down

0 comments on commit 5ac7568

Please sign in to comment.