On 30/11/2022 09:18, Xiangsheng Hou (侯祥胜) wrote: > 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? I don't think it is not easy to maintain. We have it in many, many bindings... Best regards, Krzysztof
