<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, May 27, 2013 at 12:02 PM, Steven Zakulec <span dir="ltr"><<a href="mailto:spzakulec@gmail.com" target="_blank">spzakulec@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote"><div class="im">On Mon, May 27, 2013 at 5:25 AM, Николай Николаев <span dir="ltr"><<a href="mailto:evrinoma@gmail.com" target="_blank">evrinoma@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi steven,<br><br><div class="gmail_quote"><div>2013/5/27 Steven Zakulec <span dir="ltr"><<a href="mailto:spzakulec@gmail.com" target="_blank">spzakulec@gmail.com</a>></span><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div dir="ltr"><div><div><div><div><div><div><div><div><div><div>In the last review, I mentioned the Numonyx M45PE series (10, 20, 40, 80, 16), should only have <br>

+        .feature_bits    = FEATURE_WRSR_WREN, </div>
<div>set when the command is there.  It appears I may have been unclear there, because instead of removing just the WREN flag, you removed the whole line (the chips do have a Write Status Register- WRSR).<br></div></div>


</div></div></div></div></div></div></div></div></div></blockquote><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div><div><div>

<div><div><div><div>
<div><div>

<br>Continuing the review from Numonyx M45PE16.<br></div></div></div></div></div></div></div></div></div></div></div></blockquote></div><div>Sorry i don't know which flag must be used. You right these chips have a only WREN instruction, and i think if these chips doesn't have a feature that i exclude feature_bits. Maybe we will be able to specify the details.</div>

</div></blockquote></div><div>Other people can provide more details on the general case, but in this instance, with only a WREN instruction, the FEATURE line should be:<br> +        .feature_bits    = FEATURE_WRSR</div><br>
</div></div></div></blockquote><div>I've been informed by others that your original idea of dropping the FEATURE line is actually correct, so you should do that instead of what I proposed. <br></div></div><br></div></div>