From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 1/5] ARM: bcm476x: Add platform infrastructure
Date: Fri, 19 Oct 2012 09:03:37 +0000 [thread overview]
Message-ID: <201210190903.38142.arnd@arndb.de> (raw)
In-Reply-To: <20121018154731.GA9344@glitch>
On Thursday 18 October 2012, Domenico Andreoli wrote:
> On Thu, Oct 18, 2012 at 01:48:01PM +0000, Arnd Bergmann wrote:
> > On Sunday 14 October 2012, Domenico Andreoli wrote:
> +
> +Boards with the BCM4760 SoC shall have the following properties:
> +
> +Required root node property:
> +
> +compatible = "brcm,bcm4760";
> +
> +
> +Boards with the BCM4761 SoC shall have the following properties:
> +
> +Required root node property:
> +
> +compatible = "brcm,bcm4760", "brcm,bcm4761";
>
> and the dt_mach in the board file is left only with "brcm,bcm4760" until
> required otherwise. The same applies to drivers.
Right.
> Does the order matter?
Yes, you have to have the most specific one first, and the most generic
one last, as documented in Documentation/devicetree/booting-without-of.txt.
If one chip has a functionality that the other one doesn't but is otherwise
completely compatible, then the less capable one should be put last.
For the root node, you might actually want to keep both "compatible" strings
separate as you have in the version you posted, at least if the chips are
not completely backwards compatible. For the other devices inside of the
soc, just use one.
> > For the patches that go into different directories like the clk
> > and the clocksource drivers, please Cc the respective subsystem
> > maintainers and ask them for an Ack. It certainly makes sense
> > for a new platform port to get merged through the arm-soc tree,
> > but any future improvements should normally just go through the
> > subsystem trees.
>
> I'd prefer patches but only because I've not any public git repository. If
> the git pull is much more preferred, I surely can manage it.
Ok, no problem. In the long run, it can be useful for you to set up
your own git tree, for for now, we can manage with patches just fine.
Arnd
next prev parent reply other threads:[~2012-10-19 9:03 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-14 22:14 [PATCH v2 0/5] ARM: Add support for Broadcom BCM476x SoCs Domenico Andreoli
2012-10-14 22:14 ` [PATCH v2 1/5] ARM: bcm476x: Add platform infrastructure Domenico Andreoli
2012-10-18 13:48 ` Arnd Bergmann
2012-10-18 15:47 ` Domenico Andreoli
2012-10-19 9:03 ` Arnd Bergmann [this message]
2012-10-14 22:14 ` [PATCH v2 2/5] ARM: bcm476x: Add system timer Domenico Andreoli
2012-10-14 22:14 ` [PATCH v2 3/5] ARM: bcm476x: Add ripple counter Domenico Andreoli
2012-10-14 22:14 ` [PATCH v2 4/5] ARM: bcm476x: Add stub clock driver Domenico Andreoli
2012-10-14 22:14 ` [PATCH v2 5/5] ARM: bcm476x: Add restart hook Domenico Andreoli
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=201210190903.38142.arnd@arndb.de \
--to=arnd@arndb.de \
--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 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.