Re: [PATCH 4/4] powerpc/86xx: Move gianfar mdio nodes under the ethernet nodes

[Martyn Welch <martyn.welch@gefanuc.com>]

Anton Vorontsov wrote:
> Currently it doesn't matter where the mdio nodes are placed, but with
> power management support (i.e. when sleep = <> properties will take
> effect), mdio nodes placement will become important: mdio controller
> is a part of the ethernet block, so the mdio nodes should be placed
> correctly. Otherwise we may wrongly assume that MDIO controllers are
> available during sleep.
> 
> Suggested-by: Scott Wood <scottwood@freescale.com>
> Suggested-by: Kumar Gala <galak@kernel.crashing.org>
> Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com>
> ---
>  arch/powerpc/boot/dts/gef_ppc9a.dts        |   39 ++++---
>  arch/powerpc/boot/dts/gef_sbc310.dts       |   39 ++++---
>  arch/powerpc/boot/dts/gef_sbc610.dts       |   39 ++++---
>  arch/powerpc/boot/dts/mpc8641_hpcn.dts     |  157 +++++++++++++++-------------
>  arch/powerpc/boot/dts/sbc8641d.dts         |  156 +++++++++++++++-------------
>  arch/powerpc/platforms/86xx/gef_ppc9a.c    |    1 +
>  arch/powerpc/platforms/86xx/gef_sbc310.c   |    1 +
>  arch/powerpc/platforms/86xx/gef_sbc610.c   |    1 +
>  arch/powerpc/platforms/86xx/mpc8610_hpcd.c |    1 +
>  arch/powerpc/platforms/86xx/mpc86xx_hpcn.c |    1 +
>  arch/powerpc/platforms/86xx/sbc8641d.c     |    1 +
>  11 files changed, 237 insertions(+), 199 deletions(-)
> 

Tested on ppc9a. Other GE Fanuc boards look sane.

Tested-by: Martyn Welch <martyn.welch@gefanuc.com>

Martyn

-- 
Martyn Welch MEng MPhil MIET (Principal Software Engineer)   T:+44(0)1327322748
GE Fanuc Intelligent Platforms Ltd,        |Registered in England and Wales
Tove Valley Business Park, Towcester,      |(3828642) at 100 Barbirolli Square,
Northants, NN12 6PF, UK T:+44(0)1327359444 |Manchester,M2 3AB  VAT:GB 729849476