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"); > >> > >> +

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 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 **ptr, int *in

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 *kd

[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 ---  common/kgdb.c          |   30 ++  cpu/arm720t/start.S    |    7 ++  inclu

[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 --- common/kgdb.c | 30 ++ cpu/arm720t/start.S|7 ++ inclu