From c9b7ae55d1f75640a363e85d6c318e822eb255ef Mon Sep 17 00:00:00 2001 From: Edward O'Callaghan Date: Thu, 30 Jun 2022 18:12:40 +1000 Subject: realtek_mst_i2c_spi.c: Add allow-brick=yes programmer param Currently i2c programmers do not have a safe allow listing mechanism via board_enable to facilitate fully qualified chip detection. Since i2c addresses alone can overlap a user may make the mistake of using the wrong programmer. Although unlikely, it is within the realm of possibility that a user could accidently somehow program another chip on their board. Change-Id: Ifb303989fdb67f7267002bd0425f3d050450ec93 Signed-off-by: Edward O'Callaghan Reviewed-on: https://review.coreboot.org/c/flashrom/+/65545 Reviewed-by: Thomas Heijligen Reviewed-by: Anastasia Klimchuk Tested-by: build bot (Jenkins) --- tests/realtek_mst_i2c_spi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'tests') diff --git a/tests/realtek_mst_i2c_spi.c b/tests/realtek_mst_i2c_spi.c index 10fdbe0a3..3b004cbee 100644 --- a/tests/realtek_mst_i2c_spi.c +++ b/tests/realtek_mst_i2c_spi.c @@ -56,7 +56,7 @@ void realtek_mst_basic_lifecycle_test_success(void **state) .fallback_open_state = &realtek_mst_fallback_open_state, }; - run_basic_lifecycle(state, &realtek_mst_io, &programmer_realtek_mst_i2c_spi, "bus=254,enter-isp=0"); + run_basic_lifecycle(state, &realtek_mst_io, &programmer_realtek_mst_i2c_spi, "bus=254,enter-isp=0,allow-brick=yes"); } #else SKIP_TEST(realtek_mst_basic_lifecycle_test_success) -- cgit v1.2.3