github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-09-24 06:24:02 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

usb: typec: ucsi: Use %pe to print error pointers

Browse Source 
Use the %pe format specifier instead of %ld with PTR_ERR() for printing
error pointers in ucsi_register_plug(), ucsi_register_cable(), and
ucsi_register_partner(). This prints symbolic error names (e.g.
-ENOMEM) instead of errno numbers (e.g. -12), making debug logs more
readable.

This patch fixes coccinelle reported warnings:
./typec/ucsi/ucsi.c:1026:3-10: WARNING: Consider using %pe to print PTR_ERR()
./typec/ucsi/ucsi.c:1156:3-10: WARNING: Consider using %pe to print PTR_ERR()
./typec/ucsi/ucsi.c:967:3-10: WARNING: Consider using %pe to print PTR_ERR()

Signed-off-by: Subasri S <subasris1210@gmail.com>
Link: https://patch.msgid.link/20260709-subasri-usb-ucsi-v1-v1-1-74bed201e489@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Subasri S 2026-07-09 08:13:33 +05:30 committed by Greg Kroah-Hartman
parent ef8154d8b5
commit 9f66c01d68
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
drivers/usb/typec/ucsi/ucsi.c
View File

@ -963,8 +963,8 @@ static int ucsi_register_plug(struct ucsi_connector *con)
plug = typec_register_plug(con->cable, &desc);
if (IS_ERR(plug)) {
dev_err(con->ucsi->dev,
"con%d: failed to register plug (%ld)\n", con->num,
PTR_ERR(plug));
"con%d: failed to register plug (%pe)\n", con->num,
plug);
return PTR_ERR(plug);
}
@ -1022,8 +1022,8 @@ static int ucsi_register_cable(struct ucsi_connector *con)
cable = typec_register_cable(con->port, &desc);
if (IS_ERR(cable)) {
dev_err(con->ucsi->dev,
"con%d: failed to register cable (%ld)\n", con->num,
PTR_ERR(cable));
"con%d: failed to register cable (%pe)\n", con->num,
cable);
return PTR_ERR(cable);
}
@ -1152,8 +1152,8 @@ static int ucsi_register_partner(struct ucsi_connector *con)
partner = typec_register_partner(con->port, &desc);
if (IS_ERR(partner)) {
dev_err(con->ucsi->dev,
"con%d: failed to register partner (%ld)\n", con->num,
PTR_ERR(partner));
"con%d: failed to register partner (%pe)\n", con->num,
partner);
return PTR_ERR(partner);
}