Re: [PATCH v3 1/2] cpufreq: Fix a comment in cpufreq_policy_free

On Wed, Dec 1, 2021 at 8:19 AM Viresh Kumar [off-list ref] wrote:

On 01-12-21, 15:40, Tang Yizhou wrote:

quoted

Make the comment of blocking_notifier_call_chain() easier to
understand.

Signed-off-by: Tang Yizhou <redacted>
---
 drivers/cpufreq/cpufreq.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c
index e338d2f010fe..db0b4b4258d5 100644
--- a/drivers/cpufreq/cpufreq.c
+++ b/drivers/cpufreq/cpufreq.c

@@ -1296,8 +1296,9 @@ static void cpufreq_policy_free(struct cpufreq_policy *policy)

      if (policy->max_freq_req) {
              /*
-              * CPUFREQ_CREATE_POLICY notification is sent only after
-              * successfully adding max_freq_req request.
+              * Remove max_freq_req after sending CPUFREQ_REMOVE_POLICY
+              * notification, since CPUFREQ_CREATE_POLICY notification was
+              * sent after adding max_freq_req earlier.
               */
              blocking_notifier_call_chain(&cpufreq_policy_notifier_list,
                                           CPUFREQ_REMOVE_POLICY, policy);

Acked-by: Viresh Kumar <viresh.kumar@linaro.org>

Applied as 5.16-rc material along with the [2/2[, thanks!