HRTIM1 peripheral in spin.dts is defined but not used.
While trying to add sync_in and sync_out subnodes to hrtim1 dts definition in spin.dts, I realized that at the moment hrtim.dtsi fragment is defined but not used.
Indeed ATM, hrtim.c file redefine everything it needs to configure hrtim using LL calls, not even using zephyr pinctrl-0 property.
It would be interesting to define child nodes to hrtim1 dts node, one per timing unit.
As such we would have pinctrl-0 properties for each timing unit. Interupts properties could also be spread inside timing unit child nodes.
Sync_in and Sync_out could also be defined as child nodes of hrtim1, in order to have pinctrl compatibility and supress most of sync_master_init() / sync_slave_init() LL calls.
pinctrl-1 could be used to match different versions of shields, so that sync_in sync_out would be remapped depending on shield pinout.
I would love to have your opinion on that @afarahhass @cfoucher