Re: [EXT] Re: [PATCH v6 02/11] dt-bindings: arm: fsl: add imx-se-fw binding doc

From: Krzysztof Kozlowski
Date: Wed Oct 04 2023 - 02:36:40 EST


On 04/10/2023 06:10, Pankaj Gupta wrote:

>>
>>> + $ref: /schemas/types.yaml#/definitions/phandle-array
>>> +
>>> +required:
>>> + - compatible
>>> + - mboxes
>>> + - mbox-names
>>> + - fsl,mu-id
>>
>> How is memory-region optional?
> Depending on the type of enclave IP and the FW, memory-region is configured optionally.
> For i.MX8ULP and i.MX93, it is required. For i.MX8DXL, it is not required.

Then write it in the bindings in allOf:if:then:.

>>
>>> +
>>> +additionalProperties: false
>>> +
>>> +examples:
>>> + - |
>>> + ele_fw: se-fw {
>>
>> Drop unused labels.
> I am sorry, if I am not able to understand your point here.
> But, all the below labels are getting used.
> I can remove the fsl,mu-id. Will do it in V7.

Comment was about labels, not properties.

Best regards,
Krzysztof