Daniel Campello has uploaded this change for review.

View Change

CHROMIUM: cli_classic.c: Remove deprecated options

This change removes --fast-verify and --diff options as there are other
options that implement the same logic. It also removes --ignore-fmap as
this is implicetely computed based on other supplied options by the
user.

BRANCH=none
BUG=b:162798062,b:186479007
TEST=tryjobs

Original-Signed-off-by: Daniel Campello <campello@chromium.org>
Original-Cq-Depend: chromium:2854174, chrome-internal:3789445, chromium:2854014
Original-Change-Id: I841a56d8726644cedd7d616ddfd5656b92dd7e59
Original-Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/flashrom/+/2851658
Original-Reviewed-by: Edward O'Callaghan <quasisec@chromium.org>
(cherry picked from commit 2e2a5e449229e9c9604235a98b56e5dd29bf25cf)
Change-Id: Ib329f3adb59ce1848d1540844d64b968f49eb22a
---
M util/flashrom_tester/flashrom/src/cmd.rs
M util/flashrom_tester/flashrom/src/lib.rs
2 files changed, 3 insertions(+), 10 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/90/52890/1
diff --git a/util/flashrom_tester/flashrom/src/cmd.rs b/util/flashrom_tester/flashrom/src/cmd.rs
index 8775ed4..37f8a72 100644
--- a/util/flashrom_tester/flashrom/src/cmd.rs
+++ b/util/flashrom_tester/flashrom/src/cmd.rs
@@ -129,9 +129,6 @@
if opts.flash_name {
params.push("--flash-name".to_string());
}
- if opts.ignore_fmap {
- params.push("--ignore-fmap".to_string());
- }
if opts.verbose {
params.push("-V".to_string());
}
@@ -325,11 +322,10 @@
assert_eq!(
flashrom_decode_opts(FlashromOpt {
flash_name: true,
- ignore_fmap: true,
verbose: true,
..Default::default()
}),
- &["--flash-name", "--ignore-fmap", "-V"]
+ &["--flash-name", "-V"]
);
}

diff --git a/util/flashrom_tester/flashrom/src/lib.rs b/util/flashrom_tester/flashrom/src/lib.rs
index 734e3ff..75797d3 100644
--- a/util/flashrom_tester/flashrom/src/lib.rs
+++ b/util/flashrom_tester/flashrom/src/lib.rs
@@ -91,9 +91,8 @@
pub layout: Option<&'a str>, // -l <file>
pub image: Option<&'a str>, // -i <name>

- pub flash_name: bool, // --flash-name
- pub ignore_fmap: bool, // --ignore-fmap
- pub verbose: bool, // -V
+ pub flash_name: bool, // --flash-name
+ pub verbose: bool, // -V
}

#[derive(Default)]
@@ -183,8 +182,6 @@
layout: rws.layout_file,
image: rws.name_file,

- ignore_fmap: true,
-
..Default::default()
};


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

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ib329f3adb59ce1848d1540844d64b968f49eb22a
Gerrit-Change-Number: 52890
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel Campello <campello@chromium.org>
Gerrit-MessageType: newchange