dts_fixup: Remove redefined CONFIG_WDT_0_NAME

[qianfan-Zhao]

The marco 'CONFIG_WDT_0_NAME' are defined in Kconfig, doesn't need
defined in dts_fixup.h

Signed-off-by: qianfan Zhao [email protected]

[nashif]

afaik it should be removed from kconfig or disabled for drivers enabling DTS. It should come from DTS at the end.

[codecov-io]

Codecov Report

Merging #11331 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #11331   +/-   ##
=======================================
  Coverage   51.58%   51.58%           
=======================================
  Files         216      216           
  Lines       27671    27671           
  Branches     6938     6938           
=======================================
  Hits        14274    14274           
  Misses      10648    10648           
  Partials     2749     2749

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 84f9dc7...ce202fa. Read the comment docs.

[galak]

I'm with Anas, that I'd rather we remove the Kconfig option instead of removing this from DT.

[ioannisg]

Remove option from Kconfig then?

[qianfan-Zhao]

I should waiting all the watchdog drivers support DT. Then can remove the Kconfig options.

Some soc driver doesn't support watchdog dts(such as sam)

[galak]

I should waiting all the watchdog drivers support DT. Then can remove the Kconfig options.

Some soc driver doesn't support watchdog dts(such as sam)

Just a quick look for SAM, we have dts/bindings/watchdog/atmel,sam-watchdog.yaml. Do we not use DT for it?

[qianfan-Zhao]

@galak Sorry it's my wrong.

Seems what I need to is remove the Kconfig options and add somethings like this in dts_fixup.h

#define CONFIG_WDT_0_NAME CONFIG_WDT_SAM_LABEL

[galak]

Seems what I need to is remove the Kconfig options and add somethings like this in dts_fixup.h

yeah, take a look at #11377

[nashif]

all those changes do not apply anymore.