From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephan Linz Date: Tue, 07 Aug 2012 22:10:37 +0200 Subject: [U-Boot] [PATCH v2 4/7] microblaze: intc: Coding style cleanup In-Reply-To: <1344239199-11445-4-git-send-email-monstr@monstr.eu> References: <1344239199-11445-1-git-send-email-monstr@monstr.eu> <1344239199-11445-4-git-send-email-monstr@monstr.eu> Message-ID: <1344370238.29456.89.camel@keto> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Am Montag, den 06.08.2012, 09:46 +0200 schrieb Michal Simek: > Just coding style cleanup. > Remove unneeded externs. > > Signed-off-by: Michal Simek > Acked-by: Simon Glass > Acked-by: Stephan Linz > -- > v2: Rebase - depends on previous intc patch > --- > 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 08f6bad..7f2ee64 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 > } > @@ -94,9 +93,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]; > @@ -124,7 +124,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 > } > @@ -159,7 +159,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; -- Viele Gr??e, Stephan Linz ______________________________________________________________________________ MB-Ref: http://www.li-pro.de/xilinx_mb:mbref:start OpenDCC: http://www.li-pro.net/opendcc.phtml PC/M: http://www.li-pro.net/pcm.phtml Sourceforge: http://sourceforge.net/users/slz Gitorious: https://gitorious.org/~slz