Re: [PATCH 1/5] powercap: intel_rapl: Use %pe to print error pointers symbolically
From: Krzysztof Kozlowski
Date: Sun Sep 06 2026 - 02:28:36 EST
On 06/09/2026 08:17, Sumeet Pawnikar wrote:
> trp->rp = rapl_add_package(info->package_id, &trp->priv, false);
> if (IS_ERR(trp->rp)) {
> - dev_err(&auxdev->dev, "Failed to add RAPL Domain for Package%d, %ld\n",
> - info->package_id, PTR_ERR(trp->rp));
> + dev_err(&auxdev->dev, "Failed to add RAPL Domain for Package%d, %pe\n",
> + info->package_id, trp->rp);
Instead you should use dev_err_probe and drop the last argument.
Best regards,
Krzysztof