From: Dan Carpenter <hidden> Date: 2021-02-06 00:23:08
This error path is supposed to return -EINVAL. It used to return
directly but we added some clean up and accidentally removed the
error code. Also I fixed a typo in the error message.
Fixes: c0b57581b73b ("iommu/mediatek: Add power-domain operation")
Signed-off-by: Dan Carpenter <redacted>
---
drivers/iommu/mtk_iommu.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
@@ -886,7 +886,8 @@ static int mtk_iommu_probe(struct platform_device *pdev)link=device_link_add(data->smicomm_dev,dev,DL_FLAG_STATELESS|DL_FLAG_PM_RUNTIME);if(!link){-dev_err(dev,"Unable link %s.\n",dev_name(data->smicomm_dev));+dev_err(dev,"Unable to link %s.\n",dev_name(data->smicomm_dev));+ret=-EINVAL;gotoout_runtime_disable;}
--
2.30.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
On Fri, 2021-02-05 at 15:46 +0300, Dan Carpenter wrote:
This error path is supposed to return -EINVAL. It used to return
directly but we added some clean up and accidentally removed the
error code. Also I fixed a typo in the error message.
Fixes: c0b57581b73b ("iommu/mediatek: Add power-domain operation")
Signed-off-by: Dan Carpenter <redacted>
@@ -886,7 +886,8 @@ static int mtk_iommu_probe(struct platform_device *pdev)link=device_link_add(data->smicomm_dev,dev,DL_FLAG_STATELESS|DL_FLAG_PM_RUNTIME);if(!link){-dev_err(dev,"Unable link %s.\n",dev_name(data->smicomm_dev));+dev_err(dev,"Unable to link %s.\n",dev_name(data->smicomm_dev));+ret=-EINVAL;gotoout_runtime_disable;}