Alan Green has uploaded this change for review.
flashchips.c: SFDP-capable chip formatting
To allow automated tools to manipulate flashchips.c, make the definition
of SFDP-capable chip more consistent with other definitions. This
involves
- reordering fields to match both other entries and the definition of
struct flashchip.
- reformatting comments to make them consistent with other entries.
Signed-off-by: Alan Green <avg@google.com>
Change-Id: I8708a11993822085b3e8d8c80532dfb935d39876
---
M flashchips.c
1 file changed, 10 insertions(+), 10 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/34/33834/1
diff --git a/flashchips.c b/flashchips.c
index 8ea7043..f9ad764 100644
--- a/flashchips.c
+++ b/flashchips.c
@@ -16578,7 +16578,8 @@
.read = read_memmapped,
},
- {/* W29EE011, W29EE012, W29C010M, W29C011A do not support probe_jedec according to the datasheet, but it works for newer(?) steppings. */
+ /* W29EE011, W29EE012, W29C010M, W29C011A do not support probe_jedec according to the datasheet, but it works for newer(?) steppings. */
+ {
.vendor = "Winbond",
.name = "W29C010(M)/W29C011A/W29EE011/W29EE012",
.bustype = BUS_PARALLEL,
@@ -17428,21 +17429,20 @@
.bustype = BUS_SPI,
.manufacture_id = GENERIC_MANUF_ID,
.model_id = SFDP_DEVICE_ID,
- /* We present our own "report this" text hence we do not
- * want the default "This flash part has status UNTESTED..."
- * text to be printed. */
+ .total_size = 0, /* set by probing function */
+ .page_size = 0, /* set by probing function */
+ .feature_bits = 0, /* set by probing function */
+ /* We present our own "report this" text hence we do not */
+ /* want the default "This flash part has status UNTESTED..." */
+ /* text to be printed. */
.tested = TEST_OK_PREW,
.probe = probe_spi_sfdp,
+ .block_erasers = {}, /* set by probing function */
.unlock = spi_disable_blockprotect, /* is this safe? */
+ .write = NULL, /* set by probing function */
.read = spi_chip_read,
/* FIXME: some vendor extensions define this */
.voltage = {0},
- /* Everything below will be set by the probing function. */
- .write = NULL,
- .total_size = 0,
- .page_size = 0,
- .feature_bits = 0,
- .block_erasers = {},
},
{
To view, visit change 33834. To unsubscribe, or for help writing mail filters, visit settings.