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

2010-04-09 Thread Tonny Tzeng
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

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

2010-04-09 Thread Mike Frysinger
On Friday 09 April 2010 11:32:19 Tonny Tzeng wrote: On Fri, Apr 9, 2010 at 5:36 AM, Mike Frysinger wrote: On Thursday 08 April 2010 05:40:02 Tonny Tzeng wrote: if (kgdb_setjmp(error_jmp_buf) != 0) panic(kgdb: error or fault in entry init!\n); + memset(kd, 0,

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

2010-04-08 Thread Tonny Tzeng
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 ++  

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

2010-04-08 Thread Mike Frysinger
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 **ptr, int *intValue) return (numChars); } +/* Handle the 'z' or 'Z' breakpoint remove or set packets */ +static void gdb_cmd_break(kgdb_data *kdp)

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

2010-04-07 Thread Tonny Tzeng
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 ++