* [RFC/PATCH] drivers/of/platform: Add powerpc 4xx embedded busses to default list
@ 2017-09-01 3:51 Benjamin Herrenschmidt
2017-09-01 5:10 ` Greg Kroah-Hartman
2017-09-01 15:24 ` Rob Herring
0 siblings, 2 replies; 6+ messages in thread
From: Benjamin Herrenschmidt @ 2017-09-01 3:51 UTC (permalink / raw)
To: linux-kernel; +Cc: Greg Kroah-Hartman, Rob Herring, devicetree
This allow to (slowly) migrate those embedded platforms
to of_platform_default_populate()
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
---
I'm here to collect acks (or comments :-) I'd like this to go via
the powerpc tree along with the patches converting some of the
platforms. I'll be adding more bus types if/when I start tackling
other powerpc embedded families but for now I'm dealing with 4xx.
drivers/of/platform.c | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/drivers/of/platform.c b/drivers/of/platform.c
index b19524623498..f3f2aab4eda1 100644
--- a/drivers/of/platform.c
+++ b/drivers/of/platform.c
@@ -34,6 +34,12 @@ const struct of_device_id of_default_bus_match_table[] = {
#ifdef CONFIG_ARM_AMBA
{ .compatible = "arm,amba-bus", },
#endif /* CONFIG_ARM_AMBA */
+#ifdef CONFIG_PPC
+ { .compatible = "ibm,plb3", },
+ { .compatible = "ibm,plb4", },
+ { .compatible = "ibm,opb", },
+ { .compatible = "ibm,ebc", },
+#endif
{} /* Empty terminated list */
};
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [RFC/PATCH] drivers/of/platform: Add powerpc 4xx embedded busses to default list
2017-09-01 3:51 [RFC/PATCH] drivers/of/platform: Add powerpc 4xx embedded busses to default list Benjamin Herrenschmidt
@ 2017-09-01 5:10 ` Greg Kroah-Hartman
2017-09-01 15:24 ` Rob Herring
1 sibling, 0 replies; 6+ messages in thread
From: Greg Kroah-Hartman @ 2017-09-01 5:10 UTC (permalink / raw)
To: Benjamin Herrenschmidt; +Cc: linux-kernel, Rob Herring, devicetree
On Fri, Sep 01, 2017 at 01:51:01PM +1000, Benjamin Herrenschmidt wrote:
> This allow to (slowly) migrate those embedded platforms
> to of_platform_default_populate()
>
> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
> ---
>
> I'm here to collect acks (or comments :-) I'd like this to go via
> the powerpc tree along with the patches converting some of the
> platforms. I'll be adding more bus types if/when I start tackling
> other powerpc embedded families but for now I'm dealing with 4xx.
>
> drivers/of/platform.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [RFC/PATCH] drivers/of/platform: Add powerpc 4xx embedded busses to default list
2017-09-01 3:51 [RFC/PATCH] drivers/of/platform: Add powerpc 4xx embedded busses to default list Benjamin Herrenschmidt
2017-09-01 5:10 ` Greg Kroah-Hartman
@ 2017-09-01 15:24 ` Rob Herring
2017-09-01 23:47 ` Benjamin Herrenschmidt
1 sibling, 1 reply; 6+ messages in thread
From: Rob Herring @ 2017-09-01 15:24 UTC (permalink / raw)
To: Benjamin Herrenschmidt
Cc: linux-kernel@vger.kernel.org, Greg Kroah-Hartman,
devicetree@vger.kernel.org
On Thu, Aug 31, 2017 at 10:51 PM, Benjamin Herrenschmidt
<benh@kernel.crashing.org> wrote:
> This allow to (slowly) migrate those embedded platforms
> to of_platform_default_populate()
>
> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
> ---
>
> I'm here to collect acks (or comments :-) I'd like this to go via
> the powerpc tree along with the patches converting some of the
> platforms. I'll be adding more bus types if/when I start tackling
> other powerpc embedded families but for now I'm dealing with 4xx.
Glad to see it.
Acked-by: Rob Herring <robh@kernel.org>
Rob
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [RFC/PATCH] drivers/of/platform: Add powerpc 4xx embedded busses to default list
2017-09-01 15:24 ` Rob Herring
@ 2017-09-01 23:47 ` Benjamin Herrenschmidt
2017-09-05 14:25 ` Rob Herring
0 siblings, 1 reply; 6+ messages in thread
From: Benjamin Herrenschmidt @ 2017-09-01 23:47 UTC (permalink / raw)
To: Rob Herring
Cc: linux-kernel@vger.kernel.org, Greg Kroah-Hartman,
devicetree@vger.kernel.org
On Fri, 2017-09-01 at 10:24 -0500, Rob Herring wrote:
> On Thu, Aug 31, 2017 at 10:51 PM, Benjamin Herrenschmidt
> <benh@kernel.crashing.org> wrote:
> > This allow to (slowly) migrate those embedded platforms
> > to of_platform_default_populate()
> >
> > Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
> > ---
> >
> > I'm here to collect acks (or comments :-) I'd like this to go via
> > the powerpc tree along with the patches converting some of the
> > platforms. I'll be adding more bus types if/when I start tackling
> > other powerpc embedded families but for now I'm dealing with 4xx.
>
> Glad to see it.
So my end game is to remove the #ifndef CONFIG_PPC around
of_platform_default_populate_init(void). However, for that
to work, I need to add a way to disable that on some platforms.
However, it might take time, especially when it comes to dealing
with the old Macs or some more obscure embedded platforms.
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.
Cheers,
Ben.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [RFC/PATCH] drivers/of/platform: Add powerpc 4xx embedded busses to default list
2017-09-01 23:47 ` Benjamin Herrenschmidt
@ 2017-09-05 14:25 ` Rob Herring
2017-09-05 22:00 ` Benjamin Herrenschmidt
0 siblings, 1 reply; 6+ messages in thread
From: Rob Herring @ 2017-09-05 14:25 UTC (permalink / raw)
To: Benjamin Herrenschmidt
Cc: linux-kernel@vger.kernel.org, Greg Kroah-Hartman,
devicetree@vger.kernel.org
On Fri, Sep 1, 2017 at 6:47 PM, Benjamin Herrenschmidt
<benh@kernel.crashing.org> wrote:
> On Fri, 2017-09-01 at 10:24 -0500, Rob Herring wrote:
>> On Thu, Aug 31, 2017 at 10:51 PM, Benjamin Herrenschmidt
>> <benh@kernel.crashing.org> wrote:
>> > This allow to (slowly) migrate those embedded platforms
>> > to of_platform_default_populate()
>> >
>> > Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
>> > ---
>> >
>> > I'm here to collect acks (or comments :-) I'd like this to go via
>> > the powerpc tree along with the patches converting some of the
>> > platforms. I'll be adding more bus types if/when I start tackling
>> > other powerpc embedded families but for now I'm dealing with 4xx.
>>
>> Glad to see it.
>
> So my end game is to remove the #ifndef CONFIG_PPC around
> of_platform_default_populate_init(void). However, for that
> to work, I need to add a way to disable that on some platforms.
>
> However, it might take time, especially when it comes to dealing
> with the old Macs or some more obscure embedded platforms.
>
> 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.
Rob
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [RFC/PATCH] drivers/of/platform: Add powerpc 4xx embedded busses to default list
2017-09-05 14:25 ` Rob Herring
@ 2017-09-05 22:00 ` Benjamin Herrenschmidt
0 siblings, 0 replies; 6+ messages in thread
From: Benjamin Herrenschmidt @ 2017-09-05 22:00 UTC (permalink / raw)
To: Rob Herring
Cc: linux-kernel@vger.kernel.org, Greg Kroah-Hartman,
devicetree@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.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-09-05 22:00 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-09-01 3:51 [RFC/PATCH] drivers/of/platform: Add powerpc 4xx embedded busses to default list Benjamin Herrenschmidt
2017-09-01 5:10 ` Greg Kroah-Hartman
2017-09-01 15:24 ` Rob Herring
2017-09-01 23:47 ` Benjamin Herrenschmidt
2017-09-05 14:25 ` Rob Herring
2017-09-05 22:00 ` Benjamin Herrenschmidt
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox