devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Guenter Roeck <linux@roeck-us.net>
To: Geert Uytterhoeven <geert@linux-m68k.org>,
	Grant Likely <grant.likely@secretlab.ca>
Cc: Pantelis Antoniou <pantelis.antoniou@konsulko.com>,
	Rob Herring <robherring2@gmail.com>,
	Stephen Warren <swarren@wwwdotorg.org>,
	Matt Porter <matt.porter@linaro.org>,
	Koen Kooi <koen@dominion.thruhere.net>,
	Alison Chaiken <Alison_Chaiken@mentor.com>,
	Dinh Nguyen <dinh.linux@gmail.com>, Jan Lubbe <jluebbe@lasnet.de>,
	Alexander Sverdlin <alexander.sverdlin@nsn.com>,
	Michael Stickel <ms@mycable.de>,
	Dirk Behme <dirk.behme@gmail.com>,
	Alan Tull <delicious.quinoa@gmail.com>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	Michael Bohan <mbohan@codeaurora.org>,
	Ionut Nicu <ioan.nicu.ext@nsn.com>,
	Michal Simek <monstr@monstr.eu>,
	Matt Ranostay <mranostay@gmail.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Pete Popov <pete.popov@konsulko.com>,
	Dan Malek <dan.malek@konsulko.com>, Georgi Vlaev <georgi>
Subject: Re: [PATCH v4 2/8] OF: Introduce DT overlay support.
Date: Wed, 14 May 2014 06:18:58 -0700	[thread overview]
Message-ID: <53736D42.1010302@roeck-us.net> (raw)
In-Reply-To: <CAMuHMdXfNXB7s71+joCpkZrzPCJQFyNv3+bj+=usORRScZsHZQ@mail.gmail.com>

On 05/14/2014 06:03 AM, Geert Uytterhoeven wrote:
> On Wed, May 14, 2014 at 12:08 PM, Grant Likely
> <grant.likely@secretlab.ca> wrote:
>>> +config OF_OVERLAY
>>> +     bool "OF overlay support"
>>> +     depends on OF
>>> +     select OF_DYNAMIC
>>> +     select OF_DEVICE
>>> +     select OF_RESOLVE
>>> +     help
>>> +       OpenFirmware overlay support. Allows you to modify on runtime the
>>> +       live tree using overlays.
>>
>> Should not be a user-visable option. Drivers using it should select it
>
> Why not? It's up to the (final) user to use it, or not.
>
>> or otherwise cause it to be enabled.
>
> Why should this be driver-specific? Shouldn't this just work with all drivers?
>

It does once enabled.

I think what Grant refers to is that there has to be a driver which implements
the actual overlay insertion and removal (ie calls the new API functions),
and that the Kconfig option for this driver should select OF_OVERLAY
instead of depending on it.

Guenter

  reply	other threads:[~2014-05-14 13:18 UTC|newest]

Thread overview: 50+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-04 12:43 [PATCH v4 0/8] Introducing (yet again) Device Tree Overlays Pantelis Antoniou
2014-04-04 12:43 ` [PATCH v4 1/8] OF: Introduce Device Tree resolve support Pantelis Antoniou
     [not found] ` <1396615441-29630-1-git-send-email-pantelis.antoniou-OWPKS81ov/FWk0Htik3J/w@public.gmane.org>
2014-04-04 12:43   ` [PATCH v4 2/8] OF: Introduce DT overlay support Pantelis Antoniou
2014-05-14 10:08     ` Grant Likely
     [not found]       ` <20140514100856.5198DC4153D-WNowdnHR2B42iJbIjFUEsiwD8/FfD2ys@public.gmane.org>
2014-05-14 12:11         ` Michael Stickel
     [not found]           ` <53735D88.5090108-g5CePrrZ5ROELgA04lAiVw@public.gmane.org>
2014-05-14 15:49             ` Grant Likely
     [not found]               ` <20140514154907.BC65DC40BD2-WNowdnHR2B42iJbIjFUEsiwD8/FfD2ys@public.gmane.org>
2014-05-14 16:13                 ` Guenter Roeck
2014-05-15  7:14             ` Pantelis Antoniou
2014-05-14 13:03         ` Geert Uytterhoeven
2014-05-14 13:18           ` Guenter Roeck [this message]
     [not found]             ` <53736D42.1010302-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>
2014-05-15  7:15               ` Pantelis Antoniou
2014-05-14 15:34           ` Grant Likely
2014-05-15  7:12         ` Pantelis Antoniou
     [not found]           ` <E204285C-D6CD-4518-A94F-26BF0570C1B1-OWPKS81ov/FWk0Htik3J/w@public.gmane.org>
2014-05-15  7:20             ` Geert Uytterhoeven
     [not found]               ` <CAMuHMdUWfQ=2xPhNsS36mT6nv-CfR3_1exZ+5qYgdkjbBntrWg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-05-16 10:58                 ` Grant Likely
     [not found]                   ` <20140516105814.3EA3FC403C2-WNowdnHR2B42iJbIjFUEsiwD8/FfD2ys@public.gmane.org>
