Re: [EXT] Re: [PATCH v4 2/5] dt-bindings: arm: fsl: add imx-se-fw binding doc

[Krzysztof Kozlowski <krzk@kernel.org>]

On 08/07/2024 13:54, Pankaj Gupta wrote:
> 
> 
>> -----Original Message-----
>> From: Krzysztof Kozlowski <krzk@kernel.org>
>> Sent: Monday, July 8, 2024 4:54 PM
>> To: Pankaj Gupta <pankaj.gupta@nxp.com>; Jonathan Corbet
>> <corbet@lwn.net>; Rob Herring <robh@kernel.org>; Krzysztof Kozlowski
>> <krzk+dt@kernel.org>; Conor Dooley <conor+dt@kernel.org>; Shawn Guo
>> <shawnguo@kernel.org>; Sascha Hauer <s.hauer@pengutronix.de>;
>> Pengutronix Kernel Team <kernel@pengutronix.de>; Fabio Estevam
>> <festevam@gmail.com>; Rob Herring <robh+dt@kernel.org>; Krzysztof
>> Kozlowski <krzysztof.kozlowski+dt@linaro.org>
>> Cc: linux-doc@vger.kernel.org; linux-kernel@vger.kernel.org;
>> devicetree@vger.kernel.org; imx@lists.linux.dev; linux-arm-
>> kernel@lists.infradead.org
>> Subject: Re: [EXT] Re: [PATCH v4 2/5] dt-bindings: arm: fsl: add imx-se-fw
>> binding doc
>>
>> Caution: This is an external email. Please take care when clicking links or
>> opening attachments. When in doubt, report the message using the 'Report
>> this email' button
>>
>>
>> On 08/07/2024 13:19, Pankaj Gupta wrote:
>>>>>> 4 mailboxes? This cannot be different.
>>>>> mbox-names can have any of the above mentioned 4 values.
>>>>> And  two values are minimum, required.
>>>>
>>>> No, I said it cannot be different. If two are minimum, then you can
>>>> have here 4, right? But earlier you said you can have only 2. It does not
>> make any sense.
>>>
>>> Understood.  Will fix this with following changes.
>>>
>>> -      - const: tx
>>> -      - const: rx
>>> -      - const: txdb
>>> -      - const: rxdb
>>> -    minItems: 2
>>> +      - enum:
>>> +          - tx
>>> +          - txdb
>>> +      - enum:
>>> +          - rx
>>> +          - rxdb
>>
>> OK, this is now clear and answer is also clear:
>> No, you cannot have different names, except some rare cases.
>> Please explain why this should be an exception?
> 
> The name suggests the type of protocol for:
> - sending:    tx or txdb (with doorbell).
> - receiving:  rx & rxdb  (with doorbell).
> 
> NXP IP SECO on i.MX8DXL/8QXP supports txdb & rxdb. Hence added these names.
> 
> It is just for readability to understand from platform's DTSI file.

Then no, that's not a reason to break fixed naming rule. These are
supposed to be just tx and rx.

Best regards,
Krzysztof