summaryrefslogtreecommitdiffstats
path: root/tests
diff options
context:
space:
mode:
authorNikolai Artemiev <nartemiev@google.com>2022-11-21 19:29:57 +1100
committerEdward O'Callaghan <quasisec@chromium.org>2023-03-03 05:52:20 +0000
commit29a3a09f9155e5de69d6d2a4bf582a9e09b8e0a4 (patch)
treecb0b381aa39c16acd78446f8d9f24c5d141250bc /tests
parent1d6d23bee2843b4f3a2d3676a7a75f92aa8574c4 (diff)
downloadflashrom-29a3a09f9155e5de69d6d2a4bf582a9e09b8e0a4.tar.gz
flashrom-29a3a09f9155e5de69d6d2a4bf582a9e09b8e0a4.tar.bz2
flashrom-29a3a09f9155e5de69d6d2a4bf582a9e09b8e0a4.zip
writeprotect: Add function to get register values and WP bit masks
Add a new wp_cfg_to_reg_values() function that takes a generic wp_cfg instance and returns the chip-specific values that need to be written to the chip's registers to enable the specified protection range/mode. The function returns three values for each chip register: - reg_values[reg] - Value writeprotect will write to reg - bit_masks[reg] - Bit mask for WP-related bits in reg - write_masks[reg] - Bit mask for writable WP-related bits in reg (i.e. the ones writeprotect will try to write) BUG=b:260019525,b:259013033,260020006 BRANCH=none TEST=ninja test Change-Id: Ib2a47153b230c9f82bb4eca357c335f2abbacc20 Signed-off-by: Nikolai Artemiev <nartemiev@google.com> Reviewed-on: https://review.coreboot.org/c/flashrom/+/69847 Reviewed-by: Edward O'Callaghan <quasisec@chromium.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Sergii Dmytruk <sergii.dmytruk@3mdeb.com>
Diffstat (limited to 'tests')
-rw-r--r--tests/chip_wp.c65
-rw-r--r--tests/tests.c1
-rw-r--r--tests/tests.h1
3 files changed, 67 insertions, 0 deletions
diff --git a/tests/chip_wp.c b/tests/chip_wp.c
index cb9b3ab40..9607a96ed 100644
--- a/tests/chip_wp.c
+++ b/tests/chip_wp.c
@@ -327,3 +327,68 @@ void partial_chip_erase_with_wp_dummyflasher_test_success(void **state)
flashrom_wp_cfg_release(wp_cfg);
}
+
+/* Chip register values & masks are calculated correctly by WP */
+void wp_get_register_values_and_masks(void **state)
+{
+ (void) state; /* unused */
+
+ /*
+ * Test with range: start = 0x004000, lengh = 0xffc000
+ *
+ * WP should use these bit values:
+ * WPS (S17) = 0 (write protect scheme)
+ * CMP (S14) = 1 (range complement)
+ * SRP1 (S8) = 0
+ * SRP0 (S7) = 1 (`SRP1 == 1 && SRP0 == 1` is permanent mode)
+ * SEC (S6) = 1 (base unit is a 4 KiB sector)
+ * TB (S5) = 1 (bottom up range)
+ * BP2 (S4) = 0
+ * BP1 (S3) = 1
+ * BP0 (S2) = 1 (bp: BP2-0 == 0b011 == 3)
+ *
+ * Register values:
+ * SR1 = 0b11101100 = 0xec
+ * SR2 = 0b01000000 = 0x40
+ * SR3 = 0b00000000 = 0x00
+ *
+ * Masks for WP bits in registers:
+ * SR1: 0b11111100 = 0xfc
+ * SR2: 0b01000000 = 0x41
+ * SR3: 0b00000100 = 0x04
+ *
+ * All WP bits are RW so write masks should be the same as the bit masks.
+ *
+ */
+ struct flashrom_flashctx flash = { 0 };
+ struct flashchip mock_chip = chip_W25Q128_V;
+ struct flashrom_wp_cfg *wp_cfg;
+
+ uint8_t reg_values[MAX_REGISTERS];
+ uint8_t bit_masks[MAX_REGISTERS];
+ uint8_t write_masks[MAX_REGISTERS];
+
+ setup_chip(&flash, NULL, &mock_chip, "bus=spi,emulate=W25Q128FV");
+
+ assert_int_equal(0, flashrom_wp_cfg_new(&wp_cfg));
+ flashrom_wp_set_mode(wp_cfg, FLASHROM_WP_MODE_HARDWARE);
+ flashrom_wp_set_range(wp_cfg, 0x004000, 0xffc000);
+
+ assert_int_equal(0, wp_cfg_to_reg_values(reg_values, bit_masks, write_masks, &flash, wp_cfg));
+
+ assert_int_equal(0xec, reg_values[STATUS1]);
+ assert_int_equal(0x40, reg_values[STATUS2]);
+ assert_int_equal(0x00, reg_values[STATUS3]);
+
+ assert_int_equal(0xfc, bit_masks[STATUS1]);
+ assert_int_equal(0x41, bit_masks[STATUS2]);
+ assert_int_equal(0x04, bit_masks[STATUS3]);
+
+ assert_int_equal(0xfc, write_masks[STATUS1]);
+ assert_int_equal(0x41, write_masks[STATUS2]);
+ assert_int_equal(0x04, write_masks[STATUS3]);
+
+ teardown(NULL);
+
+ flashrom_wp_cfg_release(wp_cfg);
+}
diff --git a/tests/tests.c b/tests/tests.c
index 0de01686a..911cf471a 100644
--- a/tests/tests.c
+++ b/tests/tests.c
@@ -494,6 +494,7 @@ int main(int argc, char *argv[])
cmocka_unit_test(wp_init_from_status_dummyflasher_test_success),
cmocka_unit_test(full_chip_erase_with_wp_dummyflasher_test_success),
cmocka_unit_test(partial_chip_erase_with_wp_dummyflasher_test_success),
+ cmocka_unit_test(wp_get_register_values_and_masks),
};
ret |= cmocka_run_group_tests_name("chip_wp.c tests", chip_wp_tests, NULL, NULL);
diff --git a/tests/tests.h b/tests/tests.h
index 8f63f91ba..e273e1dbf 100644
--- a/tests/tests.h
+++ b/tests/tests.h
@@ -92,6 +92,7 @@ void switch_wp_mode_dummyflasher_test_success(void **state);
void wp_init_from_status_dummyflasher_test_success(void **state);
void full_chip_erase_with_wp_dummyflasher_test_success(void **state);
void partial_chip_erase_with_wp_dummyflasher_test_success(void **state);
+void wp_get_register_values_and_masks(void **state);
/* selfcheck.c */
void selfcheck_programmer_table(void **state);