From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751497AbcBEIeN (ORCPT ); Fri, 5 Feb 2016 03:34:13 -0500 Received: from mail-pa0-f52.google.com ([209.85.220.52]:35598 "EHLO mail-pa0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751328AbcBEIeK (ORCPT ); Fri, 5 Feb 2016 03:34:10 -0500 Date: Fri, 5 Feb 2016 14:04:06 +0530 From: Viresh Kumar To: "Rafael J. Wysocki" Cc: Linux PM list , Linux Kernel Mailing List , Srinivas Pandruvada , Juri Lelli , Steve Muckle , Saravana Kannan Subject: Re: [PATCH v2 8/10] cpufreq: governor: Rename cpu_common_dbs_info to policy_dbs_info Message-ID: <20160205083406.GK21792@vireshk> References: <3705929.bslqXH980s@vostro.rjw.lan> <9008098.QDD8C89zDx@vostro.rjw.lan> <2973464.jyrWv0qA55@vostro.rjw.lan> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <2973464.jyrWv0qA55@vostro.rjw.lan> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 05-02-16, 03:20, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > The struct cpu_common_dbs_info structure represents the per-policy > part of the governor data (for the ondemand and conservative > governors), but its name doesn't reflect its purpose. > > Rename it to struct policy_dbs_info and rename variables related to > it accordingly. > > No functional changes. > > Signed-off-by: Rafael J. Wysocki > --- > drivers/cpufreq/cpufreq_governor.c | 120 ++++++++++++++++++------------------- > drivers/cpufreq/cpufreq_governor.h | 8 +- > drivers/cpufreq/cpufreq_ondemand.c | 32 ++++----- > 3 files changed, 80 insertions(+), 80 deletions(-) > > Index: linux-pm/drivers/cpufreq/cpufreq_governor.h > =================================================================== > --- linux-pm.orig/drivers/cpufreq/cpufreq_governor.h > +++ linux-pm/drivers/cpufreq/cpufreq_governor.h > @@ -132,7 +132,7 @@ static void *get_cpu_dbs_info_s(int cpu) > */ > > /* Common to all CPUs of a policy */ > -struct cpu_common_dbs_info { > +struct policy_dbs_info { > struct cpufreq_policy *policy; > /* > * Per policy mutex that serializes load evaluation from limit-change > @@ -162,7 +162,7 @@ struct cpu_dbs_info { > */ > unsigned int prev_load; > struct update_util_data update_util; > - struct cpu_common_dbs_info *shared; > + struct policy_dbs_info *shared; You should replaced shared with policy_dbs here as well ? Apart from that: Acked-by: Viresh Kumar -- viresh