From: Pawel Moll <pawel.moll@arm.com>
To: Russell King - ARM Linux <linux@arm.linux.org.uk>
Cc: Dave Martin <dave.martin@linaro.org>,
"devicetree-discuss@lists.ozlabs.org"
<devicetree-discuss@lists.ozlabs.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 3/5] ARM: vexpress: Add DT support in v2m
Date: Fri, 18 Nov 2011 12:20:49 +0000 [thread overview]
Message-ID: <1321618849.24216.323.camel@hornet.cambridge.arm.com> (raw)
In-Reply-To: <20111117155334.GT9581@n2100.arm.linux.org.uk>
On Thu, 2011-11-17 at 15:53 +0000, Russell King - ARM Linux wrote:
> You're totally failing to understand the point.
That's your opinion, I think otherwise, won't try to convince you.
> The point is that when Versatile Express first came out, it had a memory
> map. At this point in time, there was nothing 'legacy' about it, it was
> the latest and greatest thing.
That's right. And then when the new variant was created *hardware
designers* decided to call the previous one "legacy". I couldn't care
less what word was it, could be even "bublebee" from my point of view.
> Again, this is _not_ how DT is supposed to work. DT is about describing
> the hardware, not describing the state of something.
And that's exactly what I was trying to do. Describe the hardware, using
terminology *used by the hardware designers* - fact that you have chosen
to ignore, fair enough.
> So, get rid of that 'legacy' crap in DT naming. It doesn't belong. Or
> we'll start talling the Cortex-A5 stuff 'legacy' right now as well.
As the compatible values will be limited to the tile name and the
information about the motherboard "mode" will be passed through private
node property, I will follow Dave's suggestion of using "empty value"
for legacy/original/whatever variant. Hopefully this will end this topic
which is already too long.
Thanks for all comments!
Pawel
next prev parent reply other threads:[~2011-11-18 12:20 UTC|newest]
Thread overview: 49+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-11 18:27 [PATCH 0/5] Versatile Express DT support, take 2 Pawel Moll
2011-11-11 18:27 ` [PATCH 1/5] ARM: vexpress: Get rid of MMIO_P2V Pawel Moll
[not found] ` <1321036026-23411-2-git-send-email-pawel.moll-5wv7dgnIgG8@public.gmane.org>
2011-11-16 15:35 ` Dave Martin
2011-11-16 16:16 ` Pawel Moll
[not found] ` <1321460205.3137.352.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-16 17:28 ` Dave Martin
2011-11-16 17:30 ` Pawel Moll
2011-11-17 7:02 ` Ryan Harkin
2011-11-17 15:43 ` Russell King - ARM Linux
2011-11-18 12:20 ` Pawel Moll
[not found] ` <1321618848.24216.322.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-18 17:44 ` Russell King - ARM Linux
2011-11-11 18:27 ` [PATCH 2/5] ARM: vexpress: Remove platform SMP functions from ct_desc Pawel Moll
[not found] ` <1321036026-23411-3-git-send-email-pawel.moll-5wv7dgnIgG8@public.gmane.org>
2011-11-17 15:31 ` Russell King - ARM Linux
2011-11-18 12:20 ` Pawel Moll
2011-11-11 18:27 ` [PATCH 3/5] ARM: vexpress: Add DT support in v2m Pawel Moll
[not found] ` <1321036026-23411-4-git-send-email-pawel.moll-5wv7dgnIgG8@public.gmane.org>
2011-11-16 15:44 ` Dave Martin
[not found] ` <20111116154402.GE2073-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org>
2011-11-16 16:26 ` Rob Herring
2011-11-16 16:37 ` Pawel Moll
[not found] ` <1321461458.3137.374.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-16 16:59 ` Rob Herring
2011-11-16 17:07 ` Pawel Moll
2011-11-16 17:37 ` Pawel Moll
[not found] ` <1321465054.3137.386.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-16 19:14 ` Dave Martin
[not found] ` <1321463271.3137.380.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-16 17:39 ` Dave Martin
[not found] ` <20111116173945.GG2073-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org>
2011-11-16 17:50 ` Dave Martin
2011-11-16 17:55 ` Pawel Moll
2011-11-17 15:53 ` Russell King - ARM Linux
2011-11-18 12:20 ` Pawel Moll [this message]
[not found] ` <1321618849.24216.323.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-18 17:49 ` Russell King - ARM Linux
2011-11-16 16:35 ` Pawel Moll
[not found] ` <1321461315.3137.371.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-16 17:57 ` Dave Martin
2011-11-17 13:50 ` Pawel Moll
[not found] ` <1321537813.3137.475.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-17 14:41 ` Dave Martin
2011-11-17 16:05 ` Russell King - ARM Linux
[not found] ` <20111117160536.GU9581-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org>
2011-11-17 18:37 ` Dave Martin
[not found] ` <20111117183726.GE3044-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org>
2011-11-18 17:52 ` Russell King - ARM Linux
2011-11-18 12:20 ` Pawel Moll
2011-11-11 18:27 ` [PATCH 4/5] ARM: vexpress: Initial RS1 memory map support Pawel Moll
[not found] ` <1321036026-23411-5-git-send-email-pawel.moll-5wv7dgnIgG8@public.gmane.org>
2011-11-16 15:42 ` Dave Martin
2011-11-16 16:28 ` Pawel Moll
[not found] ` <1321460897.3137.364.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-16 18:03 ` Dave Martin
2011-11-17 15:36 ` Russell King - ARM Linux
2011-11-18 12:20 ` Pawel Moll
[not found] ` <1321618850.24216.324.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-18 17:56 ` Russell King - ARM Linux
2011-11-11 18:27 ` [PATCH 5/5] ARM: vexpress: DT-based support for CoreTiles Express A5x2 and A9x4 Pawel Moll
[not found] ` <1321036026-23411-6-git-send-email-pawel.moll-5wv7dgnIgG8@public.gmane.org>
2011-11-11 22:30 ` Rob Herring
2011-11-11 22:54 ` Pawel Moll
2011-11-16 15:36 ` Dave Martin
2011-11-16 16:22 ` Pawel Moll
[not found] ` <1321460559.3137.358.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-16 18:17 ` Dave Martin
[not found] ` <1321036026-23411-1-git-send-email-pawel.moll-5wv7dgnIgG8@public.gmane.org>
2011-11-16 15:33 ` [PATCH 0/5] Versatile Express DT support, take 2 Dave Martin
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=1321618849.24216.323.camel@hornet.cambridge.arm.com \
--to=pawel.moll@arm.com \
--cc=dave.martin@linaro.org \
--cc=devicetree-discuss@lists.ozlabs.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux@arm.linux.org.uk \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox