[U-Boot] [PATCH 4/5] sf: Update SST25* flash params of supported read commands

[Jagan Teki]

On 23 October 2014 19:07, Bin Meng <bmeng.cn at gmail.com> wrote:
> Explicitly list supported read commands in the flash prarmas table
> for SST25* flash parts.
>
> Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
> ---
>  drivers/mtd/spi/sf_params.c | 20 ++++++++++----------
>  include/spi_flash.h         |  3 ++-
>  2 files changed, 12 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/mtd/spi/sf_params.c b/drivers/mtd/spi/sf_params.c
> index 453edf0..d5f3597 100644
> --- a/drivers/mtd/spi/sf_params.c
> +++ b/drivers/mtd/spi/sf_params.c
> @@ -88,16 +88,16 @@ const struct spi_flash_params spi_flash_params_table[] = {
>         {"N25Q1024A",      0x20bb21, 0x0,       64 * 1024,  2048, RD_FULL, WR_QPP | E_FSR | SECT_4K},
>  #endif
>  #ifdef CONFIG_SPI_FLASH_SST            /* SST */
> -       {"SST25VF040B",    0xbf258d, 0x0,       64 * 1024,     8,       0,          SECT_4K | SST_WP},
> -       {"SST25VF080B",    0xbf258e, 0x0,       64 * 1024,    16,       0,          SECT_4K | SST_WP},
> -       {"SST25VF016B",    0xbf2541, 0x0,       64 * 1024,    32,       0,          SECT_4K | SST_WP},
> -       {"SST25VF032B",    0xbf254a, 0x0,       64 * 1024,    64,       0,          SECT_4K | SST_WP},
> -       {"SST25VF064C",    0xbf254b, 0x0,       64 * 1024,   128,       0,                   SECT_4K},
> -       {"SST25WF512",     0xbf2501, 0x0,       64 * 1024,     1,       0,          SECT_4K | SST_WP},
> -       {"SST25WF010",     0xbf2502, 0x0,       64 * 1024,     2,       0,          SECT_4K | SST_WP},
> -       {"SST25WF020",     0xbf2503, 0x0,       64 * 1024,     4,       0,          SECT_4K | SST_WP},
> -       {"SST25WF040",     0xbf2504, 0x0,       64 * 1024,     8,       0,          SECT_4K | SST_WP},
> -       {"SST25WF080",     0xbf2505, 0x0,       64 * 1024,    16,       0,          SECT_4K | SST_WP},
> +       {"SST25VF040B",    0xbf258d, 0x0,       64 * 1024,     8, RD_SLOW,          SECT_4K | SST_WP},
> +       {"SST25VF080B",    0xbf258e, 0x0,       64 * 1024,    16, RD_SLOW,          SECT_4K | SST_WP},
> +       {"SST25VF016B",    0xbf2541, 0x0,       64 * 1024,    32, RD_SLOW,          SECT_4K | SST_WP},
> +       {"SST25VF032B",    0xbf254a, 0x0,       64 * 1024,    64, RD_SLOW,          SECT_4K | SST_WP},
> +       {"SST25VF064C",    0xbf254b, 0x0,       64 * 1024,   128, RD_EXTN,                   SECT_4K},
> +       {"SST25WF512",     0xbf2501, 0x0,       64 * 1024,     1, RD_SLOW,          SECT_4K | SST_WP},
> +       {"SST25WF010",     0xbf2502, 0x0,       64 * 1024,     2, RD_SLOW,          SECT_4K | SST_WP},
> +       {"SST25WF020",     0xbf2503, 0x0,       64 * 1024,     4, RD_SLOW,          SECT_4K | SST_WP},
> +       {"SST25WF040",     0xbf2504, 0x0,       64 * 1024,     8, RD_SLOW,          SECT_4K | SST_WP},
> +       {"SST25WF080",     0xbf2505, 0x0,       64 * 1024,    16, RD_SLOW,          SECT_4K | SST_WP},
>  #endif
>  #ifdef CONFIG_SPI_FLASH_WINBOND                /* WINBOND */
>         {"W25P80",         0xef2014, 0x0,       64 * 1024,    16,       0,                         0},
> diff --git a/include/spi_flash.h b/include/spi_flash.h
> index 408a5b4..a75e030 100644
> --- a/include/spi_flash.h
> +++ b/include/spi_flash.h
> @@ -46,7 +46,8 @@ enum spi_read_cmds {
>         QUAD_OUTPUT_FAST = 1 << 3,
>         QUAD_IO_FAST = 1 << 4,
>  };
> -#define RD_EXTN                ARRAY_SLOW | DUAL_OUTPUT_FAST | DUAL_IO_FAST
> +#define RD_SLOW                ARRAY_SLOW
> +#define RD_EXTN                RD_SLOW | DUAL_OUTPUT_FAST | DUAL_IO_FAST
>  #define RD_FULL                RD_EXTN | QUAD_OUTPUT_FAST | QUAD_IO_FAST
>
>  /* Dual SPI flash memories */
> --
> 1.8.2.1
>

What is this, by defining RD_EXTN the fastest read cmd will pick based
on controller mode_rx
Why this RD_SLOW again? does this means the specific flash part will
only support slow read?

thanks!
-- 
Jagan.