mtd: spi-nor: print flash ID instead of name
authorTudor Ambarus <tudor.ambarus@linaro.org>
Fri, 15 Dec 2023 08:21:35 +0000 (10:21 +0200)
committerTudor Ambarus <tudor.ambarus@linaro.org>
Tue, 19 Dec 2023 02:43:00 +0000 (04:43 +0200)
We saw flash ID collisions which make the flash name unreliable. Print
the manufacturer and device ID instead of the flash name.

Lower the print to dev_dbg to stop polluting the kernel log.

Suggested-by: Miquel Raynal <miquel.raynal@bootlin.com>
Reviewed-by: Michael Walle <michael@walle.cc>
Link: https://lore.kernel.org/r/20231215082138.16063-2-tudor.ambarus@linaro.org
Signed-off-by: Tudor Ambarus <tudor.ambarus@linaro.org>
drivers/mtd/spi-nor/core.c

index 503fed90c2fa575b453212b1d07f79a246186a28..ca5bd93d1f17100e5c563c009576015227fae097 100644 (file)
@@ -3558,8 +3558,8 @@ int spi_nor_scan(struct spi_nor *nor, const char *name,
        /* No mtd_info fields should be used up to this point. */
        spi_nor_set_mtd_info(nor);
 
-       dev_info(dev, "%s (%lld Kbytes)\n", info->name,
-                       (long long)mtd->size >> 10);
+       dev_dbg(dev, "Manufacturer and device ID: %*phN\n",
+               SPI_NOR_MAX_ID_LEN, nor->id);
 
        dev_dbg(dev,
                "mtd .name = %s, .size = 0x%llx (%lldMiB), "