[Uboot-stm32] [PATCH 07/33] ram: stm32: migrate trace to log macro

[Patrice CHOTARD]

Hi Patrick

On 10/14/20 11:16 AM, Patrick Delaunay wrote:
> Define LOG_CATEGORY, change debug to dev_dbg and remove "%s:" __func__
> header as it is managed by dev macro (dev->name is displayed)
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay at st.com>
> ---
>
>  drivers/ram/stm32_sdram.c | 10 ++++++----
>  1 file changed, 6 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/ram/stm32_sdram.c b/drivers/ram/stm32_sdram.c
> index 3fddf4df96..9e0e70ca97 100644
> --- a/drivers/ram/stm32_sdram.c
> +++ b/drivers/ram/stm32_sdram.c
> @@ -4,6 +4,8 @@
>   * Author(s): Vikas Manocha, <vikas.manocha at st.com> for STMicroelectronics.
>   */
>  
> +#define LOG_CATEGORY UCLASS_RAM
> +
>  #include <common.h>
>  #include <clk.h>
>  #include <dm.h>
> @@ -272,7 +274,7 @@ static int stm32_fmc_ofdata_to_platdata(struct udevice *dev)
>  	ret = dev_read_phandle_with_args(dev, "st,syscfg", NULL, 0, 0,
>  						 &args);
>  	if (ret) {
> -		dev_dbg(dev, "%s: can't find syscon device (%d)\n", __func__, ret);
> +		dev_dbg(dev, "can't find syscon device (%d)\n", ret);
>  	} else {
>  		syscfg_base = (u32 *)ofnode_get_addr(args.node);
>  
> @@ -281,7 +283,7 @@ static int stm32_fmc_ofdata_to_platdata(struct udevice *dev)
>  			/* set memory mapping selection */
>  			clrsetbits_le32(syscfg_base, MEM_MODE_MASK, mem_remap);
>  		} else {
> -			dev_dbg(dev, "%s: cannot find st,mem_remap property\n", __func__);
> +			dev_dbg(dev, "cannot find st,mem_remap property\n");
>  		}
>  		
>  		swp_fmc = dev_read_u32_default(dev, "st,swp_fmc", NOT_FOUND);
> @@ -289,7 +291,7 @@ static int stm32_fmc_ofdata_to_platdata(struct udevice *dev)
>  			/* set fmc swapping selection */
>  			clrsetbits_le32(syscfg_base, SWP_FMC_MASK, swp_fmc << SWP_FMC_OFFSET);
>  		} else {
> -			dev_dbg(dev, "%s: cannot find st,swp_fmc property\n", __func__);
> +			dev_dbg(dev, "cannot find st,swp_fmc property\n");
>  		}
>  
>  		dev_dbg(dev, "syscfg %x = %x\n", (u32)syscfg_base, *syscfg_base);
> @@ -348,7 +350,7 @@ static int stm32_fmc_ofdata_to_platdata(struct udevice *dev)
>  	}
>  
>  	params->no_sdram_banks = bank;
> -	debug("%s, no of banks = %d\n", __func__, params->no_sdram_banks);
> +	dev_dbg(dev, "no of banks = %d\n", params->no_sdram_banks);
>  
>  	return 0;
>  }

Reviewed-by: Patrice Chotard <patrice.chotard at st.com>

Thanks