On Thu, Aug 18, 2011 at 4:41 AM, Tadas Slotkus devtadas@gmail.com wrote:
Overlapping regions - never support it. It's not a flashrom's job to make a good image for a chip. So do one region at a time in one place.
Agreed.
On Thu, Aug 18, 2011 at 4:41 AM, Tadas Slotkus devtadas@gmail.com wrote:
Partial write: don't read full chip. If needs erase: user supplied image must contain all the eraseregion. Or the user must explicitly mark that it needs erase somehow. I believe you won't go with me here, but I'm open for discussion :)
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.
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.
On Thu, Aug 18, 2011 at 4:41 AM, Tadas Slotkus devtadas@gmail.com wrote:
Use UNIX philosophy: do one thing and do it well.
Amen.