From: david.goodenough@btconnect.com (David Goodenough)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: dts: mvebu: add ethernet to the cm-a510 board
Date: Fri, 30 Jan 2015 12:44:56 +0000 [thread overview]
Message-ID: <1507191.iMZ4oC8rrr@stargate> (raw)
In-Reply-To: <54CB732F.7010409@gmail.com>
On Friday 30 January 2015 13:03:59 Sebastian Hesselbarth wrote:
> On 30.01.2015 12:41, Jean-Francois Moine wrote:
> > On Fri, 30 Jan 2015 12:00:16 +0100
> >
> > Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com> wrote:
> >> Adapting the .config and removing drivers is actually not an option.
> >> IMHO, introducing DT was meant for a single multi-arch kernel that
> >> can be shipped with common Linux distros. Therefore, DT is the place
> >> you enable/disable available resources. You leave most of the SoC (and
> >> SoM) nodes disabled as long as you cannot tell if there is a
> >> corresponding connector available.
> >
> > Well, I don't know too much about the hardware, and less about the
> > hardware modules (SoM?).
>
> Sorry, SoM is for System-on-Module, i.e. the CM-A510 itself.
>
> You can see from the block diagram that it comprises the Dove SoC,
> power circuitry, touch-screen controller, WiFi, GbE PHY for GbE-0, GbE
> controller on PCIe for GbE-1, I2S audio codec, RS232 Level Shifter for
> UART0, an USB2 Hub, SPI flash, NAND and RAM.
>
> That basically is what will be represented in the som.dtsi. If any of
> the functions above and the SoC will be _accessible_ on the baseboard is
> another story.
>
> > As seen in the Compulab documents, there are a lot of hardware modules.
> > For the DT, do you mean that there would be as many .dts's as the whole
> > number of connection possibilities?
>
> Nope. One dove.dtsi, one dove-cm-a510.dtsi, and one baseboard.dts
> including dove-cm-a510.dtsi for every baseboard we stumble upon.
>
> > I'd have better seen the inverted case as the actual empty cm-board dts:
> > enable every option in the (generic) .dts and let the vendor/user create
> > a specific .dts from this one for the board according to the installed
> > modules.
>
> That what dtsi's are made for with one exception: the dtsi cannot "run"
> on its own but needs at least one baseboard.dts that includes it. We
> could create a "bare"-baseboard that represents what is (easily)
> accessible on the SoM itself. Given the fact that even UART0 needs a
> baseboard that grabs it from the SoM connector, I see no value in that.
>
> > In any case, any real cm-a510 board should work with the
> > generic/full .dts even if some hardware modules are lacking. No?
>
> Nope. The cm-a510 is just an add-on for a baseboard, it does not make
> a working board. Just think of it as a feature-improved SoC.
This sounds like capes on the BeagleBoard. Are these extension boards
self-identifying? If so then the approach used with the capes might work
here too.
David
>
> Sebastian
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2015-01-30 12:44 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <E1YH5bU-0005mG-3L@bombadil.infradead.org>
2015-01-30 9:44 ` [PATCH] ARM: dts: mvebu: add ethernet to the cm-a510 board Sebastian Hesselbarth
2015-01-30 10:31 ` Jean-Francois Moine
2015-01-30 11:00 ` Sebastian Hesselbarth
2015-01-30 11:41 ` Jean-Francois Moine
2015-01-30 12:03 ` Sebastian Hesselbarth
2015-01-30 12:39 ` Jean-Francois Moine
2015-01-30 12:50 ` Sebastian Hesselbarth
[not found] ` <54CE354F.5090106@gmail.com>
2015-02-05 22:13 ` Sebastian Hesselbarth
2015-02-06 7:58 ` Jean-Francois Moine
2015-02-06 10:05 ` Sebastian Hesselbarth
2015-02-06 10:50 ` Jean-Francois Moine
2015-02-06 10:52 ` Russell King - ARM Linux
2015-02-06 10:48 ` Russell King - ARM Linux
[not found] ` <54D48711.8060903@gmail.com>
2015-02-06 10:00 ` Sebastian Hesselbarth
2015-01-30 12:44 ` David Goodenough [this message]
2015-01-30 13:07 ` Sebastian Hesselbarth
2015-01-30 13:32 ` David Goodenough
2015-01-30 6:06 Jean-Francois Moine
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=1507191.iMZ4oC8rrr@stargate \
--to=david.goodenough@btconnect.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