[flashrom] [PATCH 1/5] rayer_spi: Create list of programmer types

Kyösti Mälkki kyosti.malkki at gmail.com
Sun Mar 31 22:09:29 CEST 2013


On Sun, 2013-03-31 at 19:40 +0400, Maksim Kuleshov wrote:
> Hi.
> 
> Tested:
> byteblaster - OK
> wiggler - OK
> rayer - BAD, maybe cable too long
> 
> I broke rayer_spi.c on lpt_bitbang_spi.c and lpt_io.c.
> lpt_io.c - LPT support functions for different methods of input-output.
> lpt_bitbang_spi.c - Support rayer, xilinx, byteblaster, stk200, wiggler, willem etc.
> I also added the patch "[PATCH 0/3] rayer_spi: Add support for Willem EPROM programmer SPI" 
> from Ondrej Zary Sat Mar 16 23:42:50 CET 2013
> Please see https://bitbucket.org/mmcx/flashrom/commits/branch/lpt_bitbang_spi
> 

Hi

Please come and discuss such (massive) rewrites on #flashrom or the
mailing list. I like most approaches you had handling the parallel port,
but we need to coordinate changes to avoid duplicate work with the
Willem FWH/LPC programmer device.

Can you confirm "byteblaster" and "wiggler" to work with patches applied
upto this: http://patchwork.coreboot.org/patch/3880/

Should we use "byteblastermv" and "Altera ByteBlasterMV" ?

My plan was to have that set of patches merged as the first step.
Keeping these simple increases chances of getting a review and merge.

Handling parallel port with either direct IO or ppdev must be
independent of using SPI, there are FWH/LPC/parallel flash programmers.
And parallel port code needs to be portable to different OS too.
I did not look thoroughly, maybe these requirements were taken care of
already.

Thanks,
  Kyösti





More information about the flashrom mailing list