>> 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:

1. `contents file` sounds ambiguous, how about `reference file`?

2. 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

View Change

To view, visit change 23263. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Idf153b6955f37779ae9bfb228a434ed10c304947
Gerrit-Change-Number: 23263
Gerrit-PatchSet: 2
Gerrit-Owner: Mike Banon <mikebdp2@gmail.com>
Gerrit-Reviewer: Mike Banon <mikebdp2@gmail.com>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Comment-Date: Sat, 03 Feb 2018 13:29:17 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No