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

Reply via email to