Attention is currently required from: Nikolai Artemiev.
Felix Singer has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/69990 )
Change subject: linux_mtd: Mark Opaque chip as tested for WP ......................................................................
linux_mtd: Mark Opaque chip as tested for WP
Since linux_mtd supports write-protect, its probe function needs to mark Opaque chip as tested for WP. Programmers which are opaque masters are responsible for populating flashchip#tested struct in probe function.
Without the patch, any operation running via linux_mtd displays a message "This flash part has status UNTESTED for operations: WP". With the patch, the message is not displayed anymore.
BUG=b:258755442 BRANCH=none TEST=flashrom -p host on ARM dut Found Programmer flash chip "Opaque flash chip" (8192 kB, Programmer-specific) on host. No operations were specified.
Change-Id: Icc0521c28555a93f26ce66bdbeaa68590f10c358 Original-Signed-off-by: Anastasia Klimchuk aklm@chromium.org Original-Reviewed-on: https://review.coreboot.org/c/flashrom/+/69518 Original-Tested-by: build bot (Jenkins) no-reply@coreboot.org Original-Reviewed-by: Edward O'Callaghan quasisec@chromium.org Original-Reviewed-by: Sergii Dmytruk sergii.dmytruk@3mdeb.com
Signed-off-by: Felix Singer felixsinger@posteo.net Change-Id: Ic4017511dd953204f9de86beaecf2ab80c50b754 --- M linux_mtd.c 1 file changed, 34 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/90/69990/1
diff --git a/linux_mtd.c b/linux_mtd.c index 5a1360e..c49b14a 100644 --- a/linux_mtd.c +++ b/linux_mtd.c @@ -179,7 +179,7 @@
if (data->no_erase) flash->chip->feature_bits |= FEATURE_NO_ERASE; - flash->chip->tested = TEST_OK_PREW; + flash->chip->tested = TEST_OK_PREWB; flash->chip->total_size = data->total_size / 1024; /* bytes -> kB */ flash->chip->block_erasers[0].eraseblocks[0].size = data->erasesize; flash->chip->block_erasers[0].eraseblocks[0].count =