drivers: watchdog: esp32: Reduce number of seals/unseals

Jira: ZEP-2556
Signed-off-by: Leandro Pereira <[email protected]>

Author: lpereira

drivers/watchdog/wdt_esp32.c

@@ -124,7 +124,6 @@ static void set_interrupt_enabled(bool setting)
 {
 	volatile u32_t *intr_enable_reg = (u32_t *)TIMG_INT_ENA_TIMERS_REG(1);
 
-	wdt_esp32_unseal();
 	if (setting) {
 		*intr_enable_reg |= TIMG_WDT_INT_ENA;
 
@@ -135,7 +134,6 @@ static void set_interrupt_enabled(bool setting)
 		*intr_enable_reg &= ~TIMG_WDT_INT_ENA;
 		irq_disable(CONFIG_WDT_ESP32_IRQ);
 	}
-	wdt_esp32_seal();
 }
 
 static int wdt_esp32_set_config(struct device *dev, struct wdt_config *config)
@@ -160,29 +158,27 @@ static int wdt_esp32_set_config(struct device *dev, struct wdt_config *config)
 
 	if (config->mode == WDT_MODE_RESET) {
 		/* Warm reset on timeout */
-		v |= TIMG_WDT_STG_SEL_RESET_CPU<<TIMG_WDT_STG0_S;
+		v |= TIMG_WDT_STG_SEL_RESET_SYSTEM<<TIMG_WDT_STG0_S;
 		v |= TIMG_WDT_STG_SEL_OFF<<TIMG_WDT_STG1_S;
 
 		/* Disable interrupts for this mode. */
-		v &= ~TIMG_WDT_LEVEL_INT_EN;
-
-		set_interrupt_enabled(false);
+		v &= ~(TIMG_WDT_LEVEL_INT_EN | TIMG_WDT_EDGE_INT_EN);
 	} else if (config->mode == WDT_MODE_INTERRUPT_RESET) {
 		/* Interrupt first, and warm reset if not reloaded */
 		v |= TIMG_WDT_STG_SEL_INT<<TIMG_WDT_STG0_S;
-		v |= TIMG_WDT_STG_SEL_RESET_CPU<<TIMG_WDT_STG1_S;
+		v |= TIMG_WDT_STG_SEL_RESET_SYSTEM<<TIMG_WDT_STG1_S;
 
 		/* Use level-triggered interrupts. */
 		v |= TIMG_WDT_LEVEL_INT_EN;
-
-		set_interrupt_enabled(true);
+		v &= ~TIMG_WDT_EDGE_INT_EN;
 	} else {
 		return -EINVAL;
 	}
 
 	wdt_esp32_unseal();
 	*reg = v;
 	adjust_timeout(config->timeout & WDT_TIMEOUT_MASK);
+	set_interrupt_enabled(config->mode == WDT_MODE_INTERRUPT_RESET);
 	wdt_esp32_seal();
 
 	wdt_esp32_reload(dev);