[U-Boot] [PATCH 05/12] microblaze: intc: Coding style cleanup

Simon Glass sjg at chromium.org
Mon Jul 9 23:23:33 CEST 2012


On Mon, Jul 9, 2012 at 2:20 AM, Michal Simek <monstr at monstr.eu> wrote:

> Just coding style cleanup.
> Remove unneeded externs.
>
> Signed-off-by: Michal Simek <monstr at monstr.eu>
>

Acked-by: Simon Glass <sjg at chromium.org>


> ---
>  arch/microblaze/cpu/interrupts.c |   26 +++++++++++++-------------
>  1 files changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/arch/microblaze/cpu/interrupts.c
> b/arch/microblaze/cpu/interrupts.c
> index 871cefb..79ee96a 100644
> --- a/arch/microblaze/cpu/interrupts.c
> +++ b/arch/microblaze/cpu/interrupts.c
> @@ -32,15 +32,12 @@
>
>  #undef DEBUG_INT
>
> -extern void microblaze_disable_interrupts (void);
> -extern void microblaze_enable_interrupts (void);
> -
> -void enable_interrupts (void)
> +void enable_interrupts(void)
>  {
>         MSRSET(0x2);
>  }
>
> -int disable_interrupts (void)
> +int disable_interrupts(void)
>  {
>         unsigned int msr;
>
> @@ -58,20 +55,21 @@ microblaze_intc_t *intc;
>  /* default handler */
>  static void def_hdlr(void)
>  {
> -       puts ("def_hdlr\n");
> +       puts("def_hdlr\n");
>  }
>
>  static void enable_one_interrupt(int irq)
>  {
>         int mask;
>         int offset = 1;
> +
>         offset <<= irq;
>         mask = intc->ier;
>         intc->ier = (mask | offset);
>  #ifdef DEBUG_INT
> -       printf ("Enable one interrupt irq %x - mask %x,ier %x\n", offset,
> mask,
> +       printf("Enable one interrupt irq %x - mask %x,ier %x\n", offset,
> mask,
>                 intc->ier);
> -       printf ("INTC isr %x, ier %x, iar %x, mer %x\n", intc->isr,
> intc->ier,
> +       printf("INTC isr %x, ier %x, iar %x, mer %x\n", intc->isr,
> intc->ier,
>                 intc->iar, intc->mer);
>  #endif
>  }
> @@ -80,13 +78,14 @@ static void disable_one_interrupt(int irq)
>  {
>         int mask;
>         int offset = 1;
> +
>         offset <<= irq;
>         mask = intc->ier;
>         intc->ier = (mask & ~offset);
>  #ifdef DEBUG_INT
> -       printf ("Disable one interrupt irq %x - mask %x,ier %x\n", irq,
> mask,
> +       printf("Disable one interrupt irq %x - mask %x,ier %x\n", irq,
> mask,
>                 intc->ier);
> -       printf ("INTC isr %x, ier %x, iar %x, mer %x\n", intc->isr,
> intc->ier,
> +       printf("INTC isr %x, ier %x, iar %x, mer %x\n", intc->isr,
> intc->ier,
>                 intc->iar, intc->mer);
>  #endif
>  }
> @@ -95,9 +94,10 @@ static void disable_one_interrupt(int irq)
>  int install_interrupt_handler(int irq, interrupt_handler_t *hdlr, void
> *arg)
>  {
>         struct irq_action *act;
> +
>         /* irq out of range */
>         if ((irq < 0) || (irq > irq_no)) {
> -               puts ("IRQ out of range\n");
> +               puts("IRQ out of range\n");
>                 return -1;
>         }
>         act = &vecs[irq];
> @@ -125,7 +125,7 @@ static void intc_init(void)
>         /* XIntc_Start - hw_interrupt enable and all interrupt enable */
>         intc->mer = 0x3;
>  #ifdef DEBUG_INT
> -       printf ("INTC isr %x, ier %x, iar %x, mer %x\n", intc->isr,
> intc->ier,
> +       printf("INTC isr %x, ier %x, iar %x, mer %x\n", intc->isr,
> intc->ier,
>                 intc->iar, intc->mer);
>  #endif
>  }
> @@ -160,7 +160,7 @@ int interrupts_init(void)
>         return 0;
>  }
>
> -void interrupt_handler (void)
> +void interrupt_handler(void)
>  {
>         int irqs = intc->ivr;   /* find active interrupt */
>         int mask = 1;
> --
> 1.7.0.4
>
>


More information about the U-Boot mailing list