[U-Boot] [PATCH 3/3] dt: bcm63158: watchdog should use a 50Mhz clock
Stefan Roese
sr at denx.de
Mon May 6 07:45:12 UTC 2019
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.reynes at 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 <sr at 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 at ff800480 {
> compatible = "brcm,bcm6345-wdt";
> reg = <0x0 0xff800480 0x0 0x14>;
> - clocks = <&periph_osc>;
> + clocks = <&refclk50mhz>;
> };
>
> wdt2: watchdog at 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: sr at denx.de
More information about the U-Boot
mailing list