Thread (48 messages) 48 messages, 4 authors, 2021-11-29

Re: [PATCH v2 1/9] i2c: i801: Improve disabling runtime pm

From: Jean Delvare <hidden>
Date: 2021-08-31 11:26:13

On Tue, 31 Aug 2021 08:05:41 +0200, Heiner Kallweit wrote:
On 26.08.2021 16:00, Jean Delvare wrote:
quoted
If pm_runtime_set_autosuspend_delay() is not suitable for the task then
maybe we need a better API. I will admit I'm at a loss when it comes to
the many pm_runtime_* calls, I'm not going to claim I fully understand
what each of them is doing exactly. But don't we want to simply call
pm_runtime_dont_use_autosuspend() here?

If not and there's no suitable API for the task at the moment, then
better do not apply this patch, and instead ask the PM subsystem
maintainers if they would be willing to implement what we need.
To follow-up on this: This patch has been applied already. Therefore,
if decision is to not go with it, it would need to be reverted.
Technically it's not in Linus' tree yet ;-)

I'm still interested to know if pm_runtime_dont_use_autosuspend() is
the right call to use in this situation.

-- 
Jean Delvare
SUSE L3 Support
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help