On Sun, Jun 23, 2019 at 21:30 Fabio Estevam <festevam@xxxxxxxxx> wrote: > Hi Russell, > > On Sat, Jun 22, 2019 at 5:27 PM Russell King - ARM Linux admin > <linux@xxxxxxxxxxxxxxx> wrote: > > > > On Sat, Jun 22, 2019 at 08:26:53PM +0100, Russell King - ARM Linux admin > wrote: > > > Well, this doesn't appear to completely solve the problem either - > > > one out of four of my platforms still spat out the error (because > > > the SDMA initialisation can run on a different CPU to that which > > > receives the interrupt.) > > > > > > I've thought about using a completion, but that doesn't work either, > > > because in the case of a single CPU, the interrupts will be masked, > > > so we can't wait for completion. I think we need to eliminate that > > > spinlock around this code. > > > > It looks like iMX6 Dual does not initialise DMA properly using the 1.1 > > firmware - md5sum is: > > > > 5d4584134cc4cba62e1be2f382cd6f3a > > /lib/firmware/imx/sdma/sdma-imx6q.bin > > > > I've tried extending the timeout to 5ms, checking HI[0] (both from the > > interrupt handler and from sdma_run_channel0() to cover the case of a > > single-core setup). > > > > After boot: > > > > 60: 0 0 GPC 2 Level sdma > > > > So no interrupt was received. Looking at the registers: > > > > # /shared/bin32/devmem2 0x20ec02c > > Value at address 0x020ec02c: 0x00000000 <= H_INTRMASK # > > /shared/bin32/devmem2 0x20ec004 Value at address 0x020ec004: > > 0x00000000 <= H_INTR # /shared/bin32/devmem2 0x20ec00c Value at > > address 0x020ec00c: 0x00000000 <= H_START # /shared/bin32/devmem2 > > 0x20ec008 Value at address 0x020ec008: 0x00000001 <= H_STATSTOP > > > > Any ideas? Seems sdma script not run as expected, thus no DONE instruction involved to clear 'HE' of H_STATSTOP and notify ARM by interrupt. So this timeout happened during the first ' sdma_load_script()' phase ? > Could you please try this patch from Robin? > https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Flists.infra > dead.org%2Fpipermail%2Flinux-arm-kernel%2F2019-June%2F661914.html&a > mp;data=02%7C01%7Cyibin.gong%40nxp.com%7C7faa18517626429780d908 > d6f7ded3b6%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C636968 > 933747699843&sdata=BIipoIgBc5sMahJkz33L5ucqeuHwyYnqg09ornpeLE > 4%3D&reserved=0 This should be the different case, since in Russell King's case, no any interrupt while my patch fix the potential interrupt storm. > > Thanks
