Re: [PATCH] powerpc: populate the default bus with machine_arch_initcall

2016-08-14 Thread Benjamin Herrenschmidt
On Sun, 2016-08-14 at 17:42 -0500, Rob Herring wrote: > > Not fan of the hard #ifdef at all... it will make it hard to > convert > > platforms one by one. Why not an arch_want_default_of_probe() or > > something like this which we can then plumb into ppc_md. ? > > Yeah, I'm not either really. I di

Re: [PATCH] powerpc: populate the default bus with machine_arch_initcall

2016-08-14 Thread Rob Herring
On Sun, Aug 14, 2016 at 4:21 PM, Benjamin Herrenschmidt wrote: > On Fri, 2016-08-12 at 14:30 -0500, Rob Herring wrote: >> >> > With the commit 44a7185c2ae6 ("of/platform: Add common method to >> > populate default bus"), a default function is introduced to >> > populate >> > the default bus and th

Re: [PATCH] powerpc: populate the default bus with machine_arch_initcall

2016-08-14 Thread Benjamin Herrenschmidt
On Fri, 2016-08-12 at 14:30 -0500, Rob Herring wrote: >  > > With the commit 44a7185c2ae6 ("of/platform: Add common method to > > populate default bus"), a default function is introduced to > > populate > > the default bus and this function is invoked at the > > arch_initcall_sync > > level. But a

Re: [PATCH] powerpc: populate the default bus with machine_arch_initcall

2016-08-12 Thread Rob Herring
On Fri, Aug 12, 2016 at 12:49 AM, Kevin Hao wrote: > On Fri, Aug 12, 2016 at 02:39:32PM +1000, Michael Ellerman wrote: >> Kevin Hao writes: [...] >> Every one of those initcall changes could be introducing a bug, by >> changing the order vs other init calls. >> >> Can we just go back to the old

Re: [PATCH] powerpc: populate the default bus with machine_arch_initcall

2016-08-11 Thread Kevin Hao
On Fri, Aug 12, 2016 at 02:39:32PM +1000, Michael Ellerman wrote: > Kevin Hao writes: > > > With the commit 44a7185c2ae6 ("of/platform: Add common method to > > populate default bus"), a default function is introduced to populate > > the default bus and this function is invoked at the arch_initca

Re: [PATCH] powerpc: populate the default bus with machine_arch_initcall

2016-08-11 Thread Michael Ellerman
Kevin Hao writes: > With the commit 44a7185c2ae6 ("of/platform: Add common method to > populate default bus"), a default function is introduced to populate > the default bus and this function is invoked at the arch_initcall_sync > level. This will override the arch specific population of default

Re: [PATCH] powerpc: populate the default bus with machine_arch_initcall

2016-08-11 Thread Kevin Hao
On Thu, Aug 11, 2016 at 08:17:52AM -0500, Rob Herring wrote: > On Thu, Aug 11, 2016 at 6:09 AM, Kevin Hao wrote: > > With the commit 44a7185c2ae6 ("of/platform: Add common method to > > populate default bus"), a default function is introduced to populate > > the default bus and this function is in

Re: [PATCH] powerpc: populate the default bus with machine_arch_initcall

2016-08-11 Thread Rob Herring
On Thu, Aug 11, 2016 at 6:09 AM, Kevin Hao wrote: > With the commit 44a7185c2ae6 ("of/platform: Add common method to > populate default bus"), a default function is introduced to populate > the default bus and this function is invoked at the arch_initcall_sync > level. This will override the arch

[PATCH] powerpc: populate the default bus with machine_arch_initcall

2016-08-11 Thread Kevin Hao
With the commit 44a7185c2ae6 ("of/platform: Add common method to populate default bus"), a default function is introduced to populate the default bus and this function is invoked at the arch_initcall_sync level. This will override the arch specific population of default bus which run at a lower lev