On Thu, 5 Jun 2014, Arnd Bergmann wrote: > gcc correctly points out that hw_state can be used uninitially > in the mvsd_setup_data() function. This rearranges the function > to ensure it always contains a proper value. > > Signed-off-by: Arnd Bergmann <[email protected]> > Cc: Nicolas Pitre <[email protected]> > Cc: Chris Ball <[email protected]> > Cc: Thomas Petazzoni <[email protected]> > Cc: Jason Cooper <[email protected]> > Cc: Andrew Lunn <[email protected]> > Cc: [email protected]
Acked-by: Nicolas Pitre <[email protected]> > --- > drivers/mmc/host/mvsdio.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/mmc/host/mvsdio.c b/drivers/mmc/host/mvsdio.c > index 9377284..6b4c5ad 100644 > --- a/drivers/mmc/host/mvsdio.c > +++ b/drivers/mmc/host/mvsdio.c > @@ -79,11 +79,11 @@ static int mvsd_setup_data(struct mvsd_host *host, struct > mmc_data *data) > unsigned long t = jiffies + HZ; > unsigned int hw_state, count = 0; > do { > + hw_state = mvsd_read(MVSD_HW_STATE); > if (time_after(jiffies, t)) { > dev_warn(host->dev, "FIFO_EMPTY bit missing\n"); > break; > } > - hw_state = mvsd_read(MVSD_HW_STATE); > count++; > } while (!(hw_state & (1 << 13))); > dev_dbg(host->dev, "*** wait for FIFO_EMPTY bit " > -- > 1.8.3.2 > -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
