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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 <five231003@xxxxxxxxx>
> ---


Best regards,
Krzysztof





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux