From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dirk Behme Date: Mon, 30 Mar 2009 15:39:31 +0200 Subject: [U-Boot] [PATCH 13/19] omap3: remove duplicate interrupts code In-Reply-To: <1238360583-19791-13-git-send-email-plagnioj@jcrosoft.com> References: <1238360583-19791-1-git-send-email-plagnioj@jcrosoft.com> <1238360583-19791-2-git-send-email-plagnioj@jcrosoft.com> <1238360583-19791-3-git-send-email-plagnioj@jcrosoft.com> <1238360583-19791-4-git-send-email-plagnioj@jcrosoft.com> <1238360583-19791-5-git-send-email-plagnioj@jcrosoft.com> <1238360583-19791-6-git-send-email-plagnioj@jcrosoft.com> <1238360583-19791-7-git-send-email-plagnioj@jcrosoft.com> <1238360583-19791-8-git-send-email-plagnioj@jcrosoft.com> <1238360583-19791-9-git-send-email-plagnioj@jcrosoft.com> <1238360583-19791-10-git-send-email-plagnioj@jcrosoft.com> <1238360583-19791-11-git-send-email-plagnioj@jcrosoft.com> <1238360583-19791-12-git-send-email-plagnioj@jcrosoft.com> <1238360583-19791-13-git-send-email-plagnioj@jcrosoft.com> Message-ID: <49D0CB93.1060005@googlemail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Dear Jean-Christophe, Jean-Christophe PLAGNIOL-VILLARD wrote: > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD > --- > cpu/arm_cortexa8/omap3/interrupts.c | 128 ----------------------------------- > 1 files changed, 0 insertions(+), 128 deletions(-) It seems to me that this patch is a re-post of http://lists.denx.de/pipermail/u-boot/2009-March/049736.html ? If this is the case, it would be helpful if you could mark re-send patches with 'v2' or something similar, explain what changed (or what's unchanged) and that the previous patch is obsolete with the re-post. Dirk Btw.: A description in the body of a patch (and not subject only) going into git would be nice, too. E.g. "Remove interrupt code which is already in xxx and therefore not needed here" > diff --git a/cpu/arm_cortexa8/omap3/interrupts.c b/cpu/arm_cortexa8/omap3/interrupts.c > index 9e9817d..384ffdc 100644 > --- a/cpu/arm_cortexa8/omap3/interrupts.c > +++ b/cpu/arm_cortexa8/omap3/interrupts.c > @@ -34,137 +34,9 @@ > > #include > #include > -#include > > #define TIMER_LOAD_VAL 0 > > -#ifdef CONFIG_USE_IRQ > -/* enable IRQ interrupts */ > -void enable_interrupts(void) > -{ > - unsigned long temp; > - __asm__ __volatile__("mrs %0, cpsr\n" > - "bic %0, %0, #0x80\n" "msr cpsr_c, %0":"=r"(temp) > - ::"memory"); > -} > - > -/* > - * disable IRQ/FIQ interrupts > - * returns true if interrupts had been enabled before we disabled them > - */ > -int disable_interrupts(void) > -{ > - unsigned long old, temp; > - __asm__ __volatile__("mrs %0, cpsr\n" > - "orr %1, %0, #0xc0\n" > - "msr cpsr_c, %1":"=r"(old), "=r"(temp) > - ::"memory"); > - return (old & 0x80) == 0; > -} > -#else > -void enable_interrupts(void) > -{ > - return; > -} > -int disable_interrupts(void) > -{ > - return 0; > -} > -#endif > - > -void bad_mode(void) > -{ > - panic("Resetting CPU ...\n"); > - reset_cpu(0); > -} > - > -void show_regs(struct pt_regs *regs) > -{ > - unsigned long flags; > - const char *processor_modes[] = { > - "USER_26", "FIQ_26", "IRQ_26", "SVC_26", > - "UK4_26", "UK5_26", "UK6_26", "UK7_26", > - "UK8_26", "UK9_26", "UK10_26", "UK11_26", > - "UK12_26", "UK13_26", "UK14_26", "UK15_26", > - "USER_32", "FIQ_32", "IRQ_32", "SVC_32", > - "UK4_32", "UK5_32", "UK6_32", "ABT_32", > - "UK8_32", "UK9_32", "UK10_32", "UND_32", > - "UK12_32", "UK13_32", "UK14_32", "SYS_32", > - }; > - > - flags = condition_codes(regs); > - > - printf("pc : [<%08lx>] lr : [<%08lx>]\n" > - "sp : %08lx ip : %08lx fp : %08lx\n", > - instruction_pointer(regs), > - regs->ARM_lr, regs->ARM_sp, regs->ARM_ip, regs->ARM_fp); > - printf("r10: %08lx r9 : %08lx r8 : %08lx\n", > - regs->ARM_r10, regs->ARM_r9, regs->ARM_r8); > - printf("r7 : %08lx r6 : %08lx r5 : %08lx r4 : %08lx\n", > - regs->ARM_r7, regs->ARM_r6, regs->ARM_r5, regs->ARM_r4); > - printf("r3 : %08lx r2 : %08lx r1 : %08lx r0 : %08lx\n", > - regs->ARM_r3, regs->ARM_r2, regs->ARM_r1, regs->ARM_r0); > - printf("Flags: %c%c%c%c", > - flags & CC_N_BIT ? 'N' : 'n', > - flags & CC_Z_BIT ? 'Z' : 'z', > - flags & CC_C_BIT ? 'C' : 'c', flags & CC_V_BIT ? 'V' : 'v'); > - printf(" IRQs %s FIQs %s Mode %s%s\n", > - interrupts_enabled(regs) ? "on" : "off", > - fast_interrupts_enabled(regs) ? "on" : "off", > - processor_modes[processor_mode(regs)], > - thumb_mode(regs) ? " (T)" : ""); > -} > - > -void do_undefined_instruction(struct pt_regs *pt_regs) > -{ > - printf("undefined instruction\n"); > - show_regs(pt_regs); > - bad_mode(); > -} > - > -void do_software_interrupt(struct pt_regs *pt_regs) > -{ > - printf("software interrupt\n"); > - show_regs(pt_regs); > - bad_mode(); > -} > - > -void do_prefetch_abort(struct pt_regs *pt_regs) > -{ > - printf("prefetch abort\n"); > - show_regs(pt_regs); > - bad_mode(); > -} > - > -void do_data_abort(struct pt_regs *pt_regs) > -{ > - printf("data abort\n"); > - show_regs(pt_regs); > - bad_mode(); > -} > - > -void do_not_used(struct pt_regs *pt_regs) > -{ > - printf("not used\n"); > - show_regs(pt_regs); > - bad_mode(); > -} > - > -void do_fiq(struct pt_regs *pt_regs) > -{ > - printf("fast interrupt request\n"); > - show_regs(pt_regs); > - bad_mode(); > -} > - > -void do_irq(struct pt_regs *pt_regs) > -{ > - printf("interrupt request\n"); > - show_regs(pt_regs); > - bad_mode(); > -} > - > - > static ulong timestamp; > static ulong lastinc; > static gptimer_t *timer_base = (gptimer_t *)CONFIG_SYS_TIMERBASE;