* Mohammed, Afzal <afzal@xxxxxx> [120614 03:43]: > Hi Tony, > > On Thu, Jun 14, 2012 at 15:49:02, Tony Lindgren wrote: > > > Well I took a look at the values, and it seems the only difference is the > > static GPMC_CONFIG1_CLKACTIVATIONTIME(1) that your patch now overwrites 0. > > It seems change below should be part of $subject. > > Please let me know your comments > > Regards > Afzal > > diff --git a/arch/arm/mach-omap2/gpmc-onenand.c b/arch/arm/mach-omap2/gpmc-onenand.c > index fd4c48d..938896c 100644 > --- a/arch/arm/mach-omap2/gpmc-onenand.c > +++ b/arch/arm/mach-omap2/gpmc-onenand.c > @@ -321,6 +321,8 @@ static int omap2_onenand_set_sync_mode(struct omap_onenand_platform_data *cfg, > t.rd_cycle = gpmc_ticks_to_ns(fclk_offset + (latency + 1) * div + > ticks_cez); > > + t.clk_activation = fclk_offset_ns; > + This too should be fclk_offset, not fclk_offset_ns. Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
