Hi York,
I think it's better not to affect the original i.mx SoCs. If not in i.mx SoCs, we can enable it by define CONFIG_IMX_WATCHDOG in config file. Or do you think it's better to use "select IMX_WATCHDOG" to enable it in default for our LSCH2 SoCs/platforms?
Here is the patch I have changed:
diff --git a/drivers/watchdog/Makefile b/drivers/watchdog/Makefile index 86cf94b..ea47ccf 100644 --- a/drivers/watchdog/Makefile +++ b/drivers/watchdog/Makefile @@ -7,8 +7,12 @@
obj-$(CONFIG_AT91SAM9_WATCHDOG) += at91sam9_wdt.o obj-$(CONFIG_FTWDT010_WATCHDOG) += ftwdt010_wdt.o -ifneq (,$(filter $(SOC), mx31 mx35 mx5 mx6 mx7 vf610 fsl-layerscape)) +ifneq (,$(filter $(SOC), mx31 mx35 mx5 mx6 mx7 vf610)) obj-y += imx_watchdog.o +else +obj-$(CONFIG_IMX_WATCHDOG) += imx_watchdog.o endif
Regards, Xiaoliang Yang
-----Original Message----- From: York Sun Sent: 2018年6月1日 0:03 To: Xiaoliang Yang xiaoliang.yang_1@nxp.com; u-boot@lists.denx.de Subject: Re: [PATCH 1/2] watchdog: driver support for layerscape
Why don't move IMX_WATCHDOG to drivers/watchdog/Kconfig and enabled this option for the selected SoCs/platforms?
York