arm64: dts: broadcom: bcm4908: use proper TWD binding

[ Upstream commit 33826e9c6b ]

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: 2961f69f15 ("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>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
Rafał Miłecki 2021-12-29 11:23:14 +01:00 committed by Greg Kroah-Hartman
parent cc3a67e414
commit 17790ef9ad

View file

@ -273,9 +273,9 @@ bus@ff800000 {
#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 @@ reset-controller@2644 {
reboot {
compatible = "syscon-reboot";
regmap = <&timer>;
regmap = <&twd>;
offset = <0x34>;
mask = <1>;
};