[PATCH] imx8m: soc.c: demote some printfs to debug

[PATCH] imx8m: soc.c: demote some printfs to debug

[Rasmus Villemoes]

Getting

  Found /vpu_g1@38300000 node
  Modify /vpu_g1@38300000:status disabled
  Found /vpu_g2@38310000 node
  Modify /vpu_g2@38310000:status disabled

etc. on the console on every boot is needlessly verbose. Demote the
"Found ..." lines to debug(), which is consistent with other instances
in soc.c.

Signed-off-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk>
---
 arch/arm/mach-imx/imx8m/soc.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/mach-imx/imx8m/soc.c b/arch/arm/mach-imx/imx8m/soc.c
index 5a4f8358c9..f5c82dff35 100644
--- a/arch/arm/mach-imx/imx8m/soc.c
+++ b/arch/arm/mach-imx/imx8m/soc.c
@@ -736,7 +736,7 @@ static int disable_fdt_nodes(void *blob, const char *const nodes_path[], int siz
 		if (nodeoff < 0)
 			continue; /* Not found, skip it */
 
-		printf("Found %s node\n", nodes_path[i]);
+		debug("Found %s node\n", nodes_path[i]);
 
 add_status:
 		rc = fdt_setprop(blob, nodeoff, "status", status, strlen(status) + 1);
@@ -1265,7 +1265,7 @@ int ft_system_setup(void *blob, struct bd_info *bd)
 		if (nodeoff >= 0) {
 			const char *speed = "high-speed";
 
-			printf("Found %s node\n", usb_dwc3_path[v]);
+			debug("Found %s node\n", usb_dwc3_path[v]);
 
 usb_modify_speed:
 
-- 
2.37.2

Re: [PATCH] imx8m: soc.c: demote some printfs to debug

[Frieder Schrempf]

On 22.05.23 11:27, Rasmus Villemoes wrote:
> Getting
> 
>   Found /vpu_g1@38300000 node
>   Modify /vpu_g1@38300000:status disabled
>   Found /vpu_g2@38310000 node
>   Modify /vpu_g2@38310000:status disabled
> 
> etc. on the console on every boot is needlessly verbose. Demote the
> "Found ..." lines to debug(), which is consistent with other instances
> in soc.c.
> 
> Signed-off-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk>

Reviewed-by: Frieder Schrempf <frieder.schrempf@kontron.de>

Re: [PATCH] imx8m: soc.c: demote some printfs to debug

[Peng Fan]

On 5/22/2023 5:27 PM, Rasmus Villemoes wrote:
> Getting
> 
>    Found /vpu_g1@38300000 node
>    Modify /vpu_g1@38300000:status disabled
>    Found /vpu_g2@38310000 node
>    Modify /vpu_g2@38310000:status disabled
> 
> etc. on the console on every boot is needlessly verbose. Demote the
> "Found ..." lines to debug(), which is consistent with other instances
> in soc.c.
> 
> Signed-off-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk>

Reviewed-by: Peng Fan <peng.fan@nxp.com>

> ---
>   arch/arm/mach-imx/imx8m/soc.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/mach-imx/imx8m/soc.c b/arch/arm/mach-imx/imx8m/soc.c
> index 5a4f8358c9..f5c82dff35 100644
> --- a/arch/arm/mach-imx/imx8m/soc.c
> +++ b/arch/arm/mach-imx/imx8m/soc.c
> @@ -736,7 +736,7 @@ static int disable_fdt_nodes(void *blob, const char *const nodes_path[], int siz
>   		if (nodeoff < 0)
>   			continue; /* Not found, skip it */
>   
> -		printf("Found %s node\n", nodes_path[i]);
> +		debug("Found %s node\n", nodes_path[i]);
>   
>   add_status:
>   		rc = fdt_setprop(blob, nodeoff, "status", status, strlen(status) + 1);
> @@ -1265,7 +1265,7 @@ int ft_system_setup(void *blob, struct bd_info *bd)
>   		if (nodeoff >= 0) {
>   			const char *speed = "high-speed";
>   
> -			printf("Found %s node\n", usb_dwc3_path[v]);
> +			debug("Found %s node\n", usb_dwc3_path[v]);
>   
>   usb_modify_speed:
>   

[PATCH] imx8m: soc.c: demote some printfs to debug

[sbabic]

> Getting
>   Found /vpu_g1@38300000 node
>   Modify /vpu_g1@38300000:status disabled
>   Found /vpu_g2@38310000 node
>   Modify /vpu_g2@38310000:status disabled
> etc. on the console on every boot is needlessly verbose. Demote the
> "Found ..." lines to debug(), which is consistent with other instances
> in soc.c.
> Signed-off-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk>
> Reviewed-by: Frieder Schrempf <frieder.schrempf@kontron.de>
> Reviewed-by: Peng Fan <peng.fan@nxp.com>
Applied to u-boot-imx, master, thanks !

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,        Managing Director: Erika Unter  
HRB 165235 Munich,   Office: Kirchenstr.5, 82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de
=====================================================================