Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753031AbdGSHJK (ORCPT ); Wed, 19 Jul 2017 03:09:10 -0400 Received: from mx08-00178001.pphosted.com ([91.207.212.93]:53821 "EHLO mx07-00178001.pphosted.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751548AbdGSHJI (ORCPT ); Wed, 19 Jul 2017 03:09:08 -0400 From: Patrice CHOTARD To: Rob Herring , "Rafael J. Wysocki" CC: "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , Viresh Kumar , Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , "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 Thread-Topic: [PATCH] cpufreq: Convert to using %pOF instead of full_name Thread-Index: AQHTAA78T20oeOtRF0eOqPqvNC9MIqJamVKA 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> In-Reply-To: <20170718214339.7774-14-robh@kernel.org> Accept-Language: fr-FR, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 x-ms-exchange-messagesentrepresentingtype: 1 x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.75.127.48] Content-Type: text/plain; charset="utf-8" Content-ID: MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2017-07-19_04:,, signatures=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by nfs id v6J79GT3005433 Content-Length: 2353 Lines: 70 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