Re: [Openocd-development] [PATCH] Update ep93xx and at91rm9200 drivers

2010-10-02 Thread Luca Bruno
Zach Welch scrisse: > Since I was the one that did that work and overlooked these files, I > have reviewed and committed the fixes that you provided. Sorry for the > game delay. Thanks a lot, openocd is now properly built on all supported Debian arch: https://buildd.debian.org/pkg.cgi?pkg=openoc

Re: [Openocd-development] [PATCH] Update ep93xx and at91rm9200 drivers

2010-09-30 Thread Zach Welch
On 09/30/2010 01:50 PM, Luca Bruno wrote: > Luca Bruno scrisse: > >> ep93xx and at91rm9200 are conditionally built only on arm and were not >> updated to reflect changes in command registration handler. >> This patch makes them properly compile again, fixing a build failure >> experienced on Debia

Re: [Openocd-development] [PATCH] Update ep93xx and at91rm9200 drivers

2010-09-30 Thread Luca Bruno
Luca Bruno scrisse: > ep93xx and at91rm9200 are conditionally built only on arm and were not > updated to reflect changes in command registration handler. > This patch makes them properly compile again, fixing a build failure > experienced on Debian armel. Before integrating this patch in our dow

Re: [Openocd-development] [PATCH] Update ep93xx and at91rm9200 drivers

2010-09-29 Thread Luca Bruno
Hector Oron scrisse: > I have also been playing with this bug. Just one comment on the > ep93xx.c > > > -static int ep93xx_register_commands(struct command_context > > *cmd_ctx); > > > > -       .register_commands = ep93xx_register_commands, > > Why not just use the right value in the struct, >

Re: [Openocd-development] [PATCH] Update ep93xx and at91rm9200 drivers

2010-09-28 Thread Hector Oron
Hello Luca, I have also been playing with this bug. Just one comment on the ep93xx.c > diff --git a/src/jtag/drivers/ep93xx.c b/src/jtag/drivers/ep93xx.c > index 0959a56..09312c5 100644 > --- a/src/jtag/drivers/ep93xx.c > +++ b/src/jtag/drivers/ep93xx.c > @@ -47,7 +47,6 @@ static void ep93xx_writ

[Openocd-development] [PATCH] Update ep93xx and at91rm9200 drivers

2010-09-27 Thread Luca Bruno
ep93xx and at91rm9200 are conditionally built only on arm and were not updated to reflect changes in command registration handler. This patch makes them properly compile again, fixing a build failure experienced on Debian armel. Signed-off-by: Luca Bruno --- src/jtag/drivers/at91rm9200.c | 30