Re: [PATCH] soc: Convert to using %pOF instead of full_name
On Thu, Aug 10, 2017 at 12:09 AM, Rob Herring wrote: > On Tue, Jul 18, 2017 at 4:43 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: Scott Wood >> Cc: Qiang Zhao >> Cc: Matthias Brugger >> Cc: Simon Horman >> Cc: Magnus Damm >> Cc: Kukjin Kim >> Cc: Krzysztof Kozlowski >> Cc: Javier Martinez Canillas > Arnd, Olof, > > Can you please apply this one. Applied to next/drivers with the various Acks, thanks! Arnd
Re: [PATCH] soc: Convert to using %pOF instead of full_name
On Tue, Jul 18, 2017 at 4:43 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: 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-...@lists.ozlabs.org > Cc: linux-arm-ker...@lists.infradead.org > Cc: linux-media...@lists.infradead.org > Cc: linux-renesas-...@vger.kernel.org > Cc: linux-samsung-...@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(-) Arnd, Olof, Can you please apply this one. Rob
Re: [PATCH] soc: Convert to using %pOF instead of full_name
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-...@lists.ozlabs.org > Cc: linux-arm-ker...@lists.infradead.org > Cc: linux-media...@lists.infradead.org > Cc: linux-renesas-...@vger.kernel.org > Cc: linux-samsung-...@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(-) Renesas portions: Acked-by: Simon Horman
Re: [PATCH] soc: Convert to using %pOF instead of full_name
On 07/18/2017 11:43 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: 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-...@lists.ozlabs.org Cc: linux-arm-ker...@lists.infradead.org Cc: linux-media...@lists.infradead.org Cc: linux-renesas-...@vger.kernel.org Cc: linux-samsung-...@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 ++-- For Mediatek: Acked-by: Matthias Brugger
Re: [PATCH] soc: Convert to using %pOF instead of full_name
On Tue, Jul 18, 2017 at 11:43 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 Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
Re: [PATCH] soc: Convert to using %pOF instead of full_name
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-...@lists.ozlabs.org > Cc: linux-arm-ker...@lists.infradead.org > Cc: linux-media...@lists.infradead.org > Cc: linux-renesas-...@vger.kernel.org > Cc: linux-samsung-...@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