From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752981AbdGSGCZ (ORCPT ); Wed, 19 Jul 2017 02:02:25 -0400 Received: from mail-io0-f196.google.com ([209.85.223.196]:33999 "EHLO mail-io0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752190AbdGSGCX (ORCPT ); Wed, 19 Jul 2017 02:02:23 -0400 Date: Wed, 19 Jul 2017 08:01:53 +0200 From: Krzysztof Kozlowski To: Rob Herring Cc: arm@kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, Scott Wood , Qiang Zhao , Matthias Brugger , Simon Horman , Magnus Damm , Kukjin Kim , Javier Martinez Canillas , linuxppc-dev@lists.ozlabs.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-renesas-soc@vger.kernel.org, linux-samsung-soc@vger.kernel.org Subject: Re: [PATCH] soc: Convert to using %pOF instead of full_name Message-ID: <20170719060153.GD6246@kozik-book> References: <20170718214339.7774-49-robh@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20170718214339.7774-49-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 On Tue, Jul 18, 2017 at 04:43:29PM -0500, 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: Scott Wood > Cc: Qiang Zhao > Cc: Matthias Brugger > Cc: Simon Horman > Cc: Magnus Damm > Cc: Kukjin Kim > Cc: Krzysztof Kozlowski > Cc: Javier Martinez Canillas > Cc: linuxppc-dev@lists.ozlabs.org > Cc: linux-arm-kernel@lists.infradead.org > Cc: linux-mediatek@lists.infradead.org > Cc: linux-renesas-soc@vger.kernel.org > Cc: linux-samsung-soc@vger.kernel.org > --- > drivers/soc/fsl/qbman/bman_ccsr.c | 10 +++++----- > drivers/soc/fsl/qbman/bman_portal.c | 8 +++----- > drivers/soc/fsl/qbman/qman_ccsr.c | 12 ++++++------ > drivers/soc/fsl/qbman/qman_portal.c | 11 ++++------- > drivers/soc/fsl/qe/gpio.c | 4 ++-- > drivers/soc/mediatek/mtk-pmic-wrap.c | 4 ++-- > drivers/soc/renesas/rcar-rst.c | 4 ++-- > drivers/soc/renesas/rcar-sysc.c | 6 +++--- > drivers/soc/samsung/pm_domains.c | 8 ++++---- > 9 files changed, 31 insertions(+), 36 deletions(-) > (...) > diff --git a/drivers/soc/samsung/pm_domains.c b/drivers/soc/samsung/pm_domains.c > index a6a5d807cc2b..c630eba32066 100644 > --- a/drivers/soc/samsung/pm_domains.c > +++ b/drivers/soc/samsung/pm_domains.c > @@ -237,11 +237,11 @@ static __init int exynos4_pm_init_power_domain(void) > continue; > > if (of_genpd_add_subdomain(&parent, &child)) > - pr_warn("%s failed to add subdomain: %s\n", > - parent.np->full_name, child.np->full_name); > + pr_warn("%pOF failed to add subdomain: %pOF\n", > + parent.np, child.np); > else > - pr_info("%s has as child subdomain: %s.\n", > - parent.np->full_name, child.np->full_name); > + pr_info("%pOF has as child subdomain: %pOF.\n", > + parent.np, child.np); > } > > return 0; I guess this will go through arm-soc? For Samsung, looks good: Acked-by: Krzysztof Kozlowski Best regards, Krzysztof From mboxrd@z Thu Jan 1 00:00:00 1970 From: krzk@kernel.org (Krzysztof Kozlowski) Date: Wed, 19 Jul 2017 08:01:53 +0200 Subject: [PATCH] soc: Convert to using %pOF instead of full_name In-Reply-To: <20170718214339.7774-49-robh@kernel.org> References: <20170718214339.7774-49-robh@kernel.org> Message-ID: <20170719060153.GD6246@kozik-book> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, Jul 18, 2017 at 04:43:29PM -0500, 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: Scott Wood > Cc: Qiang Zhao > Cc: Matthias Brugger > Cc: Simon Horman > Cc: Magnus Damm > Cc: Kukjin Kim > Cc: Krzysztof Kozlowski > Cc: Javier Martinez Canillas > Cc: linuxppc-dev at lists.ozlabs.org > Cc: linux-arm-kernel at lists.infradead.org > Cc: linux-mediatek at lists.infradead.org > Cc: linux-renesas-soc at vger.kernel.org > Cc: linux-samsung-soc at vger.kernel.org > --- > drivers/soc/fsl/qbman/bman_ccsr.c | 10 +++++----- > drivers/soc/fsl/qbman/bman_portal.c | 8 +++----- > drivers/soc/fsl/qbman/qman_ccsr.c | 12 ++++++------ > drivers/soc/fsl/qbman/qman_portal.c | 11 ++++------- > drivers/soc/fsl/qe/gpio.c | 4 ++-- > drivers/soc/mediatek/mtk-pmic-wrap.c | 4 ++-- > drivers/soc/renesas/rcar-rst.c | 4 ++-- > drivers/soc/renesas/rcar-sysc.c | 6 +++--- > drivers/soc/samsung/pm_domains.c | 8 ++++---- > 9 files changed, 31 insertions(+), 36 deletions(-) > (...) > diff --git a/drivers/soc/samsung/pm_domains.c b/drivers/soc/samsung/pm_domains.c > index a6a5d807cc2b..c630eba32066 100644 > --- a/drivers/soc/samsung/pm_domains.c > +++ b/drivers/soc/samsung/pm_domains.c > @@ -237,11 +237,11 @@ static __init int exynos4_pm_init_power_domain(void) > continue; > > if (of_genpd_add_subdomain(&parent, &child)) > - pr_warn("%s failed to add subdomain: %s\n", > - parent.np->full_name, child.np->full_name); > + pr_warn("%pOF failed to add subdomain: %pOF\n", > + parent.np, child.np); > else > - pr_info("%s has as child subdomain: %s.\n", > - parent.np->full_name, child.np->full_name); > + pr_info("%pOF has as child subdomain: %pOF.\n", > + parent.np, child.np); > } > > return 0; I guess this will go through arm-soc? For Samsung, looks good: Acked-by: Krzysztof Kozlowski Best regards, Krzysztof