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

Jeremy Kerr
2025-10-24T08:06:21Z

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