[RFC PATCH 04/19] powerpc: wii: device tree

[Grant Likely]

On Sun, Nov 22, 2009 at 3:01 PM, Albert Herranz <albert_herranz at yahoo.es> wrote:
> Add a device tree source file for the Nintendo Wii video game console.
>
> Signed-off-by: Albert Herranz <albert_herranz at yahoo.es>

Same comments apply here as for the gamecube.dts file, plus a few more below.

Cheers,
g.

> ---
>  arch/powerpc/boot/dts/wii.dts |  244 +++++++++++++++++++++++++++++++++++++++++
>  1 files changed, 244 insertions(+), 0 deletions(-)
>  create mode 100644 arch/powerpc/boot/dts/wii.dts
>
> diff --git a/arch/powerpc/boot/dts/wii.dts b/arch/powerpc/boot/dts/wii.dts
> new file mode 100644
> index 0000000..a30a804
> --- /dev/null
> +++ b/arch/powerpc/boot/dts/wii.dts
> @@ -0,0 +1,244 @@
> +/*
> + * arch/powerpc/boot/dts/wii.dts
> + *
> + * Nintendo Wii platform device tree source
> + * Copyright (C) 2008-2009 The GameCube Linux Team
> + * Copyright (C) 2008,2009 Albert Herranz
> + *
> + * This program is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU General Public License
> + * as published by the Free Software Foundation; either version 2
> + * of the License, or (at your option) any later version.
> + *
> + */
> +
> +/dts-v1/;
> +
> +/memreserve/ 0x01800000 0xe800000;     /* memory hole (includes I/O area) */
> +/memreserve/ 0x10000000 0x0004000;     /* DSP RAM */

This bothers me... see below.

> +
> +/ {
> +       model = "NintendoWii";
> +       compatible = "nintendo,wii";
> +       #address-cells = <1>;
> +       #size-cells = <1>;
> +
> +       chosen {
> +               /* root filesystem on 2nd partition of SD card */
> +               bootargs = "nobats root=/dev/mmcblk0p2 rootwait udbg-immortal";
> +               linux,stdout-path = &USBGECKO0;
> +       };
> +
> +       aliases {
> +               ugecon = &USBGECKO0;
> +               hw_gpio = &gpio1;
> +       };
> +
> +       /*
> +        * The Nintendo Wii has two discontiguous RAM memory areas called
> +        * MEM1 and MEM2.
> +        * MEM1 starts at 0x00000000 and contains 24MB of 1T-SRAM.
> +        * MEM2 starts at 0x10000000 and contains 64MB of DDR2 RAM.
> +        * Between both memory address ranges there is an address space
> +        * where memory-mapped I/O registers are found.
> +        *
> +        * Currently, Linux 32-bit PowerPC does not support RAM in
> +        * discontiguous memory address spaces. Thus, in order to use
> +        * both RAM areas, we declare as RAM the range from the start of
> +        * MEM1 to the end of useable MEM2 and exclude the needed parts
> +        * with /memreserve/ statements, at the expense of wasting a bit
> +        * of memory.

Hmmm.  It's not great practice to lie about hardware in the device
tree.  Better to describe the memory correctly here, and if you have
to work around Linux deficiencies, then do so in the platform support
code (arch/platforms/*).  I won't NAK the patch over it (feel free to
add my Acked-by line) because it doesn't impact other platforms, but
it should be fixed.

> +               i2c-video {
> +                       #address-cells = <1>;
> +                       #size-cells = <0>;
> +                       compatible = "virtual,i2c-gpio";

There isn't a documented binding for this.  Is there a driver for it?

> +
> +                       gpios = <&gpio0  16 0 /* 31-15 */
> +                                &gpio0  17 0 /* 31-14 */
> +                               >;
> +                       sda-is-open-drain = <1>;
> +                       sda-enforce-dir = <1>;
> +                       scl-is-open-drain = <1>;
> +                       scl-is-output-only = <1>;
> +                       udelay = <2>;
> +
> +                       audio-video-encoder {
> +                               compatible = "nintendo,wii-ave-rvl";
> +                               reg = <0x70>;
> +                       };
> +               };
> +       };
> +};
> +
> --
> 1.6.3.3
>
> _______________________________________________
> Linuxppc-dev mailing list
> Linuxppc-dev at lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/linuxppc-dev
>



-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.