Commit 42f94bb9 authored by Krzysztof Kozlowski's avatar Krzysztof Kozlowski
Browse files

dt-bindings: memory: lpddr3: deprecate passing timings frequency as unit address 



The timings node maximum frequency was passed as an unit address, which
is actually a workaround.  Such workaround and unit address are not
needed at all, because the device memory node (parent) can contain
multiple timing nodes without unit addresses but with suffix used for
nodenames, e.g. timings-1.

LPDDR2 bindings already use such version, so unify the LPDDR3 with them.

Suggested-by: default avatarDmitry Osipenko <digetx@gmail.com>
Signed-off-by: default avatarKrzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Reviewed-by: default avatarDmitry Osipenko <digetx@gmail.com>
Reviewed-by: default avatarRob Herring <robh@kernel.org>
Link: https://lore.kernel.org/r/20220206135807.211767-7-krzysztof.kozlowski@canonical.com
parent e531932c

Documentation/devicetree/bindings/memory-controllers/ddr/jedec,lpddr3-timings.yaml

+10 −6
Original line number Diff line number Diff line
@@ -17,6 +17,13 @@ properties: 
    maxItems: 1

    description: |

      Maximum DDR clock frequency for the speed-bin, in Hz.

      Property is deprecated, use max-freq.

    deprecated: true



  max-freq:

    $ref: /schemas/types.yaml#/definitions/uint32

    description: |

      Maximum DDR clock frequency for the speed-bin, in Hz.



  min-freq:

    $ref: /schemas/types.yaml#/definitions/uint32
@@ -117,19 +124,16 @@ properties: 
required:

  - compatible

  - min-freq

  - reg

  - max-freq



additionalProperties: false



examples:

  - |

    lpddr3 {

        #address-cells = <1>;

        #size-cells = <0>;



        timings@800000000 {

        timings {

            compatible = "jedec,lpddr3-timings";

            reg = <800000000>;

            max-freq = <800000000>;

            min-freq = <100000000>;

            tCKE = <3750>;

            tCKESR = <3750>;

Documentation/devicetree/bindings/memory-controllers/ddr/jedec,lpddr3.yaml

+5 −7
Original line number Diff line number Diff line
@@ -18,6 +18,7 @@ properties: 


  '#address-cells':

    const: 1

    deprecated: true



  density:

    $ref: /schemas/types.yaml#/definitions/uint32
@@ -55,6 +56,7 @@ properties: 


  '#size-cells':

    const: 0

    deprecated: true



  tCKE-min-tck:

    $ref: /schemas/types.yaml#/definitions/uint32
@@ -191,7 +193,7 @@ properties: 
      cycles.



patternProperties:

  "^timings@[0-9a-f]+$":

  "^timings((-[0-9])+|(@[0-9a-f]+))?$":

    $ref: jedec,lpddr3-timings.yaml

    description: |

      The lpddr3 node may have one or more child nodes with timings.
@@ -201,10 +203,8 @@ patternProperties: 


required:

  - compatible

  - '#address-cells'

  - density

  - io-width

  - '#size-cells'



additionalProperties: false
@@ -214,8 +214,6 @@ examples: 
        compatible = "samsung,K3QF2F20DB", "jedec,lpddr3";

        density = <16384>;

        io-width = <32>;

        #address-cells = <1>;

        #size-cells = <0>;



        tCKE-min-tck = <2>;

        tCKESR-min-tck = <2>;
@@ -239,9 +237,9 @@ examples: 
        tXP-min-tck = <2>;

        tXSR-min-tck = <12>;



        timings@800000000 {

        timings {

            compatible = "jedec,lpddr3-timings";

            reg = <800000000>;

            max-freq = <800000000>;

            min-freq = <100000000>;

            tCKE = <3750>;

            tCKESR = <3750>;