Dear Sascha Hauer, > The flags may have to be overwritten, so add them to the CAT25_INFO > macro. > > Signed-off-by: Sascha Hauer <s.ha...@pengutronix.de> > --- > drivers/mtd/devices/m25p80.c | 16 ++++++++-------- > 1 file changed, 8 insertions(+), 8 deletions(-) > > diff --git a/drivers/mtd/devices/m25p80.c b/drivers/mtd/devices/m25p80.c > index 2f3d2a5..75d4391 100644 > --- a/drivers/mtd/devices/m25p80.c > +++ b/drivers/mtd/devices/m25p80.c > @@ -694,13 +694,13 @@ struct flash_info { > .flags = (_flags), \ > }) > > -#define CAT25_INFO(_sector_size, _n_sectors, _page_size, _addr_width) \ > +#define CAT25_INFO(_sector_size, _n_sectors, _page_size, _addr_width, > _flags) \ ((kernel_ulong_t)&(struct flash_info) { > \ > .sector_size = (_sector_size), \ > .n_sectors = (_n_sectors), \ > .page_size = (_page_size), \ > .addr_width = (_addr_width), \ > - .flags = M25P_NO_ERASE, \ > + .flags = (_flags), \ > }) > > /* NOTE: double check command sets and memory organization when you add > @@ -732,7 +732,7 @@ static const struct spi_device_id m25p_ids[] = { > { "en25qh256", INFO(0x1c7019, 0, 64 * 1024, 512, 0) }, > > /* Everspin */ > - { "mr25h256", CAT25_INFO( 32 * 1024, 1, 256, 2) }, > + { "mr25h256", CAT25_INFO( 32 * 1024, 1, 256, 2, M25P_NO_ERASE) }, > > /* GigaDevice */ > { "gd25q32", INFO(0xc84016, 0, 64 * 1024, 64, SECT_4K) }, > @@ -846,11 +846,11 @@ static const struct spi_device_id m25p_ids[] = { > { "w25q256", INFO(0xef4019, 0, 64 * 1024, 512, SECT_4K) }, > > /* Catalyst / On Semiconductor -- non-JEDEC */ > - { "cat25c11", CAT25_INFO( 16, 8, 16, 1) }, > - { "cat25c03", CAT25_INFO( 32, 8, 16, 2) }, > - { "cat25c09", CAT25_INFO( 128, 8, 32, 2) }, > - { "cat25c17", CAT25_INFO( 256, 8, 32, 2) }, > - { "cat25128", CAT25_INFO(2048, 8, 64, 2) }, > + { "cat25c11", CAT25_INFO( 16, 8, 16, 1, M25P_NO_ERASE) }, > + { "cat25c03", CAT25_INFO( 32, 8, 16, 2, M25P_NO_ERASE) }, > + { "cat25c09", CAT25_INFO( 128, 8, 32, 2, M25P_NO_ERASE) }, > + { "cat25c17", CAT25_INFO( 256, 8, 32, 2, M25P_NO_ERASE) }, > + { "cat25128", CAT25_INFO(2048, 8, 64, 2, M25P_NO_ERASE) }, > { }, > }; > MODULE_DEVICE_TABLE(spi, m25p_ids);
I can't say I like the growing macro magic (voodoo?) in this file, but your patch doesn't break anything and I don't feel like reworking this, so: Acked-by: Marek Vasut <ma...@denx.de> Best regards, Marek Vasut -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/