On 29/11/2022 03:50, Xiangsheng Hou (侯祥胜) wrote: > Hi Krzysztof, > > On Mon, 2022-11-28 at 10:00 +0100, Krzysztof Kozlowski wrote: >> On 28/11/2022 03:06, Xiangsheng Hou wrote: >>> 1. add mt7986 IC bindings >> >> Subject: drop second, redundant "bindings". > Will be fixed in next series. > >>> clocks: >>> + minItems: 2 >>> items: >>> - description: clock used for the controller >>> - description: clock used for the SPI bus >>> + - description: clock used for the AHB bus dma bus, this >>> depends on >>> + hardware design, so this is optional. >> >> Optional for which variants? For all of them? > It`s only needed for the item 3 nfi_hclk. Is it proper with this > description or any other suggestion. I understand third clock is optional. For which variants/compatibles it is optional? Add allOf:if:then restricting it. Best regards, Krzysztof