Re: [PATCH v3 6/8] dt-bindings: memory: lpddr3: deprecate passing timings frequency as unit address

From: Dmitry Osipenko <digetx@gmail.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>,
	Rob Herring <robh+dt@kernel.org>,
	Lukasz Luba <lukasz.luba@arm.com>,
	Alim Akhtar <alim.akhtar@samsung.com>,
	linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
	linux-pm@vger.kernel.org, linux-samsung-soc@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v3 6/8] dt-bindings: memory: lpddr3: deprecate passing timings frequency as unit address
Date: Sun, 6 Feb 2022 18:33:23 +0300	[thread overview]
Message-ID: <685aa09e-fa57-e082-a0e3-d385292101e6@gmail.com> (raw)
In-Reply-To: <20220206135807.211767-7-krzysztof.kozlowski@canonical.com>

06.02.2022 16:58, Krzysztof Kozlowski пишет:
> 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: Dmitry Osipenko <digetx@gmail.com>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> ---
>  .../ddr/jedec,lpddr3-timings.yaml                | 16 ++++++++++------
>  .../memory-controllers/ddr/jedec,lpddr3.yaml     | 12 +++++-------
>  2 files changed, 15 insertions(+), 13 deletions(-)

Reviewed-by: Dmitry Osipenko <digetx@gmail.com>