On 03.05.19 19:43, Philippe Reynes wrote:
The watchdog should use a clock at 50 Mhz, so
instead of using the clock osc (200 Mhz), we
define a reference clock at 50Mhz and use it
for both watchdog.

Signed-off-by: Philippe Reynes <philippe.rey...@softathome.com>

Just curious: Why is this the case? Is this also what's done in
the Linux DT version?

Other than that:

Reviewed-by: Stefan Roese <s...@denx.de>

Thanks,
Stefan

---
  arch/arm/dts/bcm63158.dtsi | 10 ++++++++--
  1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/arch/arm/dts/bcm63158.dtsi b/arch/arm/dts/bcm63158.dtsi
index 4b2eaee..175af38 100644
--- a/arch/arm/dts/bcm63158.dtsi
+++ b/arch/arm/dts/bcm63158.dtsi
@@ -66,6 +66,12 @@
                        clock-frequency = <0xbebc200>;
                        u-boot,dm-pre-reloc;
                };
+
+               refclk50mhz: refclk50mhz {
+                       compatible = "fixed-clock";
+                       #clock-cells = <0>;
+                       clock-frequency = <50000000>;
+               };
        };
ubus {
@@ -92,13 +98,13 @@
                wdt1: watchdog@ff800480 {
                        compatible = "brcm,bcm6345-wdt";
                        reg = <0x0 0xff800480 0x0 0x14>;
-                       clocks = <&periph_osc>;
+                       clocks = <&refclk50mhz>;
                };
wdt2: watchdog@ff8004c0 {
                        compatible = "brcm,bcm6345-wdt";
                        reg = <0x0 0xff8004c0 0x0 0x14>;
-                       clocks = <&periph_osc>;
+                       clocks = <&refclk50mhz>;
                };
wdt-reboot {


Viele Grüße,
Stefan

--
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: s...@denx.de
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to