Re: [PATCH 14/18] ARM: mvebu: add support for the Armada 395 SoC family

From	Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH 14/18] ARM: mvebu: add support for the Armada 395 SoC family
Date	2016-07-25 16:00 +0200
Message-ID	<rYOAh-5eA-7@gated-at.bofh.it> (permalink)
References	<rXlAl-7uo-3@gated-at.bofh.it> <rXlAm-7uo-43@gated-at.bofh.it> <rXutZ-5ec-47@gated-at.bofh.it> <rXEMF-3Qd-19@gated-at.bofh.it> <rYOqB-5bn-15@gated-at.bofh.it>
Organization	Free Electrons

Show all headers | View raw

Hello,

On Mon, 25 Jul 2016 08:47:23 -0500, Rob Herring wrote:

> > I am not sure if I get your point. The Armada-398 extends the
> > Armada-395 about 2 additional SATA ports (as you can see in commit
> > "[PATCH 15/18] ARM: mvebu: a398: update the dtsi about missing
> > interfaces"). In this example the a398-db board contains the Armada398
> > SoC, so it is a better match and goes first.  
> 
> But your patch title is adding 395 support, but you are adding the
> string to a 398 based board. It would make sense to have 395 here if
> the OS already had support for 395 and you want to support the 398
> without updating the OS. That doesn't seem to apply here.

I think the argument of Grzegorz is that the 398 is functionally a
strict superset of the 395, so that anything that applies to the 395
will also apply to 398.

Now, whether it is a good idea to consider them "compatible" in the DT
sense, I'm not sure.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

Thread

[PATCH 00/18] ARM: mvebu: misc Armada 38x/39x DT and v7 defconfig improvements Grzegorz Jaszczyk <jaz@semihalf.com> - 2016-07-21 14:50 +0200
  [PATCH 10/18] ARM: mvebu: a39x: enable watchdog for all Armada-39x SoCs Grzegorz Jaszczyk <jaz@semihalf.com> - 2016-07-21 14:50 +0200
    Re: [PATCH 10/18] ARM: mvebu: a39x: enable watchdog for all Armada-39x SoCs Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-07-28 12:20 +0200
  [PATCH 03/18] ARM: mvebu: enable E1000E in mvebu_v7_defconfig Grzegorz Jaszczyk <jaz@semihalf.com> - 2016-07-21 14:50 +0200
  [PATCH 11/18] ARM: mvebu: a39x: add missing nodes describing GPIO's Grzegorz Jaszczyk <jaz@semihalf.com> - 2016-07-21 14:50 +0200
    Re: [PATCH 11/18] ARM: mvebu: a39x: add missing nodes describing GPIO's Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-07-28 12:30 +0200
  [PATCH 14/18] ARM: mvebu: add support for the Armada 395 SoC family Grzegorz Jaszczyk <jaz@semihalf.com> - 2016-07-21 14:50 +0200
    Re: [PATCH 14/18] ARM: mvebu: add support for the Armada 395 SoC  family Rob Herring <robh@kernel.org> - 2016-07-22 00:20 +0200
      Re: [PATCH 14/18] ARM: mvebu: add support for the Armada 395 SoC family Grzegorz Jaszczyk <jaz@semihalf.com> - 2016-07-22 11:20 +0200
        Re: [PATCH 14/18] ARM: mvebu: add support for the Armada 395 SoC family Rob Herring <robh@kernel.org> - 2016-07-25 15:50 +0200
          Re: [PATCH 14/18] ARM: mvebu: add support for the Armada 395 SoC  family Thomas Petazzoni <thomas.petazzoni@free-electrons.com> - 2016-07-25 16:00 +0200
            Re: [PATCH 14/18] ARM: mvebu: add support for the Armada 395 SoC family Rob Herring <robh@kernel.org> - 2016-07-25 17:20 +0200
              Re: [PATCH 14/18] ARM: mvebu: add support for the Armada 395 SoC  family Thomas Petazzoni <thomas.petazzoni@free-electrons.com> - 2016-07-25 20:30 +0200
                Re: [PATCH 14/18] ARM: mvebu: add support for the Armada 395 SoC family Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-07-28 13:20 +0200
                Re: [PATCH 14/18] ARM: mvebu: add support for the Armada 395 SoC family Rob Herring <robh@kernel.org> - 2016-07-28 18:10 +0200
  [PATCH 06/18] ARM: mvebu: a390: add missing compatibility string and add missing bracket Grzegorz Jaszczyk <jaz@semihalf.com> - 2016-07-21 14:50 +0200
    Re: [PATCH 06/18] ARM: mvebu: a390: add missing compatibility string and add missing bracket Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-07-27 18:20 +0200
  [PATCH 08/18] ARM: mvebu: a39x: Enable PMU, CA9 MPcore SoC Controller and Coherency fabric Grzegorz Jaszczyk <jaz@semihalf.com> - 2016-07-21 14:50 +0200
    Re: [PATCH 08/18] ARM: mvebu: a39x: Enable PMU, CA9 MPcore SoC Controller and Coherency fabric Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-07-28 12:00 +0200
  [PATCH 09/18] ARM: mvebu: a39x: enable the thermal sensor in Armada-39x SoCs Grzegorz Jaszczyk <jaz@semihalf.com> - 2016-07-21 14:50 +0200
    Re: [PATCH 09/18] ARM: mvebu: a39x: enable the thermal sensor in Armada-39x SoCs Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-07-28 12:00 +0200
  [PATCH 16/18] ARM: mvebu: a398-db: enable supported usb interfaces Grzegorz Jaszczyk <jaz@semihalf.com> - 2016-07-21 14:50 +0200
    Re: [PATCH 16/18] ARM: mvebu: a398-db: enable supported usb interfaces Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-07-28 15:00 +0200
  [PATCH 13/18] ARM: mvebu: a39x: enable rtc for all Armada-39x SoCs Grzegorz Jaszczyk <jaz@semihalf.com> - 2016-07-21 14:50 +0200
  Re: [PATCH 00/18] ARM: mvebu: misc Armada 38x/39x DT and v7  defconfig improvements Andrew Lunn <andrew@lunn.ch> - 2016-07-21 15:50 +0200
    Re: [PATCH 00/18] ARM: mvebu: misc Armada 38x/39x DT and v7 defconfig improvements Grzegorz Jaszczyk <jaz@semihalf.com> - 2016-07-22 10:00 +0200

csiph-web