From mboxrd@z Thu Jan 1 00:00:00 1970 From: Patrice CHOTARD Subject: Re: [PATCH] cpufreq: Convert to using %pOF instead of full_name Date: Wed, 19 Jul 2017 07:06:06 +0000 Message-ID: <25327197-4a70-e31c-5ccc-8becf4da8395@st.com> References: <20170718214339.7774-14-robh@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20170718214339.7774-14-robh@kernel.org> Content-Language: en-US Content-ID: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Rob Herring , "Rafael J. Wysocki" Cc: "devicetree@vger.kernel.org" , "kernel@stlinux.com" , "linux-pm@vger.kernel.org" , Michael Ellerman , "linux-kernel@vger.kernel.org" , Viresh Kumar , Paul Mackerras , Benjamin Herrenschmidt , "linuxppc-dev@lists.ozlabs.org" , "linux-arm-kernel@lists.infradead.org" List-Id: devicetree@vger.kernel.org On 07/18/2017 11:42 PM, Rob Herring wrote: > Now that we have a custom printf format specifier, convert users of > full_name to use %pOF instead. This is preparation to remove storing > of the full path string for each node. > > Signed-off-by: Rob Herring > Cc: "Rafael J. Wysocki" > Cc: Viresh Kumar > Cc: Benjamin Herrenschmidt > Cc: Paul Mackerras > Cc: Michael Ellerman > Cc: Patrice Chotard > Cc: linux-pm@vger.kernel.org > Cc: linuxppc-dev@lists.ozlabs.org > Cc: linux-arm-kernel@lists.infradead.org > Cc: kernel@stlinux.com > --- > drivers/cpufreq/pmac64-cpufreq.c | 2 +- > drivers/cpufreq/sti-cpufreq.c | 8 ++++---- > 2 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/cpufreq/pmac64-cpufreq.c b/drivers/cpufreq/pmac64-cpufreq.c > index 267e0894c62d..be623dd7b9f2 100644 > --- a/drivers/cpufreq/pmac64-cpufreq.c > +++ b/drivers/cpufreq/pmac64-cpufreq.c > @@ -516,7 +516,7 @@ static int __init g5_pm72_cpufreq_init(struct device_node *cpunode) > goto bail; > } > > - DBG("cpufreq: i2c clock chip found: %s\n", hwclock->full_name); > + DBG("cpufreq: i2c clock chip found: %pOF\n", hwclock); > > /* Now get all the platform functions */ > pfunc_cpu_getfreq = > diff --git a/drivers/cpufreq/sti-cpufreq.c b/drivers/cpufreq/sti-cpufreq.c > index d2d0430d09d4..47105735df12 100644 > --- a/drivers/cpufreq/sti-cpufreq.c > +++ b/drivers/cpufreq/sti-cpufreq.c > @@ -65,8 +65,8 @@ static int sti_cpufreq_fetch_major(void) { > ret = of_property_read_u32_index(np, "st,syscfg", > MAJOR_ID_INDEX, &major_offset); > if (ret) { > - dev_err(dev, "No major number offset provided in %s [%d]\n", > - np->full_name, ret); > + dev_err(dev, "No major number offset provided in %pOF [%d]\n", > + np, ret); > return ret; > } > > @@ -92,8 +92,8 @@ static int sti_cpufreq_fetch_minor(void) > MINOR_ID_INDEX, &minor_offset); > if (ret) { > dev_err(dev, > - "No minor number offset provided %s [%d]\n", > - np->full_name, ret); > + "No minor number offset provided %pOF [%d]\n", > + np, ret); > return ret; > } > > -- > 2.11.0 > For ST cpufreq: Acked-by: Patrice Chotard Thanks