Drop useless and partly even incorrect comments (trivial).

Signed-off-by: Uwe Hermann <uwe@hermann-uwe.de>
Acked-by: Uwe Hermann <uwe@hermann-uwe.de>



git-svn-id: svn://svn.coreboot.org/coreboot/trunk@2578 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1

1 files changed, 54 insertions, 49 deletions

diff --git a/util/flashrom/flash.h b/util/flashrom/flash.h
index aa24cac391..3167fd602a 100644
--- a/util/flashrom/flash.h
+++ b/util/flashrom/flash.h
@@ -51,73 +51,78 @@ struct flashchip {
 
 extern struct flashchip flashchips[];
 
-#define AMD_ID			0x01
+/* Please keep this list sorted alphabetically by manufacturer. The first
+ * entry of each section should be the manufacturer ID, followed by the
+ * list of devices from that manufacturer (sorted by device IDs).
+ */
+
+#define AMD_ID			0x01	/* AMD */
 #define AM_29F040B		0xA4
 #define AM_29F016D		0xAD
 
-#define ASD_ID			0x25
+#define ASD_ID			0x25	/* ASD */
 #define ASD_AE49F2008		0x52
 
-#define ATMEL_ID		0x1F	/* Winbond Manufacture ID code        */
-#define AT_29C040A		0xA4	/* Winbond w29c020c device code */
+#define ATMEL_ID		0x1F	/* Atmel */
+#define AT_29C040A		0xA4
 
-#define MX_ID			0xC2
+#define MX_ID			0xC2	/* Macronix (MX) */
 #define MX_29F002		0xB0
 
-#define SHARP_ID		0xB0
+#define SHARP_ID		0xB0	/* Sharp */
 #define SHARP_LHF00L04		0xCF
 
-#define SST_ID			0xBF	/* SST Manufacturer ID code */
-#define SST_29EE020A		0x10	/* SST 29EE020 device */
-#define SST_28SF040		0x04	/* SST 29EE040 device */
-#define SST_39SF010		0xB5	/* SST 39SF010A device */
-#define SST_39SF020		0xB6	/* SST 39SF020A device */
-#define SST_39SF040		0xB7	/* SST 39SF040 device */
-#define SST_39VF020		0xD6	/* SST 39VF020 device */
-#define SST_49LF040B		0x50	/* SST 49LF040B device */
-#define SST_49LF040		0x51	/* SST 49LF040 device */
-#define SST_49LF020A		0x52	/* SST 49LF020A device */
-#define SST_49LF080A		0x5B	/* SST 48LF080A device */
-#define SST_49LF002A		0x57	/* SST 49LF002A device */
-#define SST_49LF003A		0x1B	/* SST 49LF003A device */
-#define SST_49LF004A		0x60	/* SST 49LF004A device */
-#define SST_49LF008A		0x5A	/* SST 49LF008A device */
-#define SST_49LF004C		0x54	/* SST 49LF004C device */
-#define SST_49LF008C		0x59	/* SST 49LF008C device */
-#define SST_49LF016C		0x5C	/* SST 49LF016C device */
-#define SST_49LF160C		0x4C	/* SST 49LF160C device */
-
-#define PMC_ID			0x9D	/* PMC Manufacturer ID code   */
-#define PMC_49FL002		0x6D	/* PMC 49FL002 device code      */
-#define PMC_49FL004		0x6E	/* PMC 49FL004 device code      */
-
-#define WINBOND_ID		0xDA	/* Winbond Manufacture ID code  */
-#define W_29C011		0xC1	/* Winbond w29c011 device code */
-#define W_29C020C		0x45	/* Winbond w29c020c device code */
-#define W_39V040A		0x3D	/* Winbond w39v040a device code */
-#define W_39V040B		0x54	/* Winbond w39v040b device code */
-#define W_39V080A		0xD0	/* Winbond w39v080a device code */
-#define W_49F002U		0x0B	/* Winbond w49F002u device code */
-#define W_49V002A		0xB0	/* Winbond W49V002A device code */
-#define W_49V002FA		0x32	/* Winbond W49V002FA device code */
-
-#define ST_ID			0x20
+#define SST_ID			0xBF	/* SST */
+#define SST_29EE020A		0x10
+#define SST_28SF040		0x04
+#define SST_39SF010		0xB5
+#define SST_39SF020		0xB6
+#define SST_39SF040		0xB7
+#define SST_39VF020		0xD6
+#define SST_49LF040B		0x50
+#define SST_49LF040		0x51
+#define SST_49LF020A		0x52
+#define SST_49LF080A		0x5B
+#define SST_49LF002A		0x57
+#define SST_49LF003A		0x1B
+#define SST_49LF004A		0x60
+#define SST_49LF008A		0x5A
+#define SST_49LF004C		0x54
+#define SST_49LF008C		0x59
+#define SST_49LF016C		0x5C
+#define SST_49LF160C		0x4C
+
+#define PMC_ID			0x9D	/* PMC */
+#define PMC_49FL002		0x6D
+#define PMC_49FL004		0x6E
+
+#define WINBOND_ID		0xDA	/* Winbond */
+#define W_29C011		0xC1
+#define W_29C020C		0x45
+#define W_39V040A		0x3D
+#define W_39V040B		0x54
+#define W_39V080A		0xD0
+#define W_49F002U		0x0B
+#define W_49V002A		0xB0
+#define W_49V002FA		0x32
+
+#define ST_ID			0x20	/* ST */
 #define ST_M29F040B		0xE2
 #define ST_M29F400BT		0xD5
 
-#define EMST_ID			0x8c	/* EMST - Elite Flash Storage Inc. Manufacturer ID code */
-#define EMST_F49B002UA		0x00	/* EMST F49B002UA device code */
+#define EMST_ID			0x8c	/* EMST / EFST */
+#define EMST_F49B002UA		0x00
 
-#define MSYSTEMS_ID		0x156f
+#define MSYSTEMS_ID		0x156f	/* M-Systems */
 #define MSYSTEMS_MD2200		0xdb	/* ? */
 #define MSYSTEMS_MD2800		0x30	/* hmm -- both 0x30 */
 #define MSYSTEMS_MD2802		0x30	/* hmm -- both 0x30 */
 
-#define SYNCMOS_ID		0x40	/* SyncMOS ID */
-#define S29C51001T		0x01	/* SyncMOS S29C51001T/B */
-#define S29C51002T		0x02	/* SyncMOS S29C51002T/B */
-#define S29C51004T		0x03	/* SyncMOS S29C51004T/B */
-#define S29C31004T		0x63	/* SyncMOS S29C31004T */
+#define SYNCMOS_ID		0x40	/* SyncMOS */
+#define S29C51001T		0x01
+#define S29C51002T		0x02
+#define S29C51004T		0x03
+#define S29C31004T		0x63
 
 /* function prototypes from udelay.h */