lkml.org 
[lkml]   [2019]   [Feb]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH V12 3/5] i2c: tegra: Add DMA support
From
Date
06.02.2019 17:19, Sowjanya Komatineni пишет:
>>>>> No, there are no timeout errors in PIO mode. I could post full log with the PIO-only mode if you want.
>>>>>
>>>> Yes please post full log of PIO only and DMA only Also, in above log,
>>>> lots of DMA transfer went thru fine except this one transfer where
>>>> DMA timed out.
>>>> DMA completion doesnt happen on DMA submit during this particular
>>>> transaction timeframe for some reason.
>>>
>>> Dmitry, can you provide some background information on the setup that
>>> you're using? I vaguely remember something about i2c@7000d000 being
>>> special on older Tegra devices. Perhaps there's something additional
>>> that we need to take into account?
>>>
>>> Aha! Look at tegra_i2c_reg_addr() which special-cases I2C_TX_FIFO on
>>> the DVC variant of the controller, which is exactly the one that you
>>> are seeing the timeouts on and it's exactly the register that we use
>>> as destination for the DMA transfer.
>>>
>>> I think we need to do something like this:
>>>
>>> slv_config.dst_addr = i2c_dev->base_phys +
>>> tegra_i2c_reg_addr(i2c_dev, I2C_TX_FIFO);
>>>
>>> to make sure we pass the right FIFO register address to DMA.
>>
>> I'm running it on T20 at the moment. I don't know about the DVC quirks, but sounds plausible. Please let me know if you need any other info.
>>
>> Here the full logs:
>>
>> PIO-only: http://dpaste.com/2MX3EAN.txt
>> DMA-only: http://dpaste.com/3VXZSSM.txt
>>
>> Note the "Failed to read T44 and T5 (-110)" error that happens in DMA-only. And it happens on i2c@7000c000, DVC is i2c@7000d000.
>
> Yes this is due to same issue theirry mentioned.
> Prior to transfer timeout for touch, tps6586x regulator probe failed due to i2c timeout which uses DVC (7000D000) and since DMA mode is enforced all DVC/PWR I2C transactions also go thru DMA and FIFO register for DVC offset is not accounted when configure to dma slave.
> Will fix and send so you can see for repro with updated patch..
>
>
> [ 0.925155] tegra-i2c 7000d000.i2c: starting DMA for length: 16
> [ 0.925165] tegra-i2c 7000d000.i2c: unmasked irq: 0c
> [ 1.028961] tegra-i2c 7000d000.i2c: i2c transfer timed out
> [ 1.028968] tegra-i2c 7000c000.i2c: DMA transfer timeout
> [ 1.028995] tps6586x 3-0034: Chip ID read failed: -110
> [ 1.029029] tps6586x: probe of 3-0034 failed with error -5
> [ 1.029045] atmel_mxt_ts 0-004c: __mxt_read_reg: i2c transfer failed (-110)
> [ 1.029051] atmel_mxt_ts 0-004c: Failed to read T44 and T5 (-110)
>

I think tps6586x and atmel-touch should be independent issues, but will see. Thanks!

\
 
 \ /
  Last update: 2019-02-06 15:25    [W:0.119 / U:0.148 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site