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