2014-05-16 11:52                     ` Geert Uytterhoeven
2014-05-20  5:50                       ` Grant Likely
     [not found]                         ` <20140520055026.E3A98C412DA-WNowdnHR2B42iJbIjFUEsiwD8/FfD2ys@public.gmane.org>
2014-05-20  7:38                           ` Geert Uytterhoeven
     [not found]                             ` <CAMuHMdXjWENrq0MW-DU2x7WnaCsWyipN4DO=y0ZTL+3G_fPYGg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-05-26 10:48                               ` Grant Likely
2014-05-26 10:57                                 ` Geert Uytterhoeven
     [not found]                                   ` <CAMuHMdUGdjVKBR5rRzZ274rf7p8XfztNRWEojH1RrZV8c_V1uA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-05-26 11:08                                     ` Pantelis Antoniou
2014-05-26 11:23                                   ` Grant Likely
2014-05-26 11:55                                     ` Pantelis Antoniou
2014-05-26 15:09                                       ` Sebastian Reichel
     [not found]                                         ` <20140526150942.GA26787-SfvFxonMDyemK9LvCR3Hrw@public.gmane.org>
2014-05-26 15:14                                           ` Guenter Roeck
2014-05-26 23:00                                             ` Sebastian Reichel
2014-05-26 15:14                                           ` Pantelis Antoniou
     [not found]                                       ` <A6195DFD-0447-4444-AC09-501C43F09924-OWPKS81ov/FWk0Htik3J/w@public.gmane.org>
2014-05-26 21:33                                         ` Grant Likely
     [not found]                                           ` <20140526213303.C1C73C40E11-WNowdnHR2B42iJbIjFUEsiwD8/FfD2ys@public.gmane.org>
2014-05-26 21:44                                             ` Geert Uytterhoeven
2014-05-26 23:47                                               ` Guenter Roeck
2014-05-27 12:11                                               ` Grant Likely
2014-05-26 22:36                                             ` Sebastian Reichel
     [not found]                                               ` <20140526223630.GA32691-SfvFxonMDyemK9LvCR3Hrw@public.gmane.org>
2014-05-26 23:42                                                 ` Guenter Roeck
     [not found]                                                   ` <5383D174.8070304-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>
2014-05-27  0:32                                                     ` Sebastian Reichel
     [not found]                                                       ` <20140527003219.GD32691-SfvFxonMDyemK9LvCR3Hrw@public.gmane.org>
2014-05-27  0:54                                                         ` Guenter Roeck
2014-05-27 12:12                                                   ` Grant Likely
     [not found]                                                     ` <20140527121236.3B818C40A5A-WNowdnHR2B42iJbIjFUEsiwD8/FfD2ys@public.gmane.org>
2014-05-27 12:24                                                       ` Pantelis Antoniou
     [not found]                                                         ` <9CF30A1A-5EB6-401E-A0D5-453CB99B4A83-OWPKS81ov/FWk0Htik3J/w@public.gmane.org>
2014-05-27 15:21                                                           ` Guenter Roeck
2014-05-27 15:38                                                             ` Pantelis Antoniou
     [not found]                                                             ` <20140527152115.GA30180-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>
2014-05-27 17:52                                                               ` Geert Uytterhoeven
2014-05-27 18:22                                                                 ` Guenter Roeck
2014-05-27 20:11                                                         ` Grant Likely
2014-05-20 12:27                           ` Pantelis Antoniou
2014-05-15 14:18           ` Grant Likely
2014-04-04 12:43 ` [PATCH v4 3/8] OF: DT-Overlay configfs interface Pantelis Antoniou
2014-04-04 12:43 ` [PATCH v4 4/8] OF: platform: Add overlay bus handler Pantelis Antoniou
2014-04-04 12:43 ` [PATCH v4 5/8] of: i2c: Export single device registration method Pantelis Antoniou
2014-04-04 12:43 ` [PATCH v4 6/8] OF: i2c: Add overlay bus handler Pantelis Antoniou
2014-04-04 12:44 ` [PATCH v4 7/8] OF: spi: " Pantelis Antoniou
2014-04-04 12:44 ` [PATCH v4 8/8] OF: selftest: Add overlay self-test support Pantelis Antoniou

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=53736D42.1010302@roeck-us.net \
    --to=linux@roeck-us.net \
    --cc=Alison_Chaiken@mentor.com \
    --cc=alexander.sverdlin@nsn.com \
    --cc=dan.malek@konsulko.com \
    --cc=delicious.quinoa@gmail.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dinh.linux@gmail.com \
    --cc=dirk.behme@gmail.com \
    --cc=geert@linux-m68k.org \
    --cc=grant.likely@secretlab.ca \
    --cc=ioan.nicu.ext@nsn.com \
    --cc=jluebbe@lasnet.de \
    --cc=koen@dominion.thruhere.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=matt.porter@linaro.org \
    --cc=mbohan@codeaurora.org \
    --cc=monstr@monstr.eu \
    --cc=mranostay@gmail.com \
    --cc=ms@mycable.de \
    --cc=pantelis.antoniou@konsulko.com \
    --cc=pete.popov@konsulko.com \
    --cc=robherring2@gmail.com \
    --cc=s.hauer@pengutronix.de \
    --cc=swarren@wwwdotorg.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).