[PATCH v2 2/5] devfreq: Use %pe to print error pointers symbolically
From: Sumeet Pawnikar <hidden>
Date: 2026-09-12 19:12:44
Also in:
imx, linux-arm-msm, linux-mediatek, linux-pm, linux-samsung-soc, lkml
Subsystem:
device frequency (devfreq), the rest · Maintainers:
MyungJoo Ham, Kyungmin Park, Chanwoo Choi, Linus Torvalds
Replace PTR_ERR() and %ld with %pe and pass the original pointer directly to dev_err(). The %pe format specifier prints a symbolic error name (e.g. -ENOMEM) when CONFIG_SYMBOLIC_ERRNAME is enabled, otherwise it falls back gracefully and prints the raw integer value. This makes messages more readable without any functional change. Signed-off-by: Sumeet Pawnikar <redacted> --- drivers/devfreq/imx-bus.c | 4 ++-- drivers/devfreq/imx8m-ddrc.c | 3 +-- 2 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/drivers/devfreq/imx-bus.c b/drivers/devfreq/imx-bus.c
index 49798f542d68..dba76d8b9e5a 100644
--- a/drivers/devfreq/imx-bus.c
+++ b/drivers/devfreq/imx-bus.c@@ -75,8 +75,8 @@ static int imx_bus_init_icc(struct device *dev) priv->icc_pdev = platform_device_register_data( dev, icc_driver_name, -1, NULL, 0); if (IS_ERR(priv->icc_pdev)) { - dev_err(dev, "failed to register icc provider %s: %ld\n", - icc_driver_name, PTR_ERR(priv->icc_pdev)); + dev_err(dev, "failed to register icc provider %s: %pe\n", + icc_driver_name, priv->icc_pdev); return PTR_ERR(priv->icc_pdev); }
diff --git a/drivers/devfreq/imx8m-ddrc.c b/drivers/devfreq/imx8m-ddrc.c
index 52beeb5b7d65..28d7c595fa17 100644
--- a/drivers/devfreq/imx8m-ddrc.c
+++ b/drivers/devfreq/imx8m-ddrc.c@@ -340,8 +340,7 @@ static int imx8m_ddrc_check_opps(struct device *dev) for (i = 0, freq = 0; i < opp_count; ++i, ++freq) { opp = dev_pm_opp_find_freq_ceil(dev, &freq); if (IS_ERR(opp)) { - dev_err(dev, "Failed enumerating OPPs: %ld\n", - PTR_ERR(opp)); + dev_err(dev, "Failed enumerating OPPs: %pe\n", opp); return PTR_ERR(opp); } dev_pm_opp_put(opp);
--
2.43.0