06.10.2021 18:47, Dmitry Osipenko пишет: > 06.10.2021 13:59, Krzysztof Kozlowski пишет: >> On 06/10/2021 01:00, Dmitry Osipenko wrote: >>> Add optional revision-id standard LPDDR2 properties which will help to >>> identify memory chip. >>> >>> Signed-off-by: Dmitry Osipenko <digetx@xxxxxxxxx> >>> --- >>> .../memory-controllers/ddr/jedec,lpddr2.yaml | 12 ++++++++++++ >>> 1 file changed, 12 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/memory-controllers/ddr/jedec,lpddr2.yaml b/Documentation/devicetree/bindings/memory-controllers/ddr/jedec,lpddr2.yaml >>> index d99ccad54938..01efb43c3425 100644 >>> --- a/Documentation/devicetree/bindings/memory-controllers/ddr/jedec,lpddr2.yaml >>> +++ b/Documentation/devicetree/bindings/memory-controllers/ddr/jedec,lpddr2.yaml >>> @@ -24,6 +24,16 @@ properties: >>> - enum: >>> - jedec,lpddr2-nvm >>> >>> + revision-id1: >>> + maximum: 255 >>> + description: | >>> + Revision 1 value of SDRAM chip. Obtained from device datasheet. >>> + >> >> Also type/ref for uint32 needed. > > Also unnecessary :) It's uint32 by default, inferred from the 'maximum'. > Please grep for 'maximum: ' to see that other bindings don't specify the type.
