On 1/8/20 12:25 PM, Jagan Teki wrote: > On Mon, Jan 6, 2020 at 7:44 PM Marek Vasut <ma...@denx.de> wrote: >> >> Migrate CONFIG_DESIGNWARE_WATCHDOG to Kconfig and update the headers >> accordingly, no functional change. The S10 enables the WDT only in >> SPL, but does not enable it in U-Boot itself, hence disable it in >> the config again. >> >> Signed-off-by: Marek Vasut <ma...@denx.de> >> Cc: Chin Liang See <chin.liang....@intel.com> >> Cc: Dalon Westergreen <dwest...@gmail.com> >> Cc: Dinh Nguyen <dingu...@kernel.org> >> Cc: Jagan Teki <ja...@amarulasolutions.com> >> Cc: Ley Foon Tan <ley.foon....@intel.com> >> Cc: Philipp Tomisch <philipp.tomi...@theobroma-systems.com> >> Cc: Simon Goldschmidt <simon.k.r.goldschm...@gmail.com> >> Cc: Tien Fong Chee <tien.fong.c...@intel.com> >> --- >> V2: Use non-DM watchdog in SPL on S10 >> V3: No changes >> --- >> configs/socfpga_stratix10_defconfig | 1 + >> configs/socfpga_vining_fpga_defconfig | 1 + >> drivers/watchdog/Kconfig | 7 +++++++ >> include/configs/socfpga_common.h | 4 +--- >> include/configs/socfpga_stratix10_socdk.h | 7 +++++-- >> scripts/config_whitelist.txt | 1 - >> 6 files changed, 15 insertions(+), 6 deletions(-) >> >> diff --git a/configs/socfpga_stratix10_defconfig >> b/configs/socfpga_stratix10_defconfig >> index 0665b460de..82ffedf41a 100644 >> --- a/configs/socfpga_stratix10_defconfig >> +++ b/configs/socfpga_stratix10_defconfig >> @@ -59,3 +59,4 @@ CONFIG_DM_USB=y >> CONFIG_USB_DWC2=y >> CONFIG_USB_STORAGE=y >> # CONFIG_SPL_USE_TINY_PRINTF is not set >> +CONFIG_DESIGNWARE_WATCHDOG=y >> diff --git a/configs/socfpga_vining_fpga_defconfig >> b/configs/socfpga_vining_fpga_defconfig >> index 19bed6be27..0e31e4f165 100644 >> --- a/configs/socfpga_vining_fpga_defconfig >> +++ b/configs/socfpga_vining_fpga_defconfig >> @@ -97,3 +97,4 @@ CONFIG_USB_GADGET_PRODUCT_NUM=0xa4a5 >> CONFIG_USB_GADGET_DWC2_OTG=y >> CONFIG_USB_GADGET_DOWNLOAD=y >> # CONFIG_SPL_WDT is not set >> +CONFIG_DESIGNWARE_WATCHDOG=y >> diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig >> index 8c16d69d33..5f8a070ef5 100644 >> --- a/drivers/watchdog/Kconfig >> +++ b/drivers/watchdog/Kconfig >> @@ -45,6 +45,13 @@ config ULP_WATCHDOG >> help >> Say Y here to enable i.MX7ULP watchdog driver. >> >> +config DESIGNWARE_WATCHDOG >> + bool "Designware watchdog timer support" >> + select HW_WATCHDOG >> + help >> + Enable this to support Designware Watchdog Timer IP, present e.g. >> + on Altera SoCFPGA SoCs. >> + >> config WDT >> bool "Enable driver model for watchdog timer drivers" >> depends on DM >> diff --git a/include/configs/socfpga_common.h >> b/include/configs/socfpga_common.h >> index 05bfef75c0..d1a14d23aa 100644 >> --- a/include/configs/socfpga_common.h >> +++ b/include/configs/socfpga_common.h >> @@ -104,11 +104,9 @@ >> /* >> * L4 Watchdog >> */ >> -#ifdef CONFIG_HW_WATCHDOG >> -#define CONFIG_DESIGNWARE_WATCHDOG >> #define CONFIG_DW_WDT_BASE SOCFPGA_L4WD0_ADDRESS >> #define CONFIG_DW_WDT_CLOCK_KHZ 25000 >> -#endif >> +#define CONFIG_WATCHDOG_TIMEOUT_MSECS 30000 > > We have kconfig for this, how about moving this into defconfig file? > otherwise,
I sent a V4, give it a try. It's also rebased on master.