<br><br><div class="gmail_quote">On Tue, Feb 7, 2012 at 7:25 PM, Stefan Tauner <span dir="ltr"><<a href="mailto:stefan.tauner@student.tuwien.ac.at">stefan.tauner@student.tuwien.ac.at</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Tue, 07 Feb 2012 23:31:50 +0100<br>
Carl-Daniel Hailfinger <<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</a>> wrote:<br>
<br>
> Ahem. Once we do the per-region unlock dance, this will be extremely<br>
> painful. We have three choices:<br>
> - Keep that part of the code as is. Not good.<br>
> - Spit out that message for every status register write. Not good.<br>
> - Fix all flash chip definitions. Good, but separate patch.<br>
> How do we proceed?<br>
<br>
fixing the stuff that needs fixing, of course.<br>
<br>
steve said he might look into it, so this is what has to be done:<br>
i have attached a file containing what seems to be the spi chips<br>
without correctly set feature_bits fields. we need to check the<br>
datasheets for the opcodes needed to enable writing to the status<br>
register. normally this is either WREN (write enable) or EWSR (enable<br>
write status register), or both.<br>
currently we are only interested in the non-volatile bits of the status<br>
register, but there are chips that require another write enable command<br>
for the volatile bits. if anyone creates a spreadsheet, he should<br>
probably note those too if feasible or maybe even add a comment in the<br>
patch.<br>
<br>
the field should be added after page_size and either set to:<br>
                .feature_bits   = FEATURE_WRSR_EWSR<br>
or<br>
                .feature_bits   = FEATURE_WRSR_EITHER<br>
if EWSR and WREN are allowed (all other SPI chips are already set<br>
to FEATURE_WRSR_WREN). in the case we encounter a chip that does use<br>
something else a comment should suffice for the time being.<br>
<br>
we could also split the work up, if it takes too long otherwise...<br>
<span class="HOEnZb"><font color="#888888">--<br>
Kind regards/Mit freundlichen Grüßen, Stefan Tauner<br>
</font></span></blockquote></div><br>I have tagged all of the ones in this list Stefan gave me except the Sanyo one (no datasheet- apparently not a new issue either).<br><br>If there are any questions or comments, please send them.<br>
<br>Signed-off-by: Steven Zakulec <<a href="mailto:spzakulec@gmail.com" target="_blank">spzakulec@gmail.com</a>><br>