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 _______________________________________________ flashrom mailing list [email protected] http://www.flashrom.org/mailman/listinfo/flashrom
