[flashrom-gerrit] Change in ...flashrom[master]: flashchips.c: SFDP-capable chip formatting

[Alan Green (Code Review)]

Alan Green has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/33834

Change subject: flashchips.c: SFDP-capable chip formatting
......................................................................

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 
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	= {},
    },

{