aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2022-10-14 12:25:06 +0300
committerVinod Koul <vkoul@kernel.org>2022-10-17 10:55:26 +0530
commitca1c73628f5bd0c1ef6e46073cc3be2450605b06 (patch)
treef5c31db8e7cdd49078d239153760c86a1771b9c9
parentMAINTAINERS: Update Kishon's email address in GENERIC PHY FRAMEWORK (diff)
downloadlinux-dev-ca1c73628f5bd0c1ef6e46073cc3be2450605b06.tar.xz
linux-dev-ca1c73628f5bd0c1ef6e46073cc3be2450605b06.zip
phy: stm32: fix an error code in probe
If "index > usbphyc->nphys" is true then this returns success but it should return -EINVAL. Fixes: 94c358da3a05 ("phy: stm32: add support for STM32 USB PHY Controller (USBPHYC)") Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Reviewed-by: Amelie Delaunay <amelie.delaunay@foss.st.com> Link: https://lore.kernel.org/r/Y0kq8j6S+5nDdMpr@kili Signed-off-by: Vinod Koul <vkoul@kernel.org>
-rw-r--r--drivers/phy/st/phy-stm32-usbphyc.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/phy/st/phy-stm32-usbphyc.c b/drivers/phy/st/phy-stm32-usbphyc.c
index a98c911cc37a..5bb9647b078f 100644
--- a/drivers/phy/st/phy-stm32-usbphyc.c
+++ b/drivers/phy/st/phy-stm32-usbphyc.c
@@ -710,6 +710,8 @@ static int stm32_usbphyc_probe(struct platform_device *pdev)
ret = of_property_read_u32(child, "reg", &index);
if (ret || index > usbphyc->nphys) {
dev_err(&phy->dev, "invalid reg property: %d\n", ret);
+ if (!ret)
+ ret = -EINVAL;
goto put_child;
}