On 14/05/2026 14:30, Krzysztof Kozlowski wrote: > 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. >
Heh, and now I noticed that you just merged THREE versions in one thread leading to complete mess in this entire thread. Best regards, Krzysztof
