linux-omap.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Florian Vaussard <florian.vaussard@epfl.ch>
To: Nishanth Menon <nm@ti.com>, Tero Kristo <t-kristo@ti.com>,
	Kevin Hilman <khilman@linaro.org>,
	kernel-build-reports@lists.linaro.org,
	"linaro-kernel@lists.linaro.org" <linaro-kernel@lists.linaro.org>,
	"tony@atomide.com" <tony@atomide.com>
Cc: Mike Turquette <mturquette@linaro.org>,
	linux-omap <linux-omap@vger.kernel.org>,
	Olof Johansson <olof@lixom.net>, Ash Charles <ash@gumstix.com>
Subject: Re: next boot: 34 pass, 5 fail (next-20140122)
Date: Fri, 24 Jan 2014 21:37:53 +0100	[thread overview]
Message-ID: <52E2CF21.2020304@epfl.ch> (raw)
In-Reply-To: <52E1527E.8020509@ti.com>

Hi,

On 01/23/2014 06:33 PM, Nishanth Menon wrote:
> On 01/23/2014 10:35 AM, Florian Vaussard wrote:
>>
> 
> [..]
>>
>> Ok, so with your patch and changing the include from omap34xx to
>> omap36xx, it works. Now I have to come up with a way to manage all the
>> versions without duplicating all the DT files :-(
> 
> you'd also want to be careful about the omap3_pmx_core2 Vs
> omap3_pmx_core2 split for padconf.
> 

Yes, I know that this is another problematic point. I guess that I will
end up splitting all 34xx-specific and 36xx-specific parts into
dedicated files. Unless I can figure out a way to magically compute the
offset inside the pinctrl domain, but I doubt. I will try to contain the
omap3_pmx_core2 pins to omap3-overo, away from the expansion boards.

> options that come to mind are:
> a) split omap3-overo.dtsi into omap3-overo-common.dtsi
> omap3-overo-34xx.dtsi,omap3-overo-36xx.dtsi, corresponding tobi board
> dts file include as needed
> b) only have common stuff in omap3-overo.dtsi, include omap34xx.dtsi
> or omap36xx.dtsi in corresponding tobi board dts.
> 

Yes, both are an option. I don't know if b) can work, as
omap3-overo.dtsi maybe depends on features declared in omap3yxx.dtsi (to
be verified).

The main problem with both options is that Tobi is not the only
expansion board (the only mainlined for now). I sent patches for other
expansion boards a couple of days ago [1], so at the end we may have ~10
boards if you count only Gumstix's boards, not to mention the boards
from other people (like me). So finally, we will double the number of
dts files just to support both variants. I hope that Gumstix will not
produce a third version of the Overo.

Maybe I can use dt overlay for the expansion boards, like what is done
for capes with beaglebone. The expansion board's overlay could be
applied to the correct Overo dtb at runtime, like what is done from the
hardware's point of view.

I am not familiar with overlays, but from my rapid analysis, the main
difference is: cape's overlay is fused by the userspace when the kernel
has booted, whereas Overo's expansion boards must be fused early in the
boot process (or by the bootloader), as they provide some essential
services, like the primary NIC. I need more time to explore the
feasibility of this solution.

Regards,

Florian

[1] http://thread.gmane.org/gmane.linux.ports.arm.omap/109589

  reply	other threads:[~2014-01-24 20:38 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <52e06652.c103450a.79d7.6517@mx.google.com>
2014-01-23  1:35 ` next boot: 34 pass, 5 fail (next-20140122) Kevin Hilman
2014-01-23  6:23   ` Tero Kristo
2014-01-23  8:14     ` Florian Vaussard
2014-01-23  9:00       ` Tero Kristo
2014-01-23  9:15         ` Florian Vaussard
2014-01-23  9:41           ` Florian Vaussard
2014-01-23 12:27             ` Tero Kristo
2014-01-23 16:35               ` Florian Vaussard
2014-01-23 17:33                 ` Nishanth Menon
2014-01-24 20:37                   ` Florian Vaussard [this message]
2014-01-23 17:35                 ` Tony Lindgren
2014-01-24 18:00                   ` Tony Lindgren
2014-01-24 20:51                   ` Florian Vaussard
2014-01-24 18:11           ` Tony Lindgren
2014-01-24 18:13             ` Florian Vaussard
2014-02-05 15:23               ` Kevin Hilman
2014-02-06  8:47                 ` Florian Vaussard

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=52E2CF21.2020304@epfl.ch \
    --to=florian.vaussard@epfl.ch \
    --cc=ash@gumstix.com \
    --cc=kernel-build-reports@lists.linaro.org \
    --cc=khilman@linaro.org \
    --cc=linaro-kernel@lists.linaro.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=mturquette@linaro.org \
    --cc=nm@ti.com \
    --cc=olof@lixom.net \
    --cc=t-kristo@ti.com \
    --cc=tony@atomide.com \
    /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).