Thread (14 messages) 14 messages, 2 authors, 2024-07-22

Re: [RFC PATCH 3/3] ARM: dts: davinci, keystone: correct watchdog nodenames

From: Krzysztof Kozlowski <krzk@kernel.org>
Date: 2024-07-22 08:14:06
Also in: linux-devicetree, linux-watchdog, lkml

On 21/07/2024 18:28, Kousik Sanagavarapu wrote:
Using "wdt" instead of "watchdog" for watchdog timer nodes doesn't allow
for validation with the corresponding dtschema and gives errors

	$ make CHECK_DTBS=y DT_SCHEMA_FILES=ti,davinci-wdt.yaml \
		ti/keystone/keystone-k2g-ice.dtb

	DTC_CHK arch/arm/boot/dts/ti/keystone/keystone-k2g-ice.dtb
	arch/arm/boot/dts/ti/keystone/keystone-k2g-ice.dtb:
	wdt@02250000: $nodename:0: 'wdt@02250000' does not match
	'^(timer|watchdog)(@.*|-([0-9]|[1-9][0-9]+))?$'
	from schema $id:
	http://devicetree.org/schemas/watchdog/ti,davinci-wdt.yaml#

Therefore change "wdt@" to "watchdog@".

While at it, remove "ti,davinci-wdt" compatible from the keystone dts
code.
That's entirely unrelated patch. Don't mix simple cleanups with patches
affecting ABI and users. Also, explain why.
Signed-off-by: Kousik Sanagavarapu <redacted>
---

Best regards,
Krzysztof

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help