<p dir="ltr">Sounds good to me. </p>
<p dir="ltr">Acked-by: David Hendricks <<a href="mailto:dhendrix@chromium.org">dhendrix@chromium.org</a>></p>
<div class="gmail_quote">On Jun 28, 2015 4:07 PM, "Stefan Tauner" <<a href="mailto:stefan.tauner@alumni.tuwien.ac.at">stefan.tauner@alumni.tuwien.ac.at</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sat, 27 Jun 2015 17:24:43 -0700<br>
David Hendricks <<a href="mailto:david.hendricks@gmail.com">david.hendricks@gmail.com</a>> wrote:<br>
<br>
> Mostly looks good to me, just a couple minor comments.<br>
><br>
> > +<br>
> > +enum dediprog_target {<br>
> > +       FLASH_TYPE_APPLICATION_FLASH_1  = 0,<br>
> > +       FLASH_TYPE_FLASH_CARD,<br>
> > +       FLASH_TYPE_APPLICATION_FLASH_2,<br>
><br>
><br>
> FLASH_TYPE_SOCKET,        /* SF600 */<br>
<br>
Thank you.<br>
<br>
> > +<br>
> > +enum dediprog_writemode {<br>
> > +       WRITE_MODE_PAGE_PGM                     = 1,<br>
> > +       WRITE_MODE_PAGE_WRITE                   = 2,<br>
> > +       WRITE_MODE_1B_AAI                       = 3,<br>
> > +       WRITE_MODE_2B_AAI                       = 4,<br>
> > +       WRITE_MODE_128B_PAGE                    = 5,<br>
> > +       WRITE_MODE_PAGE_AT26DF041               = 6,<br>
> > +       WRITE_MODE_SILICON_BLUE_FPGA            = 7,<br>
> > +       WRITE_MODE_64B_PAGE_NUMONYX_PCM         = 8,    /* unit of length 512 bytes */<br>
> > +       WRITE_MODE_4B_ADDR_256B_PAGE_PGM        = 9,<br>
> > +       WRITE_MODE_32B_PAGE_PGM_MXIC_512K       = 10,   /* unit of length 512 bytes */<br>
> > +       WRITE_MODE_4B_ADDR_256B_PAGE_PGM_0x12   = 11,<br>
><br>
> 11 seems to only apply for the new protocol? The old protocol spec<br>
> just says "For SF100" but I have no idea what that actually means.<br>
<br>
I don't know... and I don't think they do either. :P Since it is present<br>
in both specs I'll just leave it as is for now. The 0x12 opcode is<br>
rather clear in the flash datasheets... it is just like normal page<br>
write but with 4 mandatory bytes for the address. So it should be easy<br>
to verify if the old firmware/protocol uses that... when we implement<br>
those write modes eventually.<br>
<br>
I have not seen an acked-by line in your mail... may I add it to the<br>
respective commit?<br>
<br>
--<br>
Kind regards/Mit freundlichen Grüßen, Stefan Tauner<br>
</blockquote></div>