Hi York,

Sure, thanks for your suggestion. I'll add IMX_WATCHDOG in Kconfig, users can 
enable it for the platforms they needed.
I will push v2 patch later, please help to review it, thanks.

Regards,
Xiaoliang Yang

-----Original Message-----
From: York Sun 
Sent: 2018年6月1日 11:20
To: Xiaoliang Yang <xiaoliang.yan...@nxp.com>; u-boot@lists.denx.de
Subject: Re: [PATCH 1/2] watchdog: driver support for layerscape

On 05/31/2018 08:16 PM, Xiaoliang Yang wrote:
> 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?

It is a trend to use Kconfig instead of CONFIG_ macros. If you don't convert 
today, make sure you make it easy to convert.

If you expect this watchdog to be enabled for all LS2 platforms, you can enable 
it in LS2 config file. Otherwise, only enable it for the platforms you need.

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

Reply via email to