Hi Patrick On 10/14/20 11:16 AM, Patrick Delaunay wrote: > Change pr_* to dev_ or log_ macro and define LOG_CATEGORY. > > Signed-off-by: Patrick Delaunay <patrick.delau...@st.com> > --- > > drivers/memory/stm32-fmc2-ebi.c | 30 ++++++++++++++++-------------- > 1 file changed, 16 insertions(+), 14 deletions(-) > > diff --git a/drivers/memory/stm32-fmc2-ebi.c b/drivers/memory/stm32-fmc2-ebi.c > index d887a1e09d..e3cf75f6e3 100644 > --- a/drivers/memory/stm32-fmc2-ebi.c > +++ b/drivers/memory/stm32-fmc2-ebi.c > @@ -3,10 +3,13 @@ > * Copyright (C) STMicroelectronics 2020 > */ > > +#define LOG_CATEGORY UCLASS_NOP > + > #include <common.h> > #include <clk.h> > #include <dm.h> > #include <reset.h> > +#include <dm/device_compat.h> > #include <linux/bitfield.h> > #include <linux/err.h> > #include <linux/iopoll.h> > @@ -860,7 +863,7 @@ static int stm32_fmc2_ebi_parse_prop(struct > stm32_fmc2_ebi *ebi, > u32 setup = 0; > > if (!prop->set) { > - pr_err("property %s is not well defined\n", prop->name); > + log_err("property %s is not well defined\n", prop->name); > return -EINVAL; > } > > @@ -873,8 +876,8 @@ static int stm32_fmc2_ebi_parse_prop(struct > stm32_fmc2_ebi *ebi, > > bprop = ofnode_read_bool(node, prop->name); > if (prop->mprop && !bprop) { > - pr_err("mandatory property %s not defined in the device > tree\n", > - prop->name); > + log_err("mandatory property %s not defined in the > device tree\n", > + prop->name); > return -EINVAL; > } > > @@ -886,8 +889,8 @@ static int stm32_fmc2_ebi_parse_prop(struct > stm32_fmc2_ebi *ebi, > > ret = ofnode_read_u32(node, prop->name, &val); > if (prop->mprop && ret) { > - pr_err("mandatory property %s not defined in the device > tree\n", > - prop->name); > + log_err("mandatory property %s not defined in the > device tree\n", > + prop->name); > return ret; > } > > @@ -949,8 +952,8 @@ static int stm32_fmc2_ebi_setup_cs(struct stm32_fmc2_ebi > *ebi, > > ret = stm32_fmc2_ebi_parse_prop(ebi, node, p, cs); > if (ret) { > - pr_err("property %s could not be set: %d\n", > - p->name, ret); > + log_err("property %s could not be set: %d\n", > + p->name, ret); > return ret; > } > } > @@ -971,25 +974,24 @@ static int stm32_fmc2_ebi_parse_dt(struct udevice *dev, > dev_for_each_subnode(child, dev) { > ret = ofnode_read_u32(child, "reg", &bank); > if (ret) { > - pr_err("could not retrieve reg property: %d\n", ret); > + dev_err(dev, "could not retrieve reg property: %d\n", > ret); > return ret; > } > > if (bank >= FMC2_MAX_BANKS) { > - pr_err("invalid reg value: %d\n", bank); > + dev_err(dev, "invalid reg value: %d\n", bank); > return -EINVAL; > } > > if (ebi->bank_assigned & BIT(bank)) { > - pr_err("bank already assigned: %d\n", bank); > + dev_err(dev, "bank already assigned: %d\n", bank); > return -EINVAL; > } > > if (bank < FMC2_MAX_EBI_CE) { > ret = stm32_fmc2_ebi_setup_cs(ebi, child, bank); > if (ret) { > - pr_err("setup chip select %d failed: %d\n", > - bank, ret); > + dev_err(dev, "setup chip select %d failed: > %d\n", bank, ret); > return ret; > } > } > @@ -999,12 +1001,12 @@ static int stm32_fmc2_ebi_parse_dt(struct udevice *dev, > } > > if (!child_found) { > - pr_warn("no subnodes found, disable the driver.\n"); > + dev_warn(dev, "no subnodes found, disable the driver.\n"); > return -ENODEV; > } > > if (stm32_fmc2_ebi_nwait_used_by_ctrls(ebi)) { > - pr_err("NWAIT signal connected to EBI and NAND controllers\n"); > + dev_err(dev, "NWAIT signal connected to EBI and NAND > controllers\n"); > return -EINVAL; > } >
Reviewed-by: Patrice Chotard <patrice.chot...@st.com> Thanks