From: Jean Delvare <jdelvare@suse.de>
To: Heiner Kallweit <hkallweit1@gmail.com>
Cc: Wolfram Sang <wsa@kernel.org>,
Andy Shevchenko <andriy.shevchenko@intel.com>,
Jarkko Nikula <jarkko.nikula@linux.intel.com>,
linux-i2c@vger.kernel.org,
"Rafael J. Wysocki" <rafael@kernel.org>
Subject: Re: [PATCH v2 1/9] i2c: i801: Improve disabling runtime pm
Date: Tue, 31 Aug 2021 13:26:09 +0200 [thread overview]
Message-ID: <20210831132609.201c37f2@endymion> (raw)
In-Reply-To: <41456674-800b-a4c2-3b2c-d5af18c58e61@gmail.com>
On Tue, 31 Aug 2021 08:05:41 +0200, Heiner Kallweit wrote:
> On 26.08.2021 16:00, Jean Delvare wrote:
> > 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
next prev parent reply other threads:[~2021-08-31 11:26 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-06 21:10 [PATCH v2 0/9] i2c: i801: Series with improvements Heiner Kallweit
2021-08-06 21:12 ` [PATCH v2 1/9] i2c: i801: Improve disabling runtime pm Heiner Kallweit
2021-08-10 20:37 ` Wolfram Sang
2021-08-11 15:41 ` Andy Shevchenko
2021-08-11 20:03 ` Heiner Kallweit
2021-08-12 9:48 ` Andy Shevchenko
2021-08-17 20:15 ` Wolfram Sang
2021-08-26 14:00 ` Jean Delvare
2021-08-26 14:34 ` Andy Shevchenko
2021-08-31 6:05 ` Heiner Kallweit
2021-08-31 11:26 ` Jean Delvare [this message]
2021-08-31 20:43 ` Heiner Kallweit
2021-09-01 6:22 ` Heiner Kallweit
2021-08-06 21:13 ` [PATCH v2 2/9] i2c: i801: make p2sb_spinlock a mutex Heiner Kallweit
2021-08-10 20:38 ` Wolfram Sang
2021-08-11 15:43 ` Andy Shevchenko
2021-08-11 20:27 ` Heiner Kallweit
2021-08-12 9:53 ` Andy Shevchenko
2021-08-06 21:14 ` [PATCH v2 3/9] i2c: i801: Remove not needed debug message Heiner Kallweit
2021-08-10 20:39 ` Wolfram Sang
2021-08-06 21:15 ` [PATCH v2 4/9] i2c: i801: Improve is_dell_system_with_lis3lv02d Heiner Kallweit
2021-08-11 15:45 ` Andy Shevchenko
2021-08-11 20:28 ` Heiner Kallweit
2021-08-12 9:55 ` Andy Shevchenko
2021-08-26 14:19 ` Jean Delvare
2021-08-26 13:21 ` Jean Delvare
2021-09-29 19:38 ` Wolfram Sang
2021-08-06 21:15 ` [PATCH v2 5/9] i2c: i801: Remove not needed check for PCI_COMMAND_INTX_DISABLE Heiner Kallweit
2021-08-11 15:46 ` Andy Shevchenko
2021-08-26 15:18 ` Jean Delvare
2021-09-29 19:38 ` Wolfram Sang
2021-08-06 21:16 ` [PATCH v2 6/9] i2c: i801: Improve i801_acpi_probe/remove functions Heiner Kallweit
2021-09-29 19:38 ` Wolfram Sang
2021-08-06 21:17 ` [PATCH v2 7/9] i2c: i801: Improve i801_add_mux Heiner Kallweit
2021-09-29 19:38 ` Wolfram Sang
2021-08-06 21:18 ` [PATCH v2 8/9] i2c: i801: Improve register_dell_lis3lv02d_i2c_device Heiner Kallweit
2021-08-11 15:52 ` Andy Shevchenko
2021-08-11 21:05 ` Heiner Kallweit
2021-08-12 9:57 ` Andy Shevchenko
2021-08-27 16:21 ` Jean Delvare
2021-09-29 20:11 ` Wolfram Sang
2021-10-01 11:46 ` Jean Delvare
2021-08-06 21:18 ` [PATCH v2 9/9] i2c: i801: Improve handling platform data for tco device Heiner Kallweit
2021-08-11 15:53 ` Andy Shevchenko
2021-10-02 7:44 ` Wolfram Sang
2021-11-29 8:58 ` Wolfram Sang
2021-11-29 19:54 ` Heiner Kallweit
2021-11-29 19:53 ` [PATCH v3] " Heiner Kallweit
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210831132609.201c37f2@endymion \
--to=jdelvare@suse.de \
--cc=andriy.shevchenko@intel.com \
--cc=hkallweit1@gmail.com \
--cc=jarkko.nikula@linux.intel.com \
--cc=linux-i2c@vger.kernel.org \
--cc=rafael@kernel.org \
--cc=wsa@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.