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.yan...@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot