From mboxrd@z Thu Jan 1 00:00:00 1970 From: R, Vignesh Date: Mon, 17 Aug 2015 13:52:14 +0530 Subject: [U-Boot] [U-Boot RESEND v2 09/10] spi: ti_qspi: Use DMA to read from qspi flash In-Reply-To: References: <1439798397-1397-1-git-send-email-vigneshr@ti.com> <1439798397-1397-10-git-send-email-vigneshr@ti.com> Message-ID: <55D199B6.9070901@ti.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 8/17/2015 1:48 PM, Jagan Teki wrote: > On 17 August 2015 at 13:29, Vignesh R wrote: >> ti_qspi uses memory map mode for faster read. Enabling DMA will increase >> read speed by 3x @48MHz on DRA74 EVM. >> >> Signed-off-by: Vignesh R >> Reviewed-by: Jagan Teki >> --- >> drivers/spi/ti_qspi.c | 23 +++++++++++++++++++++++ >> 1 file changed, 23 insertions(+) >> >> diff --git a/drivers/spi/ti_qspi.c b/drivers/spi/ti_qspi.c >> index 3356c0f072e5..753d68980bd6 100644 >> --- a/drivers/spi/ti_qspi.c >> +++ b/drivers/spi/ti_qspi.c >> @@ -13,6 +13,8 @@ >> #include >> #include >> #include >> +#include >> +#include >> >> /* ti qpsi register bit masks */ >> #define QSPI_TIMEOUT 2000000 >> @@ -347,3 +349,24 @@ int spi_xfer(struct spi_slave *slave, unsigned int bitlen, const void *dout, >> >> return 0; >> } > Please add below comment here, I have asked the same on previous version patch > this will track us to the work future. > > /* TODO: control from sf layer to here through dm-spi */ Oops.. Sorry, I overlooked it.. Will add the comment and send it soon. Thanks! >> +#ifdef CONFIG_TI_EDMA3 >> +void spi_flash_copy_mmap(void *data, void *offset, size_t len) >> +{ >> + unsigned int addr = (unsigned int) (data); >> + unsigned int edma_slot_num = 1; >> + >> + /* Invalidate the area, so no writeback into the RAM races with DMA */ >> + invalidate_dcache_range(addr, addr + roundup(len, ARCH_DMA_MINALIGN)); >> + >> + /* enable edma3 clocks */ >> + enable_edma3_clocks(); >> + >> + /* Call edma3 api to do actual DMA transfer */ >> + edma3_transfer(EDMA3_BASE, edma_slot_num, data, offset, len); >> + >> + /* disable edma3 clocks */ >> + disable_edma3_clocks(); >> + >> + *((unsigned int *)offset) += len; >> +} >> +#endif >> -- >> 2.5.0 > > thanks! >