On 22/05/2025 16:39, Claudiu Beznea wrote: > > > On 22.05.2025 15:48, Krzysztof Kozlowski wrote: >> On 22/05/2025 14:46, Krzysztof Kozlowski wrote: >>>>>> >>>>>> + - if: >>>>>> + properties: >>>>>> + compatible: >>>>>> + contains: >>>>>> + const: renesas,usb2-phy-r9a08g045 >>>>>> + then: >>>>>> + required: >>>>>> + - renesas,sysc-signals >>>>> >>>>> That's ABI break. >>>> >>>> There is no in kernel device tree users of "renesas,usb2-phy-r9a08g045" >>>> compatible. It is introduced in patch 11/12 from this series. With this do >>>> you still consider it ABI break? >>> >>> Then this patch cannot be split from binding introducing the user. Don't >>> add unused/undocumented compatibles. >>> >> Or you meant DTS? > > Yes, I meant in tree DTS. Rhetorical question (because we talked about this on mailing list many times enough): what would be the point of any ABI if you can break it and adjust in-tree DTS in the next patch? Best regards, Krzysztof
