On Sun, 17 May 2009, Daniel Ribeiro wrote:

> This chip reports CFI 1.3, but the CFI PRI is like CFI 1.1. Add a quirk
> to pass probe on this chip.
> (This patch depends on "cfi_cmdset_0001.c: CFI 1.0 and CFI 1.1")
> (sent again with CC to Nicolas Pitre)
> 
> Signed-off-by: Daniel Ribeiro <drw...@gmail.com>

Acked-by: Nicolas Pitre <n...@cam.org>

> ---
>  drivers/mtd/chips/cfi_cmdset_0001.c |   12 ++++++++++++
>  1 files changed, 12 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/mtd/chips/cfi_cmdset_0001.c 
> b/drivers/mtd/chips/cfi_cmdset_0001.c
> index bf07d14..0e01ab9 100644
> --- a/drivers/mtd/chips/cfi_cmdset_0001.c
> +++ b/drivers/mtd/chips/cfi_cmdset_0001.c
> @@ -46,6 +46,7 @@
>  #define MANUFACTURER_INTEL   0x0089
>  #define I82802AB     0x00ad
>  #define I82802AC     0x00ac
> +#define PF38F4476    0x881c
>  #define MANUFACTURER_ST         0x0020
>  #define M50LPW080       0x002F
>  #define M50FLW080A   0x0080
> @@ -315,9 +316,18 @@ static struct cfi_fixup fixup_table[] = {
>       { 0, 0, NULL, NULL }
>  };
>  
> +static void cfi_fixup_major_minor(struct cfi_private *cfi,
> +                                             struct cfi_pri_intelext *extp)
> +{
> +     if (cfi->mfr == MANUFACTURER_INTEL &&
> +                     cfi->id == PF38F4476 && extp->MinorVersion == '3')
> +             extp->MinorVersion = '1';
> +}
> +
>  static inline struct cfi_pri_intelext *
>  read_pri_intelext(struct map_info *map, __u16 adr)
>  {
> +     struct cfi_private *cfi = map->fldrv_priv;
>       struct cfi_pri_intelext *extp;
>       unsigned int extra_size = 0;
>       unsigned int extp_size = sizeof(*extp);
> @@ -327,6 +337,8 @@ read_pri_intelext(struct map_info *map, __u16 adr)
>       if (!extp)
>               return NULL;
>  
> +     cfi_fixup_major_minor(cfi, extp);
> +
>       if (extp->MajorVersion != '1' ||
>           (extp->MinorVersion < '0' || extp->MinorVersion > '5')) {
>               printk(KERN_ERR "  Unknown Intel/Sharp Extended Query "
> -- 
> tg: (385157b..) mtd/pf38f4476-fix (depends on: mtd/cfi1.1-has-extra-too)
> 
> -- 
> Daniel Ribeiro
> 

Reply via email to