mirror of
https://review.coreboot.org/flashrom.git
synced 2025-07-05 15:50:41 +02:00
flashrom_tester: Drop dediprog, ec, and servo targets
None of these targets have been maintained or used for several years: dediprog: - Wasn't accepted by the argument filter in main.rs. ec: - Is incompatible with most tests because the EC only supports one protection range. servo: - Has been broken for >3 years because it uses the programmer string "ft2231_spi:type=servo-v2", where "ft2231" should be "ft2232". BUG=b:239357853 BRANCH=none TEST=flashrom_tester on dedede Change-Id: Iee94f6bb5ff8c5451acb8bcaabf28119006d0ef5 Signed-off-by: Nikolai Artemiev <nartemiev@google.com> Reviewed-on: https://review.coreboot.org/c/flashrom/+/69921 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Edward O'Callaghan <quasisec@chromium.org>
This commit is contained in:

committed by
Edward O'Callaghan

parent
ff5beeef40
commit
8b0250b32b
@ -96,19 +96,10 @@ impl<'a> TestEnv<'a> {
|
||||
}
|
||||
|
||||
pub fn run_test<T: TestCase>(&mut self, test: T) -> TestResult {
|
||||
let use_dut_control = self.chip_type == FlashChip::SERVO;
|
||||
if use_dut_control && flashrom::dut_ctrl_toggle_wp(false).is_err() {
|
||||
error!("failed to dispatch dut_ctrl_toggle_wp()!");
|
||||
}
|
||||
|
||||
let name = test.get_name();
|
||||
info!("Beginning test: {}", name);
|
||||
let out = test.run(self);
|
||||
info!("Completed test: {}; result {:?}", name, out);
|
||||
|
||||
if use_dut_control && flashrom::dut_ctrl_toggle_wp(true).is_err() {
|
||||
error!("failed to dispatch dut_ctrl_toggle_wp()!");
|
||||
}
|
||||
out
|
||||
}
|
||||
|
||||
|
Reference in New Issue
Block a user