Hi Krzysztof, On Tue, 2022-11-29 at 08:47 +0100, Krzysztof Kozlowski wrote: > On 29/11/2022 03:50, Xiangsheng Hou (侯祥胜) wrote: > > > > > > 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. The MediaTek SPI NAND controller IP used by MT7986 is the newest. In the future, there will have other SoCs. If add restricting on this, may not easy to maintain. Does this acceptable? Thanks Xiangsheng Hou