Hi,

On 5 May 2015 at 05:10, Haikun Wang <haikun.w...@freescale.com> wrote:
> Atmel DataFlash chips have commands different from common spi
> flash commands.
> Atmel DataFlash also have special page-size.
> This driver add support for accessing Atmel DataFlash.
> It is based on the Driver Model.
>
> Signed-off-by: Haikun Wang <haikun.w...@freescale.com>
> ---
> Verified with AT45DB021B.
>
> Changes in v3:
> - 1. Rename file spi_dataflash.c to sf_dataflash.c
> - 2. Add comment for array "dataflash_data"
>
> Changes in v2:
> - 1. Correct comment style
> - 2. Use get_timer in dataflash_waitready to check whether timeout
> - 3. Remove struct spi_flash * in struct dataflash, and get it from 
> udevice->uclass_priv
> - 4. Replace spi_flash_write_common with spi_flash_cmd_write
> - 5. Replace spi_flash_read with spi_flash_cmd_read
> - 6. Change type of varible "status" form char to u8 in dataflash_status
> - 7. Change add_dataflash's argument type due to <change 3>
> - 8. Add claim_bus and release_bus in erase/write/read due to <change 4 & 5>
>
> Changes in v1: None
>  drivers/mtd/spi/Makefile       |   1 +
>  drivers/mtd/spi/sf_dataflash.c | 711 
> +++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 712 insertions(+)
>  create mode 100644 drivers/mtd/spi/sf_dataflash.c

I would prefer you use dev_get_priv() and friends instead of direct
access. But anyway:

Acked-by: Simon Glass <s...@chromium.org>

Will this go through the spi tree?

Regards,
Simon
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to