From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1754228AbdIEWA0 (ORCPT <rfc822;w@1wt.eu>);
        Tue, 5 Sep 2017 18:00:26 -0400
Received: from gate.crashing.org ([63.228.1.57]:49796 "EHLO gate.crashing.org"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S1754177AbdIEWAW (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
        Tue, 5 Sep 2017 18:00:22 -0400
Message-ID: <1504648808.4073.15.camel@kernel.crashing.org>
Subject: Re: [RFC/PATCH] drivers/of/platform: Add powerpc 4xx embedded
 busses to default list
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>
Date: Wed, 06 Sep 2017 08:00:08 +1000
In-Reply-To: <CAL_JsqKv-nU7Am=Jz+PFnmpiXWfwscHNFdb0KfYdROOHmTrSHg@mail.gmail.com>
References: <1504237861.4974.59.camel@kernel.crashing.org>
         <CAL_JsqLYMeV6XsBOtZKJHFbC2+_cG+gF7L+LurH5keTtzp2wVQ@mail.gmail.com>
         <1504309672.4974.93.camel@kernel.crashing.org>
         <CAL_JsqKv-nU7Am=Jz+PFnmpiXWfwscHNFdb0KfYdROOHmTrSHg@mail.gmail.com>
Content-Type: text/plain; charset="UTF-8"
X-Mailer: Evolution 3.24.5 (3.24.5-1.fc26) 
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

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.