Mike Banon has posted comments on this change. ( https://review.coreboot.org/23263 )
Change subject: Add support for reading the current flash contents from a file ......................................................................
Patch Set 2:
Reading is faster in many cases now, is this still needed /
wanted?
Despite the small memory size (just 128 KB), all the operations with KB9012 internal flash through EDI interface are much much slower than for the regular flash chips. Sadly it takes 1 hour
for
the complete read/erase/write/read(verify) cycle.
I see. The patch generally looks good. Two things, though:
`contents file` sounds ambiguous, how about `reference file`?
The manpage needs an update. It should also be noted when the
data from the reference file is used (currently only at the begin- ning, not later, e.g. when flash content is reread after a failed erase).
1 - renamed contentsfile into referencefile, 2 - manpage update will do slightly later
This patch tries to migitate this problem, it is essential for the comfortable development of KB9012 free-as-in-freedom firmware replacement
Does "free-as-in-freedom" mean anything to you, or are you only trying to emphasize that this is important?
"free-as-in-freedom" 's freedoms mean a lot to me, especially freedom to study (I'm a bit paranoid and its very important to me to have the ability to study the source code to verify that there aren't any backdoors) . yes I'm highlighting the importance of these patches, but thats because the creation of KB9012 firmware replacement is indeed very important: all the letters (including passwords) typed on laptop's keyboard are going through this chip