Thread (29 messages) 29 messages, 4 authors, 2025-11-21

Re: [PATCH v20 1/4] dt-bindings: i2c: Split AST2600 binding into a new YAML

From: Jeremy Kerr <jk@codeconstruct.com.au>
Date: 2025-10-24 07:56:13
Also in: linux-aspeed, linux-devicetree, linux-i2c, lkml, openbmc

Hi Krzysztof,
Although now I saw next patch, so clearly this commit is incomplete.
The split that Ryan has done here - by shifting to an identical separate
binding, then making the changes explicit - allows us to review the
actual changes without losing them in the move. Sounds like a benefit to
me?
You just need allOf:if:then: section to narrow the
constraints/presence of properties.
That seems like a more complex approach. This is separate IP from the
2500 controllers, wouldn't that warrant a new binding spec?

Cheers,


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