mbox series

[U-Boot,RESEND,v2,0/2] ti_qspi: Move to spi-mem framework

Message ID 20190416160200.6262-1-vigneshr@ti.com
Headers show
Series ti_qspi: Move to spi-mem framework | expand

Message

Vignesh Raghavendra April 16, 2019, 4:01 p.m. UTC
Now that SPI NOR Supports MMIO SPI controllers via spi-mem framework,
move TI QSPI to SPI MEM framework for better performance and avoid
repeating SPI NOR generic code in the driver.

While at that get rid of non DM code, as all boards are expected to
support DM and DT by now.

Tested on AM43xx, dra7xx EVMs. Build tested for all TI platforms using
buildman

v2
* Rebase on top of latest master and Uri's patch [1] removing cl-som-am57x_defconfig
* Remove more un-needed CONFIGs related to TI QSPI that have not been
* Collect Reviewed-bys

[1] http://patchwork.ozlabs.org/patch/1085281/

Vignesh Raghavendra (2):
  spi: ti_qspi: Drop non DM code
  spi: ti_qspi: Convert to spi-mem ops

 drivers/spi/Kconfig          |  13 +-
 drivers/spi/Makefile         |   2 +-
 drivers/spi/ti_qspi.c        | 351 ++++++++++-------------------------
 include/configs/am43xx_evm.h |   6 -
 include/configs/am57xx_evm.h |   5 -
 include/configs/cm_t43.h     |   3 -
 include/configs/dra7xx_evm.h |   5 -
 scripts/config_whitelist.txt |   3 -
 8 files changed, 108 insertions(+), 280 deletions(-)

Comments

Vignesh Raghavendra April 16, 2019, 4:23 p.m. UTC | #1
Tom, Jagan

On 16/04/19 9:31 PM, Vignesh Raghavendra wrote:
> Now that SPI NOR Supports MMIO SPI controllers via spi-mem framework,
> move TI QSPI to SPI MEM framework for better performance and avoid
> repeating SPI NOR generic code in the driver.
> 
> While at that get rid of non DM code, as all boards are expected to
> support DM and DT by now.
> 
> Tested on AM43xx, dra7xx EVMs. Build tested for all TI platforms using
> buildman
> 
> v2
> * Rebase on top of latest master and Uri's patch [1] removing cl-som-am57x_defconfig
> * Remove more un-needed CONFIGs related to TI QSPI that have not been
> * Collect Reviewed-bys
> 

I am not sure how this dependency can be handled. But since [1] would go 
through Tom's TI related pull, one solution would be to take this series 
also via the same tree with Jagan's Ack/Review to avoid build issues. 
Does that sound good?

 > [1] http://patchwork.ozlabs.org/patch/1085281/

> 
> Vignesh Raghavendra (2):
>    spi: ti_qspi: Drop non DM code
>    spi: ti_qspi: Convert to spi-mem ops
> 
>   drivers/spi/Kconfig          |  13 +-
>   drivers/spi/Makefile         |   2 +-
>   drivers/spi/ti_qspi.c        | 351 ++++++++++-------------------------
>   include/configs/am43xx_evm.h |   6 -
>   include/configs/am57xx_evm.h |   5 -
>   include/configs/cm_t43.h     |   3 -
>   include/configs/dra7xx_evm.h |   5 -
>   scripts/config_whitelist.txt |   3 -
>   8 files changed, 108 insertions(+), 280 deletions(-)
>