diff mbox series

[U-Boot] dfu: dfu_nand: reduce verbosity

Message ID 20190827182819.30717-1-ralph.siemsen@linaro.org
State Accepted
Commit 0a815ff7bb22b034d15758d41816df1c3d554c14
Delegated to: Lukasz Majewski
Headers show
Series [U-Boot] dfu: dfu_nand: reduce verbosity | expand

Commit Message

Ralph Siemsen Aug. 27, 2019, 6:28 p.m. UTC
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(-)

Comments

Lukasz Majewski Aug. 27, 2019, 9:38 p.m. UTC | #1
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@denx.de
diff mbox series

Patch

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();