samples: drivers: spi flash testcase check the erased data
After erasing the sector, compare dat read with expected 0xFF pattern
to decide if erasing is successful instead of relying on the
returned code of the flash_erase function
Signed-off-by: Francois Ramu <francois.ramu@st.com>
(cherry picked from commit 325ae4d32a85947c5ee6cef570e8df8c2fa313b0)
diff --git a/samples/drivers/spi_flash/src/main.c b/samples/drivers/spi_flash/src/main.c
index 059dc7d..b29cd9c 100644
--- a/samples/drivers/spi_flash/src/main.c
+++ b/samples/drivers/spi_flash/src/main.c
@@ -30,6 +30,8 @@
#define SPI_FLASH_MULTI_SECTOR_TEST
#endif
+const uint8_t erased[] = { 0xff, 0xff, 0xff, 0xff };
+
void single_sector_test(const struct device *flash_dev)
{
const uint8_t expected[] = { 0x55, 0xaa, 0x66, 0x99 };
@@ -53,9 +55,20 @@
if (rc != 0) {
printf("Flash erase failed! %d\n", rc);
} else {
+ /* Check erased pattern */
+ memset(buf, 0, len);
+ rc = flash_read(flash_dev, SPI_FLASH_TEST_REGION_OFFSET, buf, len);
+ if (rc != 0) {
+ printf("Flash read failed! %d\n", rc);
+ return;
+ }
+ if (memcmp(erased, buf, len) != 0) {
+ printf("Flash erase failed at offset 0x%x got 0x%x\n",
+ SPI_FLASH_TEST_REGION_OFFSET, *(uint32_t *)buf);
+ return;
+ }
printf("Flash erase succeeded!\n");
}
-
printf("\nTest 2: Flash write\n");
printf("Attempting to write %zu bytes\n", len);
@@ -98,7 +111,7 @@
uint8_t buf[sizeof(expected)];
int rc;
- printf("\nPerform test on multiple consequtive sectors");
+ printf("\nPerform test on multiple consecutive sectors");
/* Write protection needs to be disabled before each write or
* erase, since the flash component turns on write protection
@@ -125,9 +138,9 @@
printf("Flash read failed! %d\n", rc);
return;
}
- if (buf[0] != 0xff) {
+ if (memcmp(erased, buf, len) != 0) {
printf("Flash erase failed at offset 0x%x got 0x%x\n",
- offs, buf[0]);
+ offs, *(uint32_t *)buf);
return;
}
offs += SPI_FLASH_SECTOR_SIZE;
