]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
IPQ40xx: clk: dont always return 0
authorRobert Marko <robert.marko@sartura.hr>
Wed, 28 Oct 2020 12:56:25 +0000 (13:56 +0100)
committerTom Rini <trini@konsulko.com>
Tue, 1 Dec 2020 19:12:28 +0000 (14:12 -0500)
Currently the driver will go through the clock ID-s and set/enable them as needed.
But if the ID is unknown it will fall through the switch case to the default case which will always return 0.

This is not correct and default cases should return a error code since clock ID is unknown.
So lets return -EINVAL instead.

Signed-off-by: Robert Marko <robert.marko@sartura.hr>
Cc: Luka Perkov <luka.perkov@sartura.hr>
arch/arm/mach-ipq40xx/clock-ipq4019.c

index ac2b830353a02dd64baae3404277ead3b913bf1c..7308563ad11952548c63db2d2ba80a3d4abf1fea 100644 (file)
@@ -26,7 +26,7 @@ ulong msm_set_rate(struct clk *clk, ulong rate)
                /* This clock is already initialized by SBL1 */
                return 0;
        default:
-               return 0;
+               return -EINVAL;
        }
 }
 
@@ -56,7 +56,7 @@ static int msm_enable(struct clk *clk)
                /* This clock is already initialized by SBL1 */
                return 0;
        default:
-               return 0;
+               return -EINVAL;
        }
 }