Re: [U-Boot] [PATCH] KGDB set / remove breakpoints

Thanks for the review, Mike. Please note, after applying this patch, those architectures, which already enabled KGDB support, have to create a new asm/kgdb.h and define the length of the break instruction (BREAK_INSTR_SIZE) in that file. i dont think breaking build is a good idea.  i would

[U-Boot] [PATCH] KGDB set / remove breakpoints

applying this patch, those architectures, which already enabled KGDB support, have to create a new asm/kgdb.h and define the length of the break instruction (BREAK_INSTR_SIZE) in that file. Signed-off-by: Tonny Tzeng tonny.tz...@gmail.com --- common/kgdb.c | 85

[U-Boot] [PATCH v2 2/4] [ARM] Add KGDB support for ARM platforms

Signed-off-by: Tonny Tzeng tonny.tz...@gmail.com --- arch/arm/lib/interrupts.c |9 + 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/arch/arm/lib/interrupts.c b/arch/arm/lib/interrupts.c index 1f2b815..6ee4309 100644 --- a/arch/arm/lib/interrupts.c +++ b/arch/arm/lib

[U-Boot] [PATCH v2 3/4] [ARM] Add KGDB support for ARM platforms

Signed-off-by: Tonny Tzeng tonny.tz...@gmail.com --- arch/arm/include/asm/kgdb.h | 66 +++ arch/arm/lib/Makefile |1 + arch/arm/lib/kgdb.c | 194 +++ 3 files changed, 261 insertions(+), 0 deletions(-) create mode 100644

[U-Boot] [PATCH v2 4/4] [ARM] Add KGDB support for ARM platforms

Signed-off-by: Tonny Tzeng tonny.tz...@gmail.com --- arch/arm/cpu/arm720t/start.S |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/arch/arm/cpu/arm720t/start.S b/arch/arm/cpu/arm720t/start.S index 90b7240..fd06298 100644 --- a/arch/arm/cpu/arm720t/start.S +++ b/arch

[U-Boot] [PATCH v2 1/4] [ARM] Add KGDB support for ARM platforms

Signed-off-by: Tonny Tzeng tonny.tz...@gmail.com --- arch/arm/lib/board.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/arch/arm/lib/board.c b/arch/arm/lib/board.c index f5660a9..f2b028c 100644 --- a/arch/arm/lib/board.c +++ b/arch/arm/lib/board.c @@ -381,6 +381,10

[U-Boot] [PATCH v2 0/4] [ARM] Add KGDB support for ARM platforms

This patchset adds KGDB support for ARM platforms. Since the KGDB support for ARM is through the undef instruction handler, we need to add code to the startup file to restore the user context after exception. Unfortunately, each ARM variant has his own start.S, so please reference the patch

Re: [U-Boot] [PATCH] Add KGDB support for ARM platforms

Mike, thanks for your review. Here need your further clarifications, thanks in advance. On Fri, Apr 9, 2010 at 5:36 AM, Mike Frysinger vap...@gentoo.org wrote: On Thursday 08 April 2010 05:40:02 Tonny Tzeng wrote: --- a/common/kgdb.c +++ b/common/kgdb.c @@ -220,6 +220,29 @@ hexToInt(char

[U-Boot] [PATCH] Add KGDB support for ARM platforms

These patches add kgdb support for ARM platforms. The add and invocation of the bad_restore_user_regs macro in cpu/arm720t/start.S should be made to other cpu/arm*/start.S files as well. Signed-off-by: Tonny Tzeng tonny.tz...@gmail.com ---  common/kgdb.c          |   30 ++  cpu/arm720t

[U-Boot] [PATCH] Add KGDB support for ARM platforms

These patches add kgdb support for ARM platforms. The add and invocation of the bad_restore_user_regs macro in cpu/arm720t/start.S should be made to other cpu/arm*/start.S files as well. Signed-off-by: Tonny Tzeng tonny.tz...@gmail.com --- common/kgdb.c | 30 ++ cpu/arm720t