From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-out.m-online.net ([212.18.0.10]) by merlin.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1V96tO-0005SR-VG for linux-mtd@lists.infradead.org; Tue, 13 Aug 2013 05:11:39 +0000 From: Marek Vasut To: Sascha Hauer Subject: Re: [PATCH 1/4] mtd: m25p80: Pass flags through CAT25_INFO macro Date: Mon, 12 Aug 2013 22:07:03 +0200 References: <1376302946-11534-1-git-send-email-s.hauer@pengutronix.de> <1376302946-11534-2-git-send-email-s.hauer@pengutronix.de> In-Reply-To: <1376302946-11534-2-git-send-email-s.hauer@pengutronix.de> MIME-Version: 1.0 Content-Type: Text/Plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Message-Id: <201308122207.04080.marex@denx.de> Cc: Artem Bityutskiy , linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org, kernel@pengutronix.de List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Dear Sascha Hauer, > The flags may have to be overwritten, so add them to the CAT25_INFO > macro. > > Signed-off-by: Sascha Hauer > --- > 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 Best regards, Marek Vasut