Re:Re: [PATCH] bus: mhi: host: pci_generic: Add autosuspend_delay customized support

[Slark Xiao]

At 2026-09-26 19:26:15, "Manivannan Sadhasivam" <mani@xxxxxxxxxx> wrote:
>On Wed, Sep 23, 2026 at 09:24:55AM +0530, Krishna Chaitanya Chundru wrote:
>> 
>> 
>> On 9/22/2026 5:09 PM, Slark Xiao wrote:
>> > At 2026-09-22 12:12:35, "Krishna Chaitanya Chundru" <krishna.chundru@xxxxxxxxxxxxxxxx> wrote:
>> >>
>> >> On 9/19/2026 3:25 PM, Slark Xiao wrote:
>> >>> For some WWAN device, it may get a SYS ERROR issue when resuming
>> >>> from suspend state frequently. Refer to the Qualcomm Windows
>> >>> driver, the default value of autosuspend_delay_ms is set as 5000
>> >>> ms. But in Linux side, all WWAN device were set as 2000ms. We
>> >>> tried to set this value to 5000ms and we can get a positive test
>> >>> result.
>> >>>
>> >>> So we add a support to allow vendor to set a difference value for
>> >>> specific WWAN device.
>> >> you can change auto suspend delay from sysfs, please use that way instead of
>> >> doing in the driver.
>> >>
>> >> - Krishna Chaitanya.
>> > Yes, we can do this by setting the sysfs API . However, not all users 
>> > are aware of this method. Additionally, this approach is only
>> > effective for a single instance; it requires reconfiguration after each
>> > restart, which is quite inconvenient.
>> you can have udev rule to set this without any user intervention.
>
>We should not rely on userspace to fix the SYS_ERROR. Userspace knob should only
>be exercised to customize the delay for power management purposes, not to fix
>firmware crash.

>
Agree. We are still working on the issue debug on the modem side. 

>First we should know whether the default delay can be increased to 5000ms for
>all devices or not. If that is not desired, then we can go for Endpoint specific
>delay.
>
>- Mani
>
For reference, MTK 5G modem T700 is using 5000ms as default value. And Qualcomm
Windows drivers side, the default value is 5000ms as well.
However, I cannot guarantee that if all MHI devices are adjusted to 5000ms, some
power-sensitive devices will not oppose this change. Additional, DELL hopes that
we only modify the SDX72 device while keeping the 2000ms setting for the SDX62
devices, as there are currently no similar issues with the SDX62 platform.

If endpoint specific delay setting is acceptable, I will commit a V2 which includes
some specific changes for DW5934e devices only.

Thanks
>-- 
>மணிவண்ணன் சதாசிவம்