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@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@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);@@ -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");@@ -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@st.com
Thanks