-----Original Message----- From: Patrice Chotard patrice.chotard@foss.st.com Sent: Tuesday, December 3, 2024 7:06 PM To: u-boot@lists.denx.de Cc: Patrice CHOTARD patrice.chotard@foss.st.com; Patrick DELAUNAY patrick.delaunay@foss.st.com; U- Boot STM32 uboot-stm32@st-md-mailman.stormreply.com; Jaehoon Chung jh80.chung@samsung.com; Jonas Karlman jonas@kwiboo.se; Marek Vasut marex@denx.de; Quentin Schulz quentin.schulz@cherry.de; Simon Glass sjg@chromium.org; Tom Rini trini@konsulko.com Subject: [PATCH v2 1/2] power: regulator: replace some debug() by dev_dbg()
Replace some debug() by dev_dbg() when dev variable is available/valid.
Signed-off-by: Patrice Chotard patrice.chotard@foss.st.com
Reviewed-by: Jaehoon Chung jh80.chung@samsung.com
Best Regards, Jaehoon Chung
Changes in v2: - rework dev_dbg() message to avoid printing twice dev->name.
drivers/power/regulator/regulator-uclass.c | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-)
diff --git a/drivers/power/regulator/regulator-uclass.c b/drivers/power/regulator/regulator-uclass.c index decd0802c84..80ea5e65d48 100644 --- a/drivers/power/regulator/regulator-uclass.c +++ b/drivers/power/regulator/regulator-uclass.c @@ -9,6 +9,7 @@ #include <errno.h> #include <dm.h> #include <log.h> +#include <dm/device_compat.h> #include <dm/uclass-internal.h> #include <linux/delay.h> #include <power/pmic.h> @@ -43,8 +44,7 @@ static void regulator_set_value_ramp_delay(struct udevice *dev, int old_uV, { int delay = DIV_ROUND_UP(abs(new_uV - old_uV), ramp_delay);
- debug("regulator %s: delay %u us (%d uV -> %d uV)\n", dev->name, delay,
old_uV, new_uV);
dev_dbg(dev, "delay %u us (%d uV -> %d uV)\n", delay, old_uV, new_uV);
udelay(delay);
} @@ -263,7 +263,7 @@ int regulator_get_by_platname(const char *plat_name, struct udevice **devp) for (ret = uclass_find_first_device(UCLASS_REGULATOR, &dev); dev; ret = uclass_find_next_device(&dev)) { if (ret) {
debug("regulator %s, ret=%d\n", dev->name, ret);
dev_dbg(dev, "ret=%d\n", ret); continue;@@ -439,16 +439,15 @@ static int regulator_post_bind(struct udevice *dev) /* Regulator's mandatory constraint */ uc_pdata->name = dev_read_string(dev, property); if (!uc_pdata->name) {
debug("%s: dev '%s' has no property '%s'\n",__func__, dev->name, property);
dev_dbg(dev, "has no property '%s'\n", property);uc_pdata->name = dev_read_name(dev); if (!uc_pdata->name) return -EINVAL; }
if (!regulator_name_is_unique(dev, uc_pdata->name)) {
debug("'%s' of dev: '%s', has nonunique value: '%s\n",property, dev->name, uc_pdata->name);
dev_dbg(dev, "'%s' has nonunique value: '%s\n",property, uc_pdata->name);-- 2.25.1