| From 33826e9c6ba76b265d4e26cb95493fa27ed78974 Mon Sep 17 00:00:00 2001 |
| From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <rafal@milecki.pl> |
| Date: Wed, 29 Dec 2021 11:23:14 +0100 |
| Subject: [PATCH] arm64: dts: broadcom: bcm4908: use proper TWD binding |
| MIME-Version: 1.0 |
| Content-Type: text/plain; charset=UTF-8 |
| Content-Transfer-Encoding: 8bit |
| |
| Block at <ff800400 0x4c> is a TWD that contains timers, watchdog and |
| reset. Actual timers happen to be at block beginning but they only span |
| across the first 0x28 registers. It means the old block description was |
| incorrect (size 0x3c). |
| |
| Drop timers binding for now and use documented TWD binding. Timers |
| should be properly documented and defined as TWD subnode. |
| |
| Fixes: 2961f69f151c ("arm64: dts: broadcom: add BCM4908 and Asus GT-AC5300 early DTS files") |
| Signed-off-by: Rafał Miłecki <rafal@milecki.pl> |
| Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> |
| --- |
| arch/arm64/boot/dts/broadcom/bcm4908/bcm4908.dtsi | 8 ++++---- |
| 1 file changed, 4 insertions(+), 4 deletions(-) |
| |
| --- a/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908.dtsi |
| +++ b/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908.dtsi |
| @@ -273,9 +273,9 @@ |
| #size-cells = <1>; |
| ranges = <0x00 0x00 0xff800000 0x3000>; |
| |
| - timer: timer@400 { |
| - compatible = "brcm,bcm6328-timer", "syscon"; |
| - reg = <0x400 0x3c>; |
| + twd: timer-mfd@400 { |
| + compatible = "brcm,bcm4908-twd", "simple-mfd", "syscon"; |
| + reg = <0x400 0x4c>; |
| }; |
| |
| gpio0: gpio-controller@500 { |
| @@ -330,7 +330,7 @@ |
| |
| reboot { |
| compatible = "syscon-reboot"; |
| - regmap = <&timer>; |
| + regmap = <&twd>; |
| offset = <0x34>; |
| mask = <1>; |
| }; |