Re: [PATCH] pinctrl: Convert to using %pOFn instead of device_node.name

2018-09-03 Thread Patrice CHOTARD
Hi Rob

On 08/28/2018 03:52 AM, Rob Herring wrote:
> In preparation to remove the node name pointer from struct device_node,
> convert printf users to use the %pOFn format specifier.
> 
> Cc: Linus Walleij 
> Cc: Dong Aisheng 
> Cc: Fabio Estevam 
> Cc: Shawn Guo 
> Cc: Stefan Agner 
> Cc: Pengutronix Kernel Team 
> Cc: Sean Wang 
> Cc: Matthias Brugger 
> Cc: Carlo Caione 
> Cc: Kevin Hilman 
> Cc: Jason Cooper 
> Cc: Andrew Lunn 
> Cc: Gregory Clement 
> Cc: Sebastian Hesselbarth 
> Cc: Jean-Christophe Plagniol-Villard 
> Cc: Nicolas Ferre 
> Cc: Alexandre Belloni 
> Cc: Heiko Stuebner 
> Cc: Tony Lindgren 
> Cc: Haojian Zhuang 
> Cc: Patrice Chotard 
> Cc: Barry Song 
> Cc: Maxime Coquelin 
> Cc: Alexandre Torgue 
> Cc: Maxime Ripard 
> Cc: Chen-Yu Tsai 
> Cc: linux-g...@vger.kernel.org
> Cc: linux-media...@lists.infradead.org
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: linux-amlo...@lists.infradead.org
> Cc: linux-rockc...@lists.infradead.org
> Cc: linux-o...@vger.kernel.org
> Signed-off-by: Rob Herring 
> ---
>   drivers/pinctrl/berlin/berlin.c   |  6 ++--
>   drivers/pinctrl/freescale/pinctrl-imx.c   |  7 ++--
>   drivers/pinctrl/freescale/pinctrl-imx1-core.c | 12 +++
>   drivers/pinctrl/mediatek/pinctrl-mtk-common.c |  4 +--
>   drivers/pinctrl/meson/pinctrl-meson.c |  2 +-
>   drivers/pinctrl/mvebu/pinctrl-mvebu.c |  4 +--
>   drivers/pinctrl/nomadik/pinctrl-nomadik.c |  6 ++--
>   drivers/pinctrl/pinctrl-at91.c|  8 ++---
>   drivers/pinctrl/pinctrl-lantiq.c  |  8 ++---
>   drivers/pinctrl/pinctrl-rockchip.c|  8 ++---
>   drivers/pinctrl/pinctrl-rza1.c|  8 ++---
>   drivers/pinctrl/pinctrl-single.c  | 32 +--
>   drivers/pinctrl/pinctrl-st.c  |  6 ++--
>   drivers/pinctrl/sirf/pinctrl-atlas7.c |  4 +--
>   drivers/pinctrl/stm32/pinctrl-stm32.c |  4 +--
>   drivers/pinctrl/sunxi/pinctrl-sunxi.c |  8 ++---
>   drivers/pinctrl/ti/pinctrl-ti-iodelay.c   |  8 ++---
>   17 files changed, 66 insertions(+), 69 deletions(-)
> 
> diff --git a/drivers/pinctrl/pinctrl-st.c b/drivers/pinctrl/pinctrl-st.c
> index 0966bb0bf71f..e66af93f2cbf 100644
> --- a/drivers/pinctrl/pinctrl-st.c
> +++ b/drivers/pinctrl/pinctrl-st.c
> @@ -817,8 +817,8 @@ static int st_pctl_dt_node_to_map(struct pinctrl_dev 
> *pctldev,
>   
>   grp = st_pctl_find_group_by_name(info, np->name);
>   if (!grp) {
> - dev_err(info->dev, "unable to find group for node %s\n",
> - np->name);
> + dev_err(info->dev, "unable to find group for node %pOFn\n",
> + np);
>   return -EINVAL;
>   }
>   
> @@ -1184,7 +1184,7 @@ static int st_pctl_dt_parse_groups(struct device_node 
> *np,
>   if (pp->length / sizeof(__be32) >= OF_GPIO_ARGS_MIN) {
>   npins++;
>   } else {
> - pr_warn("Invalid st,pins in %s node\n", np->name);
> + pr_warn("Invalid st,pins in %pOFn node\n", np);
>   return -EINVAL;
>   }
>   }

For pinctrl-st:

Acked-by: Patrice Chotard 

Thanks
Patrice


Re: [PATCH] pinctrl: Convert to using %pOFn instead of device_node.name

2018-09-03 Thread Patrice CHOTARD
Hi Rob

On 08/28/2018 03:52 AM, Rob Herring wrote:
> In preparation to remove the node name pointer from struct device_node,
> convert printf users to use the %pOFn format specifier.
> 
> Cc: Linus Walleij 
> Cc: Dong Aisheng 
> Cc: Fabio Estevam 
> Cc: Shawn Guo 
> Cc: Stefan Agner 
> Cc: Pengutronix Kernel Team 
> Cc: Sean Wang 
> Cc: Matthias Brugger 
> Cc: Carlo Caione 
> Cc: Kevin Hilman 
> Cc: Jason Cooper 
> Cc: Andrew Lunn 
> Cc: Gregory Clement 
> Cc: Sebastian Hesselbarth 
> Cc: Jean-Christophe Plagniol-Villard 
> Cc: Nicolas Ferre 
> Cc: Alexandre Belloni 
> Cc: Heiko Stuebner 
> Cc: Tony Lindgren 
> Cc: Haojian Zhuang 
> Cc: Patrice Chotard 
> Cc: Barry Song 
> Cc: Maxime Coquelin 
> Cc: Alexandre Torgue 
> Cc: Maxime Ripard 
> Cc: Chen-Yu Tsai 
> Cc: linux-g...@vger.kernel.org
> Cc: linux-media...@lists.infradead.org
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: linux-amlo...@lists.infradead.org
> Cc: linux-rockc...@lists.infradead.org
> Cc: linux-o...@vger.kernel.org
> Signed-off-by: Rob Herring 
> ---
>   drivers/pinctrl/berlin/berlin.c   |  6 ++--
>   drivers/pinctrl/freescale/pinctrl-imx.c   |  7 ++--
>   drivers/pinctrl/freescale/pinctrl-imx1-core.c | 12 +++
>   drivers/pinctrl/mediatek/pinctrl-mtk-common.c |  4 +--
>   drivers/pinctrl/meson/pinctrl-meson.c |  2 +-
>   drivers/pinctrl/mvebu/pinctrl-mvebu.c |  4 +--
>   drivers/pinctrl/nomadik/pinctrl-nomadik.c |  6 ++--
>   drivers/pinctrl/pinctrl-at91.c|  8 ++---
>   drivers/pinctrl/pinctrl-lantiq.c  |  8 ++---
>   drivers/pinctrl/pinctrl-rockchip.c|  8 ++---
>   drivers/pinctrl/pinctrl-rza1.c|  8 ++---
>   drivers/pinctrl/pinctrl-single.c  | 32 +--
>   drivers/pinctrl/pinctrl-st.c  |  6 ++--
>   drivers/pinctrl/sirf/pinctrl-atlas7.c |  4 +--
>   drivers/pinctrl/stm32/pinctrl-stm32.c |  4 +--
>   drivers/pinctrl/sunxi/pinctrl-sunxi.c |  8 ++---
>   drivers/pinctrl/ti/pinctrl-ti-iodelay.c   |  8 ++---
>   17 files changed, 66 insertions(+), 69 deletions(-)
> 
> diff --git a/drivers/pinctrl/pinctrl-st.c b/drivers/pinctrl/pinctrl-st.c
> index 0966bb0bf71f..e66af93f2cbf 100644
> --- a/drivers/pinctrl/pinctrl-st.c
> +++ b/drivers/pinctrl/pinctrl-st.c
> @@ -817,8 +817,8 @@ static int st_pctl_dt_node_to_map(struct pinctrl_dev 
> *pctldev,
>   
>   grp = st_pctl_find_group_by_name(info, np->name);
>   if (!grp) {
> - dev_err(info->dev, "unable to find group for node %s\n",
> - np->name);
> + dev_err(info->dev, "unable to find group for node %pOFn\n",
> + np);
>   return -EINVAL;
>   }
>   
> @@ -1184,7 +1184,7 @@ static int st_pctl_dt_parse_groups(struct device_node 
> *np,
>   if (pp->length / sizeof(__be32) >= OF_GPIO_ARGS_MIN) {
>   npins++;
>   } else {
> - pr_warn("Invalid st,pins in %s node\n", np->name);
> + pr_warn("Invalid st,pins in %pOFn node\n", np);
>   return -EINVAL;
>   }
>   }

For pinctrl-st:

Acked-by: Patrice Chotard 

Thanks
Patrice


Re: [PATCH] pinctrl: Convert to using %pOFn instead of device_node.name

2018-08-29 Thread Linus Walleij
On Tue, Aug 28, 2018 at 3:55 AM Rob Herring  wrote:

> In preparation to remove the node name pointer from struct device_node,
> convert printf users to use the %pOFn format specifier.
>
> Cc: Linus Walleij 

Patch applied along with the thundering hoard of ACKs.

Yours,
Linus Walleij


Re: [PATCH] pinctrl: Convert to using %pOFn instead of device_node.name

2018-08-29 Thread Linus Walleij
On Tue, Aug 28, 2018 at 3:55 AM Rob Herring  wrote:

> In preparation to remove the node name pointer from struct device_node,
> convert printf users to use the %pOFn format specifier.
>
> Cc: Linus Walleij 

Patch applied along with the thundering hoard of ACKs.

Yours,
Linus Walleij


Re: [PATCH] pinctrl: Convert to using %pOFn instead of device_node.name

2018-08-29 Thread Heiko Stübner
Am Dienstag, 28. August 2018, 03:52:41 CEST schrieb Rob Herring:
> In preparation to remove the node name pointer from struct device_node,
> convert printf users to use the %pOFn format specifier.

>  drivers/pinctrl/pinctrl-rockchip.c|  8 ++---

For the Rockchip-part
Acked-by: Heiko Stuebner 




Re: [PATCH] pinctrl: Convert to using %pOFn instead of device_node.name

2018-08-29 Thread Heiko Stübner
Am Dienstag, 28. August 2018, 03:52:41 CEST schrieb Rob Herring:
> In preparation to remove the node name pointer from struct device_node,
> convert printf users to use the %pOFn format specifier.

>  drivers/pinctrl/pinctrl-rockchip.c|  8 ++---

For the Rockchip-part
Acked-by: Heiko Stuebner 




Re: [PATCH] pinctrl: Convert to using %pOFn instead of device_node.name

2018-08-28 Thread Chen-Yu Tsai
On Tue, Aug 28, 2018 at 9:55 AM Rob Herring  wrote:
>
> In preparation to remove the node name pointer from struct device_node,
> convert printf users to use the %pOFn format specifier.
>
> Cc: Linus Walleij 
> Cc: Dong Aisheng 
> Cc: Fabio Estevam 
> Cc: Shawn Guo 
> Cc: Stefan Agner 
> Cc: Pengutronix Kernel Team 
> Cc: Sean Wang 
> Cc: Matthias Brugger 
> Cc: Carlo Caione 
> Cc: Kevin Hilman 
> Cc: Jason Cooper 
> Cc: Andrew Lunn 
> Cc: Gregory Clement 
> Cc: Sebastian Hesselbarth 
> Cc: Jean-Christophe Plagniol-Villard 
> Cc: Nicolas Ferre 
> Cc: Alexandre Belloni 
> Cc: Heiko Stuebner 
> Cc: Tony Lindgren 
> Cc: Haojian Zhuang 
> Cc: Patrice Chotard 
> Cc: Barry Song 
> Cc: Maxime Coquelin 
> Cc: Alexandre Torgue 
> Cc: Maxime Ripard 
> Cc: Chen-Yu Tsai 
> Cc: linux-g...@vger.kernel.org
> Cc: linux-media...@lists.infradead.org
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: linux-amlo...@lists.infradead.org
> Cc: linux-rockc...@lists.infradead.org
> Cc: linux-o...@vger.kernel.org
> Signed-off-by: Rob Herring 
> ---
>  drivers/pinctrl/berlin/berlin.c   |  6 ++--
>  drivers/pinctrl/freescale/pinctrl-imx.c   |  7 ++--
>  drivers/pinctrl/freescale/pinctrl-imx1-core.c | 12 +++
>  drivers/pinctrl/mediatek/pinctrl-mtk-common.c |  4 +--
>  drivers/pinctrl/meson/pinctrl-meson.c |  2 +-
>  drivers/pinctrl/mvebu/pinctrl-mvebu.c |  4 +--
>  drivers/pinctrl/nomadik/pinctrl-nomadik.c |  6 ++--
>  drivers/pinctrl/pinctrl-at91.c|  8 ++---
>  drivers/pinctrl/pinctrl-lantiq.c  |  8 ++---
>  drivers/pinctrl/pinctrl-rockchip.c|  8 ++---
>  drivers/pinctrl/pinctrl-rza1.c|  8 ++---
>  drivers/pinctrl/pinctrl-single.c  | 32 +--
>  drivers/pinctrl/pinctrl-st.c  |  6 ++--
>  drivers/pinctrl/sirf/pinctrl-atlas7.c |  4 +--
>  drivers/pinctrl/stm32/pinctrl-stm32.c |  4 +--
>  drivers/pinctrl/sunxi/pinctrl-sunxi.c |  8 ++---

For sunxi:

Acked-by: Chen-Yu Tsai 


Re: [PATCH] pinctrl: Convert to using %pOFn instead of device_node.name

2018-08-28 Thread Chen-Yu Tsai
On Tue, Aug 28, 2018 at 9:55 AM Rob Herring  wrote:
>
> In preparation to remove the node name pointer from struct device_node,
> convert printf users to use the %pOFn format specifier.
>
> Cc: Linus Walleij 
> Cc: Dong Aisheng 
> Cc: Fabio Estevam 
> Cc: Shawn Guo 
> Cc: Stefan Agner 
> Cc: Pengutronix Kernel Team 
> Cc: Sean Wang 
> Cc: Matthias Brugger 
> Cc: Carlo Caione 
> Cc: Kevin Hilman 
> Cc: Jason Cooper 
> Cc: Andrew Lunn 
> Cc: Gregory Clement 
> Cc: Sebastian Hesselbarth 
> Cc: Jean-Christophe Plagniol-Villard 
> Cc: Nicolas Ferre 
> Cc: Alexandre Belloni 
> Cc: Heiko Stuebner 
> Cc: Tony Lindgren 
> Cc: Haojian Zhuang 
> Cc: Patrice Chotard 
> Cc: Barry Song 
> Cc: Maxime Coquelin 
> Cc: Alexandre Torgue 
> Cc: Maxime Ripard 
> Cc: Chen-Yu Tsai 
> Cc: linux-g...@vger.kernel.org
> Cc: linux-media...@lists.infradead.org
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: linux-amlo...@lists.infradead.org
> Cc: linux-rockc...@lists.infradead.org
> Cc: linux-o...@vger.kernel.org
> Signed-off-by: Rob Herring 
> ---
>  drivers/pinctrl/berlin/berlin.c   |  6 ++--
>  drivers/pinctrl/freescale/pinctrl-imx.c   |  7 ++--
>  drivers/pinctrl/freescale/pinctrl-imx1-core.c | 12 +++
>  drivers/pinctrl/mediatek/pinctrl-mtk-common.c |  4 +--
>  drivers/pinctrl/meson/pinctrl-meson.c |  2 +-
>  drivers/pinctrl/mvebu/pinctrl-mvebu.c |  4 +--
>  drivers/pinctrl/nomadik/pinctrl-nomadik.c |  6 ++--
>  drivers/pinctrl/pinctrl-at91.c|  8 ++---
>  drivers/pinctrl/pinctrl-lantiq.c  |  8 ++---
>  drivers/pinctrl/pinctrl-rockchip.c|  8 ++---
>  drivers/pinctrl/pinctrl-rza1.c|  8 ++---
>  drivers/pinctrl/pinctrl-single.c  | 32 +--
>  drivers/pinctrl/pinctrl-st.c  |  6 ++--
>  drivers/pinctrl/sirf/pinctrl-atlas7.c |  4 +--
>  drivers/pinctrl/stm32/pinctrl-stm32.c |  4 +--
>  drivers/pinctrl/sunxi/pinctrl-sunxi.c |  8 ++---

For sunxi:

Acked-by: Chen-Yu Tsai 


Re: [PATCH] pinctrl: Convert to using %pOFn instead of device_node.name

2018-08-28 Thread Sean Wang
On Mon, 2018-08-27 at 20:52 -0500, Rob Herring wrote:
> In preparation to remove the node name pointer from struct device_node,
> convert printf users to use the %pOFn format specifier.
> 
> Cc: Linus Walleij 
> Cc: Dong Aisheng 
> Cc: Fabio Estevam 
> Cc: Shawn Guo 
> Cc: Stefan Agner 
> Cc: Pengutronix Kernel Team 
> Cc: Sean Wang 
> Cc: Matthias Brugger 
> Cc: Carlo Caione 
> Cc: Kevin Hilman 
> Cc: Jason Cooper 
> Cc: Andrew Lunn 
> Cc: Gregory Clement 
> Cc: Sebastian Hesselbarth 
> Cc: Jean-Christophe Plagniol-Villard 
> Cc: Nicolas Ferre 
> Cc: Alexandre Belloni 
> Cc: Heiko Stuebner 
> Cc: Tony Lindgren 
> Cc: Haojian Zhuang 
> Cc: Patrice Chotard 
> Cc: Barry Song 
> Cc: Maxime Coquelin 
> Cc: Alexandre Torgue 
> Cc: Maxime Ripard 
> Cc: Chen-Yu Tsai 
> Cc: linux-g...@vger.kernel.org
> Cc: linux-media...@lists.infradead.org
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: linux-amlo...@lists.infradead.org
> Cc: linux-rockc...@lists.infradead.org
> Cc: linux-o...@vger.kernel.org
> Signed-off-by: Rob Herring 
> ---
>  drivers/pinctrl/berlin/berlin.c   |  6 ++--
>  drivers/pinctrl/freescale/pinctrl-imx.c   |  7 ++--
>  drivers/pinctrl/freescale/pinctrl-imx1-core.c | 12 +++
>  drivers/pinctrl/mediatek/pinctrl-mtk-common.c |  4 +--

For mediatek,
Acked-by: Sean Wang 




Re: [PATCH] pinctrl: Convert to using %pOFn instead of device_node.name

2018-08-28 Thread Sean Wang
On Mon, 2018-08-27 at 20:52 -0500, Rob Herring wrote:
> In preparation to remove the node name pointer from struct device_node,
> convert printf users to use the %pOFn format specifier.
> 
> Cc: Linus Walleij 
> Cc: Dong Aisheng 
> Cc: Fabio Estevam 
> Cc: Shawn Guo 
> Cc: Stefan Agner 
> Cc: Pengutronix Kernel Team 
> Cc: Sean Wang 
> Cc: Matthias Brugger 
> Cc: Carlo Caione 
> Cc: Kevin Hilman 
> Cc: Jason Cooper 
> Cc: Andrew Lunn 
> Cc: Gregory Clement 
> Cc: Sebastian Hesselbarth 
> Cc: Jean-Christophe Plagniol-Villard 
> Cc: Nicolas Ferre 
> Cc: Alexandre Belloni 
> Cc: Heiko Stuebner 
> Cc: Tony Lindgren 
> Cc: Haojian Zhuang 
> Cc: Patrice Chotard 
> Cc: Barry Song 
> Cc: Maxime Coquelin 
> Cc: Alexandre Torgue 
> Cc: Maxime Ripard 
> Cc: Chen-Yu Tsai 
> Cc: linux-g...@vger.kernel.org
> Cc: linux-media...@lists.infradead.org
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: linux-amlo...@lists.infradead.org
> Cc: linux-rockc...@lists.infradead.org
> Cc: linux-o...@vger.kernel.org
> Signed-off-by: Rob Herring 
> ---
>  drivers/pinctrl/berlin/berlin.c   |  6 ++--
>  drivers/pinctrl/freescale/pinctrl-imx.c   |  7 ++--
>  drivers/pinctrl/freescale/pinctrl-imx1-core.c | 12 +++
>  drivers/pinctrl/mediatek/pinctrl-mtk-common.c |  4 +--

For mediatek,
Acked-by: Sean Wang 




Re: [PATCH] pinctrl: Convert to using %pOFn instead of device_node.name

2018-08-28 Thread Tony Lindgren
* Rob Herring  [180828 01:59]:
> In preparation to remove the node name pointer from struct device_node,
> convert printf users to use the %pOFn format specifier.
...
>  drivers/pinctrl/pinctrl-single.c  | 32 +--
>  drivers/pinctrl/ti/pinctrl-ti-iodelay.c   |  8 ++---

Acked-by: Tony Lindgren 


Re: [PATCH] pinctrl: Convert to using %pOFn instead of device_node.name

2018-08-28 Thread Tony Lindgren
* Rob Herring  [180828 01:59]:
> In preparation to remove the node name pointer from struct device_node,
> convert printf users to use the %pOFn format specifier.
...
>  drivers/pinctrl/pinctrl-single.c  | 32 +--
>  drivers/pinctrl/ti/pinctrl-ti-iodelay.c   |  8 ++---

Acked-by: Tony Lindgren 


Re: [PATCH] pinctrl: Convert to using %pOFn instead of device_node.name

2018-08-28 Thread Alexandre Belloni
On 27/08/2018 20:52:41-0500, Rob Herring wrote:
> In preparation to remove the node name pointer from struct device_node,
> convert printf users to use the %pOFn format specifier.
> 
> Cc: Linus Walleij 
> Cc: Dong Aisheng 
> Cc: Fabio Estevam 
> Cc: Shawn Guo 
> Cc: Stefan Agner 
> Cc: Pengutronix Kernel Team 
> Cc: Sean Wang 
> Cc: Matthias Brugger 
> Cc: Carlo Caione 
> Cc: Kevin Hilman 
> Cc: Jason Cooper 
> Cc: Andrew Lunn 
> Cc: Gregory Clement 
> Cc: Sebastian Hesselbarth 
> Cc: Jean-Christophe Plagniol-Villard 
> Cc: Nicolas Ferre 
> Cc: Alexandre Belloni 
> Cc: Heiko Stuebner 
> Cc: Tony Lindgren 
> Cc: Haojian Zhuang 
> Cc: Patrice Chotard 
> Cc: Barry Song 
> Cc: Maxime Coquelin 
> Cc: Alexandre Torgue 
> Cc: Maxime Ripard 
> Cc: Chen-Yu Tsai 
> Cc: linux-g...@vger.kernel.org
> Cc: linux-media...@lists.infradead.org
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: linux-amlo...@lists.infradead.org
> Cc: linux-rockc...@lists.infradead.org
> Cc: linux-o...@vger.kernel.org
> Signed-off-by: Rob Herring 

For at91:
Reviewed-by: Alexandre Belloni 


> ---
>  drivers/pinctrl/berlin/berlin.c   |  6 ++--
>  drivers/pinctrl/freescale/pinctrl-imx.c   |  7 ++--
>  drivers/pinctrl/freescale/pinctrl-imx1-core.c | 12 +++
>  drivers/pinctrl/mediatek/pinctrl-mtk-common.c |  4 +--
>  drivers/pinctrl/meson/pinctrl-meson.c |  2 +-
>  drivers/pinctrl/mvebu/pinctrl-mvebu.c |  4 +--
>  drivers/pinctrl/nomadik/pinctrl-nomadik.c |  6 ++--
>  drivers/pinctrl/pinctrl-at91.c|  8 ++---
>  drivers/pinctrl/pinctrl-lantiq.c  |  8 ++---
>  drivers/pinctrl/pinctrl-rockchip.c|  8 ++---
>  drivers/pinctrl/pinctrl-rza1.c|  8 ++---
>  drivers/pinctrl/pinctrl-single.c  | 32 +--
>  drivers/pinctrl/pinctrl-st.c  |  6 ++--
>  drivers/pinctrl/sirf/pinctrl-atlas7.c |  4 +--
>  drivers/pinctrl/stm32/pinctrl-stm32.c |  4 +--
>  drivers/pinctrl/sunxi/pinctrl-sunxi.c |  8 ++---
>  drivers/pinctrl/ti/pinctrl-ti-iodelay.c   |  8 ++---
>  17 files changed, 66 insertions(+), 69 deletions(-)
> 
> diff --git a/drivers/pinctrl/berlin/berlin.c b/drivers/pinctrl/berlin/berlin.c
> index b5903fffb3d0..b17a03cf87be 100644
> --- a/drivers/pinctrl/berlin/berlin.c
> +++ b/drivers/pinctrl/berlin/berlin.c
> @@ -64,16 +64,14 @@ static int berlin_pinctrl_dt_node_to_map(struct 
> pinctrl_dev *pctrl_dev,
>   ret = of_property_read_string(node, "function", _name);
>   if (ret) {
>   dev_err(pctrl->dev,
> - "missing function property in node %s\n",
> - node->name);
> + "missing function property in node %pOFn\n", node);
>   return -EINVAL;
>   }
>  
>   ngroups = of_property_count_strings(node, "groups");
>   if (ngroups < 0) {
>   dev_err(pctrl->dev,
> - "missing groups property in node %s\n",
> - node->name);
> + "missing groups property in node %pOFn\n", node);
>   return -EINVAL;
>   }
>  
> diff --git a/drivers/pinctrl/freescale/pinctrl-imx.c 
> b/drivers/pinctrl/freescale/pinctrl-imx.c
> index b04edc22dad7..4e8cf0e357c6 100644
> --- a/drivers/pinctrl/freescale/pinctrl-imx.c
> +++ b/drivers/pinctrl/freescale/pinctrl-imx.c
> @@ -69,8 +69,7 @@ static int imx_dt_node_to_map(struct pinctrl_dev *pctldev,
>*/
>   grp = imx_pinctrl_find_group_by_name(pctldev, np->name);
>   if (!grp) {
> - dev_err(ipctl->dev, "unable to find group for node %s\n",
> - np->name);
> + dev_err(ipctl->dev, "unable to find group for node %pOFn\n", 
> np);
>   return -EINVAL;
>   }
>  
> @@ -434,7 +433,7 @@ static int imx_pinctrl_parse_groups(struct device_node 
> *np,
>   int i;
>   u32 config;
>  
> - dev_dbg(ipctl->dev, "group(%d): %s\n", index, np->name);
> + dev_dbg(ipctl->dev, "group(%d): %pOFn\n", index, np);
>  
>   if (info->flags & SHARE_MUX_CONF_REG)
>   pin_size = FSL_PIN_SHARE_SIZE;
> @@ -544,7 +543,7 @@ static int imx_pinctrl_parse_functions(struct device_node 
> *np,
>   struct group_desc *grp;
>   u32 i = 0;
>  
> - dev_dbg(pctl->dev, "parse function(%d): %s\n", index, np->name);
> + dev_dbg(pctl->dev, "parse function(%d): %pOFn\n", index, np);
>  
>   func = pinmux_generic_get_function(pctl, index);
>   if (!func)
> diff --git a/drivers/pinctrl/freescale/pinctrl-imx1-core.c 
> b/drivers/pinctrl/freescale/pinctrl-imx1-core.c
> index deb7870b3d1a..7e29e3fecdb2 100644
> --- a/drivers/pinctrl/freescale/pinctrl-imx1-core.c
> +++ b/drivers/pinctrl/freescale/pinctrl-imx1-core.c
> @@ -233,8 +233,8 @@ static int imx1_dt_node_to_map(struct pinctrl_dev 
> *pctldev,
>*/
>   grp = imx1_pinctrl_find_group_by_name(info, np->name);
>   if (!grp) {
> - dev_err(info->dev, "unable 

Re: [PATCH] pinctrl: Convert to using %pOFn instead of device_node.name

2018-08-28 Thread Alexandre Belloni
On 27/08/2018 20:52:41-0500, Rob Herring wrote:
> In preparation to remove the node name pointer from struct device_node,
> convert printf users to use the %pOFn format specifier.
> 
> Cc: Linus Walleij 
> Cc: Dong Aisheng 
> Cc: Fabio Estevam 
> Cc: Shawn Guo 
> Cc: Stefan Agner 
> Cc: Pengutronix Kernel Team 
> Cc: Sean Wang 
> Cc: Matthias Brugger 
> Cc: Carlo Caione 
> Cc: Kevin Hilman 
> Cc: Jason Cooper 
> Cc: Andrew Lunn 
> Cc: Gregory Clement 
> Cc: Sebastian Hesselbarth 
> Cc: Jean-Christophe Plagniol-Villard 
> Cc: Nicolas Ferre 
> Cc: Alexandre Belloni 
> Cc: Heiko Stuebner 
> Cc: Tony Lindgren 
> Cc: Haojian Zhuang 
> Cc: Patrice Chotard 
> Cc: Barry Song 
> Cc: Maxime Coquelin 
> Cc: Alexandre Torgue 
> Cc: Maxime Ripard 
> Cc: Chen-Yu Tsai 
> Cc: linux-g...@vger.kernel.org
> Cc: linux-media...@lists.infradead.org
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: linux-amlo...@lists.infradead.org
> Cc: linux-rockc...@lists.infradead.org
> Cc: linux-o...@vger.kernel.org
> Signed-off-by: Rob Herring 

For at91:
Reviewed-by: Alexandre Belloni 


> ---
>  drivers/pinctrl/berlin/berlin.c   |  6 ++--
>  drivers/pinctrl/freescale/pinctrl-imx.c   |  7 ++--
>  drivers/pinctrl/freescale/pinctrl-imx1-core.c | 12 +++
>  drivers/pinctrl/mediatek/pinctrl-mtk-common.c |  4 +--
>  drivers/pinctrl/meson/pinctrl-meson.c |  2 +-
>  drivers/pinctrl/mvebu/pinctrl-mvebu.c |  4 +--
>  drivers/pinctrl/nomadik/pinctrl-nomadik.c |  6 ++--
>  drivers/pinctrl/pinctrl-at91.c|  8 ++---
>  drivers/pinctrl/pinctrl-lantiq.c  |  8 ++---
>  drivers/pinctrl/pinctrl-rockchip.c|  8 ++---
>  drivers/pinctrl/pinctrl-rza1.c|  8 ++---
>  drivers/pinctrl/pinctrl-single.c  | 32 +--
>  drivers/pinctrl/pinctrl-st.c  |  6 ++--
>  drivers/pinctrl/sirf/pinctrl-atlas7.c |  4 +--
>  drivers/pinctrl/stm32/pinctrl-stm32.c |  4 +--
>  drivers/pinctrl/sunxi/pinctrl-sunxi.c |  8 ++---
>  drivers/pinctrl/ti/pinctrl-ti-iodelay.c   |  8 ++---
>  17 files changed, 66 insertions(+), 69 deletions(-)
> 
> diff --git a/drivers/pinctrl/berlin/berlin.c b/drivers/pinctrl/berlin/berlin.c
> index b5903fffb3d0..b17a03cf87be 100644
> --- a/drivers/pinctrl/berlin/berlin.c
> +++ b/drivers/pinctrl/berlin/berlin.c
> @@ -64,16 +64,14 @@ static int berlin_pinctrl_dt_node_to_map(struct 
> pinctrl_dev *pctrl_dev,
>   ret = of_property_read_string(node, "function", _name);
>   if (ret) {
>   dev_err(pctrl->dev,
> - "missing function property in node %s\n",
> - node->name);
> + "missing function property in node %pOFn\n", node);
>   return -EINVAL;
>   }
>  
>   ngroups = of_property_count_strings(node, "groups");
>   if (ngroups < 0) {
>   dev_err(pctrl->dev,
> - "missing groups property in node %s\n",
> - node->name);
> + "missing groups property in node %pOFn\n", node);
>   return -EINVAL;
>   }
>  
> diff --git a/drivers/pinctrl/freescale/pinctrl-imx.c 
> b/drivers/pinctrl/freescale/pinctrl-imx.c
> index b04edc22dad7..4e8cf0e357c6 100644
> --- a/drivers/pinctrl/freescale/pinctrl-imx.c
> +++ b/drivers/pinctrl/freescale/pinctrl-imx.c
> @@ -69,8 +69,7 @@ static int imx_dt_node_to_map(struct pinctrl_dev *pctldev,
>*/
>   grp = imx_pinctrl_find_group_by_name(pctldev, np->name);
>   if (!grp) {
> - dev_err(ipctl->dev, "unable to find group for node %s\n",
> - np->name);
> + dev_err(ipctl->dev, "unable to find group for node %pOFn\n", 
> np);
>   return -EINVAL;
>   }
>  
> @@ -434,7 +433,7 @@ static int imx_pinctrl_parse_groups(struct device_node 
> *np,
>   int i;
>   u32 config;
>  
> - dev_dbg(ipctl->dev, "group(%d): %s\n", index, np->name);
> + dev_dbg(ipctl->dev, "group(%d): %pOFn\n", index, np);
>  
>   if (info->flags & SHARE_MUX_CONF_REG)
>   pin_size = FSL_PIN_SHARE_SIZE;
> @@ -544,7 +543,7 @@ static int imx_pinctrl_parse_functions(struct device_node 
> *np,
>   struct group_desc *grp;
>   u32 i = 0;
>  
> - dev_dbg(pctl->dev, "parse function(%d): %s\n", index, np->name);
> + dev_dbg(pctl->dev, "parse function(%d): %pOFn\n", index, np);
>  
>   func = pinmux_generic_get_function(pctl, index);
>   if (!func)
> diff --git a/drivers/pinctrl/freescale/pinctrl-imx1-core.c 
> b/drivers/pinctrl/freescale/pinctrl-imx1-core.c
> index deb7870b3d1a..7e29e3fecdb2 100644
> --- a/drivers/pinctrl/freescale/pinctrl-imx1-core.c
> +++ b/drivers/pinctrl/freescale/pinctrl-imx1-core.c
> @@ -233,8 +233,8 @@ static int imx1_dt_node_to_map(struct pinctrl_dev 
> *pctldev,
>*/
>   grp = imx1_pinctrl_find_group_by_name(info, np->name);
>   if (!grp) {
> - dev_err(info->dev, "unable 

RE: [PATCH] pinctrl: Convert to using %pOFn instead of device_node.name

2018-08-27 Thread A.s. Dong
> -Original Message-
> From: Rob Herring [mailto:r...@kernel.org]
> Sent: Tuesday, August 28, 2018 9:53 AM
> To: linux-kernel@vger.kernel.org
> Cc: Linus Walleij ; A.s. Dong
> ; Fabio Estevam ; Shawn
> Guo ; Stefan Agner ; Pengutronix
> Kernel Team ; Sean Wang
> ; Matthias Brugger ;
> Carlo Caione ; Kevin Hilman ;
> Jason Cooper ; Andrew Lunn ;
> Gregory Clement ; Sebastian Hesselbarth
> ; Jean-Christophe Plagniol-Villard
> ; Nicolas Ferre ;
> Alexandre Belloni ; Heiko Stuebner
> ; Tony Lindgren ; Haojian Zhuang
> ; Patrice Chotard ;
> Barry Song ; Maxime Coquelin
> ; Alexandre Torgue
> ; Maxime Ripard ;
> Chen-Yu Tsai ; linux-g...@vger.kernel.org;
> linux-media...@lists.infradead.org; linux-arm-ker...@lists.infradead.org;
> linux-amlo...@lists.infradead.org; linux-rockc...@lists.infradead.org;
> linux-o...@vger.kernel.org
> Subject: [PATCH] pinctrl: Convert to using %pOFn instead of device_node.name
> 
> In preparation to remove the node name pointer from struct device_node,
> convert printf users to use the %pOFn format specifier.
> 
> Cc: Linus Walleij 
> Cc: Dong Aisheng 
> Cc: Fabio Estevam 
> Cc: Shawn Guo 
> Cc: Stefan Agner 
> Cc: Pengutronix Kernel Team 
> Cc: Sean Wang 
> Cc: Matthias Brugger 
> Cc: Carlo Caione 
> Cc: Kevin Hilman 
> Cc: Jason Cooper 
> Cc: Andrew Lunn 
> Cc: Gregory Clement 
> Cc: Sebastian Hesselbarth 
> Cc: Jean-Christophe Plagniol-Villard 
> Cc: Nicolas Ferre 
> Cc: Alexandre Belloni 
> Cc: Heiko Stuebner 
> Cc: Tony Lindgren 
> Cc: Haojian Zhuang 
> Cc: Patrice Chotard 
> Cc: Barry Song 
> Cc: Maxime Coquelin 
> Cc: Alexandre Torgue 
> Cc: Maxime Ripard 
> Cc: Chen-Yu Tsai 
> Cc: linux-g...@vger.kernel.org
> Cc: linux-media...@lists.infradead.org
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: linux-amlo...@lists.infradead.org
> Cc: linux-rockc...@lists.infradead.org
> Cc: linux-o...@vger.kernel.org
> Signed-off-by: Rob Herring 
> ---
>  drivers/pinctrl/berlin/berlin.c   |  6 ++--
>  drivers/pinctrl/freescale/pinctrl-imx.c   |  7 ++--
>  drivers/pinctrl/freescale/pinctrl-imx1-core.c | 12 +++

For i.MX,
Acked-by: Dong Aisheng 

Regards
Dong Aisheng


RE: [PATCH] pinctrl: Convert to using %pOFn instead of device_node.name

2018-08-27 Thread A.s. Dong
> -Original Message-
> From: Rob Herring [mailto:r...@kernel.org]
> Sent: Tuesday, August 28, 2018 9:53 AM
> To: linux-kernel@vger.kernel.org
> Cc: Linus Walleij ; A.s. Dong
> ; Fabio Estevam ; Shawn
> Guo ; Stefan Agner ; Pengutronix
> Kernel Team ; Sean Wang
> ; Matthias Brugger ;
> Carlo Caione ; Kevin Hilman ;
> Jason Cooper ; Andrew Lunn ;
> Gregory Clement ; Sebastian Hesselbarth
> ; Jean-Christophe Plagniol-Villard
> ; Nicolas Ferre ;
> Alexandre Belloni ; Heiko Stuebner
> ; Tony Lindgren ; Haojian Zhuang
> ; Patrice Chotard ;
> Barry Song ; Maxime Coquelin
> ; Alexandre Torgue
> ; Maxime Ripard ;
> Chen-Yu Tsai ; linux-g...@vger.kernel.org;
> linux-media...@lists.infradead.org; linux-arm-ker...@lists.infradead.org;
> linux-amlo...@lists.infradead.org; linux-rockc...@lists.infradead.org;
> linux-o...@vger.kernel.org
> Subject: [PATCH] pinctrl: Convert to using %pOFn instead of device_node.name
> 
> In preparation to remove the node name pointer from struct device_node,
> convert printf users to use the %pOFn format specifier.
> 
> Cc: Linus Walleij 
> Cc: Dong Aisheng 
> Cc: Fabio Estevam 
> Cc: Shawn Guo 
> Cc: Stefan Agner 
> Cc: Pengutronix Kernel Team 
> Cc: Sean Wang 
> Cc: Matthias Brugger 
> Cc: Carlo Caione 
> Cc: Kevin Hilman 
> Cc: Jason Cooper 
> Cc: Andrew Lunn 
> Cc: Gregory Clement 
> Cc: Sebastian Hesselbarth 
> Cc: Jean-Christophe Plagniol-Villard 
> Cc: Nicolas Ferre 
> Cc: Alexandre Belloni 
> Cc: Heiko Stuebner 
> Cc: Tony Lindgren 
> Cc: Haojian Zhuang 
> Cc: Patrice Chotard 
> Cc: Barry Song 
> Cc: Maxime Coquelin 
> Cc: Alexandre Torgue 
> Cc: Maxime Ripard 
> Cc: Chen-Yu Tsai 
> Cc: linux-g...@vger.kernel.org
> Cc: linux-media...@lists.infradead.org
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: linux-amlo...@lists.infradead.org
> Cc: linux-rockc...@lists.infradead.org
> Cc: linux-o...@vger.kernel.org
> Signed-off-by: Rob Herring 
> ---
>  drivers/pinctrl/berlin/berlin.c   |  6 ++--
>  drivers/pinctrl/freescale/pinctrl-imx.c   |  7 ++--
>  drivers/pinctrl/freescale/pinctrl-imx1-core.c | 12 +++

For i.MX,
Acked-by: Dong Aisheng 

Regards
Dong Aisheng


[PATCH] pinctrl: Convert to using %pOFn instead of device_node.name

2018-08-27 Thread Rob Herring
In preparation to remove the node name pointer from struct device_node,
convert printf users to use the %pOFn format specifier.

Cc: Linus Walleij 
Cc: Dong Aisheng 
Cc: Fabio Estevam 
Cc: Shawn Guo 
Cc: Stefan Agner 
Cc: Pengutronix Kernel Team 
Cc: Sean Wang 
Cc: Matthias Brugger 
Cc: Carlo Caione 
Cc: Kevin Hilman 
Cc: Jason Cooper 
Cc: Andrew Lunn 
Cc: Gregory Clement 
Cc: Sebastian Hesselbarth 
Cc: Jean-Christophe Plagniol-Villard 
Cc: Nicolas Ferre 
Cc: Alexandre Belloni 
Cc: Heiko Stuebner 
Cc: Tony Lindgren 
Cc: Haojian Zhuang 
Cc: Patrice Chotard 
Cc: Barry Song 
Cc: Maxime Coquelin 
Cc: Alexandre Torgue 
Cc: Maxime Ripard 
Cc: Chen-Yu Tsai 
Cc: linux-g...@vger.kernel.org
Cc: linux-media...@lists.infradead.org
Cc: linux-arm-ker...@lists.infradead.org
Cc: linux-amlo...@lists.infradead.org
Cc: linux-rockc...@lists.infradead.org
Cc: linux-o...@vger.kernel.org
Signed-off-by: Rob Herring 
---
 drivers/pinctrl/berlin/berlin.c   |  6 ++--
 drivers/pinctrl/freescale/pinctrl-imx.c   |  7 ++--
 drivers/pinctrl/freescale/pinctrl-imx1-core.c | 12 +++
 drivers/pinctrl/mediatek/pinctrl-mtk-common.c |  4 +--
 drivers/pinctrl/meson/pinctrl-meson.c |  2 +-
 drivers/pinctrl/mvebu/pinctrl-mvebu.c |  4 +--
 drivers/pinctrl/nomadik/pinctrl-nomadik.c |  6 ++--
 drivers/pinctrl/pinctrl-at91.c|  8 ++---
 drivers/pinctrl/pinctrl-lantiq.c  |  8 ++---
 drivers/pinctrl/pinctrl-rockchip.c|  8 ++---
 drivers/pinctrl/pinctrl-rza1.c|  8 ++---
 drivers/pinctrl/pinctrl-single.c  | 32 +--
 drivers/pinctrl/pinctrl-st.c  |  6 ++--
 drivers/pinctrl/sirf/pinctrl-atlas7.c |  4 +--
 drivers/pinctrl/stm32/pinctrl-stm32.c |  4 +--
 drivers/pinctrl/sunxi/pinctrl-sunxi.c |  8 ++---
 drivers/pinctrl/ti/pinctrl-ti-iodelay.c   |  8 ++---
 17 files changed, 66 insertions(+), 69 deletions(-)

diff --git a/drivers/pinctrl/berlin/berlin.c b/drivers/pinctrl/berlin/berlin.c
index b5903fffb3d0..b17a03cf87be 100644
--- a/drivers/pinctrl/berlin/berlin.c
+++ b/drivers/pinctrl/berlin/berlin.c
@@ -64,16 +64,14 @@ static int berlin_pinctrl_dt_node_to_map(struct pinctrl_dev 
*pctrl_dev,
ret = of_property_read_string(node, "function", _name);
if (ret) {
dev_err(pctrl->dev,
-   "missing function property in node %s\n",
-   node->name);
+   "missing function property in node %pOFn\n", node);
return -EINVAL;
}
 
ngroups = of_property_count_strings(node, "groups");
if (ngroups < 0) {
dev_err(pctrl->dev,
-   "missing groups property in node %s\n",
-   node->name);
+   "missing groups property in node %pOFn\n", node);
return -EINVAL;
}
 
diff --git a/drivers/pinctrl/freescale/pinctrl-imx.c 
b/drivers/pinctrl/freescale/pinctrl-imx.c
index b04edc22dad7..4e8cf0e357c6 100644
--- a/drivers/pinctrl/freescale/pinctrl-imx.c
+++ b/drivers/pinctrl/freescale/pinctrl-imx.c
@@ -69,8 +69,7 @@ static int imx_dt_node_to_map(struct pinctrl_dev *pctldev,
 */
grp = imx_pinctrl_find_group_by_name(pctldev, np->name);
if (!grp) {
-   dev_err(ipctl->dev, "unable to find group for node %s\n",
-   np->name);
+   dev_err(ipctl->dev, "unable to find group for node %pOFn\n", 
np);
return -EINVAL;
}
 
@@ -434,7 +433,7 @@ static int imx_pinctrl_parse_groups(struct device_node *np,
int i;
u32 config;
 
-   dev_dbg(ipctl->dev, "group(%d): %s\n", index, np->name);
+   dev_dbg(ipctl->dev, "group(%d): %pOFn\n", index, np);
 
if (info->flags & SHARE_MUX_CONF_REG)
pin_size = FSL_PIN_SHARE_SIZE;
@@ -544,7 +543,7 @@ static int imx_pinctrl_parse_functions(struct device_node 
*np,
struct group_desc *grp;
u32 i = 0;
 
-   dev_dbg(pctl->dev, "parse function(%d): %s\n", index, np->name);
+   dev_dbg(pctl->dev, "parse function(%d): %pOFn\n", index, np);
 
func = pinmux_generic_get_function(pctl, index);
if (!func)
diff --git a/drivers/pinctrl/freescale/pinctrl-imx1-core.c 
b/drivers/pinctrl/freescale/pinctrl-imx1-core.c
index deb7870b3d1a..7e29e3fecdb2 100644
--- a/drivers/pinctrl/freescale/pinctrl-imx1-core.c
+++ b/drivers/pinctrl/freescale/pinctrl-imx1-core.c
@@ -233,8 +233,8 @@ static int imx1_dt_node_to_map(struct pinctrl_dev *pctldev,
 */
grp = imx1_pinctrl_find_group_by_name(info, np->name);
if (!grp) {
-   dev_err(info->dev, "unable to find group for node %s\n",
-   np->name);
+   dev_err(info->dev, "unable to find group for node %pOFn\n",
+   np);
return -EINVAL;
}
 
@@ -466,7 +466,7 @@ static int 

[PATCH] pinctrl: Convert to using %pOFn instead of device_node.name

2018-08-27 Thread Rob Herring
In preparation to remove the node name pointer from struct device_node,
convert printf users to use the %pOFn format specifier.

Cc: Linus Walleij 
Cc: Dong Aisheng 
Cc: Fabio Estevam 
Cc: Shawn Guo 
Cc: Stefan Agner 
Cc: Pengutronix Kernel Team 
Cc: Sean Wang 
Cc: Matthias Brugger 
Cc: Carlo Caione 
Cc: Kevin Hilman 
Cc: Jason Cooper 
Cc: Andrew Lunn 
Cc: Gregory Clement 
Cc: Sebastian Hesselbarth 
Cc: Jean-Christophe Plagniol-Villard 
Cc: Nicolas Ferre 
Cc: Alexandre Belloni 
Cc: Heiko Stuebner 
Cc: Tony Lindgren 
Cc: Haojian Zhuang 
Cc: Patrice Chotard 
Cc: Barry Song 
Cc: Maxime Coquelin 
Cc: Alexandre Torgue 
Cc: Maxime Ripard 
Cc: Chen-Yu Tsai 
Cc: linux-g...@vger.kernel.org
Cc: linux-media...@lists.infradead.org
Cc: linux-arm-ker...@lists.infradead.org
Cc: linux-amlo...@lists.infradead.org
Cc: linux-rockc...@lists.infradead.org
Cc: linux-o...@vger.kernel.org
Signed-off-by: Rob Herring 
---
 drivers/pinctrl/berlin/berlin.c   |  6 ++--
 drivers/pinctrl/freescale/pinctrl-imx.c   |  7 ++--
 drivers/pinctrl/freescale/pinctrl-imx1-core.c | 12 +++
 drivers/pinctrl/mediatek/pinctrl-mtk-common.c |  4 +--
 drivers/pinctrl/meson/pinctrl-meson.c |  2 +-
 drivers/pinctrl/mvebu/pinctrl-mvebu.c |  4 +--
 drivers/pinctrl/nomadik/pinctrl-nomadik.c |  6 ++--
 drivers/pinctrl/pinctrl-at91.c|  8 ++---
 drivers/pinctrl/pinctrl-lantiq.c  |  8 ++---
 drivers/pinctrl/pinctrl-rockchip.c|  8 ++---
 drivers/pinctrl/pinctrl-rza1.c|  8 ++---
 drivers/pinctrl/pinctrl-single.c  | 32 +--
 drivers/pinctrl/pinctrl-st.c  |  6 ++--
 drivers/pinctrl/sirf/pinctrl-atlas7.c |  4 +--
 drivers/pinctrl/stm32/pinctrl-stm32.c |  4 +--
 drivers/pinctrl/sunxi/pinctrl-sunxi.c |  8 ++---
 drivers/pinctrl/ti/pinctrl-ti-iodelay.c   |  8 ++---
 17 files changed, 66 insertions(+), 69 deletions(-)

diff --git a/drivers/pinctrl/berlin/berlin.c b/drivers/pinctrl/berlin/berlin.c
index b5903fffb3d0..b17a03cf87be 100644
--- a/drivers/pinctrl/berlin/berlin.c
+++ b/drivers/pinctrl/berlin/berlin.c
@@ -64,16 +64,14 @@ static int berlin_pinctrl_dt_node_to_map(struct pinctrl_dev 
*pctrl_dev,
ret = of_property_read_string(node, "function", _name);
if (ret) {
dev_err(pctrl->dev,
-   "missing function property in node %s\n",
-   node->name);
+   "missing function property in node %pOFn\n", node);
return -EINVAL;
}
 
ngroups = of_property_count_strings(node, "groups");
if (ngroups < 0) {
dev_err(pctrl->dev,
-   "missing groups property in node %s\n",
-   node->name);
+   "missing groups property in node %pOFn\n", node);
return -EINVAL;
}
 
diff --git a/drivers/pinctrl/freescale/pinctrl-imx.c 
b/drivers/pinctrl/freescale/pinctrl-imx.c
index b04edc22dad7..4e8cf0e357c6 100644
--- a/drivers/pinctrl/freescale/pinctrl-imx.c
+++ b/drivers/pinctrl/freescale/pinctrl-imx.c
@@ -69,8 +69,7 @@ static int imx_dt_node_to_map(struct pinctrl_dev *pctldev,
 */
grp = imx_pinctrl_find_group_by_name(pctldev, np->name);
if (!grp) {
-   dev_err(ipctl->dev, "unable to find group for node %s\n",
-   np->name);
+   dev_err(ipctl->dev, "unable to find group for node %pOFn\n", 
np);
return -EINVAL;
}
 
@@ -434,7 +433,7 @@ static int imx_pinctrl_parse_groups(struct device_node *np,
int i;
u32 config;
 
-   dev_dbg(ipctl->dev, "group(%d): %s\n", index, np->name);
+   dev_dbg(ipctl->dev, "group(%d): %pOFn\n", index, np);
 
if (info->flags & SHARE_MUX_CONF_REG)
pin_size = FSL_PIN_SHARE_SIZE;
@@ -544,7 +543,7 @@ static int imx_pinctrl_parse_functions(struct device_node 
*np,
struct group_desc *grp;
u32 i = 0;
 
-   dev_dbg(pctl->dev, "parse function(%d): %s\n", index, np->name);
+   dev_dbg(pctl->dev, "parse function(%d): %pOFn\n", index, np);
 
func = pinmux_generic_get_function(pctl, index);
if (!func)
diff --git a/drivers/pinctrl/freescale/pinctrl-imx1-core.c 
b/drivers/pinctrl/freescale/pinctrl-imx1-core.c
index deb7870b3d1a..7e29e3fecdb2 100644
--- a/drivers/pinctrl/freescale/pinctrl-imx1-core.c
+++ b/drivers/pinctrl/freescale/pinctrl-imx1-core.c
@@ -233,8 +233,8 @@ static int imx1_dt_node_to_map(struct pinctrl_dev *pctldev,
 */
grp = imx1_pinctrl_find_group_by_name(info, np->name);
if (!grp) {
-   dev_err(info->dev, "unable to find group for node %s\n",
-   np->name);
+   dev_err(info->dev, "unable to find group for node %pOFn\n",
+   np);
return -EINVAL;
}
 
@@ -466,7 +466,7 @@ static int