Re: [PATCH v3 2/2] mtd: spi: nor: force mtd name to "nor%d"

2021-09-17 Thread Marek Vasut
On 9/17/21 3:06 PM, Patrick DELAUNAY wrote: Hi Marek, > Marek VasutSept. 16, 2021, 5:27 p.m. UTC | #3 > On 9/16/21 4:01 PM, Patrick Delaunay wrote: > [...] > > @@ -3664,6 +3666,11 @@ int spi_nor_scan(struct spi_nor *nor) > >       struct mtd_info *mtd = >mtd; > >       struct spi_slave

Re: [PATCH v3 2/2] mtd: spi: nor: force mtd name to "nor%d"

2021-09-17 Thread Patrick DELAUNAY
Hi Marek, > Marek VasutSept. 16, 2021, 5:27 p.m. UTC | #3 > On 9/16/21 4:01 PM, Patrick Delaunay wrote: > [...] > > @@ -3664,6 +3666,11 @@ int spi_nor_scan(struct spi_nor *nor) > >       struct mtd_info *mtd = >mtd; > >       struct spi_slave *spi = nor->spi; > >       int ret; > > +    int

Re: [PATCH v3 2/2] mtd: spi: nor: force mtd name to "nor%d"

2021-09-16 Thread Marek Behún
On Thu, 16 Sep 2021 16:01:18 +0200 Patrick Delaunay wrote: > Force the mtd name of spi-nor to "nor" + the driver sequence number: > "nor0", "nor1"... beginning after the existing nor devices. > > This patch is coherent with existing "nand" and "spi-nand" > mtd device names. > > When CFI MTD

Re: [PATCH v3 2/2] mtd: spi: nor: force mtd name to "nor%d"

2021-09-16 Thread Marek Vasut
On 9/16/21 4:01 PM, Patrick Delaunay wrote: [...] @@ -3664,6 +3666,11 @@ int spi_nor_scan(struct spi_nor *nor) struct mtd_info *mtd = >mtd; struct spi_slave *spi = nor->spi; int ret; + int cfi_mtd_nb = 0; + +#ifdef CONFIG_SYS_MAX_FLASH_BANKS + cfi_mtd_nb =

Re: [PATCH v3 2/2] mtd: spi: nor: force mtd name to "nor%d"

2021-09-16 Thread Marek Behún
Hi Marek, Patrick, the patches that made this change also include patch https://source.denx.de/u-boot/u-boot/-/commit/dcb9a80359d699cf659c95b9b6e6604e2d68adb9 This patch was added so that when there are multiple identical SPI-NOR chips on the board, you can still select between them in the `mtd`

[PATCH v3 2/2] mtd: spi: nor: force mtd name to "nor%d"

2021-09-16 Thread Patrick Delaunay
Force the mtd name of spi-nor to "nor" + the driver sequence number: "nor0", "nor1"... beginning after the existing nor devices. This patch is coherent with existing "nand" and "spi-nand" mtd device names. When CFI MTD NOR device are supported, the spi-nor index is chosen after the last CFI