Hi,

On Thu Apr 16, 2026 at 11:03 AM CEST, tkuw584924 wrote:
> --- a/drivers/mtd/spi-nor/core.h
> +++ b/drivers/mtd/spi-nor/core.h
> @@ -124,23 +124,22 @@
>  /* Keep these in sync with the list in debugfs.c */
>  enum spi_nor_option_flags {
>  	SNOR_F_HAS_SR_TB	= BIT(0),
> -	SNOR_F_NO_OP_CHIP_ERASE	= BIT(1),

Can we just drop this line and leave the rest as is? It's already
hard to follow any development in spi-nor due to code churn. I don't
think one hole in the flags hurts. The next new flag can then just
reuse it.

-michael

> -	SNOR_F_BROKEN_RESET	= BIT(2),
> -	SNOR_F_4B_OPCODES	= BIT(3),
> -	SNOR_F_HAS_4BAIT	= BIT(4),
> -	SNOR_F_HAS_LOCK		= BIT(5),
> -	SNOR_F_HAS_16BIT_SR	= BIT(6),
> -	SNOR_F_NO_READ_CR	= BIT(7),
> -	SNOR_F_HAS_SR_TB_BIT6	= BIT(8),
> -	SNOR_F_HAS_4BIT_BP      = BIT(9),
> -	SNOR_F_HAS_SR_BP3_BIT6  = BIT(10),
> -	SNOR_F_IO_MODE_EN_VOLATILE = BIT(11),
> -	SNOR_F_SOFT_RESET	= BIT(12),
> -	SNOR_F_SWP_IS_VOLATILE	= BIT(13),
> -	SNOR_F_RWW		= BIT(14),
> -	SNOR_F_ECC		= BIT(15),
> -	SNOR_F_NO_WP		= BIT(16),
> -	SNOR_F_SWAP16		= BIT(17),
> +	SNOR_F_BROKEN_RESET	= BIT(1),
> +	SNOR_F_4B_OPCODES	= BIT(2),
> +	SNOR_F_HAS_4BAIT	= BIT(3),
> +	SNOR_F_HAS_LOCK		= BIT(4),
> +	SNOR_F_HAS_16BIT_SR	= BIT(5),
> +	SNOR_F_NO_READ_CR	= BIT(6),
> +	SNOR_F_HAS_SR_TB_BIT6	= BIT(7),
> +	SNOR_F_HAS_4BIT_BP      = BIT(8),
> +	SNOR_F_HAS_SR_BP3_BIT6  = BIT(9),
> +	SNOR_F_IO_MODE_EN_VOLATILE = BIT(10),
> +	SNOR_F_SOFT_RESET	= BIT(11),
> +	SNOR_F_SWP_IS_VOLATILE	= BIT(12),
> +	SNOR_F_RWW		= BIT(13),
> +	SNOR_F_ECC		= BIT(14),
> +	SNOR_F_NO_WP		= BIT(15),
> +	SNOR_F_SWAP16		= BIT(16),