[Openocd-development] [PATCH 04/12] at91rm9200: use register_commands()

2009-11-21 Thread Zachary T Welch
Use register_commands() with command registration array. --- This module was broken by previous changes, but no one has complained. Are there still users for this modules? Signed-off-by: Zachary T Welch z...@superlucidity.net --- src/jtag/at91rm9200.c | 15 --- 1 files changed, 12

Re: [Openocd-development] [PATCH 04/12] at91rm9200: use register_commands()

2009-11-21 Thread David Brownell
On Saturday 21 November 2009, Zachary T Welch wrote: +static const struct command_registration at91rm9200_command_handlers[] = { +   { +   .name = at91rm9200_device, +   .handler = at91rm9200_handle_device_command, +   

Re: [Openocd-development] [PATCH 04/12] at91rm9200: use register_commands()

2009-11-21 Thread Zach Welch
On Sat, 2009-11-21 at 16:24 -0700, David Brownell wrote: On Saturday 21 November 2009, Zachary T Welch wrote: +static const struct command_registration at91rm9200_command_handlers[] = { + { + .name = at91rm9200_device, + .handler =

Re: [Openocd-development] [PATCH 04/12] at91rm9200: use register_commands()

2009-11-21 Thread David Brownell
On Saturday 21 November 2009, Zach Welch wrote: Could we get one less level of indent throughout? There a couple of arguments for using this style at the top-level: - It's consistent with the other indenting.  The statement has not been completed, so the code should not return to the