Skip to contentSkip to conversation

aspeed: watchdog: Set bootstatus during probe

PATCH In review Eddie James 2 participants 2 messages Mar 27, 2018

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH v2 1/1 aspeed: watchdog: Set bootstatus during probe

Check the aspeed timeout status register to see if the system has booted from the secondary boot source. If so, set the watchdog device bootstatus flag for "Card previously reset the CPU."

---
Changes since v1:
- use bootstatus instead of status
- use WDIOF_CARDRESET instead of raw register contents

 drivers/watchdog/aspeed_wdt.c | 7 +++++++
 1 file changed, 7 insertions(+)
drivers/watchdog/aspeed_wdt.c +7 -0
index a5b8eb2..1abe4d0 100644
@@ -55,6 +55,8 @@ struct aspeed_wdt_config {
 #define   WDT_CTRL_WDT_INTR		BIT(2)
 #define   WDT_CTRL_RESET_SYSTEM		BIT(1)
 #define   WDT_CTRL_ENABLE		BIT(0)
+#define WDT_TIMEOUT_STATUS	0x10
+#define   WDT_TIMEOUT_STATUS_BOOT_SECONDARY	BIT(1)
 
 /*
  * WDT_RESET_WIDTH controls the characteristics of the external pulse (if
@@ -192,6 +194,7 @@ static int aspeed_wdt_probe(struct platform_device *pdev)
 	struct device_node *np;
 	const char *reset_type;
 	u32 duration;
+	u32 status;
 	int ret;
 
 	wdt = devm_kzalloc(&pdev->dev, sizeof(*wdt), GFP_KERNEL);
@@ -307,6 +310,10 @@ static int aspeed_wdt_probe(struct platform_device *pdev)
 		writel(duration - 1, wdt->base + WDT_RESET_WIDTH);
 	}
 
+	status = readl(wdt->base + WDT_TIMEOUT_STATUS);
+	if (status & WDT_TIMEOUT_STATUS_BOOT_SECONDARY)
+		wdt->wdd.bootstatus = WDIOF_CARDRESET;
+
 	ret = devm_watchdog_register_device(&pdev->dev, &wdt->wdd);
 	if (ret) {
 		dev_err(&pdev->dev, "failed to register\n");
Signature · 1 line
1.8.3.1
maintainer Reviewed-by on v2 1/1
#1 lore raw
Quoted 5 lines · Eddie James

Check the aspeed timeout status register to see if the system has booted from the secondary boot source. If so, set the watchdog device bootstatus flag for "Card previously reset the CPU."

Quoted 41 lines

---
Changes since v1:
- use bootstatus instead of status
- use WDIOF_CARDRESET instead of raw register contents

  drivers/watchdog/aspeed_wdt.c | 7 +++++++
  1 file changed, 7 insertions(+)
drivers/watchdog/aspeed_wdt.c +7 -0
index a5b8eb2..1abe4d0 100644
@@ -55,6 +55,8 @@ struct aspeed_wdt_config {
  #define   WDT_CTRL_WDT_INTR		BIT(2)
  #define   WDT_CTRL_RESET_SYSTEM		BIT(1)
  #define   WDT_CTRL_ENABLE		BIT(0)
+#define WDT_TIMEOUT_STATUS	0x10
+#define   WDT_TIMEOUT_STATUS_BOOT_SECONDARY	BIT(1)
 
  /*
   * WDT_RESET_WIDTH controls the characteristics of the external pulse (if
@@ -192,6 +194,7 @@ static int aspeed_wdt_probe(struct platform_device *pdev)
  	struct device_node *np;
  	const char *reset_type;
  	u32 duration;
+	u32 status;
  	int ret;
 
  	wdt = devm_kzalloc(&pdev->dev, sizeof(*wdt), GFP_KERNEL);
@@ -307,6 +310,10 @@ static int aspeed_wdt_probe(struct platform_device *pdev)
  		writel(duration - 1, wdt->base + WDT_RESET_WIDTH);
  	}
 
+	status = readl(wdt->base + WDT_TIMEOUT_STATUS);
+	if (status & WDT_TIMEOUT_STATUS_BOOT_SECONDARY)
+		wdt->wdd.bootstatus = WDIOF_CARDRESET;
+
  	ret = devm_watchdog_register_device(&pdev->dev, &wdt->wdd);
  	if (ret) {
  		dev_err(&pdev->dev, "failed to register\n");