hi Ben and Sergei, On Thu, Feb 3, 2011 at 7:21 PM, Ben Gardiner <[email protected]>wrote:
> > On 03-02-2011 15:16, sughosh ganu wrote: > > > >> > diff --git a/arch/arm/mach-davinci/Kconfig > >> > b/arch/arm/mach-davinci/Kconfig > >> > index 32f1479..e13b7e0 100644 > >> > --- a/arch/arm/mach-davinci/Kconfig > >> > +++ b/arch/arm/mach-davinci/Kconfig > >> > @@ -213,6 +213,16 @@ config MACH_MITYOMAPL138 > >> > config MACH_OMAPL138_HAWKBOARD > >> > bool "TI AM1808 / OMAPL-138 Hawkboard > >> > platform" > >> > depends on ARCH_DAVINCI_DA850 > >> > + select MTD > >> > + select MTD_PARTITIONS > >> > + select MTD_CHAR > >> > + select MTD_BLKDEVS > >> > + select MTD_BLOCK > >> > + select MTD_PHYSMAP > >> > + select MTD_CFI > <snip> Sergei, perhaps it would be acceptable for the Hawkboard > MACH_OMAPL138_HAWKBOARD bool to be the 'default' for the necessary MTD > settings? This would be similar to KEYBOARD_GPIO_POLLED in > 9cc3049911d0166d54247cf2471802eb5293c66c and GPIO_PCA953X in > d5539ca0a5c9abd5cbf9ddcbbc1c2a507b5fd63c. > > Sugosh, this means you would make an entry in > arch/arm/mach-davinci/KConfig for each of the 'selects' above. > Something like: > > config MTD > default MACH_OMAPL138_HAWKBOARD > config MTD_PARTITIONS > default MACH_OMAPL138_HAWKBOARD > [etc.] > > Thanks a lot for your hints. I am ready to go this way too. Let me test this and get back. > I agree that MTD_CFI and MTD_PHYSMAP are not needed and so should not > be included in the 'defaults.' > No, this was an error, these options should be removed. @Sergei We have a common da8xx_omapl_defconfig which is being currently used by multiple boards(including hawkboard). Not all boards have a nand flash mounted as default. So i cannot add this in the above defconfig file. So then, the flash driver, along with the neccessary mtd configs will have to be included everytime through the menuconfig, inspite of the fact that the flash is a default component on the board. -sughosh
_______________________________________________ Davinci-linux-open-source mailing list [email protected] http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
