[U-Boot] [PATCH 12/12] sf: Pass rd_qeb_req variable as 0 for status and config reg reads
Simon Glass
sjg at chromium.org
Fri Jan 11 03:21:40 CET 2013
Hi Jagannadha,
On Mon, Dec 31, 2012 at 4:14 AM, Jagannadha Sutradharudu Teki
<jagannadh.teki at gmail.com> wrote:
> Config and Status register reads doesn't require to set
> the quad enable bit, hence rd_qeb_req variable passed as 0.
>
> Signed-off-by: Jagannadha Sutradharudu Teki <jagannadh.teki at gmail.com>
A few more general comments:
Please can you look at enhancing 'sf test' to support your modes. Then
we can run 'sf test' and test which ones work.
I wonder if we should something to help with things like
spi_flash_read_common(flash, &cmd, sizeof(cmd), (void *)data, 1);
Perhaps you should define spi_flash_xfer_byte(flash, int cmd, int *resp)
which writes a single byte command and reads a single byte response?
Regards,
Simon
> ---
> drivers/mtd/spi/spi_flash.c | 6 ++++--
> 1 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/mtd/spi/spi_flash.c b/drivers/mtd/spi/spi_flash.c
> index 15ad05c..7a15a69 100644
> --- a/drivers/mtd/spi/spi_flash.c
> +++ b/drivers/mtd/spi/spi_flash.c
> @@ -293,7 +293,8 @@ int spi_flash_cmd_read_status(struct spi_flash *flash, void *data)
> int ret;
>
> cmd = CMD_READ_STATUS;
> - ret = spi_flash_read_common(flash, &cmd, sizeof(cmd), (void *)data, 1);
> + ret = spi_flash_read_common(flash, &cmd, 0,
> + sizeof(cmd), (void *)data, 1);
> if (ret < 0) {
> debug("SF: fail to read status register\n");
> return ret;
> @@ -343,7 +344,8 @@ int spi_flash_cmd_read_config(struct spi_flash *flash, void *data)
> int ret;
>
> cmd = CMD_READ_CONFIG;
> - ret = spi_flash_read_common(flash, &cmd, sizeof(cmd), (void *)data, 1);
> + ret = spi_flash_read_common(flash, &cmd, 0,
> + sizeof(cmd), (void *)data, 1);
> if (ret < 0) {
> debug("SF: fail to read config register\n");
> return ret;
> --
> 1.7.0.4
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
More information about the U-Boot
mailing list