Am 05.10.2019 um 22:12 schrieb Marek Vasut:
On 10/5/19 10:10 PM, Simon Goldschmidt wrote:
These boards don't have a watchdog enabled in SPL, so make sure
CONFIG_SPL_WDT is not enabled.

Fixes: commit 6874cb72204f ("watchdog: Split WDT from SPL_WDT")
---


While I think the more fundamental problem is selecting
SPL_WATCHDOG_SUPPORT in arch/arm/Kconfig, let's fix it this way
for the upcoming v2019.10 release to prevent breaking s10.


Indeed, we need some sort of bandaid for my flub.

[...]

diff --git a/configs/socfpga_arria10_defconfig 
b/configs/socfpga_arria10_defconfig
index a2d555b9b4..7cb214d655 100644
--- a/configs/socfpga_arria10_defconfig
+++ b/configs/socfpga_arria10_defconfig
@@ -46,3 +46,4 @@ CONFIG_SPI=y
  CONFIG_TIMER=y
  CONFIG_SPL_TIMER=y
  CONFIG_DESIGNWARE_APB_TIMER=y
+# CONFIG_SPL_WDT is not set

[...]

diff --git a/configs/socfpga_vining_fpga_defconfig 
b/configs/socfpga_vining_fpga_defconfig
index 6b2266fdfd..c52afddd20 100644
--- a/configs/socfpga_vining_fpga_defconfig
+++ b/configs/socfpga_vining_fpga_defconfig
@@ -91,3 +91,4 @@ CONFIG_USB_GADGET_VENDOR_NUM=0x0525
  CONFIG_USB_GADGET_PRODUCT_NUM=0xa4a5
  CONFIG_USB_GADGET_DWC2_OTG=y
  CONFIG_USB_GADGET_DOWNLOAD=y
+# CONFIG_SPL_WDT is not set

I think these two boards enabled WDT in SPL, no ?


Hmm, they might. Although I haven't found CONFIG_HW_WATCHDOG enabled with them?

Anyway, that would make my commit message wrong, but the patch would still be correct since none of them uses the DM watchdog framework, or do they?

Do you want me to update the commit message?

Regards,
Simon
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to