Darren wrote:
... the ARM64 guys added a default platform_bus_init routine which was preventing any
later platform bus calls from working, in the X1000's case this stopped the
sdc bus, which has the ethernet phy on, and the local bus which has the CF
on.
On checking my email, which was building up while I was at work this week, a
similar fix to mine has been applied else where.
*** Forwarded message, originally written by Rob Herring on 12/08/2016 ***
On Fri, Aug 12, 2016 at 12:49 AM, Kevin Hao <
haokexin@gmail.com> wrote:
> On Fri, Aug 12, 2016 at 02:39:32PM +1000, Michael Ellerman wrote:
>> Kevin Hao <
haokexin@gmail.com> 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 behaviour on ppc?
>
> Sure. How about this one?
>
> From 4362b4cdd8a6198df4cc46c628473f0d44e03fa8 Mon Sep 17 00:00:00 2001
> From: Kevin Hao <
haokexin@gmail.com>
> Date: Fri, 12 Aug 2016 13:30:03 +0800
> Subject: [PATCH v2] of/platform: disable the
> of_platform_default_populate_init() for all the ppc boards
>
> 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 lot of ppc boards use machine_device_initcall() to
> populate the default bus. This means that the default populate function
> has higher priority and would override the arch specific population of
> the bus. The side effect is that some arch specific bus are not probed,
> then cause various malfunction due to the miss of some devices. Since
> it is very possible to introduce bugs if we simply change the initcall
> level for all these boards(about 30+). This just disable this default
> function for all the ppc boards.
>
> Signed-off-by: Kevin Hao <
haokexin@gmail.com>
> ---
> drivers/of/platform.c | 2 ++
> 1 file changed, 2 insertions(+)
I've applied this one.
Rob
*** End of forwarded message ***
Kind regards
Darren