From: thomas.petazzoni@free-electrons.com (Thomas Petazzoni)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 3/4] arm64: dts: marvell: Clean up armada-7040-db
Date: Thu, 24 Mar 2016 11:40:37 +0100 [thread overview]
Message-ID: <20160324114037.1204b46e@free-electrons.com> (raw)
In-Reply-To: <87oaa4npbk.fsf@free-electrons.com>
Hello,
On Thu, 24 Mar 2016 11:27:27 +0100, Gregory CLEMENT wrote:
> > We haven't used this solution on Marvell Armada 32-bits SoCs, so there
> > needs to be a discussion on whether we want to go in this direction for
> > the 64 bits SoCs.
>
> At first view using the label helps to have simpler dts files.
>
> Is there any cons by using it?
The only minor drawback from my point of view is that you don't see
where in the hierarchy the device you're enabling is. But it's really a
minor drawback and not everybody agrees that it is actually a drawback.
So I'm fine.
> I agree that converting the Marvell Armada 32-bits SoCs would produce a
> lot of churn. But if some binding are common there is no file at all are
> in common, so we could use this solution for the 64 bits SoCs only.
Yes, we could. I'm fine with it. I was merely pointing out that it is
moving away from our 32 bits way of doing things.
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
next prev parent reply other threads:[~2016-03-24 10:40 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-23 22:24 [PATCH 0/4] ARM64: mvebu: Armada 3700/7K/8K DT cleanups Andreas Färber
2016-03-23 22:24 ` [PATCH 1/4] arm64: dts: marvell: Clean up armada-3720-db Andreas Färber
2016-03-24 16:11 ` Gregory CLEMENT
2016-03-24 16:46 ` Andreas Färber
2016-03-25 8:16 ` Gregory CLEMENT
2016-04-07 19:00 ` Gregory CLEMENT
2016-03-23 22:24 ` [PATCH 2/4] arm64: dts: marvell: Rename armada-37xx USB node Andreas Färber
2016-03-23 22:24 ` [PATCH 3/4] arm64: dts: marvell: Clean up armada-7040-db Andreas Färber
2016-03-24 8:18 ` Thomas Petazzoni
2016-03-24 10:27 ` Gregory CLEMENT
2016-03-24 10:40 ` Thomas Petazzoni [this message]
2016-03-24 13:37 ` Andrew Lunn
2016-03-24 13:59 ` Jason Cooper
2016-03-23 22:24 ` [PATCH 4/4] arm64: dts: marvell: Rename armada-ap806 XOR nodes Andreas Färber
2016-03-24 8:17 ` Thomas Petazzoni
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=20160324114037.1204b46e@free-electrons.com \
--to=thomas.petazzoni@free-electrons.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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;
as well as URLs for NNTP newsgroup(s).