<div class="gmail_quote">On Thu, Aug 18, 2011 at 4:41 AM, Tadas Slotkus <span dir="ltr"><<a href="mailto:devtadas@gmail.com">devtadas@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 class="HOEnZb"><div></div><div class="h5">Overlapping regions - never support it. It's not a flashrom's job to</div></div>
make a good image for a chip. So do one region at a time in one place.<br></blockquote><div><br></div><div>Agreed.</div><div><br></div><div>On Thu, Aug 18, 2011 at 4:41 AM, Tadas Slotkus <span dir="ltr"><<a href="mailto:devtadas@gmail.com">devtadas@gmail.com</a>></span> wrote: </div>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Partial write: don't read full chip. If needs erase: user supplied image<br>
must contain all the eraseregion. Or the user must explicitly mark that<br>
it needs erase somehow. I believe you won't go with me here, but I'm<br>
open for discussion :)<br></blockquote><div><br></div><div>I'll have to disagree with you here. That makes the logic involved to utilize partial writes much more complicated. I think the best default action (and overwhelming usage case) for Flashrom is to handle erase regions automagically as it does now.</div>

<div><br></div><div>We'll need to improve the partial write logic a bit so it calculates all the necessary erase regions and checks to ensure they are all readable and erasable. Much of the work will be done to address issues mentioned earlier in Carl-Daniel's list.</div>

<div><br></div><div>On Thu, Aug 18, 2011 at 4:41 AM, Tadas Slotkus <span dir="ltr"><<a href="mailto:devtadas@gmail.com">devtadas@gmail.com</a>></span> wrote: </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

Use UNIX philosophy: do one thing and do it well.<br></blockquote><div><br></div><div>Amen.</div><div><br></div></div>-- <br>David Hendricks (dhendrix)<br>Systems Software Engineer, Google Inc.<br>