Re: [PATCH 02/12] powerpc: wiiu: device tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Rob,
Thanks for the review.

On 3/3/22 00:36, Rob Herring wrote:
On Tue, Mar 1, 2022 at 10:44 PM Ash Logan <ash@xxxxxxxxxxxx> wrote:

Add a device tree source file for the Nintendo Wii U video game console.

Test this with 'make W=1 dtbs_checks'.

Does make W=1 ARCH=powerpc wiiu_defconfig dtbs_check seem reasonable? I ran it, and saw LINT/CHKDT/UPD/SCHEMA/COPY steps, but if I put garbage in the .dts it gives no warnings.


Signed-off-by: Ash Logan <ash@xxxxxxxxxxxx>
Co-developed-by: Roberto Van Eeden <rw-r-r-0644@xxxxxxxxxxxxxx>
Signed-off-by: Roberto Van Eeden <rw-r-r-0644@xxxxxxxxxxxxxx>
Co-developed-by: Emmanuel Gil Peyrot <linkmauve@xxxxxxxxxxxx>
Signed-off-by: Emmanuel Gil Peyrot <linkmauve@xxxxxxxxxxxx>
---
  arch/powerpc/boot/dts/wiiu.dts | 327 +++++++++++++++++++++++++++++++++
  1 file changed, 327 insertions(+)
  create mode 100644 arch/powerpc/boot/dts/wiiu.dts

diff --git a/arch/powerpc/boot/dts/wiiu.dts b/arch/powerpc/boot/dts/wiiu.dts
new file mode 100644
index 000000000000..aaf264963f61
--- /dev/null
+++ b/arch/powerpc/boot/dts/wiiu.dts
@@ -0,0 +1,327 @@
+// SPDX-License-Identifier: GPL-2.0

What about non-GPL environments?

The other powerpc dts files are all GPL-2.0(-or-later), is there a preferred license for devicetrees?

