"Mark A. Greer" <[email protected]> writes: > 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].
I'll apply 1/1 today. Sudhakar, for 2/2, what's the reason to wait for u-boot support to go upstream. Is the u-boot support going to change? If most folks are using recent u-boot anyways, it seems that this should get merged as well. Kevin _______________________________________________ Davinci-linux-open-source mailing list [email protected] http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
