On Tue, 2023-06-06 at 11:22 +0300, Dan Carpenter wrote:
> The devm_ioremap_resource() function returns error pointers on error,
> it never returns NULL. Update the check accordingly.
>
> Fixes: 9eef7f9da928 ("powercap: intel_rapl: Introduce RAPL TPMI
> interface driver")
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
Acked-by: Zhang Rui <rui.zhang@xxxxxxxxx>
thanks,
rui
> ---
> drivers/powercap/intel_rapl_tpmi.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/powercap/intel_rapl_tpmi.c
> b/drivers/powercap/intel_rapl_tpmi.c
> index c016127b3497..4f4f13ded225 100644
> --- a/drivers/powercap/intel_rapl_tpmi.c
> +++ b/drivers/powercap/intel_rapl_tpmi.c
> @@ -255,8 +255,8 @@ static int intel_rapl_tpmi_probe(struct
> auxiliary_device *auxdev,
> }
>
> trp->base = devm_ioremap_resource(&auxdev->dev, res);
> - if (!trp->base) {
> - ret = -ENOMEM;
> + if (IS_ERR(trp->base)) {
> + ret = PTR_ERR(trp->base);
> goto err;
> }
>
