Re: [flashrom] [PATCH] Add Altera USB-Blaster SPI programmer.

[Kyösti Mälkki]

On Mon, 2012-10-29 at 01:33 +0100, Stefan Tauner wrote:
> Adds support for the Altera USB-Blaster programming dongle in
> Active Serial (AS) mode.
> Tested on both original product and a clone dongle.
>
> Signed-off-by: James Laird 
> Signed-off-by: Kyösti Mälkki 
> Signed-off-by: Stefan Tauner
>
> Rebased on Kyösti's version.
> Fixed Makefile and whitespace, cleaned up a few things,
> added a manpage... please retest on hardware.
>
> Can someone please explain to me why send_read() does what it does?

Thanks, your clean-up looks good.

Tested on both a clone with C8051F321 and the original product.

The blaster design is described here:
http://sourceforge.net/apps/mediawiki/urjtag/index.php?title=Cable_Altera_US...

As for send_read(), to read a byte from SPI, one must first write a byte
to the USB. Stupid, but that is the design. This hardware has
particularly bad performance penalty when reading in small chunks of
pagesize or 256 bytes.

With non-blocking bulk submits in libusb-1.0, the raw read speed can be
increased by factor of 2 for clone and by 5 for original product.

Regards,
KM

[PATCH] Add Altera USB-Blaster SPI programmer.

[Stefan Tauner]

On Sun, 04 Nov 2012 23:48:41 +0200
Kyösti Mälkki  wrote:
> Thanks, your clean-up looks good.
>
> Tested on both a clone with C8051F321 and the original product.

great.
the only remaining problem now is the makefile. the current scheme does
not work as intended.
without the patch this works:
make distclean; make -j1 CONFIG_INTERNAL=no CONFIG_SERPROG=no CONFIG_RAYER_SPI=no CONFIG_PONY_SPI=no CONFIG_NIC3COM=no CONFIG_GFXNVIDIA=no CONFIG_SATASII=no  CONFIG_DUMMY=no CONFIG_DRKAISER=no CONFIG_NICREALTEK=no CONFIG_NICINTEL=no CONFIG_NICINTEL_SPI=no CONFIG_OGP_SPI=no CONFIG_BUSPIRATE_SPI=no CONFIG_SATAMV=no CONFIG_LINUX_SPI=no CONFIG_FT2232_SPI=yes CONFIG_USBBLASTER_SPI=no
with the patch it does not.