<br><br><div class="gmail_quote">On Mon, Jul 11, 2011 at 4:16 AM, 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 Sun, 10 Jul 2011 18:37:27 -0400<br>
<div class="im">Steven Zakulec <<a href="mailto:spzakulec@gmail.com">spzakulec@gmail.com</a>> wrote:<br>
<br>
</div><div class="im">> Here's a new version rebased on r1368, with no extraneous changes.<br>
<br>
</div>ok that's fixed, thank you. sorry for being so picky, but i noticed<br>
that the phrasing in your comments for identical features changes a lot.<br>
<br>
for the high voltage mode we have:<br>
<div class="im">/* 12 V fast program mode */<br>
</div>/* 5.0V +-10% read, 12V fast program & erase- +-5% standard, +-10% option */<br>
(that dash should be a comma above btw?)<br>
/* Also offers a 12V fast program */<br>
/* Also has 12V fast program & erase */<br>
<br>
and for the ranges that are different in one speed grade we have:<br>
/* x speed is +-5%, all others +-10% */<br>
* x speed is 3.0-3.6V, others 2.7-3.6V */<br>
/* 3.0-3.6V (x), 2.7-3.6V for others */<br>
<br>
and for datasheets that define two ranges without giving any indication<br>
how to the chips can be distinguished we have:<br>
<div class="im">/* Datasheet says 2.5-3.6 V or 2.7-3.6 V */<br>
</div><div class="im">/* Datasheet says there's a 2.3-3.6V & 2.7-3.6V model */<br>
</div>/* Datasheet says some are only 4.75-5.25 V */ (not sure if this is a speed grade case or belongs in this section)<br>
<br>
it would be good if those are unified so that it is easier to find them<br>
later, for example when we have a new feature flag or field that<br>
indicates a property directly in code instead of comments.<br>
the exact phrasing is not important as long as it is understandable for<br>
developers, but they should be equal (apart from the concrete values).<br>
this of course applies to past and future changes too.<br>
<font color="#888888">--<br>
</font><div><div></div><div>Kind regards/Mit freundlichen Grüßen, Stefan Tauner<br>
<br>
_______________________________________________<br>
flashrom mailing list<br>
<a href="mailto:flashrom@flashrom.org">flashrom@flashrom.org</a><br>
<a href="http://www.flashrom.org/mailman/listinfo/flashrom" target="_blank">http://www.flashrom.org/mailman/listinfo/flashrom</a> <br></div></div></blockquote><div>I believe I've adjusted the phrasings so there's one style for each comment type.<br>
The only issue I've come across is the Intel 28F002BC/BL/BV/BX-T.<br>From my reading of the datasheets, the later two are compatible with each other voltage-wise, but the BC and BL versions are definitely not- they have non-overlapping ranges for reading.<br>
<br>Any thoughts?<br></div></div>