2012/7/6 Rob Herring <span dir="ltr"><<a href="mailto:robherring2@gmail.com" target="_blank">robherring2@gmail.com</a>></span><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div class="im">On 07/06/2012 05:38 AM, <a href="mailto:jgq516@gmail.com">jgq516@gmail.com</a> wrote:<br>
> From: Xiao Jiang <<a href="mailto:jgq516@gmail.com">jgq516@gmail.com</a>><br>
><br>
> Since more and more arm chips support device tree, it'd be better add PROC_DEVICETREE<br>
> in arch/arm/Kconfig to avoid duplicate code.<br>
<br>
</div>I think this should remain user choice. If its going to be selected,<br>
then you might as well just remove the option altogether. Perhaps just<br>
make the option default to yes.<br>
<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div>Hmm, sounds reasonable from your point. So the better choice is to set the option default to Y if the board has dt support just like tegra_defconfig and at91_dt_defconfig,  right? thanks.<br>
<br>Regards,<br>Xiao<br><br></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="HOEnZb"><font color="#888888">
Rob<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
> Signed-off-by: Xiao Jiang <<a href="mailto:jgq516@gmail.com">jgq516@gmail.com</a>><br>
> ---<br>
>  arch/arm/Kconfig                   |    1 +<br>
>  arch/arm/configs/at91_dt_defconfig |    1 -<br>
>  arch/arm/configs/tegra_defconfig   |    1 -<br>
>  arch/arm/plat-omap/Kconfig         |    1 -<br>
>  4 files changed, 1 insertions(+), 3 deletions(-)<br>
><br>
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig<br>
> index a91009c..f68896e 100644<br>
> --- a/arch/arm/Kconfig<br>
> +++ b/arch/arm/Kconfig<br>
> @@ -1864,6 +1864,7 @@ config USE_OF<br>
>       select OF<br>
>       select OF_EARLY_FLATTREE<br>
>       select IRQ_DOMAIN<br>
> +     select PROC_DEVICETREE if PROC_FS<br>
>       help<br>
>         Include support for flattened device tree machine descriptions.<br>
><br>
> diff --git a/arch/arm/configs/at91_dt_defconfig b/arch/arm/configs/at91_dt_defconfig<br>
> index 67bc571..b856cb6 100644<br>
> --- a/arch/arm/configs/at91_dt_defconfig<br>
> +++ b/arch/arm/configs/at91_dt_defconfig<br>
> @@ -64,7 +64,6 @@ CONFIG_MTD_NAND=y<br>
>  CONFIG_MTD_NAND_ATMEL=y<br>
>  CONFIG_MTD_UBI=y<br>
>  CONFIG_MTD_UBI_GLUEBI=y<br>
> -CONFIG_PROC_DEVICETREE=y<br>
>  CONFIG_BLK_DEV_LOOP=y<br>
>  CONFIG_BLK_DEV_RAM=y<br>
>  CONFIG_BLK_DEV_RAM_COUNT=4<br>
> diff --git a/arch/arm/configs/tegra_defconfig b/arch/arm/configs/tegra_defconfig<br>
> index 1198dd6..43e7b65 100644<br>
> --- a/arch/arm/configs/tegra_defconfig<br>
> +++ b/arch/arm/configs/tegra_defconfig<br>
> @@ -72,7 +72,6 @@ CONFIG_IPV6_TUNNEL=y<br>
>  CONFIG_IPV6_MULTIPLE_TABLES=y<br>
>  # CONFIG_WIRELESS is not set<br>
>  # CONFIG_FIRMWARE_IN_KERNEL is not set<br>
> -CONFIG_PROC_DEVICETREE=y<br>
>  CONFIG_BLK_DEV_LOOP=y<br>
>  CONFIG_AD525X_DPOT=y<br>
>  CONFIG_AD525X_DPOT_I2C=y<br>
> diff --git a/arch/arm/plat-omap/Kconfig b/arch/arm/plat-omap/Kconfig<br>
> index ad95c7a..4e2cb17 100644<br>
> --- a/arch/arm/plat-omap/Kconfig<br>
> +++ b/arch/arm/plat-omap/Kconfig<br>
> @@ -27,7 +27,6 @@ config ARCH_OMAP2PLUS<br>
>       select GENERIC_IRQ_CHIP<br>
>       select OMAP_DM_TIMER<br>
>       select USE_OF<br>
> -     select PROC_DEVICETREE if PROC_FS<br>
>       help<br>
>         "Systems based on OMAP2, OMAP3 or OMAP4"<br>
><br>
><br>
<br>
<br>
</div></div></blockquote></div><br>