All of lore.kernel.org
 help / color / mirror / Atom feed
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Rob Herring <robh@kernel.org>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>
Subject: Re: [RFC/PATCH] drivers/of/platform: Add powerpc 4xx embedded busses to default list
Date: Wed, 06 Sep 2017 08:00:08 +1000	[thread overview]
Message-ID: <1504648808.4073.15.camel@kernel.crashing.org> (raw)
In-Reply-To: <CAL_JsqKv-nU7Am=Jz+PFnmpiXWfwscHNFdb0KfYdROOHmTrSHg@mail.gmail.com>

On Tue, 2017-09-05 at 09:25 -0500, Rob Herring wrote:
> > So in the meantime, I'm thinking adding some kind of runtime way
> > of disabling this default populate. Would you be ok with that ?
> > 
> > Something like:
> > 
> >          bool arch_wants_of_platform_defaults(void)
> > 
> > With a weak implementation returning true.
> 
> That seems fine. The intention was that you can call
> of_platform_default_populate multiple times with arch specific match
> lists and the default. There could be ordering issues both with
> initcall levels and device creation ordering.

Right but I want to skip the default completely on things like old macs
because they manually create their own cherry-picks and I worry about
conflicts here. Also Apple has history of random crap in the root of
the DT.

Cheers,
Ben.

      reply	other threads:[~2017-09-05 22:00 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-09-01  3:51 [RFC/PATCH] drivers/of/platform: Add powerpc 4xx embedded busses to default list Benjamin Herrenschmidt
2017-09-01  3:51 ` Benjamin Herrenschmidt
     [not found] ` <1504237861.4974.59.camel-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
2017-09-01  5:10   ` Greg Kroah-Hartman
2017-09-01  5:10     ` Greg Kroah-Hartman
2017-09-01 15:24 ` Rob Herring
     [not found]   ` <CAL_JsqLYMeV6XsBOtZKJHFbC2+_cG+gF7L+LurH5keTtzp2wVQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-09-01 23:47     ` Benjamin Herrenschmidt
2017-09-01 23:47       ` Benjamin Herrenschmidt
     [not found]       ` <1504309672.4974.93.camel-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
2017-09-05 14:25         ` Rob Herring
2017-09-05 14:25           ` Rob Herring
2017-09-05 22:00           ` Benjamin Herrenschmidt [this message]

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=1504648808.4073.15.camel@kernel.crashing.org \
    --to=benh@kernel.crashing.org \
    --cc=devicetree@vger.kernel.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=robh@kernel.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.