[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