All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gregory CLEMENT <gregory.clement@free-electrons.com>
To: Rob Herring <robh@kernel.org>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>,
	Grzegorz Jaszczyk <jaz@semihalf.com>,
	"devicetree\@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-kernel\@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-arm-kernel\@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	Mark Rutland <mark.rutland@arm.com>,
	Jason Cooper <jason@lakedaemon.net>, Andrew Lunn <andrew@lunn.ch>,
	Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
	Russell King - ARM Linux <linux@armlinux.org.uk>,
	Marcin Wojtas <mw@semihalf.com>, Lior Amsalem <alior@marvell.com>
Subject: Re: [PATCH 14/18] ARM: mvebu: add support for the Armada 395 SoC family
Date: Thu, 28 Jul 2016 13:19:31 +0200	[thread overview]
Message-ID: <87y44mj9h8.fsf@free-electrons.com> (raw)
In-Reply-To: <20160725202209.5fae4d98@free-electrons.com> (Thomas Petazzoni's message of "Mon, 25 Jul 2016 20:22:09 +0200")

Hi Rob,
 
 On lun., juil. 25 2016, Thomas Petazzoni <thomas.petazzoni@free-electrons.com> wrote:

> Hello,
>
> On Mon, 25 Jul 2016 10:12:43 -0500, Rob Herring wrote:
>
>> Yes, I get that, but that is only meaningful if you want to run an OS
>> that is only aware of 395 on a 398 SoC/board (though I'd guess the 390
>> compat is enough for that). Otherwise, that property is not really
>> meaningful as the additional nodes are enough to handle what is the
>> superset.
>> 
>> I would agree both are fine if both chips are in fact the same die,
>> just fused or packaged differently. I've seen a lot of chips that are
>> supposed to be sub/supersets of each other, but have different errata
>> lists because they are different die.
>
> Unfortunately HW vendors are rarely willing to publicly indicate whether
> the different chips in their families are actually the same die fused
> differently, or really different dies.

So do you want that we keep both "marvell,armada398" and
"marvell,armada395" or do you xant we use only "marvell,armada398" ?

Thanks,

Gregory


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

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

WARNING: multiple messages have this Message-ID (diff)
From: Gregory CLEMENT <gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
To: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Cc: Thomas Petazzoni
	<thomas.petazzoni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
	Grzegorz Jaszczyk <jaz-nYOzD4b6Jr9Wk0Htik3J/w@public.gmane.org>,
	"devicetree@vger.kernel.org"
	<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"linux-kernel@vger.kernel.org"
	<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
	Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
	Jason Cooper <jason-NLaQJdtUoK4Be96aLqz0jA@public.gmane.org>,
	Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org>,
	Sebastian Hesselbarth
	<sebastian.hesselbarth-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Russell King - ARM Linux
	<linux-I+IVW8TIWO2tmTQ+vhA3Yw@public.gmane.org>,
	Marcin Wojtas <mw-nYOzD4b6Jr9Wk0Htik3J/w@public.gmane.org>,
	Lior Amsalem <alior-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org>
Subject: Re: [PATCH 14/18] ARM: mvebu: add support for the Armada 395 SoC family
Date: Thu, 28 Jul 2016 13:19:31 +0200	[thread overview]
Message-ID: <87y44mj9h8.fsf@free-electrons.com> (raw)
In-Reply-To: <20160725202209.5fae4d98-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> (Thomas Petazzoni's message of "Mon, 25 Jul 2016 20:22:09 +0200")

Hi Rob,
 
 On lun., juil. 25 2016, Thomas Petazzoni <thomas.petazzoni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> wrote:

> Hello,
>
> On Mon, 25 Jul 2016 10:12:43 -0500, Rob Herring wrote:
>
>> Yes, I get that, but that is only meaningful if you want to run an OS
>> that is only aware of 395 on a 398 SoC/board (though I'd guess the 390
>> compat is enough for that). Otherwise, that property is not really
>> meaningful as the additional nodes are enough to handle what is the
>> superset.
>> 
>> I would agree both are fine if both chips are in fact the same die,
>> just fused or packaged differently. I've seen a lot of chips that are
>> supposed to be sub/supersets of each other, but have different errata
>> lists because they are different die.
>
> Unfortunately HW vendors are rarely willing to publicly indicate whether
> the different chips in their families are actually the same die fused
> differently, or really different dies.

So do you want that we keep both "marvell,armada398" and
"marvell,armada395" or do you xant we use only "marvell,armada398" ?

Thanks,

Gregory


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

-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

WARNING: multiple messages have this Message-ID (diff)
From: gregory.clement@free-electrons.com (Gregory CLEMENT)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 14/18] ARM: mvebu: add support for the Armada 395 SoC family
Date: Thu, 28 Jul 2016 13:19:31 +0200	[thread overview]
Message-ID: <87y44mj9h8.fsf@free-electrons.com> (raw)
In-Reply-To: <20160725202209.5fae4d98@free-electrons.com> (Thomas Petazzoni's message of "Mon, 25 Jul 2016 20:22:09 +0200")

Hi Rob,
 
 On lun., juil. 25 2016, Thomas Petazzoni <thomas.petazzoni@free-electrons.com> wrote:

> Hello,
>
> On Mon, 25 Jul 2016 10:12:43 -0500, Rob Herring wrote:
>
>> Yes, I get that, but that is only meaningful if you want to run an OS
>> that is only aware of 395 on a 398 SoC/board (though I'd guess the 390
>> compat is enough for that). Otherwise, that property is not really
>> meaningful as the additional nodes are enough to handle what is the
>> superset.
>> 
>> I would agree both are fine if both chips are in fact the same die,
>> just fused or packaged differently. I've seen a lot of chips that are
>> supposed to be sub/supersets of each other, but have different errata
>> lists because they are different die.
>
> Unfortunately HW vendors are rarely willing to publicly indicate whether
> the different chips in their families are actually the same die fused
> differently, or really different dies.

So do you want that we keep both "marvell,armada398" and
"marvell,armada395" or do you xant we use only "marvell,armada398" ?

Thanks,

Gregory


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

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

  reply	other threads:[~2016-07-28 11:19 UTC|newest]

Thread overview: 129+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-21 12:43 [PATCH 00/18] ARM: mvebu: misc Armada 38x/39x DT and v7 defconfig improvements Grzegorz Jaszczyk
2016-07-21 12:43 ` Grzegorz Jaszczyk
2016-07-21 12:43 ` Grzegorz Jaszczyk
2016-07-21 12:43 ` [PATCH 01/18] ARM: mvebu: enable MTD command line partition table in mvebu_v7_defconfig Grzegorz Jaszczyk
2016-07-21 12:43   ` Grzegorz Jaszczyk
2016-07-21 12:43   ` Grzegorz Jaszczyk
2016-07-27 15:57   ` Gregory CLEMENT
2016-07-27 15:57     ` Gregory CLEMENT
2016-07-21 12:43 ` [PATCH 02/18] ARM: mvebu: enable UBI and UBIFS " Grzegorz Jaszczyk
2016-07-21 12:43   ` Grzegorz Jaszczyk
2016-07-21 12:43   ` Grzegorz Jaszczyk
2016-07-27 15:58   ` Gregory CLEMENT
2016-07-27 15:58     ` Gregory CLEMENT
2016-07-21 12:43 ` [PATCH 03/18] ARM: mvebu: enable E1000E " Grzegorz Jaszczyk
2016-07-21 12:43   ` Grzegorz Jaszczyk
2016-07-21 12:44   ` Russell King - ARM Linux
2016-07-21 12:44     ` Russell King - ARM Linux
2016-07-21 12:46     ` Thomas Petazzoni
2016-07-21 12:46       ` Thomas Petazzoni
2016-07-21 13:52       ` Grzegorz Jaszczyk
2016-07-21 13:52         ` Grzegorz Jaszczyk
2016-07-21 12:44 ` [PATCH 04/18] ARM: mvebu: a385-db-ap: enable USB (orion-ehci) port Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-27 16:00   ` Gregory CLEMENT
2016-07-27 16:00     ` Gregory CLEMENT
2016-07-27 16:00     ` Gregory CLEMENT
2016-07-27 16:09     ` Gregory CLEMENT
2016-07-27 16:09       ` Gregory CLEMENT
2016-07-21 12:44 ` [PATCH 05/18] ARM: mvebu: a385-db-ap: add default partition description for NAND Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-27 16:04   ` Gregory CLEMENT
2016-07-27 16:04     ` Gregory CLEMENT
2016-07-27 16:04     ` Gregory CLEMENT
2016-07-27 16:10     ` Gregory CLEMENT
2016-07-27 16:10       ` Gregory CLEMENT
2016-07-21 12:44 ` [PATCH 06/18] ARM: mvebu: a390: add missing compatibility string and add missing bracket Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-27 16:13   ` Gregory CLEMENT
2016-07-27 16:13     ` Gregory CLEMENT
2016-07-27 16:13     ` Gregory CLEMENT
2016-07-21 12:44 ` [PATCH 07/18] ARM: mvebu: a39x: update the SDHCI node on Armada 39x Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-28  9:39   ` Gregory CLEMENT
2016-07-28  9:39     ` Gregory CLEMENT
2016-07-21 12:44 ` [PATCH 08/18] ARM: mvebu: a39x: enable PMU, CA9 MPcore SoC Controller and Coherency fabric Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-28  9:44   ` Gregory CLEMENT
2016-07-28  9:44     ` Gregory CLEMENT
2016-07-28  9:44     ` Gregory CLEMENT
2016-07-21 12:44 ` [PATCH 08/18] ARM: mvebu: a39x: Enable " Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-28  9:51   ` Gregory CLEMENT
2016-07-28  9:51     ` Gregory CLEMENT
2016-07-21 12:44 ` [PATCH 09/18] ARM: mvebu: a39x: enable the thermal sensor in Armada-39x SoCs Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-28  9:52   ` Gregory CLEMENT
2016-07-28  9:52     ` Gregory CLEMENT
2016-07-21 12:44 ` [PATCH 10/18] ARM: mvebu: a39x: enable watchdog for all " Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-28 10:18   ` Gregory CLEMENT
2016-07-28 10:18     ` Gregory CLEMENT
2016-07-28 10:18     ` Gregory CLEMENT
2016-07-21 12:44 ` [PATCH 11/18] ARM: mvebu: a39x: add missing nodes describing GPIO's Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-28 10:25   ` Gregory CLEMENT
2016-07-28 10:25     ` Gregory CLEMENT
2016-07-28 10:25     ` Gregory CLEMENT
2016-07-21 12:44 ` [PATCH 12/18] ARM: mvebu: a39x: add missing nodes describing USB2.0 and USB3.0 Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-28 10:34   ` Gregory CLEMENT
2016-07-28 10:34     ` Gregory CLEMENT
2016-07-21 12:44 ` [PATCH 13/18] ARM: mvebu: a39x: enable rtc for all Armada-39x SoCs Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-28 10:53   ` Gregory CLEMENT
2016-07-28 10:53     ` Gregory CLEMENT
2016-07-28 10:53     ` Gregory CLEMENT
2016-07-21 12:44 ` [PATCH 14/18] ARM: mvebu: add support for the Armada 395 SoC family Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-21 22:16   ` Rob Herring
2016-07-21 22:16     ` Rob Herring
2016-07-22  9:10     ` Grzegorz Jaszczyk
2016-07-22  9:10       ` Grzegorz Jaszczyk
2016-07-22  9:10       ` Grzegorz Jaszczyk
2016-07-25 13:47       ` Rob Herring
2016-07-25 13:47         ` Rob Herring
2016-07-25 13:47         ` Rob Herring
2016-07-25 13:50         ` Thomas Petazzoni
2016-07-25 13:50           ` Thomas Petazzoni
2016-07-25 13:50           ` Thomas Petazzoni
2016-07-25 15:12           ` Rob Herring
2016-07-25 15:12             ` Rob Herring
2016-07-25 15:12             ` Rob Herring
2016-07-25 18:22             ` Thomas Petazzoni
2016-07-25 18:22               ` Thomas Petazzoni
2016-07-25 18:22               ` Thomas Petazzoni
2016-07-28 11:19               ` Gregory CLEMENT [this message]
2016-07-28 11:19                 ` Gregory CLEMENT
2016-07-28 11:19                 ` Gregory CLEMENT
2016-07-28 15:59                 ` Rob Herring
2016-07-28 15:59                   ` Rob Herring
2016-07-28 15:59                   ` Rob Herring
2016-07-21 12:44 ` [PATCH 15/18] ARM: mvebu: a398: update the dtsi about missing interfaces Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-28 12:55   ` Gregory CLEMENT
2016-07-28 12:55     ` Gregory CLEMENT
2016-07-21 12:44 ` [PATCH 16/18] ARM: mvebu: a398-db: enable supported usb interfaces Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-28 12:55   ` Gregory CLEMENT
2016-07-28 12:55     ` Gregory CLEMENT
2016-07-28 12:55     ` Gregory CLEMENT
2016-07-21 12:44 ` [PATCH 17/18] ARM: mvebu: a390-db: add support for the Armada 390 DB board Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-28 13:01   ` Gregory CLEMENT
2016-07-28 13:01     ` Gregory CLEMENT
2016-07-28 13:01     ` Gregory CLEMENT
2016-07-21 12:44 ` [PATCH 18/18] ARM: mvebu: a395-gp: add support for the Armada 395 GP Board Grzegorz Jaszczyk
2016-07-21 12:44   ` Grzegorz Jaszczyk
2016-07-28 13:15   ` Gregory CLEMENT
2016-07-28 13:15     ` Gregory CLEMENT
2016-07-21 13:43 ` [PATCH 00/18] ARM: mvebu: misc Armada 38x/39x DT and v7 defconfig improvements Andrew Lunn
2016-07-21 13:43   ` Andrew Lunn
2016-07-22  7:51   ` Grzegorz Jaszczyk
2016-07-22  7:51     ` Grzegorz Jaszczyk
2016-07-22  7:51     ` Grzegorz Jaszczyk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87y44mj9h8.fsf@free-electrons.com \
    --to=gregory.clement@free-electrons.com \
    --cc=alior@marvell.com \
    --cc=andrew@lunn.ch \
    --cc=devicetree@vger.kernel.org \
    --cc=jason@lakedaemon.net \
    --cc=jaz@semihalf.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@armlinux.org.uk \
    --cc=mark.rutland@arm.com \
    --cc=mw@semihalf.com \
    --cc=robh@kernel.org \
    --cc=sebastian.hesselbarth@gmail.com \
    --cc=thomas.petazzoni@free-electrons.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.