spi: ti_qspi: Fix compiler warning when DEBUG macro is set
authorVignesh R <vigneshr@ti.com>
Fri, 22 Jul 2016 05:25:49 +0000 (10:55 +0530)
committerJagan Teki <jteki@openedev.com>
Fri, 29 Jul 2016 18:45:00 +0000 (00:15 +0530)
clk_div is uninitialized at the beginning of ti_spi_set_speed(), move
debug() print after clk_div calculation to avoid compiler warning and to
have proper value of clk_div printed during debugging.

Signed-off-by: Vignesh R <vigneshr@ti.com>
Reviewed-by: Jagan Teki <jteki@openedev.com>
Reviewed-by: Mugunthan V N <mugunthanvnm@ti.com>
drivers/spi/ti_qspi.c

index a850aa26ec2b8219d8050cc9d03e2f43aa5d7158..56ae29a3ee7c592f10f383eb7e6ae4a926f5853c 100644 (file)
@@ -110,13 +110,13 @@ static void ti_spi_set_speed(struct ti_qspi_priv *priv, uint hz)
 {
        uint clk_div;
 
-       debug("ti_spi_set_speed: hz: %d, clock divider %d\n", hz, clk_div);
-
        if (!hz)
                clk_div = 0;
        else
                clk_div = (QSPI_FCLK / hz) - 1;
 
+       debug("ti_spi_set_speed: hz: %d, clock divider %d\n", hz, clk_div);
+
        /* disable SCLK */
        writel(readl(&priv->base->clk_ctrl) & ~QSPI_CLK_EN,
               &priv->base->clk_ctrl);