IPQ40xx: clk: drop breaks in switch case
authorRobert Marko <robert.marko@sartura.hr>
Wed, 28 Oct 2020 12:56:24 +0000 (13:56 +0100)
committerTom Rini <trini@konsulko.com>
Tue, 1 Dec 2020 19:12:28 +0000 (14:12 -0500)
There is no point in having break statements in the switch case as there is already a return before break.

So lets drop them from the driver.

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

index 1f38566..ac2b830 100644 (file)
@@ -25,7 +25,6 @@ ulong msm_set_rate(struct clk *clk, ulong rate)
        case GCC_BLSP1_UART1_APPS_CLK: /*UART1*/
                /* This clock is already initialized by SBL1 */
                return 0;
-               break;
        default:
                return 0;
        }
@@ -53,11 +52,9 @@ static int msm_enable(struct clk *clk)
        case GCC_BLSP1_QUP1_SPI_APPS_CLK: /*SPI1*/
                /* This clock is already initialized by SBL1 */
                return 0;
-               break;
        case GCC_PRNG_AHB_CLK: /*PRNG*/
                /* This clock is already initialized by SBL1 */
                return 0;
-               break;
        default:
                return 0;
        }