On 08/04/2024 10:01, Shengjiu Wang wrote: >>> + >>> +anyOf: >>> + - required: >>> + - spdif-in >>> + - required: >>> + - spdif-out >>> + - required: >>> + - spdif-out >>> + - spdif-in >> >> Do you need the last required block? > > Yes, one of them or both are required. And? It's already there: that's the meaning of any. It is not oneOf... Before answering please test your changes and ideas. I pointed issue here and you responded just to close my comment. That does not make me happy, just wastes my time. > >> >>> + >>> +additionalProperties: false >>> + >>> +examples: >>> + - | >>> + sound { >> >> That's a random change... >> >> Instead of sending two patches per day, please carefully address the >> feedback. > > In v1 you suggest to change it to spdif? but spdif may conflict > with the fsl,spdif.yaml. so which name I should use? I don't understand where is the conflict. That's a different binding. Best regards, Krzysztof