Without further confirmation of this patch's correctness, I do not want
to add it until after the 0.2.0 branch release.  I have been trying to
push that out for most of the week, but it would be good to include this
fix if it is indeed required.

Can you perhaps provide a little more explanation regarding why you
believe this patch to be correct?  Sorry for having to ask, but I need
to learn the details in order to make an informed decision about it.

On Fri, 2009-07-03 at 22:16 +0200, Ferdinand Postema wrote:
> Hello,
> 
> I found the bug by myself. I have made a patch file to correct it.
> Is it possible to incorporate this patch into the source code?
> 
> Ferdinand Postema
> 
> 
> Ferdinand Postema schreef:
> > First I want to say that I am very happy with the OpenOCD-software! I 
> > like it very much.
> >
> > I have a Chameleon POD from Amontec. This dongle can be programmed to 
> > act as a Wiggler-cable, but also as a JTAG Accelerator interface.
> > I use it in combination with an ARM processor and a FPGA. Both are 
> > supplied by Propox.
> >
> > When I use the Wiggler JTAG interface, I get the following information:
> >    Info : JTAG tap: at91sam9260.cpu tap/device found: 0x0792603f 
> > (Manufacturer: 0x01f, Part: 0x7926, Version: 0x0)
> > When I use the Amontec JTAG Accelerator Interface, I get the following 
> > information:
> >    Info : JTAG tap: at91sam9260.cpu tap/device found: 0x03c9301f 
> > (Manufacturer: 0x00f, Part: 0x3c93, Version: 0x0)
> > It looks like the whole word is shifted 1 bit. I think the Wiggler 
> > interface is correct.
> >
> > I also tried my FPGA module and got the following ID's:
> >                    Manuf.  Chip        wiggler     amtjtagaccel
> >    Processor:      Atmel   AT91SAM9260 0x0792603f  0x03c9301f
> >    Platform Flash: Xilinx  XCF01S      0xF5044093  0x7A822049
> >    FPGA:           Xilinx  XC3S200     0x01414093  0x80A0A049
> >
> > The ID of the FPGA is not only shifted 1 bit to the right, but is also 
> > OR-ed with 0x80000000
> > The wiggler ID is correct
> >
> > Can you correct this?
> >
> > Kind regards,
> >
> > Ferdinand Postema
> > (The Netherlands)
> >
> plain text document attachment (openocd_wrong_id_code.patch)
> Index: src/jtag/amt_jtagaccel.c
> ===================================================================
> --- src/jtag/amt_jtagaccel.c  (revision 2461)
> +++ src/jtag/amt_jtagaccel.c  (working copy)
> @@ -92,8 +92,8 @@
>  static uint8_t amt_jtagaccel_tap_move[6][6][2] =
>  {
>       /*         RESET         IDLE        DRSHIFT       DRPAUSE       
> IRSHIFT       IRPAUSE             */
> -     {{0x1f, 0x00}, {0x0f, 0x00}, {0x8a, 0x04}, {0x0a, 0x00}, {0x06, 0x00}, 
> {0x96, 0x00}},   /* RESET */
> -     {{0x1f, 0x00}, {0x00, 0x00}, {0x85, 0x08}, {0x05, 0x00}, {0x8b, 0x08}, 
> {0x0b, 0x00}},   /* IDLE */
> +     {{0x1f, 0x00}, {0x0f, 0x00}, {0x05, 0x00}, {0x0a, 0x00}, {0x06, 0x00}, 
> {0x96, 0x00}},   /* RESET */
> +     {{0x1f, 0x00}, {0x00, 0x00}, {0x04, 0x00}, {0x05, 0x00}, {0x06, 0x00}, 
> {0x0b, 0x00}},   /* IDLE */
>       {{0x1f, 0x00}, {0x0d, 0x00}, {0x00, 0x00}, {0x01, 0x00}, {0x8f, 0x09}, 
> {0x8f, 0x01}},   /* DRSHIFT  */
>       {{0x1f, 0x00}, {0x0c, 0x00}, {0x08, 0x00}, {0x00, 0x00}, {0x8f, 0x09}, 
> {0x8f, 0x01}},   /* DRPAUSE  */
>       {{0x1f, 0x00}, {0x0d, 0x00}, {0x07, 0x00}, {0x97, 0x00}, {0x00, 0x00}, 
> {0x01, 0x00}},   /* IRSHIFT  */
> _______________________________________________
> Openocd-development mailing list
> Openocd-development@lists.berlios.de
> https://lists.berlios.de/mailman/listinfo/openocd-development

_______________________________________________
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to