[PATCH 3/5] ARM: vexpress: Add DT support in v2m

Pawel Moll pawel.moll at arm.com
Fri Nov 18 07:20:49 EST 2011


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





More information about the linux-arm-kernel mailing list