[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250902102101.378809-1-daniel.baluta@nxp.com>
Date: Tue, 2 Sep 2025 13:21:00 +0300
From: Daniel Baluta <daniel.baluta@....com>
To: broonie@...nel.org
Cc: linux-sound@...r.kernel.org,
imx@...ts.linux.dev,
linux-kernel@...r.kernel.org,
Frank.Li@....com,
robh@...nel.org,
laurentiu.mihalcea@....com,
dan.carpenter@...aro.org,
waqar.hameed@...s.com,
festevam@...il.com,
kernel@...gutronix.de,
lgirdwood@...il.com,
peter.ujfalusi@...ux.intel.com,
yung-chuan.liao@...ux.intel.com,
ranjani.sridharan@...ux.intel.com,
kai.vehmanen@...ux.intel.com,
Daniel Baluta <daniel.baluta@....com>
Subject: [PATCH v2] ASoC: SOF: imx: Fix devm_ioremap_resource check
devm_ioremap_resource does not return NULL on error
but an error pointer so we need to use IS_ERR to check
the return code.
While at it also pass the error code to dev_err_probe
to improve logging.
Fixes: bc163baef570 ("ASoC: Use of_reserved_mem_region_to_resource() for "memory-region"")
Signed-off-by: Daniel Baluta <daniel.baluta@....com>
---
Changes since v1:
- pass the error code to dev_err_probe
sound/soc/sof/imx/imx-common.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound/soc/sof/imx/imx-common.c b/sound/soc/sof/imx/imx-common.c
index 40debc541bf9..e787d3932fbb 100644
--- a/sound/soc/sof/imx/imx-common.c
+++ b/sound/soc/sof/imx/imx-common.c
@@ -316,9 +316,9 @@ static int imx_parse_ioremap_memory(struct snd_sof_dev *sdev)
}
sdev->bar[blk_type] = devm_ioremap_resource(sdev->dev, res);
- if (!sdev->bar[blk_type])
+ if (IS_ERR(sdev->bar[blk_type]))
return dev_err_probe(sdev->dev,
- -ENOMEM,
+ PTR_ERR(sdev->bar[blk_type]),
"failed to ioremap %s region\n",
chip_info->memory[i].name);
}
--
2.45.2
Powered by blists - more mailing lists