From nobody Fri Oct 2 10:07:49 2026 Received: from mail-pl1-f178.google.com (mail-pl1-f178.google.com [209.85.214.178]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 64C261F30BB for ; Sun, 2 Aug 2026 11:20:44 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.178 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785669646; cv=none; b=lBJVZKgrMMg+7Pzmd6MigqiG0yuPj/tO0u9fUEwd6Ga0nupaTJfSHkqy8VqJxkzQpRsYY9pgbKyKPbGFfoURR5Ou2Z4F+KCvnIUhsr/2YPAMHH3N08b7OxAYsanmgmqttDFdQDcnLXk3D5jqjslzrZMb/OfTL7KYNNtxdkUEztY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785669646; c=relaxed/simple; bh=LFPcp6CXA/x3F71q2TIdQYdeiBvgEWE8aizjnFiqyiE=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=hKCOe5EdSKhiLgBfLyjr/P6Qo/DY8/wBcvOy4XK3qx0T6uBHHL91TPhv0kVg7MB6ASQVvF5qaFwXie1rY8nBpxT4Z3KbgDP8kmwrjrO5HKqVd5UEQviIHw3OR8XV07NAGhw3wIw/qbsc+OtVU/ZB/BC/ZFoUwWLNuUEshildyNs= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=THK+NQ2A; arc=none smtp.client-ip=209.85.214.178 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="THK+NQ2A" Received: by mail-pl1-f178.google.com with SMTP id d9443c01a7336-2cacf197759so36117365ad.2 for ; Sun, 02 Aug 2026 04:20:44 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1785669644; x=1786274444; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to:content-type; bh=XXturRtkPIePk9i4DZbCbMO/flEYuSWL+nIK+cXwWV4=; b=THK+NQ2AqMQaUyVQ+zto2DrZ/4dCt2MQCBFg7f6U0JYORe96xTgZsIjmB2zVUXyGxd iQL+8YY4Gb/nXU0H7oqSfCxEEXdcyvNF6GiKZXCTaRBux5BT5xAM8d4tPD3TsBoRQPGd 9LziQi+TGS1kqtEglsjVsdvJ/3oGHXmPmpOLLNO9Piwcss1mLBjxq4nbCTjRoJUlzfSt YLpcjL6jOKQwKiMAH4aWVpXG4irJlMjdUqaVTVJSKzwqaSvaxJ/0bm+TT88F68Hg6sXZ KBwmoQxIRizu6b/S7st0AuZIm+INUUzU9guI/Ga968pJX5V029/a/luCGgWDHMzTN+3B nTgQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1785669644; x=1786274444; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=XXturRtkPIePk9i4DZbCbMO/flEYuSWL+nIK+cXwWV4=; b=M07RJmHgdCw7dXRMaUchbjf+OyrOAcf4fpI1UeXcEtHuc+HYUVwET2PYwcfZVPAIEE nTNjB0hZ7EP4i8nuymVLDRhNQJfY7FZdKeW2ox60/nkGRDYKclnygYJoFFiW1K12csDV 1m65tN1LuWKfRay8uJjwttP3I0EciuG1K0QRdEEu5YWEn3+ai4La5qcp1Tt2zHpCAdC3 WEdfxwnPmS3sZC4bc42RUaKRbPnAkn+IiHkab9FSP2W0ZR55DgkrlY6ZCtHuIXsol2k5 sXmHZb4hEMT8qXxogU3C1/NB8kdSBxwdGBStiXji82UTvzcvCWwl21b8hzWQqz0QMtx8 G7Mw== X-Forwarded-Encrypted: i=1; AHgh+RrnQUO/in9Tp7lq1RoOh2z7SYNvc+Rb0tpyHfEloUGknwLfpt0sGmTMq5LtJLvoMgpgzs0hi7AGUu0whLg=@vger.kernel.org X-Gm-Message-State: AOJu0YwbTiisSbxLtJ8nA/P5Xd6JfIFP0xVIPWaAc4SrTCK28HdRwRo1 vqk5Pv2mo7rd30BjVEnX6niM4nEVALf1MF4rgtGvc+kN96cNUZHfydgn X-Gm-Gg: AR+sD139+TNPobLQfeQRk1JBHhUjEvbtIU/P/tLWmgYrY9/4hf28OPuh+grO3eN27Dk Ta6Qwuo2orUb5zhKyy7uRDn+4OQGwEHr1hReRrRC1JcFM1xBQJYiIEiX9COq45R96NvCKCxdv+f JhA/pH2LKKKAAHeRza1D94yLuBC3Me+H5WcBgcKAObiHCREp3KsnitbcjgGWJQgUmcLuP25bF9y 48iPXSthDilpE5J77vKtcAqiASN1H5ZJltqka5lHpm8jU7mCD174ZybljHCtOQiYBA38GU7jwgx WfNR0q3RvWGr/oy1nDti6DFFLChR0szfQdXKw2VcDvnY12ClySJt+M/N3yfVT9DGjt8fV9rompz l8r50kuGDH0dYC4H5DqHwcpaI5v5XaTRBw6LI1RJlsdxP3iTu1p8gybFOFurHc5TP4lWqs8CRzz dHFFFLP2E957Tv+w9LMk0SZ+CmEjRcGbrxS0yEo78lOn7Ng3KY0YAH9mGLdiHHBhVP1MehHCGiC mPCJiwOThQsiPOZ1/HqoWaH3kGZ X-Received: by 2002:a17:902:da81:b0:2c9:97a9:2097 with SMTP id d9443c01a7336-2d05241c768mr70935345ad.43.1785669643654; Sun, 02 Aug 2026 04:20:43 -0700 (PDT) Received: from acer-nitro-anv15-41.entro.com ([118.34.230.2]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2d04b0eb486sm25568915ad.53.2026.08.02.04.20.41 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 02 Aug 2026 04:20:42 -0700 (PDT) From: Shaikh Kamaluddin To: Davidlohr Bueso , Jonathan Cameron , Dave Jiang , Alison Schofield , Vishal Verma , Dan Williams , Ira Weiny , Li Ming , Robert Richter , Gregory Price Cc: linux-cxl@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v2] cxl: Use %pe to print error pointers Date: Sun, 2 Aug 2026 16:50:29 +0530 Message-ID: <20260802112029.28767-1-shaikhkamal2012@gmail.com> X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Use the %pe printk format specifier to report error pointers directly instead of printing PTR_ERR() as a long value. A failed dport addition then reports -EBUSY rather than -16, which is easier to follow when tracing port and region setup with dynamic debug enabled. Convert the five affected sites in drivers/cxl/core/port.c and drivers/cxl/core/region.c. PTR_ERR() uses in return statements are unaffected and left unchanged. drivers/cxl was scanned in full; these are the only conversion candidates. Found by: make coccicheck MODE=3Dreport M=3Ddrivers/cxl/ No functional change intended. Signed-off-by: Shaikh Kamaluddin Reviewed-by: Alison Schofield Reviewed-by: Richard Cheng --- Compile tested only with cxl config on x86. v2: - Reword commit message: lead with the code improvement, note coccinelle only as the tool used to locate the sites (Alison) - Capitalise subject per CXL convention (Alison) - Note that drivers/cxl was scanned in full and these are the only candidates (Alison) - Clarify that the patch is compile tested only - No code changes Link to v1: https://lore.kernel.org/linux-cxl/20260801101217.12778-1-shaikh= kamal2012@gmail.com/ drivers/cxl/core/port.c | 13 ++++++------- drivers/cxl/core/region.c | 8 ++++---- 2 files changed, 10 insertions(+), 11 deletions(-) diff --git a/drivers/cxl/core/port.c b/drivers/cxl/core/port.c index 1215ee4f4035..c9875e6af9f6 100644 --- a/drivers/cxl/core/port.c +++ b/drivers/cxl/core/port.c @@ -932,11 +932,10 @@ struct cxl_port *devm_cxl_add_port(struct device *hos= t, =20 parent_port =3D parent_dport ? parent_dport->port : NULL; if (IS_ERR(port)) { - dev_dbg(uport_dev, "Failed to add%s%s%s: %ld\n", + dev_dbg(uport_dev, "Failed to add%s%s%s: %pe\n", parent_port ? " port to " : "", parent_port ? dev_name(&parent_port->dev) : "", - parent_port ? "" : " root port", - PTR_ERR(port)); + parent_port ? "" : " root port", port); } else { dev_dbg(uport_dev, "%s added%s%s%s\n", dev_name(&port->dev), @@ -1271,8 +1270,8 @@ struct cxl_dport *devm_cxl_add_dport(struct cxl_port = *port, dport =3D __devm_cxl_add_dport(port, dport_dev, port_id, component_reg_phys, CXL_RESOURCE_NONE); if (IS_ERR(dport)) { - dev_dbg(dport_dev, "failed to add dport to %s: %ld\n", - dev_name(&port->dev), PTR_ERR(dport)); + dev_dbg(dport_dev, "failed to add dport to %s: %pe\n", + dev_name(&port->dev), dport); } else { dev_dbg(dport_dev, "dport added to %s\n", dev_name(&port->dev)); @@ -1305,8 +1304,8 @@ struct cxl_dport *devm_cxl_add_rch_dport(struct cxl_p= ort *port, dport =3D __devm_cxl_add_dport(port, dport_dev, port_id, CXL_RESOURCE_NONE, rcrb); if (IS_ERR(dport)) { - dev_dbg(dport_dev, "failed to add RCH dport to %s: %ld\n", - dev_name(&port->dev), PTR_ERR(dport)); + dev_dbg(dport_dev, "failed to add RCH dport to %s: %pe\n", + dev_name(&port->dev), dport); } else { dev_dbg(dport_dev, "RCH dport added to %s\n", dev_name(&port->dev)); diff --git a/drivers/cxl/core/region.c b/drivers/cxl/core/region.c index 1e211542b6b6..894df68a6074 100644 --- a/drivers/cxl/core/region.c +++ b/drivers/cxl/core/region.c @@ -682,8 +682,8 @@ static int alloc_hpa(struct cxl_region *cxlr, resource_= size_t size) dev_name(&cxlr->dev)); if (IS_ERR(res)) { dev_dbg(&cxlr->dev, - "HPA allocation error (%ld) for size:%pap in %s %pr\n", - PTR_ERR(res), &size, cxlrd->res->name, cxlrd->res); + "HPA allocation error (%pe) for size:%pap in %s %pr\n", + res, &size, cxlrd->res->name, cxlrd->res); return PTR_ERR(res); } =20 @@ -3773,9 +3773,9 @@ static struct cxl_region *construct_region(struct cxl= _root_decoder *cxlrd, =20 if (IS_ERR(cxlr)) { dev_err(cxlmd->dev.parent, - "%s:%s: %s failed assign region: %ld\n", + "%s:%s: %s failed assign region: %pe\n", dev_name(&cxlmd->dev), dev_name(&cxled->cxld.dev), - __func__, PTR_ERR(cxlr)); + __func__, cxlr); return cxlr; } =20 base-commit: f5098b6bae761e346ebcd9da7f95622c04733cff --=20 2.43.0