Am Freitag, 3. Juli 2026, 09:26:43 Ostafrikanische Zeit schrieb Krzysztof Kozlowski: > Also, this cannot be placed in soc. Clock and reset controllers DO NOT > go to the soc directory. Place in it clocks. Even when they contain other functionality like hwlock, usb phy and reboot? Here is the previous discussion that made me move it to soc: https://lore.kernel.org/linux-arm-kernel/20260621-plywood-straddle-b01f0c45f27c@spud/ Likewise, what about the node name: syscon@ or clock-controller@ ? Conor advised me to go with syscon@ Cheers, Stefan --- Quoting the relevant parts from the above link: > At what point does it make sense to move the bindings from bindings/clock to > bindings/mfd? The controllers are still very clock-heavy. allwinner,*- > prcm.yaml look like clock, reset, misc controllers in mfd/ whereas > ingenic,cgu.yaml, sprd,sc9863a-clk.yaml and da8xx-cfgchip.txt are clock + misc > drivers in clock/. Yeah, to bindings/mfd or bindings/soc/. Which I think is mostly a judgement call. Two of your devices have at least three functions, which I think is enough to make the claim that it's not just a clock controller. > Likewise for the node names: syscon@ or clock-controller@? If you have syscon in the compatible, then I think it should be syscon in the node name as it's more general and makes it clear the device isn't just a clock controller.