Re: [PATCHv2] dt-bindings: arm: socionext: add Synquacer platforms

From: Krzysztof Kozlowski
Date: Tue Jun 20 2023 - 13:54:51 EST


On 20/06/2023 19:24, Jassi Brar wrote:
>>> +properties:
>>> + $nodename:
>>> + const: '/'
>>> + compatible:
>>> + oneOf:
>>> + - items:
>>> + - enum:
>>> + - socionext,developer-box
>>> + - socionext,synquacer
>>> + - const: socionext,sc2a11b
>>
>> That's quite different change.
>>
> So it is not carrying your ack.
>
>> What is synquacer in this case? You claim
>> now it is a board, but based on previous discussions and U-Boot source
>> it does not look like such.
>>
> I never made that claim. I said Kojima-san will confirm. He informed
> Synquacer is a brand name.
>
> Currently no code internally or externally differentiates between
> SC2A11B and Synquacer and we might as well keep living with Synquacer
> only. This patch is an attempt to be accurate.

Then the patch is not correct, because synquacer is not a board. We
should anyway choose only one for adding to documentation.

Best regards,
Krzysztof