From: Hans de Goede <hdegoede@redhat.com> To: Andy Shevchenko <andy.shevchenko@gmail.com> Cc: Lee Jones <lee.jones@linaro.org>, Cezary Rojewski <cezary.rojewski@intel.com>, Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>, Liam Girdwood <liam.r.girdwood@linux.intel.com>, Jie Yang <yang.jie@linux.intel.com>, Mark Brown <broonie@kernel.org>, patches@opensource.cirrus.com, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Charles Keepax <ckeepax@opensource.cirrus.com>, ALSA Development Mailing List <alsa-devel@alsa-project.org> Subject: Re: [PATCH v2 07/12] ASoC: arizona-jack: Use arizona->dev for runtime-pm Date: Fri, 22 Jan 2021 01:03:14 +0100 [thread overview] Message-ID: <d0189116-d110-f91a-afd9-70f323833e32@redhat.com> (raw) In-Reply-To: <CAHp75Vc92WoJm5T1jbF7UUjCNrVZr2as8ccEWxCZ9aP7z+ZyLA@mail.gmail.com> Hi, On 1/18/21 1:02 PM, Andy Shevchenko wrote: > On Sun, Jan 17, 2021 at 6:06 PM Hans de Goede <hdegoede@redhat.com> wrote: >> >> Use arizona->dev for runtime-pm as the main shared/libray code from >> sound/soc/codecs/arizona.c does. > > Can you elaborate switchings from get() to get_sync() in few places Sorry, those 2 changes really should have been in a separate commit. I've put the 2 get -> get_sync() changed in their own commit now with the following commit-msg: """ extcon: arizona: Always use pm_runtime_get_sync() when we need the device to be awake Before this commit the extcon-arizona code was mixing pm_runtime_get() and pm_runtime_get_sync() in different places. In all cases where either function is called we make use of the device immediately afterwards. This means that we should always use pm_runtime_get_sync(). """ > along with moving disable()? The enable / disable calls are not moved, they are removed. Here is a new commit msg which hopefully explains this better which I plan to use for v3: """ Drivers for MFD child-devices such as the arizona codec drivers and the arizona-extcon driver can choose to either make runtime_pm_get/_put calls on their own child-device, which will then be propagated to their parent; or they can make them directly on their MFD parent-device. The arizona-extcon code was using runtime_pm_get/_put calls on its own child-device where as the codec drivers are using runtime_pm_get/_put calls on their parent. The arizona-extcon MFD cell/child-device has been removed and this commit is part of refactoring the arizona-extcon code into a library to be used directly from the codec drivers. Specifically this commit moves the code over to make runtime_pm_get/_put calls on the parent device (on arizona->dev) bringing the code inline with how the codec drivers do this. Note this also removes the pm_runtime_enable/_disable calls as pm_runtime support has already been enabled on the parent-device by the arizona MFD driver. """ Regards, Hans
WARNING: multiple messages have this Message-ID (diff)
From: Hans de Goede <hdegoede@redhat.com> To: Andy Shevchenko <andy.shevchenko@gmail.com> Cc: Cezary Rojewski <cezary.rojewski@intel.com>, Charles Keepax <ckeepax@opensource.cirrus.com>, ALSA Development Mailing List <alsa-devel@alsa-project.org>, patches@opensource.cirrus.com, Jie Yang <yang.jie@linux.intel.com>, Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Liam Girdwood <liam.r.girdwood@linux.intel.com>, Mark Brown <broonie@kernel.org>, Lee Jones <lee.jones@linaro.org> Subject: Re: [PATCH v2 07/12] ASoC: arizona-jack: Use arizona->dev for runtime-pm Date: Fri, 22 Jan 2021 01:03:14 +0100 [thread overview] Message-ID: <d0189116-d110-f91a-afd9-70f323833e32@redhat.com> (raw) In-Reply-To: <CAHp75Vc92WoJm5T1jbF7UUjCNrVZr2as8ccEWxCZ9aP7z+ZyLA@mail.gmail.com> Hi, On 1/18/21 1:02 PM, Andy Shevchenko wrote: > On Sun, Jan 17, 2021 at 6:06 PM Hans de Goede <hdegoede@redhat.com> wrote: >> >> Use arizona->dev for runtime-pm as the main shared/libray code from >> sound/soc/codecs/arizona.c does. > > Can you elaborate switchings from get() to get_sync() in few places Sorry, those 2 changes really should have been in a separate commit. I've put the 2 get -> get_sync() changed in their own commit now with the following commit-msg: """ extcon: arizona: Always use pm_runtime_get_sync() when we need the device to be awake Before this commit the extcon-arizona code was mixing pm_runtime_get() and pm_runtime_get_sync() in different places. In all cases where either function is called we make use of the device immediately afterwards. This means that we should always use pm_runtime_get_sync(). """ > along with moving disable()? The enable / disable calls are not moved, they are removed. Here is a new commit msg which hopefully explains this better which I plan to use for v3: """ Drivers for MFD child-devices such as the arizona codec drivers and the arizona-extcon driver can choose to either make runtime_pm_get/_put calls on their own child-device, which will then be propagated to their parent; or they can make them directly on their MFD parent-device. The arizona-extcon code was using runtime_pm_get/_put calls on its own child-device where as the codec drivers are using runtime_pm_get/_put calls on their parent. The arizona-extcon MFD cell/child-device has been removed and this commit is part of refactoring the arizona-extcon code into a library to be used directly from the codec drivers. Specifically this commit moves the code over to make runtime_pm_get/_put calls on the parent device (on arizona->dev) bringing the code inline with how the codec drivers do this. Note this also removes the pm_runtime_enable/_disable calls as pm_runtime support has already been enabled on the parent-device by the arizona MFD driver. """ Regards, Hans
next prev parent reply other threads:[~2021-01-22 0:05 UTC|newest] Thread overview: 70+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-01-17 16:05 [PATCH v2 00/12] MFD/extcon/ASoC: Rework arizona codec jack-detect support Hans de Goede 2021-01-17 16:05 ` Hans de Goede 2021-01-17 16:05 ` [PATCH v2 01/12] mfd: arizona: Drop arizona-extcon cells Hans de Goede 2021-01-17 16:05 ` Hans de Goede 2021-01-17 16:05 ` [PATCH v2 02/12] ASoC: arizona-jack: Add arizona-jack.c Hans de Goede 2021-01-17 16:05 ` Hans de Goede 2021-01-18 11:57 ` Andy Shevchenko 2021-01-18 11:57 ` Andy Shevchenko 2021-01-17 16:05 ` [PATCH v2 03/12] ASoC: arizona-jack: Fix some issues when HPDET IRQ fires after the jack has been unplugged Hans de Goede 2021-01-17 16:05 ` Hans de Goede 2021-01-18 12:47 ` Mark Brown 2021-01-18 12:47 ` Mark Brown 2021-01-21 15:58 ` Hans de Goede 2021-01-21 15:58 ` Hans de Goede 2021-01-22 11:07 ` Charles Keepax 2021-01-22 11:07 ` Charles Keepax 2021-01-17 16:05 ` [PATCH v2 04/12] ASoC: arizona-jack: Fix various races on driver unbind Hans de Goede 2021-01-17 16:05 ` Hans de Goede 2021-01-17 16:05 ` [PATCH v2 05/12] ASoC: arizona-jack: Fix flags parameter to the gpiod_get("wlf,micd-pol") call Hans de Goede 2021-01-17 16:05 ` [PATCH v2 05/12] ASoC: arizona-jack: Fix flags parameter to the gpiod_get("wlf, micd-pol") call Hans de Goede 2021-01-17 16:05 ` [PATCH v2 06/12] ASoC: arizona-jack: Move jack-detect variables to struct arizona_priv Hans de Goede 2021-01-17 16:05 ` Hans de Goede 2021-01-18 12:01 ` Andy Shevchenko 2021-01-18 12:01 ` Andy Shevchenko 2021-01-22 11:12 ` Charles Keepax 2021-01-22 11:12 ` Charles Keepax 2021-01-17 16:05 ` [PATCH v2 07/12] ASoC: arizona-jack: Use arizona->dev for runtime-pm Hans de Goede 2021-01-17 16:05 ` Hans de Goede 2021-01-18 12:02 ` Andy Shevchenko 2021-01-18 12:02 ` Andy Shevchenko 2021-01-22 0:03 ` Hans de Goede [this message] 2021-01-22 0:03 ` Hans de Goede 2021-01-22 9:38 ` Andy Shevchenko 2021-01-22 9:38 ` Andy Shevchenko 2021-01-22 13:56 ` Hans de Goede 2021-01-22 13:56 ` Hans de Goede 2021-01-17 16:05 ` [PATCH v2 08/12] ASoC: arizona-jack: convert into a helper library for codec drivers Hans de Goede 2021-01-17 16:05 ` Hans de Goede 2021-01-18 17:24 ` Andy Shevchenko 2021-01-18 17:24 ` Andy Shevchenko 2021-01-19 9:51 ` Richard Fitzgerald 2021-01-19 9:51 ` Richard Fitzgerald 2021-01-21 16:55 ` Hans de Goede 2021-01-21 16:55 ` Hans de Goede 2021-01-22 11:26 ` Charles Keepax 2021-01-22 11:26 ` Charles Keepax 2021-01-22 12:23 ` Hans de Goede 2021-01-22 12:23 ` Hans de Goede 2021-01-22 13:04 ` Charles Keepax 2021-01-22 13:04 ` Charles Keepax 2021-01-22 13:36 ` Hans de Goede 2021-01-22 13:36 ` Hans de Goede 2021-01-22 13:21 ` Charles Keepax 2021-01-22 13:21 ` Charles Keepax 2021-01-17 16:05 ` [PATCH v2 09/12] ASoC: arizona-jack: Use snd_soc_jack to report jack events Hans de Goede 2021-01-17 16:05 ` Hans de Goede 2021-01-17 16:05 ` [PATCH v2 10/12] ASoC: arizona: Make the wm5102, wm5110, wm8997 and wm8998 drivers use the new jack library Hans de Goede 2021-01-17 16:05 ` Hans de Goede 2021-01-17 16:05 ` [PATCH v2 11/12] ASoC: Intel: bytcr_wm5102: Add jack detect support Hans de Goede 2021-01-17 16:05 ` Hans de Goede 2021-01-17 16:05 ` [PATCH v2 12/12] extcon: arizona: Drop the arizona extcon driver Hans de Goede 2021-01-17 16:05 ` Hans de Goede 2021-01-18 12:13 ` Andy Shevchenko 2021-01-18 12:13 ` Andy Shevchenko 2021-01-18 9:55 ` [PATCH v2 00/12] MFD/extcon/ASoC: Rework arizona codec jack-detect support Lee Jones 2021-01-18 9:55 ` Lee Jones 2021-01-18 10:28 ` Hans de Goede 2021-01-18 10:28 ` Hans de Goede 2021-01-18 10:47 ` Lee Jones 2021-01-18 10:47 ` Lee Jones
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=d0189116-d110-f91a-afd9-70f323833e32@redhat.com \ --to=hdegoede@redhat.com \ --cc=alsa-devel@alsa-project.org \ --cc=andy.shevchenko@gmail.com \ --cc=broonie@kernel.org \ --cc=cezary.rojewski@intel.com \ --cc=ckeepax@opensource.cirrus.com \ --cc=lee.jones@linaro.org \ --cc=liam.r.girdwood@linux.intel.com \ --cc=linux-kernel@vger.kernel.org \ --cc=patches@opensource.cirrus.com \ --cc=pierre-louis.bossart@linux.intel.com \ --cc=yang.jie@linux.intel.com \ /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: linkBe 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.