Hi, > -----Original Message----- > From: Jagan Teki [mailto:ja...@openedev.com] > Sent: Wednesday, November 16, 2016 9:33 AM > To: u-boot@lists.denx.de > Cc: Jagan Teki <ja...@openedev.com>; Bin Meng <bmeng...@gmail.com>; > York Sun <york....@nxp.com>; Vignesh R <vigne...@ti.com>; Mugunthan V > N <mugunthan...@ti.com>; Michal Simek <michal.si...@xilinx.com>; Siva > Durga Prasad Paladugu <siva...@xilinx.com> > Subject: [PATCH v6 08/21] sf: Add INFO6 flash_info macro > > INFO6 is for tabulating 6 byte flash parts, Ex: S25FS256S_64K > > Cc: Bin Meng <bmeng...@gmail.com> > Cc: York Sun <york....@nxp.com> > Cc: Vignesh R <vigne...@ti.com> > Cc: Mugunthan V N <mugunthan...@ti.com> > Cc: Michal Simek <michal.si...@xilinx.com> > Cc: Siva Durga Prasad Paladugu <siva...@xilinx.com> > Signed-off-by: Jagan Teki <ja...@openedev.com> > Reviewed-by: Simon Glass <s...@chromium.org> > Reviewed-by: Jagan Teki <ja...@openedev.com> > Tested-by: Jagan Teki <ja...@openedev.com> > --- > drivers/mtd/spi/sf_params.c | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > > diff --git a/drivers/mtd/spi/sf_params.c b/drivers/mtd/spi/sf_params.c index > d46a276..d0c978e 100644 > --- a/drivers/mtd/spi/sf_params.c > +++ b/drivers/mtd/spi/sf_params.c > @@ -27,6 +27,21 @@ > .page_size = 256, \ > .flags = (_flags), > > +#define INFO6(_jedec_id, _ext_id, _sector_size, _n_sectors, _flags) \ > + .id = { \ > + ((_jedec_id) >> 16) & 0xff, \ > + ((_jedec_id) >> 8) & 0xff, \ > + (_jedec_id) & 0xff, \ > + ((_ext_id) >> 16) & 0xff, \ > + ((_ext_id) >> 8) & 0xff, \ > + (_ext_id) & 0xff, \ > + }, \ > + .id_len = 6, Why cant you use macro which you defined in 07/21.
Thanks, Siva \ > + .sector_size = (_sector_size), \ > + .n_sectors = (_n_sectors), \ > + .page_size = 256, \ > + .flags = (_flags), > + > /* SPI/QSPI flash device params structure */ const struct spi_flash_info > spi_flash_ids[] = { > #ifdef CONFIG_SPI_FLASH_ATMEL /* ATMEL */ > -- > 1.9.1 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot