Change in flashrom[master]: layout: Add -i <region>[:<file>] support. - flashrom-gerrit

8 Feb 2018


      Nico Huber has posted comments on this change. ( https://review.coreboot.org/23021 )
Change subject: layout: Add -i <region>[:<file>] support.
......................................................................
Patch Set 13:
In the light of the shy ML discussion, what do you think about my
alternative syntax proposal [1]? Especially the last one would be
most user-friendly, IMHO, because you just can't do confusing stuff.
I assume, it's unlikely that regular users would want to mix a full
flash sized file with per-region files.
[1] https://flashrom.org/Per_region_file_arguments
-- 
To view, visit https://review.coreboot.org/23021
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic5465659605d8431d931053967b40290195cfd99
Gerrit-Change-Number: 23021
Gerrit-PatchSet: 13
Gerrit-Owner: David Hendricks david.hendricks@gmail.com
Gerrit-Reviewer: Arthur Heymans arthur@aheymans.xyz
Gerrit-Reviewer: Carl-Daniel Hailfinger c-d.hailfinger.devel.2006@gmx.net
Gerrit-Reviewer: David Hendricks david.hendricks@gmail.com
Gerrit-Reviewer: Nico Huber nico.h@gmx.de
Gerrit-Reviewer: Nicola Corna nicola@corna.info
Gerrit-Reviewer: Paul Menzel paulepanter@users.sourceforge.net
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-Comment-Date: Thu, 08 Feb 2018 22:06:48 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No