<p dir="ltr">I hope my rebase and my patches would be useful - see branch layout_descriptor in <a href="https://github.com/XVilka/flashrom">https://github.com/XVilka/flashrom</a> </p>
<div class="gmail_quote">On Apr 25, 2016 10:08 PM, "Nico Huber" <<a href="mailto:nico.huber@secunet.com">nico.huber@secunet.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
I've looked at many patches now and feel pretty dazed.<br>
<br>
While it's not ready yet (most probably because it didn't get a review),<br>
I'm in clear favor of Alexander's approach:<br>
> <a href="https://www.flashrom.org/pipermail/flashrom/2014-October/012967.html" rel="noreferrer" target="_blank">https://www.flashrom.org/pipermail/flashrom/2014-October/012967.html</a><br>
<br>
It does what I'd expect of layout support, takes the erase blocks into<br>
account, and leaves the whole code in a better shape (gets rid of the<br>
ugly flow in doit()).<br>
<br>
The separate functions for erase / read / verify reminded me of lib-<br>
flashrom, and that I actually wanted to rewrite doit() years ago to<br>
use the libflashrom functions. Which brings me to the following pro-<br>
posal:<br>
<br>
1st take the modified walk_eraseregions() from Alexander's patch,<br>
2nd rewrite the libflashrom patch to use it,<br>
3rd rewrite doit() using libflashrom.<br>
<br>
So far this would suffice my use case. However, it would overrun major<br>
features of Stefan's layout patch series: The new layout format and<br>
the `-i <image>[:<file>]` support. Well, the former seems orthogonal to<br>
the other changes, and the latter would require more work for a similar<br>
feature in libflashrom anyway.<br>
<br>
So my main concern is that if I don't push forward libflashrom now,<br>
we will maintain two code paths until libflashrom is used internally<br>
by flashrom.<br>
<br>
Any comments?<br>
<br>
Best regards,<br>
Nico<br>
<br>
--<br>
M. Sc. Nico Huber<br>
Senior Berater SINA-Softwareentwicklung<br>
Netzwerk- & Client-Sicherheit / Network & Client Security<br>
Division Öffentliche Auftraggeber / Public Authorities<br>
secunet Security Networks AG<br>
<br>
Tel.: <a href="tel:%2B49-201-5454-3635" value="+4920154543635">+49-201-5454-3635</a>, Fax: <a href="tel:%2B49-201-5454-1325" value="+4920154541325">+49-201-5454-1325</a><br>
E-Mail: <a href="mailto:nico.huber@secunet.com">nico.huber@secunet.com</a><br>
Mergenthalerallee 77, 65760 Eschborn, Deutschland<br>
<a href="http://www.secunet.com" rel="noreferrer" target="_blank">www.secunet.com</a><br>
______________________________________________________________________<br>
<br>
Sitz: Kurfürstenstraße 58, 45138 Essen, Deutschland<br>
Amtsgericht Essen HRB 13615<br>
Vorstand: Dr. Rainer Baumgart (Vors.), Thomas Pleines<br>
Aufsichtsratsvorsitzender: Dr. Peter Zattler<br>
______________________________________________________________________<br>
</blockquote></div>