[PATCH 5/5] power: Use %pe to print error pointers symbolically
From: Sumeet Pawnikar <hidden>
Date: 2026-09-06 06:19:14
Also in:
imx, linux-arm-msm, linux-mediatek, linux-pm, linux-samsung-soc, lkml
Subsystem:
arm/apple machine support, arm/qualcomm mailing list, cellwise cw2015 battery driver, power supply class/subsystem and drivers, qualcomm smb charger driver, system reset/shutdown drivers, the rest · Maintainers:
Sven Peter, Janne Grunau, Tobias Schrammm, Sebastian Reichel, Casey Connolly, Linus Torvalds
Replace PTR_ERR() and %ld/%li with %pe and pass the original pointer directly to dev_info(), dev_warn(), dev_err() and dev_dbg(). 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/power/reset/macsmc-reboot.c | 4 ++-- drivers/power/supply/axp20x_battery.c | 4 ++-- drivers/power/supply/cpcap-charger.c | 4 ++-- drivers/power/supply/cw2015_battery.c | 4 ++-- drivers/power/supply/intel_dc_ti_battery.c | 2 +- drivers/power/supply/qcom_smbx.c | 2 +- 6 files changed, 10 insertions(+), 10 deletions(-)
diff --git a/drivers/power/reset/macsmc-reboot.c b/drivers/power/reset/macsmc-reboot.c
index 9fc36fa68676..7b8460a55e9d 100644
--- a/drivers/power/reset/macsmc-reboot.c
+++ b/drivers/power/reset/macsmc-reboot.c@@ -221,8 +221,8 @@ static int macsmc_reboot_probe(struct platform_device *pdev) if (IS_ERR(cell)) { if (PTR_ERR(cell) == -EPROBE_DEFER) return -EPROBE_DEFER; - dev_warn(&pdev->dev, "Missing NVMEM cell %s (%ld)\n", - nvmem_names[i], PTR_ERR(cell)); + dev_warn(&pdev->dev, "Missing NVMEM cell %s (%pe)\n", + nvmem_names[i], cell); /* Non fatal, we'll deal with it */ cell = NULL; }
diff --git a/drivers/power/supply/axp20x_battery.c b/drivers/power/supply/axp20x_battery.c
index ee8701a6e907..915b6f6881db 100644
--- a/drivers/power/supply/axp20x_battery.c
+++ b/drivers/power/supply/axp20x_battery.c@@ -1123,8 +1123,8 @@ static int axp20x_power_probe(struct platform_device *pdev) axp20x_batt->data->bat_ps_desc, &psy_cfg); if (IS_ERR(axp20x_batt->batt)) { - dev_err(&pdev->dev, "failed to register power supply: %ld\n", - PTR_ERR(axp20x_batt->batt)); + dev_err(&pdev->dev, "failed to register power supply: %pe\n", + axp20x_batt->batt); return PTR_ERR(axp20x_batt->batt); }
diff --git a/drivers/power/supply/cpcap-charger.c b/drivers/power/supply/cpcap-charger.c
index 34a57dd5c3cf..d423a8bdc0c2 100644
--- a/drivers/power/supply/cpcap-charger.c
+++ b/drivers/power/supply/cpcap-charger.c@@ -808,8 +808,8 @@ static void cpcap_charger_init_optional_gpios(struct cpcap_charger_ddata *ddata) ddata->gpio[i] = devm_gpiod_get_index(ddata->dev, "mode", i, GPIOD_OUT_HIGH); if (IS_ERR(ddata->gpio[i])) { - dev_info(ddata->dev, "no mode change GPIO%i: %li\n", - i, PTR_ERR(ddata->gpio[i])); + dev_info(ddata->dev, "no mode change GPIO%i: %pe\n", + i, ddata->gpio[i]); ddata->gpio[i] = NULL; } }
diff --git a/drivers/power/supply/cw2015_battery.c b/drivers/power/supply/cw2015_battery.c
index 1bcb60cf763d..48643f7bf503 100644
--- a/drivers/power/supply/cw2015_battery.c
+++ b/drivers/power/supply/cw2015_battery.c@@ -658,8 +658,8 @@ static int cw_bat_probe(struct i2c_client *client) cw_bat->regmap = devm_regmap_init_i2c(client, &cw2015_regmap_config); if (IS_ERR(cw_bat->regmap)) { - dev_err(cw_bat->dev, "Failed to allocate regmap: %ld\n", - PTR_ERR(cw_bat->regmap)); + dev_err(cw_bat->dev, "Failed to allocate regmap: %pe\n", + cw_bat->regmap); return PTR_ERR(cw_bat->regmap); }
diff --git a/drivers/power/supply/intel_dc_ti_battery.c b/drivers/power/supply/intel_dc_ti_battery.c
index cb6fa8d88b43..5819df23a543 100644
--- a/drivers/power/supply/intel_dc_ti_battery.c
+++ b/drivers/power/supply/intel_dc_ti_battery.c@@ -349,7 +349,7 @@ static int dc_ti_battery_probe(struct platform_device *pdev) chip->vbat_channel = devm_iio_channel_get(dev, "VBAT"); if (IS_ERR(chip->vbat_channel)) { - dev_dbg(dev, "devm_iio_channel_get() ret %ld\n", PTR_ERR(chip->vbat_channel)); + dev_dbg(dev, "devm_iio_channel_get() ret %pe\n", chip->vbat_channel); return dev_err_probe(dev, -EPROBE_DEFER, "Waiting for VBAT IIO channel\n"); }
diff --git a/drivers/power/supply/qcom_smbx.c b/drivers/power/supply/qcom_smbx.c
index 67fdb4335338..c8e242341d04 100644
--- a/drivers/power/supply/qcom_smbx.c
+++ b/drivers/power/supply/qcom_smbx.c@@ -613,7 +613,7 @@ static int smb_get_iio_chan(struct smb_chip *chip, struct iio_channel *chan, } if (IS_ERR(chan)) { - dev_err(chip->dev, "Failed to chan, err = %li", PTR_ERR(chan)); + dev_err(chip->dev, "Failed to chan, err = %pe", chan); return PTR_ERR(chan); }
--
2.43.0