On Wednesday 22 July 2009 14:03:04 Niklaus Giger wrote:
> see http://www.jedec.org/download/search/jep106Z.pdf
> Add some second source legacy flash chips 256x8.

Please add an empty line here.

> Signed-off-by: Niklaus Giger <niklaus.gi...@member.fsf.org>

And still some more white space cleanup comments below (sorry, I didn't catch 
them before).

> ---
>  drivers/mtd/cfi_flash.c   |   15 +++++++++-
>  drivers/mtd/jedec_flash.c |   67
> +++++++++++++++++++++++++++++++++++++++++++++ include/flash.h           |  
> 10 ++++++-
>  3 files changed, 89 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/mtd/cfi_flash.c b/drivers/mtd/cfi_flash.c
> index 81ac5d3..4a4b697 100644
> --- a/drivers/mtd/cfi_flash.c
> +++ b/drivers/mtd/cfi_flash.c
> @@ -106,6 +106,8 @@
>  #define ATM_CMD_SOFTLOCK_START               0x80
>  #define ATM_CMD_LOCK_SECT            0x40
>
> +#define FLASH_CONTINUATION_CODE         0x7F

                                  ^^^^^^^^

Indentation by tab's please.

> +
>  #define FLASH_OFFSET_MANUFACTURER_ID 0x00
>  #define FLASH_OFFSET_DEVICE_ID               0x01
>  #define FLASH_OFFSET_DEVICE_ID2              0x0E
> @@ -1541,13 +1543,22 @@ static int cmdset_intel_init(flash_info_t *info,
> struct cfi_qry *qry)
>
>  static void cmdset_amd_read_jedec_ids(flash_info_t *info)
>  {
> +     ushort bankId = 0;
> +     uchar  manuId;
> +
>       flash_write_cmd(info, 0, 0, AMD_CMD_RESET);
>       flash_unlock_seq(info, 0);
>       flash_write_cmd(info, 0, info->addr_unlock1, FLASH_CMD_READ_ID);
>       udelay(1000); /* some flash are slow to respond */
>
> -     info->manufacturer_id = flash_read_uchar (info,
> -                                     FLASH_OFFSET_MANUFACTURER_ID);
> +     manuId = flash_read_uchar (info, FLASH_OFFSET_MANUFACTURER_ID);
> +     /* JEDEC JEP106Z specifies ID codes up to bank 7 */
> +     while (manuId == FLASH_CONTINUATION_CODE && bankId < 0x800) {
> +             bankId += 0x100;
> +             manuId = flash_read_uchar (info,
> +                     bankId | FLASH_OFFSET_MANUFACTURER_ID);
> +     }
> +     info->manufacturer_id = manuId;
>
>       switch (info->chipwidth){
>       case FLASH_CFI_8BIT:
> diff --git a/drivers/mtd/jedec_flash.c b/drivers/mtd/jedec_flash.c
> index e48acec..376a323 100644
> --- a/drivers/mtd/jedec_flash.c
> +++ b/drivers/mtd/jedec_flash.c
> @@ -68,6 +68,17 @@
>  #define SST39SF010A  0x00B5
>  #define SST39SF020A  0x00B6
>
> +/* MXIC */
> +#define MX29LV040    0x004F
> +
> +/* WINBOND */
> +#define W39L040A        0x00D6

tabs again.

> +
> +/* AMIC */
> +#define A29L040         0x0092

and here.

> +
> +/* EON */
> +#define EN29LV040A      0x004F

and here.

Thanks.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: off...@denx.de
=====================================================================
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to