On Thu, May 14, 2026 at 07:45:33PM +0800, Arnab Layek wrote: > The MT8188 SCP requires support for 1-2 reserved memory regions, while > other MediaTek SoCs use only a single memory region. > > The schema uses a permissive base with restrictive conditionals: > 1) Base schema allows all devices minItems: 1, maxItems: 2 > 2) Non-MT8188 devices (mt8183, mt8186, mt8192, mt8195, mt8195-dual) are > restricted to maxItems: 1, overriding the base > 3) MT8188 devices (mt8188, mt8188-dual) set minItems: 1 with item > descriptions, inheriting maxItems: 2 from base, making the second > L1TCM region optional > > This follows the same pattern as other MediaTek dt-bindings such as > mediatek,jpeg-encoder.yaml which uses conditional schemas to support > different numbers of iommus per device variant. >
So I just reviewed v2, because it appeared in patchwork thread... because you just threaded v3 there. No, really, this was repeated also to Mediatek so many times. Implement v2 review. Best regards, Krzysztof
