Re: [PATCH 03/10] arm64: add Armada 3700 architecture entry

From	Gregory CLEMENT <gregory.clement@free-electrons.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH 03/10] arm64: add Armada 3700 architecture entry
Date	2016-02-03 12:20 +0100
Message-ID	<qY3DA-1cM-9@gated-at.bofh.it> (permalink)
References	<qXNyN-6U3-3@gated-at.bofh.it> <qXZ6W-6G0-17@gated-at.bofh.it> <qY0w2-7rK-9@gated-at.bofh.it> <qY1Lr-8sb-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hi Arnd,
 
 On mer., févr. 03 2016, Arnd Bergmann <arnd@arndb.de> wrote:

> On Wednesday 03 February 2016 08:55:22 Gregory CLEMENT wrote:
>> >> 
>> >> diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
>> >> index 21074f674bde..7da341f7b90d 100644
>> >> --- a/arch/arm64/Kconfig.platforms
>> >> +++ b/arch/arm64/Kconfig.platforms
>> >> @@ -1,5 +1,11 @@
>> >>  menu "Platform selection"
>> >>  
>> >> +config ARCH_ARMADA_3700
>> >
>> > Would it be better to use ARCH_MVEBU? IMHO, there will be more ARMv8 SoCs
>> > from our EBU, do we plan to add one CONFIG_ARMADA_yyzz for each SoC?
>> 
>> Not for each SoC but for each family, yes. As we did for the other mvebu
>> SoC families.
>> 
>
> We tend to use a little more general config strings for arm64 than we have
> for arm32. I think just using ARCH_MVEBU is fine here, no need to list
> the 3700 series separately.

OK, for the arm32 mvebu SoC, the CONFIG_ARMADA_ were barely used, and
CONFIG_ARCH_MVEBU is mostly use as a "depends on" and do not force
anything.

The only annoying point was with the irqchip, but thanks to the last
series sent by Thomas irq-armada-370-xp.c won't be inconditionnaly built
if CONFIG_ARCH_MVEBU is selected.

So OK I will use CONFIG_ARCH_MVEBU.

Thanks,

Gregory



> 	Arnd

-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH 00/10] Add support for the Armada 3700 SoC an mvebu ARM64 based Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-02-02 19:10 +0100
  [PATCH 01/10] serial: mvebu-uart: initial support for Armada-3700 serial port Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-02-02 19:10 +0100
    Re: [PATCH 01/10] serial: mvebu-uart: initial support for  Armada-3700 serial port Mark Rutland <mark.rutland@arm.com> - 2016-02-02 19:30 +0100
      Re: [PATCH 01/10] serial: mvebu-uart: initial support for  Armada-3700 serial port Mark Rutland <mark.rutland@arm.com> - 2016-02-02 19:30 +0100
    Re: [PATCH 01/10] serial: mvebu-uart: initial support for  Armada-3700 serial port One Thousand Gnomes <gnomes@lxorguk.ukuu.org.uk> - 2016-02-02 19:30 +0100
      Re: [PATCH 01/10] serial: mvebu-uart: initial support for Armada-3700 serial port Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-02-03 17:10 +0100
    Re: [PATCH 01/10] serial: mvebu-uart: initial support for  Armada-3700 serial port Mark Rutland <mark.rutland@arm.com> - 2016-02-02 19:40 +0100
      Re: [PATCH 01/10] serial: mvebu-uart: initial support for Armada-3700 serial port Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-02-03 17:40 +0100
      Re: [PATCH 01/10] serial: mvebu-uart: initial support for Armada-3700 serial port Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-02-03 18:00 +0100
        Re: [PATCH 01/10] serial: mvebu-uart: initial support for  Armada-3700 serial port Mark Rutland <mark.rutland@arm.com> - 2016-02-03 18:10 +0100
    Re: [PATCH 01/10] serial: mvebu-uart: initial support for Armada-3700 serial port Arnd Bergmann <arnd@arndb.de> - 2016-02-02 22:10 +0100
      Re: [PATCH 01/10] serial: mvebu-uart: initial support for Armada-3700 serial port Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-02-03 17:50 +0100
    Re: [PATCH 01/10] serial: mvebu-uart: initial support for  Armada-3700 serial port Rob Herring <robh@kernel.org> - 2016-02-02 22:20 +0100
    Re: [PATCH 01/10] serial: mvebu-uart: initial support for Armada-3700  serial port Marcin Wojtas <mw@semihalf.com> - 2016-02-03 09:10 +0100
      Re: [PATCH 01/10] serial: mvebu-uart: initial support for Armada-3700 serial port Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-02-03 17:50 +0100
  [PATCH 04/10] Documentation: dt-bindings: Add a new compatible for the Armada 3700 Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-02-02 19:20 +0100
    Re: [PATCH 04/10] Documentation: dt-bindings: Add a new compatible  for the Armada 3700 Rob Herring <robh@kernel.org> - 2016-02-02 22:30 +0100
  [PATCH 08/10] MAINTAINERS: Extend dts entry for ARM64 mvebu files Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-02-02 19:20 +0100
  [PATCH 07/10] arm64: dts: add the Marvell Armada 3700 family and a development board Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-02-02 19:20 +0100
    Re: [PATCH 07/10] arm64: dts: add the Marvell Armada 3700 family and  a development board Mark Rutland <mark.rutland@arm.com> - 2016-02-02 19:40 +0100
      Re: [PATCH 07/10] arm64: dts: add the Marvell Armada 3700 family and a development board Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-02-03 11:50 +0100
    Re: [PATCH 07/10] arm64: dts: add the Marvell Armada 3700 family  and a development board Jisheng Zhang <jszhang@marvell.com> - 2016-02-03 07:50 +0100
  [PATCH 02/10] ata: ahci_mvebu: add support for Armada 3700 variant Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-02-02 19:20 +0100
  [PATCH 09/10] Documentation: arm: update supported Marvell EBU processors Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-02-02 19:20 +0100
  [PATCH 10/10] arm64: defconfig: enable Armada 3700 related config Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-02-02 19:20 +0100
  [PATCH 06/10] devicetree: bindings: add DT binding for the Marvell Armada 3700 SoC family Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-02-02 19:20 +0100
    Re: [PATCH 06/10] devicetree: bindings: add DT binding for the  Marvell Armada 3700 SoC family Rob Herring <robh@kernel.org> - 2016-02-02 22:30 +0100
      Re: [PATCH 06/10] devicetree: bindings: add DT binding for the Marvell Armada 3700 SoC family Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-02-03 09:10 +0100
  [PATCH 03/10] arm64: add Armada 3700 architecture entry Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-02-02 19:20 +0100
    Re: [PATCH 03/10] arm64: add Armada 3700 architecture entry Jisheng Zhang <jszhang@marvell.com> - 2016-02-03 07:30 +0100
      Re: [PATCH 03/10] arm64: add Armada 3700 architecture entry Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-02-03 09:00 +0100
        Re: [PATCH 03/10] arm64: add Armada 3700 architecture entry Arnd Bergmann <arnd@arndb.de> - 2016-02-03 10:20 +0100
          Re: [PATCH 03/10] arm64: add Armada 3700 architecture entry Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-02-03 12:20 +0100

csiph-web