Le 15/01/2015 22:58, Alexandre Belloni a écrit :
> Only use SOC_AT91SAM9 for all the at91sam9 SoCs.
> 
> It enables HAVE_AT91_SMD and HAVE_AT91_UTMI for all the sam9 SoCs but this 
> only
> represents 96 bytes of uncompressed kernel code.

After this patch that removes all the CONFIG_SOC_AT91SAM9xxxx options
excepted CONFIG_SOC_AT91SAM9, I still see a lot of them remaining in the
source tree.

So, this patch is not complete in my opinion. Even if you remove these
options later on, why not suppress them here?

> Signed-off-by: Alexandre Belloni <[email protected]>
> ---
>  arch/arm/configs/at91_dt_defconfig | 10 +----
>  arch/arm/mach-at91/Kconfig         | 92 
> +++++++++++---------------------------
>  2 files changed, 27 insertions(+), 75 deletions(-)
> 
> diff --git a/arch/arm/configs/at91_dt_defconfig 
> b/arch/arm/configs/at91_dt_defconfig
> index 8116d122e098..811e72bbe642 100644
> --- a/arch/arm/configs/at91_dt_defconfig
> +++ b/arch/arm/configs/at91_dt_defconfig
> @@ -15,15 +15,7 @@ CONFIG_MODULE_UNLOAD=y
>  # CONFIG_IOSCHED_CFQ is not set
>  CONFIG_ARCH_AT91=y
>  CONFIG_SOC_AT91RM9200=y
> -CONFIG_SOC_AT91SAM9260=y
> -CONFIG_SOC_AT91SAM9261=y
> -CONFIG_SOC_AT91SAM9263=y
> -CONFIG_SOC_AT91SAM9RL=y
> -CONFIG_SOC_AT91SAM9G45=y
> -CONFIG_SOC_AT91SAM9X5=y
> -CONFIG_SOC_AT91SAM9N12=y
> -CONFIG_MACH_AT91RM9200_DT=y
> -CONFIG_MACH_AT91SAM9_DT=y
> +CONFIG_SOC_AT91SAM9=y
>  CONFIG_AT91_TIMER_HZ=128
>  CONFIG_AEABI=y
>  CONFIG_UACCESS_WITH_MEMCPY=y
> diff --git a/arch/arm/mach-at91/Kconfig b/arch/arm/mach-at91/Kconfig
> index c6956b863b9d..c6740e359a44 100644
> --- a/arch/arm/mach-at91/Kconfig
> +++ b/arch/arm/mach-at91/Kconfig
> @@ -16,15 +16,6 @@ config HAVE_AT91_SMD
>  config HAVE_AT91_H32MX
>       bool
>  
> -config SOC_AT91SAM9
> -     bool
> -     select ATMEL_AIC_IRQ
> -     select COMMON_CLK_AT91
> -     select CPU_ARM926T
> -     select GENERIC_CLOCKEVENTS
> -     select MEMORY
> -     select ATMEL_SDRAMC
> -
>  config SOC_SAMA5
>       bool
>       select ATMEL_AIC5_IRQ
> @@ -92,67 +83,36 @@ config SOC_AT91RM9200
>       select GENERIC_CLOCKEVENTS
>       select HAVE_AT91_USB_CLK
>  
> -config SOC_AT91SAM9260
> -     bool "AT91SAM9260, AT91SAM9XE or AT91SAM9G20"
> -     select SOC_AT91SAM9
> -     select HAVE_AT91_USB_CLK
> -     help
> -       Select this if you are using one of Atmel's AT91SAM9260, AT91SAM9XE
> -       or AT91SAM9G20 SoC.
> -
> -config SOC_AT91SAM9261
> -     bool "AT91SAM9261 or AT91SAM9G10"
> -     select HAVE_FB_ATMEL
> -     select SOC_AT91SAM9
> -     select HAVE_AT91_USB_CLK
> -     help
> -       Select this if you are using one of Atmel's AT91SAM9261 or 
> AT91SAM9G10 SoC.
> -
> -config SOC_AT91SAM9263
> -     bool "AT91SAM9263"
> -     select HAVE_FB_ATMEL
> -     select SOC_AT91SAM9
> -     select HAVE_AT91_USB_CLK
> -
> -config SOC_AT91SAM9RL
> -     bool "AT91SAM9RL"
> -     select HAVE_FB_ATMEL
> -     select SOC_AT91SAM9
> -     select HAVE_AT91_UTMI
> -
> -config SOC_AT91SAM9G45
> -     bool "AT91SAM9G45 or AT91SAM9M10 families"
> -     select HAVE_FB_ATMEL
> -     select SOC_AT91SAM9
> -     select HAVE_AT91_UTMI
> -     select HAVE_AT91_USB_CLK
> -     help
> -       Select this if you are using one of Atmel's AT91SAM9G45 family SoC.
> -       This support covers AT91SAM9G45, AT91SAM9G46, AT91SAM9M10 and 
> AT91SAM9M11.
> -
> -config SOC_AT91SAM9X5
> -     bool "AT91SAM9x5 family"
> -     select HAVE_FB_ATMEL
> -     select SOC_AT91SAM9
> -     select HAVE_AT91_UTMI
> +config SOC_AT91SAM9
> +     bool "AT91SAM9"
> +     select ATMEL_AIC_IRQ
> +     select ATMEL_SDRAMC
> +     select COMMON_CLK_AT91
> +     select CPU_ARM926T
> +     select GENERIC_CLOCKEVENTS
>       select HAVE_AT91_SMD
>       select HAVE_AT91_USB_CLK
> -     help
> -       Select this if you are using one of Atmel's AT91SAM9x5 family SoC.
> -       This means that your SAM9 name finishes with a '5' (except if it is
> -       AT91SAM9G45!).
> -       This support covers AT91SAM9G15, AT91SAM9G25, AT91SAM9X25, AT91SAM9G35
> -       and AT91SAM9X35.
> -
> -config SOC_AT91SAM9N12
> -     bool "AT91SAM9N12 family"
> +     select HAVE_AT91_UTMI
>       select HAVE_FB_ATMEL
> -     select SOC_AT91SAM9
> -     select HAVE_AT91_USB_CLK
> +     select MEMORY
>       help
> -       Select this if you are using Atmel's AT91SAM9N12 SoC.
> -
> -# ----------------------------------------------------------
> +       Select this if you are using one of those Atmel SoC:
> +         AT91SAM9260
> +         AT91SAM9261
> +         AT91SAM9263
> +         AT91SAM9G15
> +         AT91SAM9G20
> +         AT91SAM9G25
> +         AT91SAM9G35
> +         AT91SAM9G45
> +         AT91SAM9G46
> +         AT91SAM9M10
> +         AT91SAM9M11
> +         AT91SAM9N12
> +         AT91SAM9RL
> +         AT91SAM9X25
> +         AT91SAM9X35
> +         AT91SAM9XE
>  endif # SOC_SAM_V4_V5
>  
>  comment "AT91 Feature Selections"
> 


-- 
Nicolas Ferre
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to