Hello.
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
>
>
>
> I don't think CFI is necessary for NAND?
>
>
>
> Neither is the 'physmap' driver. And all this part of the
> patch looks completely bogus.
>
>
>
> I think MTD_CFI and MTD_PHYSMAP are not needed. I need the rest of
the
> options for mtd related stuff. I guess i can include these in the
same
> patch. Will come up with an updated revision. Thanks.
Why do you need to select these through Kconfig. Why cant these
be enabled/disabled by users using menuconfig?
Since the hawkboard comes with a default Nand chip, i thought
these could
be part of the default config. But i see that these configs are not
defined for for other ARM boards(except PXA). In that case, i can
remove
these configs from the deault. Thanks.
Looking through the defconfigs, looks like a lot of boards define the
configs
for MTD as default.
So what? Defconfigs are not created by forcing the options via Kconfig,
they are created by manually configuring in the needed options.
Hmm. So then how do you suggest me to get the MTD configurations for the
board. I think it makes sense to get these as NAND comes mounted by default(as
i pointed in my earlier mail). The da8xx_omapl_defconfig is shared by multiple
boards, and this is the way i see things being configured for individual
boards, e.g MACH_DAVINCI_DA830_EVM.
I don't understand, what's the way? DA830 EVM doesn't force unnecessary
options.
If there is any better way, let me know.
If you really want to enable the NAND related options, just change
da8xx_omapl_defconfig.
-sughosh
WBR, Sergei
_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
