[PATCH 03/10] mtd: spi-nor: add SPI NOR manufacturer IDs

Jagan Teki jteki at openedev.com
Thu Sep 24 13:17:18 PDT 2015


On 2 September 2015 at 01:27, Brian Norris <computersforpeace at gmail.com> wrote:
> These are often similar for CFI (parallel NOR) and for SPI NOR, but they
> aren't always the same, for various reasons (different namespaces,
> company acquisitions and renames, etc.). And some don't have CFI_MFR_*
> entries at all.
>
> So let's make a proper place to list the SPI NOR IDs, with all the SPI
> NOR specific assumptions and comments.
>
> Signed-off-by: Brian Norris <computersforpeace at gmail.com>
> ---
>  include/linux/mtd/spi-nor.h | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
>
> diff --git a/include/linux/mtd/spi-nor.h b/include/linux/mtd/spi-nor.h
> index 321a055bc266..8558793cc0f7 100644
> --- a/include/linux/mtd/spi-nor.h
> +++ b/include/linux/mtd/spi-nor.h
> @@ -11,6 +11,21 @@
>  #define __LINUX_MTD_SPI_NOR_H
>
>  #include <linux/bitops.h>
> +#include <linux/mtd/cfi.h>
> +
> +/*
> + * Manufacturer IDs
> + *
> + * The first byte returned from the flash after sending opcode SPINOR_OP_RDID.
> + * Sometimes these are the same as CFI IDs, but sometimes they aren't.
> + */
> +#define SNOR_MFR_ATMEL         CFI_MFR_ATMEL
> +#define SNOR_MFR_INTEL         CFI_MFR_INTEL
> +#define SNOR_MFR_MICRON                CFI_MFR_ST /* ST Micro <--> Micron */
> +#define SNOR_MFR_MACRONIX      CFI_MFR_MACRONIX
> +#define SNOR_MFR_SPANSION      CFI_MFR_AMD
> +#define SNOR_MFR_SST           CFI_MFR_SST
> +#define SNOR_MFR_WINBOND       0xef

Ideally company names same but looks like winbond is the only diff as
of now, why can't we define WIN and re-use the same from CFI?

>
>  /*
>   * Note on opcode nomenclature: some opcodes have a format like
> --
> 2.5.0.457.gab17608

thanks!
-- 
Jagan | openedev.



More information about the linux-mtd mailing list