+/*
+ * Nintendo Wii U Device Tree Source
+ *
+ * Copyright (C) 2022 The linux-wiiu Team
+ */
+
+/dts-v1/;
+#include <dt-bindings/gpio/gpio.h>
+#include <dt-bindings/input/input.h>
+
+/ {
+       model = "nintendo,wiiu";
+       compatible = "nintendo,wiiu";
+
+       #address-cells = <1>;
+       #size-cells = <1>;
+
+       chosen {
+               bootargs = "root=/dev/sda1 rootwait";
+       };
+
+       memory {
+               device_type = "memory";
+               reg = <0x00000000 0x02000000    /* MEM1 - 32MiB */
+                      0x08000000 0x00300000    /* MEM0 - 3MiB  */
+                      0x10000000 0x80000000>;  /* MEM2 - 2GiB  */
+       };
+
+       cpus {
+               #address-cells = <1>;
+               #size-cells = <0>;
+
+               /* TODO: Add SMP */
+               PowerPC,espresso@0 {
+                       device_type = "cpu";
+                       reg = <0>;
+                       clock-frequency = <1243125000>;         /* 1.243125GHz */
+                       bus-frequency = <248625000>;            /* 248.625MHz core-to-bus 5x */
+                       timebase-frequency = <62156250>;        /* 1/4 of the bus clock */
+                       i-cache-size = <32768>; /* 32K icache */
+                       i-cache-line-size = <32>;
+                       i-cache-block-size = <32>;
+                       i-cache-sets = <128>;
+                       d-cache-size = <32768>; /* 32K dcache */
+                       d-cache-line-size = <32>;
+                       d-cache-block-size = <32>;
+                       d-cache-sets = <128>;
+                       next-level-cache = <&L2_0>;
+                       L2_0:l2-cache {
+                               compatible = "cache";
+                               cache-level = <2>;
+                               cache-unified;
+                               cache-size = <0x80000>; /* 512KB L2 */
+                               cache-line-size = <64>;
+                               cache-block-size = <32>;
+                               cache-sets = <2048>;
+                       };
+               };
+       };
+
+       latte {
+               #address-cells = <1>;
+               #size-cells = <1>;
+               compatible = "nintendo,latte";
+               ranges = <0x0c000000 0x0c000000 0x00400000      /* Espresso-only registers */
+                         0x0d000000 0x0d000000 0x00200000      /* Latte AHB deivces */
+                         0x0d800000 0x0d800000 0x00800000>;    /* Latte SoC registers */
+
+               gpu7@c200000 {

gpu@...

+                       compatible = "nintendo,latte-gpu7";
+                       reg = <0x0c200000 0x80000>;
+                       interrupts = <2>;
+                       interrupt-parent = <&espresso_pic>;
+               };
+
+               espresso_pic: pic@c000078 {
+                       #interrupt-cells = <1>;
+                       interrupt-controller;
+
+                       compatible = "nintendo,espresso-pic";
+                       reg = <0x0c000078 0x18>;
+               };
+
+               latte_dsp: dsp@c005000 {
+                       compatible = "nintendo,latte-dsp";
+                       reg = <0x0c005000 0x200>;
+               };
+
+               ehci_0: usb@d040000 {
+                       compatible = "nintendo,latte-usb-ehci", "usb-ehci";
+                       reg = <0x0d040000 0x100>;
+                       interrupts = <4>;
+                       interrupt-parent = <&latte_pic>;
+                       big-endian-regs;
+               };
+
+               ohci_0_0: usb@d050000 {
+                       compatible = "nintendo,latte-usb-ohci";
+                       reg = <0x0d050000 0x100>;
+                       interrupts = <5>;
+                       interrupt-parent = <&latte_pic>;
+
+                       big-endian-regs;
+               };
+
+               ohci_0_1: usb@d060000 {
+                       compatible = "nintendo,latte-usb-ohci";
+                       reg = <0x0d060000 0x100>;
+                       interrupts = <6>;
+                       interrupt-parent = <&latte_pic>;
+
+                       big-endian-regs;
+               };
+
+               ehci_1: usb@d120000 {
+                       compatible = "nintendo,latte-usb-ehci", "usb-ehci";
+                       reg = <0x0d120000 0x100>;
+                       interrupts = <16>;
+                       interrupt-parent = <&latte_pic>;
+                       big-endian-regs;
+               };
+
+               ohci_1_0: usb@d130000 {
+                       compatible = "nintendo,latte-usb-ohci";
+                       reg = <0x0d130000 0x100>;
+                       interrupts = <35>;
+                       interrupt-parent = <&latte_pic>;
+
+                       big-endian-regs;
+               };
+
+               ehci_2: usb@d140000 {
+                       compatible = "nintendo,latte-usb-ehci", "usb-ehci";
+                       reg = <0x0d140000 0x100>;
+                       interrupts = <36>;
+                       interrupt-parent = <&latte_pic>;
+                       big-endian-regs;
+               };
+
+               ohci_2_0: usb@d150000 {
+                       compatible = "nintendo,latte-usb-ohci";
+                       reg = <0x0d150000 0x100>;
+                       interrupts = <37>;
+                       interrupt-parent = <&latte_pic>;
+
+                       big-endian-regs;
+               };
+
+               sdcard_0: sdhci@d070000 {
+                       compatible = "nintendo,latte-sdhci","sdhci";
+                       reg = <0x0d070000 0x200>;
+                       interrupts = <7>;
+                       interrupt-parent = <&latte_pic>;
+               };
+
+               wifi_0: sdhci@d080000 {
+                       compatible = "nintendo,latte-sdhci","sdhci";
+                       reg = <0x0d080000 0x200>;
+                       interrupts = <8>;
+                       interrupt-parent = <&latte_pic>;
+               };
+
+               legacy_ipc: ipc@d800000 {
+                       compatible = "nintendo,latte-ipc","nintendo,hollywood-ipc";

space between compatibles needed.

+                       reg = <0x0d800000 0x10>;
+                       interrupts = <30 31>;
+                       interrupt-parent = <&latte_pic>;
+               };
+
+               latte_otp: otp@d8001ec {
+                       compatible = "nintendo,latte-otp";
+                       reg = <0x0d8001ec 0x8>;
+               };
+
+               sata: ahci@d160400 {
+                       compatible = "nintendo,latte-ahci";
+                       reg = <0x0d160400 0x808>;
+
+                       interrupt-parent = <&latte_pic>;
+                       interrupts = <38 28>;
+               };
+
+               latte_pic: pic@d800440 {
+                       #interrupt-cells = <1>;
+                       interrupt-controller;
+
+                       compatible = "nintendo,latte-pic";
+                       reg = <0x0d800440 0x30>;
+                       interrupt-parent = <&espresso_pic>;
+                       interrupts = <24>;
+               };
+
+               gpio: gpio@d8000c0 {
+                       #gpio-cells = <2>;
+                       compatible = "nintendo,latte-gpio", "nintendo,hollywood-gpio";
+
+                       reg = <0x0d8000c0 0x40>;
+                       gpio-controller;
+                       /* TODO: There are actually 31 AHBALL GPIOs */
+                       ngpios = <24>;
+
+                       gpio-line-names =
+                               "POWER", "DWIFI", "FAN", "DC_DC",
+                               "", "Esp10WorkAround", "", "",
+                               "PADPD", "", "EEP_CS", "EEP_CLK",
+                               "EEP_MOSI", "EEP_MISO", "AVE_SCL", "AVE_SDA",
+                               "DEBUG0", "DEBUG1", "DEBUG2", "DEBUG3",
+                               "DEBUG4", "DEBUG5", "DEBUG6", "DEBUG7";
+
+                       interrupt-controller;
+                       #interrupt-cells = <2>;
+                       interrupts = <10>;
+                       interrupt-parent = <&latte_pic>;
+               };
+
+               spi_gpio: spi-gpio {

Move this to root level. It's not on the 'latte' bus.

+                       compatible = "spi-gpio";
+                       #address-cells = <1>;
+                       #size-cells = <0>;
+                       status = "okay";

Not needed, that's the default.

+
+                       cs-gpios = <&gpio 10 GPIO_ACTIVE_HIGH>;
+                       gpio-sck = <&gpio 11 GPIO_ACTIVE_HIGH>;
+                       gpio-mosi = <&gpio 12 GPIO_ACTIVE_HIGH>;
+                       gpio-miso = <&gpio 13 GPIO_ACTIVE_HIGH>;
+                       num-chipselects = <1>;
+
+                       seeprom@0 {

eeprom@0

+                               compatible = "atmel,at93c66";
+                               reg = <0>;
+                               spi-max-frequency = <1000000>;
+                               spi-cs-high;
+                               /* TODO: wiiubrew says this is 16-bit, but I only get the correct
+                                * values in 8-bit...
+                                */
+                               data-size = <8>;
+                               read-only;
+
+                               #address-cells = <1>;
+                               #size-cells = <1>;
+
+                               /* https://wiiubrew.org/wiki/Hardware/SEEPROM */
+                               rng_seed: rng@12 { reg = <0x012 8>; };
+                               ppc_pvr: pvr@20 { reg = <0x020 4>; };
+                               seeprom_ver_str: sver-str@24 { reg = <0x024 6>; };
+                               seeprom_ver: sver@2a { reg = <0x02A 2>; };
+                               otp_ver: over@2c { reg = <0x02C 2>; };
+                               otp_rev: orev@2e { reg = <0x02E 2>; };
+                               otp_ver_str: over-str@30 { reg = <0x030 8>; };
+
+                               bc_crc: bc-crc@38 { reg = <0x038 4>; };
+                               bc_sz: bc-sz@3c { reg = <0x03C 2>; };
+                               bc_ver: bc-ver@3e { reg = <0x03E 2>; };
+                               bc_boardtype: boardtype@42 { reg = <0x042 2>; };
+                               bc_boardrev: boardrev@44 { reg = <0x044 2>; };
+                               bc_bootsource: bootsource@46 { reg = <0x046 2>; };
+                               bc_ddr3size: ddr3size@48 { reg = <0x048 2>; };
+                               bc_ddr3speed: ddr3speed@4a { reg = <0x04A 2>; };
+                               bc_ppcclockmul: ppcclockmul@4c { reg = <0x04C 2>; };
+                               bc_iopclockmul: iopclockmul@46 { reg = <0x04E 2>; };
+                               bc_video1080p: video1080p@50 { reg = <0x050 2>; };
+                               bc_ddr3vendor: ddr3vendor@52 { reg = <0x052 2>; };
+                               bc_movpassivereset: movpassivereset@54 { reg = <0x054 2>; };
+                               bc_syspllspd: syspllspd@56 { reg = <0x056 2>; };
+                               bc_satadevice: satadevice@58 { reg = <0x058 2>; };
+                               bc_consoletype: consoletype@5a { reg = <0x05A 2>; };
+                               bc_deviceprescence: deviceprescence@5c { reg = <0x05C 2>; };
+
+                               drive_key: drvkey@80 { reg = <0x080 16>; };
+                               factory_key: fackey@90 { reg = <0x090 16>; };
+                               shdd_key: shddkey@a0 { reg = <0x0A0 16>; };
+                               usb_key_seed: usbkeyseed@b0 { reg = <0x0B0 16>; };
+                               drive_key_flag: drvkeyf@c0 { reg = <0x0C0 2>; };
+                               usb_key_flag: udbkeyf@c2 { reg = <0x0C2 2>; };
+                               shdd_key_flag: shddkeyf@c4 { reg = <0x0C4 2>; };
+
+                               sysprod_version: sp_ver@140 { reg = <0x140 4>; };
+                               sysprod_eeprom_version: sp_ee_ver@144 { reg = <0x144 4>; };
+                               sysprod_product_area: sp_parea@148 { reg = <0x148 4>; };
+                               sysprod_game_region: sp_region@14c { reg = <0x14C 4>; };
+                               sysprod_ntsc_pal: sp_ntscpal@150 { reg = <0x150 4>; };
+                               sysprod_5ghz_country: sp_5ghz_c@154 { reg = <0x154 2>; };
+                               sysprod_5ghz_country_rev: sp_5ghz_crev@156 { reg = <0x156 2>; };
+                               sysprod_code: sp_code@158 { reg = <0x158 8>; };
+                               sysprod_serial: sp_serial@160 { reg = <0x160 16>; };
+                               sysprod_model: sp_model@170 { reg = <0x170 16>; };
+
+                               prod_year: pyear@188 { reg = <0x188 2>; };
+                               prod_date: pdate@18a { reg = <0x18A 2>; };
+                               prod_time: ptime@18c { reg = <0x18C 2>; };
+
+                               boot_params: boot_params@1c0 { reg = <0x1C0 48>; };
+                       };
+               };
+
+               /* TODO make this gpio-keyed once hollywood-gpio supports interrupts */
+               gpio-keys-polled {

This too.

+                       poll-interval = <50>;
+                       compatible = "gpio-keys-polled";
+
+                       power {
+                               label = "Power Button";
+                               gpios = <&gpio 0 GPIO_ACTIVE_HIGH>;
+                               linux,code = <KEY_POWER>;
+                       };
+               };
+
+               gpio2: gpio2@d8000c0 {

gpio@d800520

+                       #gpio-cells = <2>;
+                       compatible = "nintendo,latte-gpio", "nintendo,hollywood-gpio";
+
+                       reg = <0x0d800520 0x40>;
+                       gpio-controller;
+                       ngpios = <7>;
+
+                       gpio-line-names =
+                               "FANSPEED", "SMC_SCL", "SMC_SDA", "DC_DC2",
+                               "AVE_INT", "", "AVE_RESET";
+
+                       interrupt-controller;
+                       #interrupt-cells = <2>;
+                       interrupts = <10>;
+                       interrupt-parent = <&latte_pic>;
+               };
+       };
+};
--
2.35.1


Will do in v2.

Thanks,
Ash



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux