diff mbox series

ARM: dts: ast2600: Make WDT by default disabled

Message ID 20210916061009.10382-1-chiawei_wang@aspeedtech.com
State Accepted
Commit 8e1ebdcfd71a089a5aa7a69664555a75153a9b9e
Delegated to: Tom Rini
Headers show
Series ARM: dts: ast2600: Make WDT by default disabled | expand

Commit Message

ChiaWei Wang Sept. 16, 2021, 6:10 a.m. UTC
The WDT devices described in the general .dtsi file
should be marked as "disabled" by default.

A WDT should be then enabled in the board specific
.dts file on demands.

Signed-off-by: Chia-Wei Wang <chiawei_wang@aspeedtech.com>
---
 arch/arm/dts/ast2600.dtsi | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Ryan Chen Sept. 16, 2021, 7:11 a.m. UTC | #1
> -----Original Message-----
> From: ChiaWei Wang <chiawei_wang@aspeedtech.com>
> Sent: Thursday, September 16, 2021 2:10 PM
> To: u-boot@lists.denx.de
> Cc: Ryan Chen <ryan_chen@aspeedtech.com>
> Subject: [PATCH] ARM: dts: ast2600: Make WDT by default disabled
> 
> The WDT devices described in the general .dtsi file should be marked as
> "disabled" by default.
> 
> A WDT should be then enabled in the board specific .dts file on demands.
> 
> Signed-off-by: Chia-Wei Wang <chiawei_wang@aspeedtech.com>
Reviewed-by: Ryan Chen <ryan_chen@aspeedtech.com>
> ---
>  arch/arm/dts/ast2600.dtsi | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm/dts/ast2600.dtsi b/arch/arm/dts/ast2600.dtsi index
> ac0f08b7ea..f121f547e6 100644
> --- a/arch/arm/dts/ast2600.dtsi
> +++ b/arch/arm/dts/ast2600.dtsi
> @@ -474,21 +474,25 @@
>  			wdt1: watchdog@1e785000 {
>  				compatible = "aspeed,ast2600-wdt";
>  				reg = <0x1e785000 0x40>;
> +				status = "disabled";
>  			};
> 
>  			wdt2: watchdog@1e785040 {
>  				compatible = "aspeed,ast2600-wdt";
>  				reg = <0x1e785040 0x40>;
> +				status = "disabled";
>  			};
> 
>  			wdt3: watchdog@1e785080 {
>  				compatible = "aspeed,ast2600-wdt";
>  				reg = <0x1e785080 0x40>;
> +				status = "disabled";
>  			};
> 
>  			wdt4: watchdog@1e7850C0 {
>  				compatible = "aspeed,ast2600-wdt";
>  				reg = <0x1e7850C0 0x40>;
> +				status = "disabled";
>  			};
> 
>  			lpc: lpc@1e789000 {
> --
> 2.17.1
Tom Rini Oct. 3, 2021, 11:33 p.m. UTC | #2
On Thu, Sep 16, 2021 at 02:10:09PM +0800, Chia-Wei Wang wrote:

> The WDT devices described in the general .dtsi file
> should be marked as "disabled" by default.
> 
> A WDT should be then enabled in the board specific
> .dts file on demands.
> 
> Signed-off-by: Chia-Wei Wang <chiawei_wang@aspeedtech.com>
> Reviewed-by: Ryan Chen <ryan_chen@aspeedtech.com>

Applied to u-boot/next, thanks!
diff mbox series

Patch

diff --git a/arch/arm/dts/ast2600.dtsi b/arch/arm/dts/ast2600.dtsi
index ac0f08b7ea..f121f547e6 100644
--- a/arch/arm/dts/ast2600.dtsi
+++ b/arch/arm/dts/ast2600.dtsi
@@ -474,21 +474,25 @@ 
 			wdt1: watchdog@1e785000 {
 				compatible = "aspeed,ast2600-wdt";
 				reg = <0x1e785000 0x40>;
+				status = "disabled";
 			};
 
 			wdt2: watchdog@1e785040 {
 				compatible = "aspeed,ast2600-wdt";
 				reg = <0x1e785040 0x40>;
+				status = "disabled";
 			};
 
 			wdt3: watchdog@1e785080 {
 				compatible = "aspeed,ast2600-wdt";
 				reg = <0x1e785080 0x40>;
+				status = "disabled";
 			};
 
 			wdt4: watchdog@1e7850C0 {
 				compatible = "aspeed,ast2600-wdt";
 				reg = <0x1e7850C0 0x40>;
+				status = "disabled";
 			};
 
 			lpc: lpc@1e789000 {