]> git.ipfire.org Git - thirdparty/u-boot.git/commitdiff
phy: cadence: torrent: Set an error code for return
authorAndrew Goodbody <andrew.goodbody@linaro.org>
Mon, 18 Aug 2025 10:44:29 +0000 (11:44 +0100)
committerJerome Forissier <jerome.forissier@linaro.org>
Mon, 18 Aug 2025 13:47:58 +0000 (15:47 +0200)
In cdns_torrent_phy_probe the test for too many lanes configured does
not set an error code before taking the error path. This could lead to a
silent failure if the calling code does not detect the error. Add the
code to return -EINVAL in this case.

This issue was found by Smatch.

Signed-off-by: Andrew Goodbody <andrew.goodbody@linaro.org>
drivers/phy/cadence/phy-cadence-torrent.c

index 1f566d082f998f131bfce87b906f046c42220d09..28fe026223c179e68cf919220f1f8865734574f7 100644 (file)
@@ -719,6 +719,7 @@ static int cdns_torrent_phy_probe(struct udevice *dev)
 
        if (total_num_lanes > MAX_NUM_LANES) {
                dev_err(dev, "Invalid lane configuration\n");
+               ret = -EINVAL;
                goto put_lnk_rst;
        }