[PATCH 1/5] powercap: intel_rapl: Use %pe to print error pointers symbolically

Krzysztof Kozlowski krzk at kernel.org
Sat Sep 5 23:28:10 PDT 2026


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



More information about the Linux-mediatek mailing list