Steve,

Steve McKown wrote:
> PFLASH programming works with the patch below.

Please see the file HACKING in the source tree for instructions on
how you push the commit into Gerrit.


> diff --git c/src/jtag/drivers/osbdm.c w/src/jtag/drivers/osbdm.c
> index 2560db1..15ffad9 100644
> --- c/src/jtag/drivers/osbdm.c
> +++ w/src/jtag/drivers/osbdm.c
> @@ -130,8 +130,8 @@ static struct queue *queue_alloc(void)
> 
>   /* Lists of valid VID/PID pairs
>    */
> -static const uint16_t osbdm_vid[] = { 0x15a2, 0x15a2, 0 };
> -static const uint16_t osbdm_pid[] = { 0x0042, 0x0058, 0 };
> +static const uint16_t osbdm_vid[] = { 0x15a2, 0x15a2, 0x15a2, 0 };
> +static const uint16_t osbdm_pid[] = { 0x0042, 0x0058, 0x005e, 0 };

Please kill this madness right here. Change this to be one entry per
line, so that the next vidpid does not have to clobber all previous
history. It's just bad style to keep something broken like this. It's
completely unneccessary. It would be great if you could fix it since
you are touching this code anyway. Thanks!


//Peter

------------------------------------------------------------------------------
Precog is a next-generation analytics platform capable of advanced
analytics on semi-structured data. The platform includes APIs for building
apps and a phenomenal toolset for data science. Developers can use
our toolset for easy data analysis & visualization. Get a free account!
http://www2.precog.com/precogplatform/slashdotnewsletter
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to