This is wrong, but needed in order to avoid overlapping ranges with the
OTP area added in the next commit.  A refactor of this part of the
device tree is needed: according to Wiibrew[1], this area starts at
0x0d800000 and spans 0x400 bytes (that is, 0x100 32-bit registers),
encompassing PIC and GPIO registers, amongst the ones already exposed in
this device tree, which should become children of the control@d800000
node.

[1] https://wiibrew.org/wiki/Hardware/Hollywood_Registers

Signed-off-by: Emmanuel Gil Peyrot <linkma...@linkmauve.fr>
---
 arch/powerpc/boot/dts/wii.dts | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/arch/powerpc/boot/dts/wii.dts b/arch/powerpc/boot/dts/wii.dts
index aaa381da1906..c5fb54f8cc02 100644
--- a/arch/powerpc/boot/dts/wii.dts
+++ b/arch/powerpc/boot/dts/wii.dts
@@ -216,7 +216,13 @@ AVE: audio-video-encoder@70 {
 
                control@d800100 {
                        compatible = "nintendo,hollywood-control";
-                       reg = <0x0d800100 0x300>;
+                       /*
+                        * Both the address and length are wrong, according to
+                        * Wiibrew this should be <0x0d800000 0x400>, but it
+                        * requires refactoring the PIC1 and GPIO nodes before
+                        * changing that.
+                        */
+                       reg = <0x0d800100 0xa0>;
                };
 
                disk@d806000 {
-- 
2.32.0

Reply via email to