If all branches of a switch statement have a return instruction, all
subsequent lines are unreachable.

Identified with cppcheck.

Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de>
---
 drivers/clk/intel/clk_intel.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/clk/intel/clk_intel.c b/drivers/clk/intel/clk_intel.c
index d2e15491a3..b633934d90 100644
--- a/drivers/clk/intel/clk_intel.c
+++ b/drivers/clk/intel/clk_intel.c
@@ -11,8 +11,6 @@

 static ulong intel_clk_get_rate(struct clk *clk)
 {
-       ulong rate;
-
        switch (clk->id) {
        case CLK_I2C:
                /* Hard-coded to 133MHz on current platforms */
@@ -20,8 +18,6 @@ static ulong intel_clk_get_rate(struct clk *clk)
        default:
                return -ENODEV;
        }
-
-       return rate;
 }

 static struct clk_ops intel_clk_ops = {
--
2.25.0

Reply via email to