[PATCH] watchdog: ulp_wdog: only compile the code of DM driver when CONFIG_WDT=y

[PATCH] watchdog: ulp_wdog: only compile the code of DM driver when CONFIG_WDT=y

[Ye Li]

From: Alice Guo <alice.guo@nxp.com>

DM watchdog timer driver and non-DM driver exist in ulp_wdog.c at the
same time. Add a compilation restriction that only compile the DM driver
part when CONFIG_WDT=y.

Signed-off-by: Alice Guo <alice.guo@nxp.com>
Reviewed-by: Ye Li <ye.li@nxp.com>
---
 drivers/watchdog/ulp_wdog.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/watchdog/ulp_wdog.c b/drivers/watchdog/ulp_wdog.c
index e081054..8f8182d 100644
--- a/drivers/watchdog/ulp_wdog.c
+++ b/drivers/watchdog/ulp_wdog.c
@@ -164,6 +164,7 @@ void reset_cpu(void)
 	while (1);
 }
 
+#if CONFIG_IS_ENABLED(WDT)
 static int ulp_wdt_start(struct udevice *dev, u64 timeout_ms, ulong flags)
 {
 	struct ulp_wdt_priv *priv = dev_get_priv(dev);
@@ -222,3 +223,4 @@ U_BOOT_DRIVER(ulp_wdt) = {
 	.probe		= ulp_wdt_probe,
 	.ops	= &ulp_wdt_ops,
 };
+#endif
-- 
2.7.4

Re: [PATCH] watchdog: ulp_wdog: only compile the code of DM driver when CONFIG_WDT=y

[Stefan Roese]

On 1/31/23 07:59, Ye Li wrote:
> From: Alice Guo <alice.guo@nxp.com>
> 
> DM watchdog timer driver and non-DM driver exist in ulp_wdog.c at the
> same time. Add a compilation restriction that only compile the DM driver
> part when CONFIG_WDT=y.
> 
> Signed-off-by: Alice Guo <alice.guo@nxp.com>
> Reviewed-by: Ye Li <ye.li@nxp.com>

Reviewed-by: Stefan Roese <sr@denx.de>

Thanks,
Stefan

> ---
>   drivers/watchdog/ulp_wdog.c | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/watchdog/ulp_wdog.c b/drivers/watchdog/ulp_wdog.c
> index e081054..8f8182d 100644
> --- a/drivers/watchdog/ulp_wdog.c
> +++ b/drivers/watchdog/ulp_wdog.c
> @@ -164,6 +164,7 @@ void reset_cpu(void)
>   	while (1);
>   }
>   
> +#if CONFIG_IS_ENABLED(WDT)
>   static int ulp_wdt_start(struct udevice *dev, u64 timeout_ms, ulong flags)
>   {
>   	struct ulp_wdt_priv *priv = dev_get_priv(dev);
> @@ -222,3 +223,4 @@ U_BOOT_DRIVER(ulp_wdt) = {
>   	.probe		= ulp_wdt_probe,
>   	.ops	= &ulp_wdt_ops,
>   };
> +#endif

Viele Grüße,
Stefan Roese

-- 
DENX Software Engineering GmbH,      Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de