Thread (41 messages) 41 messages, 4 authors, 2021-07-13

RE: [PATCH v14 00/12] add ecspi ERR009165 for i.mx6/7 soc family

From: Robin Gong <hidden>
Date: 2021-06-15 02:23:27
Also in: dmaengine, linux-devicetree, linux-spi, lkml

On 06/11/21 21:51 Fabio Estevam [off-list ref] wrote: 
Hi Robin,

On Wed, Apr 7, 2021 at 4:15 AM Robin Gong [off-list ref] wrote:
quoted
There is ecspi ERR009165 on i.mx6/7 soc family, which cause FIFO
transfer to be send twice in DMA mode. Please get more information from:
https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.
nxp.com%2Fdocs%2Fen%2Ferrata%2FIMX6DQCE.pdf&amp;data=04%7C01%7
Cyibin.g
quoted
ong%40nxp.com%7Cab59d4a2e95a4351e48708d92cdffc09%7C686ea1d3bc2
b4c6fa92
quoted
cd99c5c301635%7C0%7C1%7C637590162781662129%7CUnknown%7CTWF
pbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVC
I6Mn0%3D%7C1000&amp;sdata=DIbrrAYvpoJ4lKrkDJYvyoixR9DcMCgVDwuW
gI5fuVw%3D&amp;reserved=0. The workaround is adding new sdma ram
script which works in XCH  mode as PIO inside sdma instead of SMC mode,
meanwhile, 'TX_THRESHOLD' should be 0. The issue should be exist on all
legacy i.mx6/7 soc family before i.mx6ul.
quoted
NXP fix this design issue from i.mx6ul, so newer chips including
i.mx6ul/ 6ull/6sll do not need this workaroud anymore. All other
i.mx6/7/8 chips still need this workaroud. This patch set add new
'fsl,imx6ul-ecspi'
quoted
for ecspi driver and 'ecspi_fixed' in sdma driver to choose if need
errata or not.
The first two reverted patches should be the same issue, though, it
seems 'fixed' by changing to other shp script. Hope Sean or Sascha
could have the chance to test this patch set if could fix their issues.
Besides, enable sdma support for i.mx8mm/8mq and fix ecspi1 not work
on i.mx8mm because the event id is zero.

PS:
   Please get sdma firmware from below linux-firmware and copy it to
your local rootfs /lib/firmware/imx/sdma.
https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit.
kernel.org%2Fpub%2Fscm%2Flinux%2Fkernel%2Fgit%2Ffirmware%2Flinux-fir
mw
quoted
are.git%2Ftree%2Fimx%2Fsdma&amp;data=04%7C01%7Cyibin.gong%40nxp.c
om%7C
quoted
ab59d4a2e95a4351e48708d92cdffc09%7C686ea1d3bc2b4c6fa92cd99c5c301
635%7C
quoted
0%7C1%7C637590162781662129%7CUnknown%7CTWFpbGZsb3d8eyJWIjoi
MC4wLjAwMDA
quoted
iLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=uL
%2F
quoted
6w%2F7JF76dqTfKYk4BT%2F5flWr0d3U2O86ABSq2UhI%3D&amp;reserved=0

Without this series, SPI DMA does not work on i.MX8MM:

 [   41.315984] spi_master spi1: I/O Error in DMA RX

I applied your series and SPI DMA works now:

Reviewed-by: Fabio Estevam <festevam@gmail.com>
Thanks Fabio.
Hello Vinod, Mark, 
Is my patch set good enough to merge? I remember someone else are
requesting it from last year like Fabio. 


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help