Re: [PATCH v4 2/7] dt-bindings: memory: lpddr2: Convert to schema

From: Krzysztof Kozlowski
Date: Wed Oct 06 2021 - 13:24:02 EST


On 06/10/2021 17:44, Dmitry Osipenko wrote:
> 06.10.2021 18:41, Dmitry Osipenko пишет:
>> 06.10.2021 13:57, Krzysztof Kozlowski пишет:
>>>> + density:
>>>> + description: |
>>>> + Density in megabits of SDRAM chip. Obtained from device datasheet.
>>> You need here a type/ref, so uint32.
>>>
>>
>> The type is uint32 by default. I can add it, but it's not really necessary.
>>
>
> You may grep bindings for 'enum: [' to see that nobody is specifying the
> type.
>

Just because everyone makes a mistake, is not a proof it should be done
like that. Please see example schema and vendor,int-property.

AFAIR, only properties defined by schema (directly or by unit suffix,
e.g. microvolt) do not need types.

Best regards,
Krzysztof