From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH 3/3] ARM: OMAP2+: gpmc: handle additional timings Date: Wed, 13 Jun 2012 05:42:15 -0700 Message-ID: <20120613124215.GT12766@atomide.com> References: <4FD63DBF.9000200@ti.com> <4FD77E35.3050703@ti.com> <20120613113217.GK12766@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mho-02-ewr.mailhop.org ([204.13.248.72]:27726 "EHLO mho-02-ewr.mailhop.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753152Ab2FMMmS (ORCPT ); Wed, 13 Jun 2012 08:42:18 -0400 Content-Disposition: inline In-Reply-To: Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: "Mohammed, Afzal" Cc: "paul@pwsan.com" , "linux-omap@vger.kernel.org" , "Hunter, Jon" , "linux-arm-kernel@lists.infradead.org" * Mohammed, Afzal [120613 05:43]: > Hi Tony, > > On Wed, Jun 13, 2012 at 17:02:17, Tony Lindgren wrote: > > * Mohammed, Afzal [120612 22:00]: > > > Yes, that would be better except for too much logging, if Tony > > > prefers that way I will add. > > > > If there's a chance it causing file system corruption, we should > > test it carefully on the boards before applying. If that's done, > > then there's probably no need for warnings. It's safer to disable > > NAND for untested boards if there's a chance of breaking the timings. > > By disabling NAND, I understand that you are suggesting to remove > nand initialization done in board file, right ? Yes, but before we do that, let's fix things first for cases that we can test, like tusb6010 DMA. > And boards that can be tested here are omap3evm & beagleboard, > both of which can't be tested for this change. > > Or should additional timings be achieved without affecting old > interface, but that it seems would necessitate more code > duplication. We should just keep the same timings as before, with values added for the newly added registers. Regards, Tony