From: Dan Carpenter <dan.carpen...@oracle.com>

We refactored this code but accidentally left out a break statement so
QUARK_X1000_SSP isn't handled correctly.

Fixes: 025ffe88ee60 ('spi: pxa2xx: shift clk_div in one place')
Tested-by: Andy Shevchenko <andriy.shevche...@linux.intel.com>
Signed-off-by: Dan Carpenter <dan.carpen...@oracle.com>
---
 drivers/spi/spi-pxa2xx.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/spi/spi-pxa2xx.c b/drivers/spi/spi-pxa2xx.c
index 60526a5..e3223ac 100644
--- a/drivers/spi/spi-pxa2xx.c
+++ b/drivers/spi/spi-pxa2xx.c
@@ -791,8 +791,10 @@ static unsigned int pxa2xx_ssp_get_clk_div(struct 
driver_data *drv_data,
        switch (drv_data->ssp_type) {
        case QUARK_X1000_SSP:
                clk_div = quark_x1000_get_clk_div(rate, &chip->dds_rate);
+               break;
        default:
                clk_div = ssp_get_clk_div(drv_data, rate);
+               break;
        }
        return clk_div << 8;
 }
-- 
2.1.4

--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to