Re: [PATCH] MTD: m25p80: fix inconsistency in m25p_ids for Micron devices

From: Brian Norris
Date: Tue Mar 03 2015 - 02:07:24 EST


On Mon, Mar 02, 2015 at 10:26:51AM +0100, Mike Looijmans wrote:
> As stated in a5b7616c5, "mtd: m25p80,spi-nor: Fix module aliases for
> m25p80", m25p_ids[] in m25p80.c needs to be kept in sync with
> spi_nor_ids[] in spi-nor.c.
>
> This patch fixes the mismatches for the Micron devices, the
> "n25q256a" and "n25q512a" do not exist in the spi_nor_ids, so

Huh?

$ git grep -n 'n25q512a' drivers/mtd
drivers/mtd/devices/m25p80.c:269: {"n25q512a"}, {"n25q512ax3"}, {"n25q00"},
drivers/mtd/spi-nor/spi-nor.c:569: { "n25q512a", INFO(0x20bb20, 0, 64 * 1024, 1024, SECT_4K | USE_FSR | SPI_NOR_QUAD_READ) },
drivers/mtd/spi-nor/spi-nor.c:570: { "n25q512ax3", INFO(0x20ba20, 0, 64 * 1024, 1024, SECT_4K | USE_FSR | SPI_NOR_QUAD_READ) },
$ git grep -n 'n25q256a' drivers/mtd
drivers/mtd/devices/m25p80.c:268: {"n25q064"}, {"n25q128a11"}, {"n25q128a13"}, {"n25q256a"},
drivers/mtd/spi-nor/spi-nor.c:568: { "n25q256a", INFO(0x20ba19, 0, 64 * 1024, 512, SECT_4K | SPI_NOR_QUAD_READ) },

> replace them with the correct names for these chips.
>
> This repairs the disappearance of NOR flash on the Miami boards since 3.18.

$ git grep -n 'n25q256a' v3.18 -- drivers/mtd
v3.18:drivers/mtd/devices/m25p80.c:283: {"n25q064"}, {"n25q128a11"}, {"n25q128a13"}, {"n25q256a"},
v3.18:drivers/mtd/spi-nor/spi-nor.c:538: { "n25q256a", INFO(0x20ba19, 0, 64 * 1024, 512, SECT_4K) },
$ git grep -n 'n25q512a' v3.18 -- drivers/mtd
v3.18:drivers/mtd/devices/m25p80.c:284: {"n25q512a"}, {"n25q512ax3"}, {"n25q00"},
v3.18:drivers/mtd/spi-nor/spi-nor.c:539: { "n25q512a", INFO(0x20bb20, 0, 64 * 1024, 1024, SECT_4K) },
v3.18:drivers/mtd/spi-nor/spi-nor.c:540: { "n25q512ax3", INFO(0x20ba20, 0, 64 * 1024, 1024, USE_FSR) },

Perhaps you're looking at a modified 3.18 kernel from your vendor?

> Signed-off-by: Mike Looijmans <mike.looijmans@xxxxxxxx>
> ---
> drivers/mtd/devices/m25p80.c | 7 +++++--
> 1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/mtd/devices/m25p80.c b/drivers/mtd/devices/m25p80.c
> index b2b33e4..402172c 100644
> --- a/drivers/mtd/devices/m25p80.c
> +++ b/drivers/mtd/devices/m25p80.c
> @@ -280,8 +280,11 @@ static const struct spi_device_id m25p_ids[] = {
> {"mx25l3205d"}, {"mx25l3255e"}, {"mx25l6405d"}, {"mx25l12805d"},
> {"mx25l12855e"},{"mx25l25635e"},{"mx25l25655e"},{"mx66l51235l"},
> {"mx66l1g55g"},
> - {"n25q064"}, {"n25q128a11"}, {"n25q128a13"}, {"n25q256a"},
> - {"n25q512a"}, {"n25q512ax3"}, {"n25q00"},
> + {"n25q064"},
> + {"n25q128a11"}, {"n25q128a13"},
> + {"n25q256a11"}, {"n25q256a13"},
> + {"n25q512a11"}, {"n25q512a13"}, {"n25q512ax3"},
> + {"n25q00"},

Even if I were to take your change (which I will not), please don't make
arbitrary whitespace changes.

> {"pm25lv512"}, {"pm25lv010"}, {"pm25lq032"},
> {"s25sl032p"}, {"s25sl064p"}, {"s25fl256s0"}, {"s25fl256s1"},
> {"s25fl512s"}, {"s70fl01gs"}, {"s25sl12800"}, {"s25sl12801"},

Brian
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/