On 06/10/15 23:07, Uwe Kleine-König wrote: > When gpmc_cs_show_timings is called in gpmc_cs_set_timings() > gpmc_cs_program_settings() was already run which modifies the CONFIG1 > register. So to be more useful do the "before" dump earlier. > > Signed-off-by: Uwe Kleine-König <[email protected]>
Acked-by: Roger Quadros <[email protected]> > --- > drivers/memory/omap-gpmc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/memory/omap-gpmc.c b/drivers/memory/omap-gpmc.c > index 32ac049f2bc4..6515dfc2b805 100644 > --- a/drivers/memory/omap-gpmc.c > +++ b/drivers/memory/omap-gpmc.c > @@ -696,7 +696,6 @@ int gpmc_cs_set_timings(int cs, const struct gpmc_timings > *t, > int div; > u32 l; > > - gpmc_cs_show_timings(cs, "before gpmc_cs_set_timings"); > div = gpmc_calc_divider(t->sync_clk); > if (div < 0) > return div; > @@ -1988,6 +1987,7 @@ static int gpmc_probe_generic_child(struct > platform_device *pdev, > if (ret < 0) > goto err; > > + gpmc_cs_show_timings(cs, "before gpmc_cs_program_settings"); > ret = gpmc_cs_program_settings(cs, &gpmc_s); > if (ret < 0) > goto err; > cheers, -roger -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
