On Mon, Oct 01, 2007 at 02:53:35PM +0200, Carl-Daniel Hailfinger wrote: > Index: util/flashrom/flash.h > =================================================================== > --- util/flashrom/flash.h (Revision 2814) > +++ util/flashrom/flash.h (Arbeitskopie) > @@ -55,6 +55,8 @@ > /* 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). > + * All LPC/FWH parts (parallel flash) have 8-bit device IDs. > + * All SPI parts have 16-bit device IDs. > */ > > #define AMD_ID 0x01 /* AMD */ > @@ -68,8 +70,31 @@ > #define AT_29C040A 0xA4 > #define AT_29C020 0xDA > > +#define EON_ID 0x1C > +/* EN25 chips are SPI, first byte of device id is memory type, > + second byte of device id is log(bitsize)-9 */
id -> ID
> +#define EN_25B05 0x2010 /* 2^19 kbit or 2^16 kByte */
> +#define EN_25B10 0x2011
> +#define EN_25B20 0x2012
> +#define EN_25B40 0x2013
> +#define EN_25B80 0x2014
> +#define EN_25B16 0x2015
> +#define EN_25B32 0x2016
> +
> #define MX_ID 0xC2 /* Macronix (MX) */
> #define MX_29F002 0xB0
> +/* MX25L chips are SPI, first byte of device id is memory type,
> + second byte of device id is log(bitsize)-9 */
id -> ID
> +#define MX_25L512 0x2010 /* 2^19 kbit or 2^16 kByte */
> +#define MX_25L1005 0x2011
> +#define MX_25L2005 0x2012
> +#define MX_25L4005 0x2013 /* MX25L4005{,A} */
> +#define MX_25L8005 0x2014
> +#define MX_25L1605 0x2015 /* MX25L1605{,A,D} */
> +#define MX_25L3205 0x2016 /* MX25L3205{,A} */
> +#define MX_25L6405 0x2017 /* MX25L3205{,D} */
> +#define MX_25L1635D 0x2415
> +#define MX_25L3235D 0x2416
>
> #define SHARP_ID 0xB0 /* Sharp */
> #define SHARP_LHF00L04 0xCF
> @@ -182,6 +207,8 @@
> int linuxbios_init(void);
> extern char *lb_part, *lb_vendor;
>
> +int probe_spi(struct flashchip *flash);
> +
> /* 82802ab.c */
> int probe_82802ab(struct flashchip *flash);
> int erase_82802ab(struct flashchip *flash);
> Index: util/flashrom/board_enable.c
> ===================================================================
> --- util/flashrom/board_enable.c (Revision 2814)
> +++ util/flashrom/board_enable.c (Arbeitskopie)
> @@ -30,6 +30,15 @@
> #include <string.h>
> #include "flash.h"
>
> +#define ITE_SUPERIO_PORT1 0x2e
> +#define ITE_SUPERIO_PORT2 0x4e
> +
> +#define JEDEC_RDID {0x9f}
> +#define JEDEC_RDID_OUTSIZE 0x01
> +#define JEDEC_RDID_INSIZE 0x03
> +
> +static uint16_t it8716f_flashport = 0;
> +
> /* Generic Super I/O helper functions */
> uint8_t regval(uint16_t port, uint8_t reg)
> {
> @@ -51,7 +60,7 @@
> outb(0x87, port);
> outb(0x01, port);
> outb(0x55, port);
> - if (port == 0x2e)
> + if (port == ITE_SUPERIO_PORT1)
> outb(0x55, port);
> else
> outb(0xaa, port);
> @@ -96,35 +105,97 @@
> return flashport;
> }
>
> -static void it8716_serial_rdid(uint16_t port)
> +/* The IT8716F only supports commands with length 1,2,4,5 bytes including
> + command byte and can not read more than 3 bytes from the device.
> + This function expects writearr[0] to be the first byte sent to the device,
> + whereas the IT8716F splits commands internally into address and
> non-address
> + commands with the address in inverse wire order. That's why the register
> + ordering in case 4 and 5 may seem strange. */
Please make all multi-line comments look the same as in the rest of the
code:
/* Foo
* Bar
*/
Or, if we want to use the exact Linux-style:
/*
* Foo
* Bar
*/
> +static int it8716f_spi_command(uint16_t port, unsigned char writecnt,
> unsigned char readcnt, const unsigned char *writearr, unsigned char *readarr)
As per another mail, writecnt/readcnt/writearr/readarr seems to have a
dont-care size, so I think we can make them all 'int' (or 'unsigned int'
if they can't or shouldn't get negative).
I think using the "native" int type is the best solution in general, it
gives the compiler maximum freedom to optimize (not that it would matter
much here).
> Index: util/flashrom/flashchips.c
> ===================================================================
> --- util/flashrom/flashchips.c (Revision 2814)
> +++ util/flashrom/flashchips.c (Arbeitskopie)
> @@ -38,6 +38,8 @@
> probe_jedec, erase_chip_jedec, write_jedec},
> {"Mx29f002", MX_ID, MX_29F002, 256, 64 * 1024,
> probe_29f002, erase_29f002, write_29f002},
> + {"MX25L4005", MX_ID, MX_25L4005, 512, 4 * 1024,
> + probe_spi, NULL, NULL},
All the other chips from above are not added?
> {"SST29EE020A", SST_ID, SST_29EE020A, 256, 128,
> probe_jedec, erase_chip_jedec, write_jedec},
> {"SST28SF040A", SST_ID, SST_28SF040, 512, 256,
With the above fixes:
Acked-by: Uwe Hermann <[EMAIL PROTECTED]>
Uwe.
--
http://www.hermann-uwe.de | http://www.holsham-traders.de
http://www.crazy-hacks.org | http://www.unmaintained-free-software.org
signature.asc
Description: Digital signature
-- linuxbios mailing list [email protected] http://www.linuxbios.org/mailman/listinfo/linuxbios
