Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753100AbdGSGip (ORCPT ); Wed, 19 Jul 2017 02:38:45 -0400 Received: from mail-pg0-f53.google.com ([74.125.83.53]:35936 "EHLO mail-pg0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752849AbdGSGin (ORCPT ); Wed, 19 Jul 2017 02:38:43 -0400 Date: Wed, 19 Jul 2017 12:08:40 +0530 From: Viresh Kumar To: Rob Herring Cc: "Rafael J. Wysocki" , linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , Patrice Chotard , linux-pm@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-arm-kernel@lists.infradead.org, kernel@stlinux.com Subject: Re: [PATCH] cpufreq: Convert to using %pOF instead of full_name Message-ID: <20170719063840.GD352@vireshk-i7> References: <20170718214339.7774-14-robh@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170718214339.7774-14-robh@kernel.org> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2296 Lines: 64 On 18-07-17, 16:42, 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; > } Acked-by: Viresh Kumar -- viresh