All of lore.kernel.org
 help / color / mirror / Atom feed
From: Linus Walleij <linus.walleij@linaro.org>
To: Gabriel Fernandez <gabriel.fernandez.st@gmail.com>
Cc: Lee Jones <lee.jones@linaro.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Olivier Clergeaud <olivier.clergeaud@st.com>,
	Gabriel Fernandez <gabriel.fernandez@st.com>
Subject: Re: [PATCH 1/4] ARM: ux540: Add ccu8540-uib-v3 Device Tree Machine
Date: Tue, 4 Jun 2013 10:57:49 +0200	[thread overview]
Message-ID: <CACRpkdbThH7LysDjrPcFjxg0sMDm1myTfmXkJnkY3QSS3F_GGQ@mail.gmail.com> (raw)
In-Reply-To: <CAGo1xzDB-uGsMANghtqX-rX6GuhmHKo1Z=ywtC=TXfx6eM_ALQ@mail.gmail.com>

On Thu, May 30, 2013 at 3:20 PM, Gabriel Fernandez
<gabriel.fernandez.st@gmail.com> wrote:

[Lee]
>> Is it a new board, or just a UIB?
>
> For me it's a new board.
>
> ccu8540.dts is a board without uib.
> You can can plug on this board some different uibs.
> That's why, i created a ccu8540-uib-v3.dts board in which i included a
> common board ccu8540.dts

So that means that ccu8540.dts will boot the system even
without any UIB if I understand correctly. Anything that
pertains to the UIB should be in the UIB overlay.

> Another solution is to create one *.dtsi file with all uibs for
> ccu8540 and makes a dynamique detection.
> Linus, Lee  what do you think about that ?

I'm lost on this, dynamic detection would be done in the
boot loader, usually this just encodes one, static tree.

The first approach seems OK with me.

Yours,
Linus Walleij

  reply	other threads:[~2013-06-04  8:57 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-29 13:33 [PATCH 0/4] Introduce ccu8540 uib-v3 Gabriel Fernandez
2013-05-29 13:33 ` [PATCH 1/4] ARM: ux540: Add ccu8540-uib-v3 Device Tree Machine Gabriel Fernandez
2013-05-29 15:50   ` Lee Jones
2013-05-30 13:20     ` Gabriel Fernandez
2013-06-04  8:57       ` Linus Walleij [this message]
2013-05-29 13:33 ` [PATCH 2/4] ARM: u8540: Add device tree support for lp5521 leds Gabriel Fernandez
2013-05-29 15:52   ` Lee Jones
2013-05-29 13:33 ` [PATCH 3/4] ARM: u8540: Add device tree support for bh1780 Gabriel Fernandez
2013-05-29 15:54   ` Lee Jones
2013-05-29 13:33 ` [PATCH 4/4] ARM: u8540: Add DT probing for SFH7741 Proximity Sensor Gabriel Fernandez

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=CACRpkdbThH7LysDjrPcFjxg0sMDm1myTfmXkJnkY3QSS3F_GGQ@mail.gmail.com \
    --to=linus.walleij@linaro.org \
    --cc=gabriel.fernandez.st@gmail.com \
    --cc=gabriel.fernandez@st.com \
    --cc=lee.jones@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=olivier.clergeaud@st.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 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.