Hello.

sughosh ganu wrote:

hi Ben and Sergei,

On Thu, Feb 3, 2011 at 7:21 PM, Ben Gardiner <[email protected] <mailto:[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.

   Don't even waste your time, it won't be acceptable.

    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.

Why? If the board doesn't have a NAND flash, it doesn't register the platform device and the driver doesn't see it. What's the issue?

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.

Note that a NAND flash is a default component of DA850 EVM board too, nevertheless the NAND driver is not enabled in the defconfig.

-sughosh

WBR, Sergei
_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to