Re: [PATCH] USB: dwc2: write HCINT with INTMASK applied

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Ivan,

On 11/28/23 18:43, Ivan Ivanov wrote:
> 
> Hi Minas,
> 
> On Tue, 2023-11-28 at 11:48 +0000, Minas Harutyunyan wrote:
>>
>> Does this "spurious" interrupt broke your tests?
> 
> It is not just some kind of synthetic test case that was broken.
> but real world usage. You can find complains about this error on
> various internet forums, just search for dwc2_hc_chhltd_intr_dma
> and it is not so difficult to reproduce.
> 
> Without databook I am not sure we can create better fix, but if
> you develop different solution I will gladly tested it.
> 
> Regards,
> Ivan
1. In addition to HCCHARx and ep_type printing please add printing of 
GRXFSTSR if EP is IN or GNPTXSTS if EP is OUT, and provide dmesg with 
error case.
2. In function dwc2_hc_chhltd_intr_dma() remove "goto error;" after 
printing "ChHltd set, but reason is unknown".

Questions.
1. Do you see in the dmesg "Transaction Error--" before "ChHltd set, but 
reason is unknown"?
2. Your driver built for Host mode only or for both Device/Host?
3. If you build driver with debug and verbose debug prints enabled, 
issue still seen?
4. Andrea wrote: "we are atrss testing via ping flooding
through an LTE modem attached". This about USB2.0 ping flood in USB 
trace or just "ping" user command?

Thanks,
Minas




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux