From: <Tudor.Ambarus@microchip.com> To: <michael@walle.cc>, <linux-mtd@lists.infradead.org>, <linux-kernel@vger.kernel.org> Cc: <p.yadav@ti.com>, <miquel.raynal@bootlin.com>, <richard@nod.at>, <vigneshr@ti.com> Subject: Re: [PATCH v1 08/14] mtd: spi-nor: micron-st: convert USE_FSR to a manufacturer flag Date: Thu, 10 Feb 2022 03:37:45 +0000 [thread overview] Message-ID: <0eee21f3-8c03-3d82-dc5f-6d9d6fc7fc45@microchip.com> (raw) In-Reply-To: <20220202145853.4187726-9-michael@walle.cc> On 2/2/22 16:58, Michael Walle wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > Now that all functions using that flag are local to the micron module, > we can convert the flag to a manufacturer one. > > Signed-off-by: Michael Walle <michael@walle.cc> > --- > drivers/mtd/spi-nor/core.c | 3 -- > drivers/mtd/spi-nor/core.h | 3 -- > drivers/mtd/spi-nor/micron-st.c | 92 +++++++++++++++++++++------------ > 3 files changed, 59 insertions(+), 39 deletions(-) > > diff --git a/drivers/mtd/spi-nor/core.c b/drivers/mtd/spi-nor/core.c > index e9d9880149d2..be65aaa954ca 100644 > --- a/drivers/mtd/spi-nor/core.c > +++ b/drivers/mtd/spi-nor/core.c > @@ -2499,9 +2499,6 @@ static void spi_nor_init_flags(struct spi_nor *nor) > > if (flags & USE_CLSR) > nor->flags |= SNOR_F_USE_CLSR; > - > - if (flags & USE_FSR) > - nor->flags |= SNOR_F_USE_FSR; > } > > /** > diff --git a/drivers/mtd/spi-nor/core.h b/drivers/mtd/spi-nor/core.h > index fabc01ae9a81..a02bf54289fb 100644 > --- a/drivers/mtd/spi-nor/core.h > +++ b/drivers/mtd/spi-nor/core.h > @@ -12,7 +12,6 @@ > #define SPI_NOR_MAX_ID_LEN 6 > > enum spi_nor_option_flags { > - SNOR_F_USE_FSR = BIT(0), > SNOR_F_HAS_SR_TB = BIT(1), > SNOR_F_NO_OP_CHIP_ERASE = BIT(2), > SNOR_F_USE_CLSR = BIT(4), > @@ -349,7 +348,6 @@ struct spi_nor_fixups { > * NO_CHIP_ERASE: chip does not support chip erase. > * SPI_NOR_NO_FR: can't do fastread. > * USE_CLSR: use CLSR command. > - * USE_FSR: use flag status register > * > * @no_sfdp_flags: flags that indicate support that can be discovered via SFDP. > * Used when SFDP tables are not defined in the flash. These > @@ -401,7 +399,6 @@ struct flash_info { > #define NO_CHIP_ERASE BIT(7) > #define SPI_NOR_NO_FR BIT(8) > #define USE_CLSR BIT(9) > -#define USE_FSR BIT(10) > > u8 no_sfdp_flags; > #define SPI_NOR_SKIP_SFDP BIT(0) > diff --git a/drivers/mtd/spi-nor/micron-st.c b/drivers/mtd/spi-nor/micron-st.c > index c66580e8aa00..33531c101ccb 100644 > --- a/drivers/mtd/spi-nor/micron-st.c > +++ b/drivers/mtd/spi-nor/micron-st.c > @@ -8,6 +8,8 @@ > > #include "core.h" > > +#define USE_FSR BIT(0) please add a description and inform the reader that this is a manufacturer specific flash_info flag. Reviewed-by: Tudor Ambarus <tudor.ambarus@microchip.com> > + > #define SPINOR_OP_RDFSR 0x70 /* Read flag status register */ > #define SPINOR_OP_CLFSR 0x50 /* Clear flag status register */ > #define SPINOR_OP_MT_DTR_RD 0xfd /* Fast Read opcode in DTR mode */ > @@ -140,15 +142,17 @@ static const struct spi_nor_fixups mt35xu512aba_fixups = { > > static const struct flash_info micron_parts[] = { > { "mt35xu512aba", INFO(0x2c5b1a, 0, 128 * 1024, 512) > - FLAGS(USE_FSR) > NO_SFDP_FLAGS(SECT_4K | SPI_NOR_OCTAL_READ | > SPI_NOR_OCTAL_DTR_READ | SPI_NOR_OCTAL_DTR_PP) > FIXUP_FLAGS(SPI_NOR_4B_OPCODES | SPI_NOR_IO_MODE_EN_VOLATILE) > - .fixups = &mt35xu512aba_fixups}, > + MFR_FLAGS(USE_FSR) > + .fixups = &mt35xu512aba_fixups > + }, > { "mt35xu02g", INFO(0x2c5b1c, 0, 128 * 1024, 2048) > - FLAGS(USE_FSR) > NO_SFDP_FLAGS(SECT_4K | SPI_NOR_OCTAL_READ) > - FIXUP_FLAGS(SPI_NOR_4B_OPCODES) }, > + FIXUP_FLAGS(SPI_NOR_4B_OPCODES) > + MFR_FLAGS(USE_FSR) > + }, > }; > > static const struct flash_info st_parts[] = { > @@ -164,57 +168,79 @@ static const struct flash_info st_parts[] = { > NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) }, > { "n25q128a11", INFO(0x20bb18, 0, 64 * 1024, 256) > FLAGS(SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB | SPI_NOR_4BIT_BP | > - SPI_NOR_BP3_SR_BIT6 | USE_FSR) > - NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) }, > + SPI_NOR_BP3_SR_BIT6) > + NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) > + MFR_FLAGS(USE_FSR) > + }, > { "n25q128a13", INFO(0x20ba18, 0, 64 * 1024, 256) > FLAGS(SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB | SPI_NOR_4BIT_BP | > - SPI_NOR_BP3_SR_BIT6 | USE_FSR) > - NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) }, > + SPI_NOR_BP3_SR_BIT6) > + NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) > + MFR_FLAGS(USE_FSR) > + }, > { "mt25ql256a", INFO6(0x20ba19, 0x104400, 64 * 1024, 512) > - FLAGS(USE_FSR) > NO_SFDP_FLAGS(SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) > - FIXUP_FLAGS(SPI_NOR_4B_OPCODES) }, > + FIXUP_FLAGS(SPI_NOR_4B_OPCODES) > + MFR_FLAGS(USE_FSR) > + }, > { "n25q256a", INFO(0x20ba19, 0, 64 * 1024, 512) > - FLAGS(USE_FSR) > NO_SFDP_FLAGS(SECT_4K | SPI_NOR_DUAL_READ | > - SPI_NOR_QUAD_READ) }, > + SPI_NOR_QUAD_READ) > + MFR_FLAGS(USE_FSR) > + }, > { "mt25qu256a", INFO6(0x20bb19, 0x104400, 64 * 1024, 512) > - FLAGS(USE_FSR) > NO_SFDP_FLAGS(SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) > - FIXUP_FLAGS(SPI_NOR_4B_OPCODES) }, > + FIXUP_FLAGS(SPI_NOR_4B_OPCODES) > + MFR_FLAGS(USE_FSR) > + }, > { "n25q256ax1", INFO(0x20bb19, 0, 64 * 1024, 512) > - FLAGS(USE_FSR) > - NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) }, > + NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) > + MFR_FLAGS(USE_FSR) > + }, > { "mt25ql512a", INFO6(0x20ba20, 0x104400, 64 * 1024, 1024) > - FLAGS(USE_FSR) > NO_SFDP_FLAGS(SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) > - FIXUP_FLAGS(SPI_NOR_4B_OPCODES) }, > + FIXUP_FLAGS(SPI_NOR_4B_OPCODES) > + MFR_FLAGS(USE_FSR) > + }, > { "n25q512ax3", INFO(0x20ba20, 0, 64 * 1024, 1024) > FLAGS(SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB | SPI_NOR_4BIT_BP | > - SPI_NOR_BP3_SR_BIT6 | USE_FSR) > - NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) }, > + SPI_NOR_BP3_SR_BIT6) > + NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) > + MFR_FLAGS(USE_FSR) > + }, > { "mt25qu512a", INFO6(0x20bb20, 0x104400, 64 * 1024, 1024) > - FLAGS(USE_FSR) > NO_SFDP_FLAGS(SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) > - FIXUP_FLAGS(SPI_NOR_4B_OPCODES) }, > + FIXUP_FLAGS(SPI_NOR_4B_OPCODES) > + MFR_FLAGS(USE_FSR) > + }, > { "n25q512a", INFO(0x20bb20, 0, 64 * 1024, 1024) > FLAGS(SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB | SPI_NOR_4BIT_BP | > - SPI_NOR_BP3_SR_BIT6 | USE_FSR) > - NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) }, > + SPI_NOR_BP3_SR_BIT6) > + NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) > + MFR_FLAGS(USE_FSR) > + }, > { "n25q00", INFO(0x20ba21, 0, 64 * 1024, 2048) > FLAGS(SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB | SPI_NOR_4BIT_BP | > - SPI_NOR_BP3_SR_BIT6 | NO_CHIP_ERASE | USE_FSR) > - NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) }, > + SPI_NOR_BP3_SR_BIT6 | NO_CHIP_ERASE) > + NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) > + MFR_FLAGS(USE_FSR) > + }, > { "n25q00a", INFO(0x20bb21, 0, 64 * 1024, 2048) > - FLAGS(NO_CHIP_ERASE | USE_FSR) > - NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) }, > + FLAGS(NO_CHIP_ERASE) > + NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) > + MFR_FLAGS(USE_FSR) > + }, > { "mt25ql02g", INFO(0x20ba22, 0, 64 * 1024, 4096) > - FLAGS(NO_CHIP_ERASE | USE_FSR) > - NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) }, > + FLAGS(NO_CHIP_ERASE) > + NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) > + MFR_FLAGS(USE_FSR) > + }, > { "mt25qu02g", INFO(0x20bb22, 0, 64 * 1024, 4096) > - FLAGS(NO_CHIP_ERASE | USE_FSR) > + FLAGS(NO_CHIP_ERASE) > NO_SFDP_FLAGS(SECT_4K | SPI_NOR_DUAL_READ | > - SPI_NOR_QUAD_READ) }, > + SPI_NOR_QUAD_READ) > + MFR_FLAGS(USE_FSR) > + }, > > { "m25p05", INFO(0x202010, 0, 32 * 1024, 2) }, > { "m25p10", INFO(0x202011, 0, 32 * 1024, 4) }, > @@ -406,7 +432,7 @@ static void micron_st_default_init(struct spi_nor *nor) > nor->params->quad_enable = NULL; > nor->params->set_4byte_addr_mode = st_micron_set_4byte_addr_mode; > > - if (nor->flags & SNOR_F_USE_FSR) > + if (nor->info->mfr_flags & USE_FSR) > nor->params->ready = spi_nor_fsr_ready; > } > > -- > 2.30.2 >
WARNING: multiple messages have this Message-ID (diff)
From: <Tudor.Ambarus@microchip.com> To: <michael@walle.cc>, <linux-mtd@lists.infradead.org>, <linux-kernel@vger.kernel.org> Cc: <p.yadav@ti.com>, <miquel.raynal@bootlin.com>, <richard@nod.at>, <vigneshr@ti.com> Subject: Re: [PATCH v1 08/14] mtd: spi-nor: micron-st: convert USE_FSR to a manufacturer flag Date: Thu, 10 Feb 2022 03:37:45 +0000 [thread overview] Message-ID: <0eee21f3-8c03-3d82-dc5f-6d9d6fc7fc45@microchip.com> (raw) In-Reply-To: <20220202145853.4187726-9-michael@walle.cc> On 2/2/22 16:58, Michael Walle wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > Now that all functions using that flag are local to the micron module, > we can convert the flag to a manufacturer one. > > Signed-off-by: Michael Walle <michael@walle.cc> > --- > drivers/mtd/spi-nor/core.c | 3 -- > drivers/mtd/spi-nor/core.h | 3 -- > drivers/mtd/spi-nor/micron-st.c | 92 +++++++++++++++++++++------------ > 3 files changed, 59 insertions(+), 39 deletions(-) > > diff --git a/drivers/mtd/spi-nor/core.c b/drivers/mtd/spi-nor/core.c > index e9d9880149d2..be65aaa954ca 100644 > --- a/drivers/mtd/spi-nor/core.c > +++ b/drivers/mtd/spi-nor/core.c > @@ -2499,9 +2499,6 @@ static void spi_nor_init_flags(struct spi_nor *nor) > > if (flags & USE_CLSR) > nor->flags |= SNOR_F_USE_CLSR; > - > - if (flags & USE_FSR) > - nor->flags |= SNOR_F_USE_FSR; > } > > /** > diff --git a/drivers/mtd/spi-nor/core.h b/drivers/mtd/spi-nor/core.h > index fabc01ae9a81..a02bf54289fb 100644 > --- a/drivers/mtd/spi-nor/core.h > +++ b/drivers/mtd/spi-nor/core.h > @@ -12,7 +12,6 @@ > #define SPI_NOR_MAX_ID_LEN 6 > > enum spi_nor_option_flags { > - SNOR_F_USE_FSR = BIT(0), > SNOR_F_HAS_SR_TB = BIT(1), > SNOR_F_NO_OP_CHIP_ERASE = BIT(2), > SNOR_F_USE_CLSR = BIT(4), > @@ -349,7 +348,6 @@ struct spi_nor_fixups { > * NO_CHIP_ERASE: chip does not support chip erase. > * SPI_NOR_NO_FR: can't do fastread. > * USE_CLSR: use CLSR command. > - * USE_FSR: use flag status register > * > * @no_sfdp_flags: flags that indicate support that can be discovered via SFDP. > * Used when SFDP tables are not defined in the flash. These > @@ -401,7 +399,6 @@ struct flash_info { > #define NO_CHIP_ERASE BIT(7) > #define SPI_NOR_NO_FR BIT(8) > #define USE_CLSR BIT(9) > -#define USE_FSR BIT(10) > > u8 no_sfdp_flags; > #define SPI_NOR_SKIP_SFDP BIT(0) > diff --git a/drivers/mtd/spi-nor/micron-st.c b/drivers/mtd/spi-nor/micron-st.c > index c66580e8aa00..33531c101ccb 100644 > --- a/drivers/mtd/spi-nor/micron-st.c > +++ b/drivers/mtd/spi-nor/micron-st.c > @@ -8,6 +8,8 @@ > > #include "core.h" > > +#define USE_FSR BIT(0) please add a description and inform the reader that this is a manufacturer specific flash_info flag. Reviewed-by: Tudor Ambarus <tudor.ambarus@microchip.com> > + > #define SPINOR_OP_RDFSR 0x70 /* Read flag status register */ > #define SPINOR_OP_CLFSR 0x50 /* Clear flag status register */ > #define SPINOR_OP_MT_DTR_RD 0xfd /* Fast Read opcode in DTR mode */ > @@ -140,15 +142,17 @@ static const struct spi_nor_fixups mt35xu512aba_fixups = { > > static const struct flash_info micron_parts[] = { > { "mt35xu512aba", INFO(0x2c5b1a, 0, 128 * 1024, 512) > - FLAGS(USE_FSR) > NO_SFDP_FLAGS(SECT_4K | SPI_NOR_OCTAL_READ | > SPI_NOR_OCTAL_DTR_READ | SPI_NOR_OCTAL_DTR_PP) > FIXUP_FLAGS(SPI_NOR_4B_OPCODES | SPI_NOR_IO_MODE_EN_VOLATILE) > - .fixups = &mt35xu512aba_fixups}, > + MFR_FLAGS(USE_FSR) > + .fixups = &mt35xu512aba_fixups > + }, > { "mt35xu02g", INFO(0x2c5b1c, 0, 128 * 1024, 2048) > - FLAGS(USE_FSR) > NO_SFDP_FLAGS(SECT_4K | SPI_NOR_OCTAL_READ) > - FIXUP_FLAGS(SPI_NOR_4B_OPCODES) }, > + FIXUP_FLAGS(SPI_NOR_4B_OPCODES) > + MFR_FLAGS(USE_FSR) > + }, > }; > > static const struct flash_info st_parts[] = { > @@ -164,57 +168,79 @@ static const struct flash_info st_parts[] = { > NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) }, > { "n25q128a11", INFO(0x20bb18, 0, 64 * 1024, 256) > FLAGS(SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB | SPI_NOR_4BIT_BP | > - SPI_NOR_BP3_SR_BIT6 | USE_FSR) > - NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) }, > + SPI_NOR_BP3_SR_BIT6) > + NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) > + MFR_FLAGS(USE_FSR) > + }, > { "n25q128a13", INFO(0x20ba18, 0, 64 * 1024, 256) > FLAGS(SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB | SPI_NOR_4BIT_BP | > - SPI_NOR_BP3_SR_BIT6 | USE_FSR) > - NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) }, > + SPI_NOR_BP3_SR_BIT6) > + NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) > + MFR_FLAGS(USE_FSR) > + }, > { "mt25ql256a", INFO6(0x20ba19, 0x104400, 64 * 1024, 512) > - FLAGS(USE_FSR) > NO_SFDP_FLAGS(SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) > - FIXUP_FLAGS(SPI_NOR_4B_OPCODES) }, > + FIXUP_FLAGS(SPI_NOR_4B_OPCODES) > + MFR_FLAGS(USE_FSR) > + }, > { "n25q256a", INFO(0x20ba19, 0, 64 * 1024, 512) > - FLAGS(USE_FSR) > NO_SFDP_FLAGS(SECT_4K | SPI_NOR_DUAL_READ | > - SPI_NOR_QUAD_READ) }, > + SPI_NOR_QUAD_READ) > + MFR_FLAGS(USE_FSR) > + }, > { "mt25qu256a", INFO6(0x20bb19, 0x104400, 64 * 1024, 512) > - FLAGS(USE_FSR) > NO_SFDP_FLAGS(SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) > - FIXUP_FLAGS(SPI_NOR_4B_OPCODES) }, > + FIXUP_FLAGS(SPI_NOR_4B_OPCODES) > + MFR_FLAGS(USE_FSR) > + }, > { "n25q256ax1", INFO(0x20bb19, 0, 64 * 1024, 512) > - FLAGS(USE_FSR) > - NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) }, > + NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) > + MFR_FLAGS(USE_FSR) > + }, > { "mt25ql512a", INFO6(0x20ba20, 0x104400, 64 * 1024, 1024) > - FLAGS(USE_FSR) > NO_SFDP_FLAGS(SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) > - FIXUP_FLAGS(SPI_NOR_4B_OPCODES) }, > + FIXUP_FLAGS(SPI_NOR_4B_OPCODES) > + MFR_FLAGS(USE_FSR) > + }, > { "n25q512ax3", INFO(0x20ba20, 0, 64 * 1024, 1024) > FLAGS(SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB | SPI_NOR_4BIT_BP | > - SPI_NOR_BP3_SR_BIT6 | USE_FSR) > - NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) }, > + SPI_NOR_BP3_SR_BIT6) > + NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) > + MFR_FLAGS(USE_FSR) > + }, > { "mt25qu512a", INFO6(0x20bb20, 0x104400, 64 * 1024, 1024) > - FLAGS(USE_FSR) > NO_SFDP_FLAGS(SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) > - FIXUP_FLAGS(SPI_NOR_4B_OPCODES) }, > + FIXUP_FLAGS(SPI_NOR_4B_OPCODES) > + MFR_FLAGS(USE_FSR) > + }, > { "n25q512a", INFO(0x20bb20, 0, 64 * 1024, 1024) > FLAGS(SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB | SPI_NOR_4BIT_BP | > - SPI_NOR_BP3_SR_BIT6 | USE_FSR) > - NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) }, > + SPI_NOR_BP3_SR_BIT6) > + NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) > + MFR_FLAGS(USE_FSR) > + }, > { "n25q00", INFO(0x20ba21, 0, 64 * 1024, 2048) > FLAGS(SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB | SPI_NOR_4BIT_BP | > - SPI_NOR_BP3_SR_BIT6 | NO_CHIP_ERASE | USE_FSR) > - NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) }, > + SPI_NOR_BP3_SR_BIT6 | NO_CHIP_ERASE) > + NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) > + MFR_FLAGS(USE_FSR) > + }, > { "n25q00a", INFO(0x20bb21, 0, 64 * 1024, 2048) > - FLAGS(NO_CHIP_ERASE | USE_FSR) > - NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) }, > + FLAGS(NO_CHIP_ERASE) > + NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) > + MFR_FLAGS(USE_FSR) > + }, > { "mt25ql02g", INFO(0x20ba22, 0, 64 * 1024, 4096) > - FLAGS(NO_CHIP_ERASE | USE_FSR) > - NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) }, > + FLAGS(NO_CHIP_ERASE) > + NO_SFDP_FLAGS(SECT_4K | SPI_NOR_QUAD_READ) > + MFR_FLAGS(USE_FSR) > + }, > { "mt25qu02g", INFO(0x20bb22, 0, 64 * 1024, 4096) > - FLAGS(NO_CHIP_ERASE | USE_FSR) > + FLAGS(NO_CHIP_ERASE) > NO_SFDP_FLAGS(SECT_4K | SPI_NOR_DUAL_READ | > - SPI_NOR_QUAD_READ) }, > + SPI_NOR_QUAD_READ) > + MFR_FLAGS(USE_FSR) > + }, > > { "m25p05", INFO(0x202010, 0, 32 * 1024, 2) }, > { "m25p10", INFO(0x202011, 0, 32 * 1024, 4) }, > @@ -406,7 +432,7 @@ static void micron_st_default_init(struct spi_nor *nor) > nor->params->quad_enable = NULL; > nor->params->set_4byte_addr_mode = st_micron_set_4byte_addr_mode; > > - if (nor->flags & SNOR_F_USE_FSR) > + if (nor->info->mfr_flags & USE_FSR) > nor->params->ready = spi_nor_fsr_ready; > } > > -- > 2.30.2 > ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/
next prev parent reply other threads:[~2022-02-10 3:37 UTC|newest] Thread overview: 126+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-02-02 14:58 [PATCH v1 00/14] mtd: spi-nor: move vendor specific code into vendor modules Michael Walle 2022-02-02 14:58 ` Michael Walle 2022-02-02 14:58 ` [PATCH v1 01/14] mtd: spi-nor: export more function to be used in " Michael Walle 2022-02-02 14:58 ` Michael Walle 2022-02-10 3:13 ` Tudor.Ambarus 2022-02-10 3:13 ` Tudor.Ambarus 2022-02-15 18:30 ` Pratyush Yadav 2022-02-15 18:30 ` Pratyush Yadav 2022-02-02 14:58 ` [PATCH v1 02/14] mtd: spi-nor: slightly refactor the spi_nor_setup() Michael Walle 2022-02-02 14:58 ` Michael Walle 2022-02-10 3:00 ` Tudor.Ambarus 2022-02-10 3:00 ` Tudor.Ambarus 2022-02-10 8:01 ` Michael Walle 2022-02-10 8:01 ` Michael Walle 2022-02-10 8:05 ` Tudor.Ambarus 2022-02-10 8:05 ` Tudor.Ambarus 2022-02-15 18:32 ` Pratyush Yadav 2022-02-15 18:32 ` Pratyush Yadav 2022-02-02 14:58 ` [PATCH v1 03/14] mtd: spi-nor: allow a flash to define its own ready() function Michael Walle 2022-02-02 14:58 ` Michael Walle 2022-02-10 3:05 ` Tudor.Ambarus 2022-02-10 3:05 ` Tudor.Ambarus 2022-02-15 18:36 ` Pratyush Yadav 2022-02-15 18:36 ` Pratyush Yadav 2022-02-02 14:58 ` [PATCH v1 04/14] mtd: spi-nor: move all xilinx specifics into xilinx.c Michael Walle 2022-02-02 14:58 ` Michael Walle 2022-02-10 3:04 ` Tudor.Ambarus 2022-02-10 3:04 ` Tudor.Ambarus 2022-02-15 18:57 ` Pratyush Yadav 2022-02-15 18:57 ` Pratyush Yadav 2022-02-02 14:58 ` [PATCH v1 05/14] mtd: spi-nor: xilinx: rename vendor specific functions and defines Michael Walle 2022-02-02 14:58 ` Michael Walle 2022-02-02 17:14 ` kernel test robot 2022-02-02 17:14 ` kernel test robot 2022-02-02 17:14 ` kernel test robot 2022-02-02 20:07 ` kernel test robot 2022-02-02 20:07 ` kernel test robot 2022-02-02 20:07 ` kernel test robot 2022-02-10 3:08 ` Tudor.Ambarus 2022-02-10 3:08 ` Tudor.Ambarus 2022-02-10 8:04 ` Michael Walle 2022-02-10 8:04 ` Michael Walle 2022-02-10 8:06 ` Tudor.Ambarus 2022-02-10 8:06 ` Tudor.Ambarus 2022-02-15 8:25 ` Michael Walle 2022-02-15 8:25 ` Michael Walle 2022-02-15 8:52 ` Tudor.Ambarus 2022-02-15 8:52 ` Tudor.Ambarus 2022-02-15 9:58 ` Michael Walle 2022-02-15 9:58 ` Michael Walle 2022-02-15 10:12 ` Tudor.Ambarus 2022-02-15 10:12 ` Tudor.Ambarus 2022-02-15 19:04 ` Pratyush Yadav 2022-02-15 19:04 ` Pratyush Yadav 2022-02-02 14:58 ` [PATCH v1 06/14] mtd: spi-nor: xilinx: correct the debug message Michael Walle 2022-02-02 14:58 ` Michael Walle 2022-02-10 3:11 ` Tudor.Ambarus 2022-02-10 3:11 ` Tudor.Ambarus 2022-02-15 19:05 ` Pratyush Yadav 2022-02-15 19:05 ` Pratyush Yadav 2022-02-02 14:58 ` [PATCH v1 07/14] mtd: spi-nor: move all micron-st specifics into micron-st.c Michael Walle 2022-02-02 14:58 ` Michael Walle 2022-02-10 3:18 ` Tudor.Ambarus 2022-02-10 3:18 ` Tudor.Ambarus 2022-02-15 19:13 ` Pratyush Yadav 2022-02-15 19:13 ` Pratyush Yadav 2022-02-02 14:58 ` [PATCH v1 08/14] mtd: spi-nor: micron-st: convert USE_FSR to a manufacturer flag Michael Walle 2022-02-02 14:58 ` Michael Walle 2022-02-10 3:37 ` Tudor.Ambarus [this message] 2022-02-10 3:37 ` Tudor.Ambarus 2022-02-15 19:16 ` Pratyush Yadav 2022-02-15 19:16 ` Pratyush Yadav 2022-02-02 14:58 ` [PATCH v1 09/14] mtd: spi-nor: micron-st: fix micron_st prefix Michael Walle 2022-02-02 14:58 ` Michael Walle 2022-02-10 3:23 ` Tudor.Ambarus 2022-02-10 3:23 ` Tudor.Ambarus 2022-02-15 19:16 ` Pratyush Yadav 2022-02-15 19:16 ` Pratyush Yadav 2022-02-02 14:58 ` [PATCH v1 10/14] mtd: spi-nor: micron-st: rename vendor specific functions and defines Michael Walle 2022-02-02 14:58 ` Michael Walle 2022-02-10 3:23 ` Tudor.Ambarus 2022-02-10 3:23 ` Tudor.Ambarus 2022-02-02 14:58 ` [PATCH v1 11/14] mtd: spi-nor: spansion: slightly rework control flow in late_init() Michael Walle 2022-02-02 14:58 ` Michael Walle 2022-02-10 3:26 ` Tudor.Ambarus 2022-02-10 3:26 ` Tudor.Ambarus 2022-02-10 8:16 ` Michael Walle 2022-02-10 8:16 ` Michael Walle 2022-02-10 8:42 ` Tudor.Ambarus 2022-02-10 8:42 ` Tudor.Ambarus 2022-02-14 18:59 ` Pratyush Yadav 2022-02-14 18:59 ` Pratyush Yadav 2022-02-02 14:58 ` [PATCH v1 12/14] mtd: spi-nor: move all spansion specifics into spansion.c Michael Walle 2022-02-02 14:58 ` Michael Walle 2022-02-02 18:15 ` kernel test robot 2022-02-02 18:15 ` kernel test robot 2022-02-02 18:15 ` kernel test robot 2022-02-02 20:07 ` kernel test robot 2022-02-02 20:07 ` kernel test robot 2022-02-02 20:07 ` kernel test robot 2022-02-02 21:38 ` [RFC PATCH] mtd: spi-nor: spi_nor_sr_ready_and_clear() can be static kernel test robot 2022-02-02 21:38 ` kernel test robot 2022-02-02 21:38 ` kernel test robot 2022-02-02 21:48 ` [PATCH v1 12/14] mtd: spi-nor: move all spansion specifics into spansion.c kernel test robot 2022-02-02 21:48 ` kernel test robot 2022-02-02 21:48 ` kernel test robot 2022-02-10 3:32 ` Tudor.Ambarus 2022-02-10 3:32 ` Tudor.Ambarus 2022-02-15 19:23 ` Pratyush Yadav 2022-02-15 19:23 ` Pratyush Yadav 2022-02-02 14:58 ` [PATCH v1 13/14] mtd: spi-nor: spansion: convert USE_CLSR to a manufacturer flag Michael Walle 2022-02-02 14:58 ` Michael Walle 2022-02-10 3:34 ` Tudor.Ambarus 2022-02-10 3:34 ` Tudor.Ambarus 2022-02-15 19:25 ` Pratyush Yadav 2022-02-15 19:25 ` Pratyush Yadav 2022-02-02 14:58 ` [PATCH v1 14/14] mtd: spi-nor: renumber flags Michael Walle 2022-02-02 14:58 ` Michael Walle 2022-02-10 3:38 ` Tudor.Ambarus 2022-02-10 3:38 ` Tudor.Ambarus 2022-02-15 19:27 ` Pratyush Yadav 2022-02-15 19:27 ` Pratyush Yadav 2022-02-10 3:42 ` [PATCH v1 00/14] mtd: spi-nor: move vendor specific code into vendor modules Tudor.Ambarus 2022-02-10 3:42 ` Tudor.Ambarus 2022-02-17 7:31 ` Tudor.Ambarus 2022-02-17 7:31 ` Tudor.Ambarus
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=0eee21f3-8c03-3d82-dc5f-6d9d6fc7fc45@microchip.com \ --to=tudor.ambarus@microchip.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mtd@lists.infradead.org \ --cc=michael@walle.cc \ --cc=miquel.raynal@bootlin.com \ --cc=p.yadav@ti.com \ --cc=richard@nod.at \ --cc=vigneshr@ti.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.