On Tue, Sep 22, 2009 at 06:10:41PM +0530, Sudhakar Rajashekhara wrote: > On Sat, Sep 19, 2009 at 02:40:04, Mark A. Greer wrote: > > From: Mark A. Greer <[email protected]> > > > > The existing NAND infrastructure allows the default main and > > mirror bad block tables to be overridden in nand_default_bbt(). > > However, the davinci_nand driver does not support this. So, > > add fields to the davinci driver's platform data so platform > > code can pass in their own bbt's and make the driver honor > > those overrides. > > > > Signed-off-by: Mark A. Greer <[email protected]> > > --- > > This is required by the da830 evm (see following patch) which requires > > different 'offs' and 'veroffs' values than the default. This seemed > > like the solution that fit best with the existing infratructure. If > > anyone has a better solution, please speak up. > > > > Mark, > > Are you trying to align with the OOB layout being followed in U-Boot?
Yes. That and to match what both TI and MV have been supporting for quite a while. > U-Boot for da830/omap-l137 is not in mainline yet. I am planning to > start working on that shortly. We can re-visit this when da830/omap-l137 > support is present in u-boot. Hmm, okay... So what about all the platforms that are already out there? Even if it changes, I'd recommend the driver patch [1/2] still go in so that the functionality is there if/when its ever needed (and its a really simple patch). I can alway redo the platform patch [2/2]. Mark -- _______________________________________________ Davinci-linux-open-source mailing list [email protected] http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
