On 19/12/2024 17:13, Fabien Parent wrote:
quoted
quoted
+
+ compatible:
+ const: onnn,ncv6336
+
+ reg:
+ maxItems: 1
+
+ buck:
+ description: buck regulator description
Why do you need "buck" node? Just merge the properties into this device
node.
I decided to move the properties into a "buck" node to make the
upstream process of the driver
a little bit simpler. The driver is written in Rust, and if I want to
move the properties to the device
node I will need to provide a Rust abstraction for "struct
device_node". I decided to avoid this
buck is already a device node, so I don't quite get how this design
avoids such abstraction, but anyway driver design choices like this do
not shape DT.
to keep the patch series simpler by having one less abstraction to review.
If you think that's a problem, let me know and I will implement it the
way you are suggesting for v2.
Best regards,
Krzysztof