All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] devres: no print device_node.name in devres_log()
@ 2018-11-24 14:25 Yangtao Li
  2018-11-24 14:56 ` Frank Lee
  0 siblings, 1 reply; 2+ messages in thread
From: Yangtao Li @ 2018-11-24 14:25 UTC (permalink / raw)
  To: robh, gregkh, rafael; +Cc: linux-kernel, Yangtao Li

In preparation to remove the node name pointer from struct device_node,
avoid to printf node name.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/base/devres.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/base/devres.c b/drivers/base/devres.c
index 4aaf00d2098b..34c07e9faaf9 100644
--- a/drivers/base/devres.c
+++ b/drivers/base/devres.c
@@ -52,8 +52,8 @@ static void devres_log(struct device *dev, struct devres_node *node,
 		       const char *op)
 {
 	if (unlikely(log_devres))
-		dev_err(dev, "DEVRES %3s %p %s (%lu bytes)\n",
-			op, node, node->name, (unsigned long)node->size);
+		dev_err(dev, "DEVRES %3s %p (%lu bytes)\n",
+			op, node, (unsigned long)node->size);
 }
 #else /* CONFIG_DEBUG_DEVRES */
 #define set_node_dbginfo(node, n, s)	do {} while (0)
-- 
2.17.0


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] devres: no print device_node.name in devres_log()
  2018-11-24 14:25 [PATCH] devres: no print device_node.name in devres_log() Yangtao Li
@ 2018-11-24 14:56 ` Frank Lee
  0 siblings, 0 replies; 2+ messages in thread
From: Frank Lee @ 2018-11-24 14:56 UTC (permalink / raw)
  To: robh, gregkh, rafael; +Cc: linux-kernel

On Sat, Nov 24, 2018 at 10:25 PM Yangtao Li <tiny.windzz@gmail.com> wrote:
>
> In preparation to remove the node name pointer from struct device_node,
> avoid to printf node name.
>
> Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
> ---
>  drivers/base/devres.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/base/devres.c b/drivers/base/devres.c
> index 4aaf00d2098b..34c07e9faaf9 100644
> --- a/drivers/base/devres.c
> +++ b/drivers/base/devres.c
> @@ -52,8 +52,8 @@ static void devres_log(struct device *dev, struct devres_node *node,
>                        const char *op)
>  {
>         if (unlikely(log_devres))
> -               dev_err(dev, "DEVRES %3s %p %s (%lu bytes)\n",
> -                       op, node, node->name, (unsigned long)node->size);
The modification of this place is a bit problematic, ignoring this email.

Yours,
Yangtao
> +               dev_err(dev, "DEVRES %3s %p (%lu bytes)\n",
> +                       op, node, (unsigned long)node->size);
>  }
>  #else /* CONFIG_DEBUG_DEVRES */
>  #define set_node_dbginfo(node, n, s)   do {} while (0)
> --
> 2.17.0
>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2018-11-24 14:57 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-11-24 14:25 [PATCH] devres: no print device_node.name in devres_log() Yangtao Li
2018-11-24 14:56 ` Frank Lee

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.