mirror of
https://review.coreboot.org/flashrom.git
synced 2025-04-28 07:23:43 +02:00
CHROMIUM: flashrom_tester: Remove --ignore-fmap option
This change removes --ignore-fmap as this is implicitly computed based on other supplied options by the user. 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 Reviewed-on: https://review.coreboot.org/c/flashrom/+/52890 Reviewed-by: Peter Marheine <pmarheine@chromium.org> Reviewed-by: Edward O'Callaghan <quasisec@chromium.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
parent
e7155dda0a
commit
1a9441f748
@ -129,9 +129,6 @@ fn flashrom_decode_opts(opts: FlashromOpt) -> Vec<String> {
|
|||||||
if opts.flash_name {
|
if opts.flash_name {
|
||||||
params.push("--flash-name".to_string());
|
params.push("--flash-name".to_string());
|
||||||
}
|
}
|
||||||
if opts.ignore_fmap {
|
|
||||||
params.push("--ignore-fmap".to_string());
|
|
||||||
}
|
|
||||||
if opts.verbose {
|
if opts.verbose {
|
||||||
params.push("-V".to_string());
|
params.push("-V".to_string());
|
||||||
}
|
}
|
||||||
@ -325,11 +322,10 @@ mod tests {
|
|||||||
assert_eq!(
|
assert_eq!(
|
||||||
flashrom_decode_opts(FlashromOpt {
|
flashrom_decode_opts(FlashromOpt {
|
||||||
flash_name: true,
|
flash_name: true,
|
||||||
ignore_fmap: true,
|
|
||||||
verbose: true,
|
verbose: true,
|
||||||
..Default::default()
|
..Default::default()
|
||||||
}),
|
}),
|
||||||
&["--flash-name", "--ignore-fmap", "-V"]
|
&["--flash-name", "-V"]
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -91,9 +91,8 @@ pub struct FlashromOpt<'a> {
|
|||||||
pub layout: Option<&'a str>, // -l <file>
|
pub layout: Option<&'a str>, // -l <file>
|
||||||
pub image: Option<&'a str>, // -i <name>
|
pub image: Option<&'a str>, // -i <name>
|
||||||
|
|
||||||
pub flash_name: bool, // --flash-name
|
pub flash_name: bool, // --flash-name
|
||||||
pub ignore_fmap: bool, // --ignore-fmap
|
pub verbose: bool, // -V
|
||||||
pub verbose: bool, // -V
|
|
||||||
}
|
}
|
||||||
|
|
||||||
#[derive(Default)]
|
#[derive(Default)]
|
||||||
@ -183,8 +182,6 @@ pub fn write_file_with_layout(
|
|||||||
layout: rws.layout_file,
|
layout: rws.layout_file,
|
||||||
image: rws.name_file,
|
image: rws.name_file,
|
||||||
|
|
||||||
ignore_fmap: true,
|
|
||||||
|
|
||||||
..Default::default()
|
..Default::default()
|
||||||
};
|
};
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user