All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] dfu: dfu_nand: reduce verbosity
@ 2019-08-27 18:28 Ralph Siemsen
  2019-08-27 21:38 ` Lukasz Majewski
  0 siblings, 1 reply; 2+ messages in thread
From: Ralph Siemsen @ 2019-08-27 18:28 UTC (permalink / raw)
  To: u-boot

In combination with multiple partitions in NAND, this printf() ends up
being more noise than helpful. Change it to debug() instead.

Signed-off-by: Ralph Siemsen <ralph.siemsen@linaro.org>
---

 drivers/dfu/dfu_nand.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/dfu/dfu_nand.c b/drivers/dfu/dfu_nand.c
index 0bfdbf9428..b812a3dfb1 100644
--- a/drivers/dfu/dfu_nand.c
+++ b/drivers/dfu/dfu_nand.c
@@ -214,7 +214,7 @@ int dfu_fill_entity_nand(struct dfu_entity *dfu, char *devstr, char *s)
 		part = simple_strtoul(s, &s, 10);
 
 		sprintf(mtd_id, "%s%d,%d", "nand", dev, part - 1);
-		printf("using id '%s'\n", mtd_id);
+		debug("using id '%s'\n", mtd_id);
 
 		mtdparts_init();
 
-- 
2.17.1

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

* [U-Boot] [PATCH] dfu: dfu_nand: reduce verbosity
  2019-08-27 18:28 [U-Boot] [PATCH] dfu: dfu_nand: reduce verbosity Ralph Siemsen
@ 2019-08-27 21:38 ` Lukasz Majewski
  0 siblings, 0 replies; 2+ messages in thread
From: Lukasz Majewski @ 2019-08-27 21:38 UTC (permalink / raw)
  To: u-boot

Hi Ralph,

> In combination with multiple partitions in NAND, this printf() ends up
> being more noise than helpful. Change it to debug() instead.
> 
> Signed-off-by: Ralph Siemsen <ralph.siemsen@linaro.org>
> ---
> 
>  drivers/dfu/dfu_nand.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/dfu/dfu_nand.c b/drivers/dfu/dfu_nand.c
> index 0bfdbf9428..b812a3dfb1 100644
> --- a/drivers/dfu/dfu_nand.c
> +++ b/drivers/dfu/dfu_nand.c
> @@ -214,7 +214,7 @@ int dfu_fill_entity_nand(struct dfu_entity *dfu,
> char *devstr, char *s) part = simple_strtoul(s, &s, 10);
>  
>  		sprintf(mtd_id, "%s%d,%d", "nand", dev, part - 1);
> -		printf("using id '%s'\n", mtd_id);
> +		debug("using id '%s'\n", mtd_id);
>  
>  		mtdparts_init();
>  

Acked-by: Lukasz Majewski <lukma@denx.de>


Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190827/ac5da14d/attachment.sig>

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

end of thread, other threads:[~2019-08-27 21:38 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-27 18:28 [U-Boot] [PATCH] dfu: dfu_nand: reduce verbosity Ralph Siemsen
2019-08-27 21:38 ` Lukasz Majewski

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.