Re: [PATCH soc] ARM: use ARM_SINGLE_ARMV7M for ARMv7-M platforms

[Joachim Eastwood]

Hi Stefan,

On 21 May 2015 at 00:35, Stefan Agner <stefan@xxxxxxxx> wrote:
> Use the new config symbol ARM_SINGLE_ARMV7M which groups config
> symbols used by modern ARMv7-M platforms. This allows supporting
> multiple ARMv7-M platforms in one kernel image. However, a common
> kernel image requires the combined platforms to share the same
> main memory layout to be bootable.
>
> Acked-by: Uwe Kleine-KÃnig <u.kleine-koenig@xxxxxxxxxxxxxx>
> Signed-off-by: Stefan Agner <stefan@xxxxxxxx>

You should have your sign-off on the top and not Uwe's ack.

> ---
> Since this is essentially only a shift of config symbols, it
> should not change runtime behavior, at least when selecting
> only one platform.
>
> Uwe, this is essentially the same I had in my patchset, just
> converting the other platforms too. I was bold and added your
> Ack anyway...
>
> Joachim, Maxime, I test compiled with your defconfigs, compiled
> fine on my machine.
>
>  arch/arm/Kconfig       | 86 ++++++++++++++++++--------------------------------
>  arch/arm/Kconfig.debug |  5 ++-
>  2 files changed, 32 insertions(+), 59 deletions(-)
>
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index 75920ed..9b777e3 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -334,6 +334,7 @@ config ARM_SINGLE_ARMV7M
>         depends on !MMU
>         select ARCH_WANT_OPTIONAL_GPIOLIB
>         select ARM_NVIC
> +       select AUTO_ZRELADDR
>         select CLKSRC_OF
>         select COMMON_CLK
>         select CPU_V7M
> @@ -411,24 +412,6 @@ config ARCH_EBSA110
>           Ethernet interface, two PCMCIA sockets, two serial ports and a
>           parallel port.
[...]
> -config ARCH_LPC18XX
> -       bool "NXP LPC18xx/LPC43xx"
> -       depends on !MMU
> -       select ARCH_HAS_RESET_CONTROLLER
> -       select ARCH_REQUIRE_GPIOLIB
> -       select ARM_AMBA
> -       select ARM_NVIC
> -       select AUTO_ZRELADDR
> -       select CLKSRC_LPC32XX
> -       select COMMON_CLK
> -       select CPU_V7M
> -       select GENERIC_CLOCKEVENTS
> -       select NO_IOPORT_MAP
> -       select PINCTRL
> -       select SPARSE_IRQ
> -       select USE_OF
> -       help
> -         Support for NXP's LPC18xx Cortex-M3 and LPC43xx Cortex-M4
> -         high performance microcontrollers.
> -
[...]
> +config ARCH_LPC18XX
> +       bool "NXP LPC18xx/LPC43xx"
> +       depends on ARM_SINGLE_ARMV7M
> +       select ARCH_HAS_RESET_CONTROLLER
> +       select ARM_AMBA
> +       select CLKSRC_LPC32XX
> +       select PINCTRL
> +       help
> +         Support for NXP's LPC18xx Cortex-M3 and LPC43xx Cortex-M4
> +         high performance microcontrollers.

The LPC18xx parts look good and it still builds and boots on my devkit so;

Acked-by: Joachim Eastwood <manabian@xxxxxxxxx>

regards,
Joachim Eastwood
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/