Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Tue, 27 Oct 2020 14:22:41 +0100
From:   Thierry Reding <thierry.reding@gmail.com>
To:     Dmitry Osipenko <digetx@gmail.com>
Cc:     Jonathan Hunter <jonathanh@nvidia.com>,
        Georgi Djakov <georgi.djakov@linaro.org>,
        Rob Herring <robh+dt@kernel.org>,
        Michael Turquette <mturquette@baylibre.com>,
        Stephen Boyd <sboyd@kernel.org>,
        Peter De Schrijver <pdeschrijver@nvidia.com>,
        MyungJoo Ham <myungjoo.ham@samsung.com>,
        Kyungmin Park <kyungmin.park@samsung.com>,
        Chanwoo Choi <cw00.choi@samsung.com>,
        Mikko Perttunen <cyndis@kapsi.fi>,
        Viresh Kumar <vireshk@kernel.org>,
        Peter Geis <pgwipeout@gmail.com>,
        Nicolas Chauvet <kwizart@gmail.com>,
        Krzysztof Kozlowski <krzk@kernel.org>,
        linux-tegra@vger.kernel.org, linux-pm@vger.kernel.org,
        linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
        devicetree@vger.kernel.org
Subject: Re: [PATCH v6 08/52] dt-bindings: memory: tegra20: emc: Document
 mfd-simple compatible and statistics sub-device
Message-ID: <20201027132241.GG1822510@ulmo>
References: <20201025221735.3062-1-digetx@gmail.com>
 <20201025221735.3062-9-digetx@gmail.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="8bBEDOJVaa9YlTAt"
Content-Disposition: inline
In-Reply-To: <20201025221735.3062-9-digetx@gmail.com>
User-Agent: Mutt/1.14.7 (2020-08-29)
Precedence: bulk


--8bBEDOJVaa9YlTAt
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Mon, Oct 26, 2020 at 01:16:51AM +0300, Dmitry Osipenko wrote:
> External Memory Controller can gather various hardware statistics that
> are intended to be used for debugging purposes and for dynamic frequency
> scaling of memory bus.
>=20
> Document the new mfd-simple compatible and EMC statistics sub-device.
> The subdev contains EMC DFS OPP table and interconnect paths to be used
> for dynamic scaling of system's memory bandwidth based on EMC utilization
> statistics.
>=20
> Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
> ---
>  .../memory-controllers/nvidia,tegra20-emc.txt | 43 +++++++++++++++++--
>  1 file changed, 40 insertions(+), 3 deletions(-)

Why does this have to be modelled as a separate device? Isn't this just
using a couple of registers out of the EMC register range? If so, this
would better just be integrated into the parent node and implemented as
part of the EMC driver. No need to further complicate things by adding
a dummy child.

Thierry

--8bBEDOJVaa9YlTAt
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAl+YHyEACgkQ3SOs138+
s6EvOA/9GFUCJRzhojvUyJHzwHHS7x6T3glDA/XF4WmogDP9ijUclThlH5xzRkCP
Lg0hU+S5WFRyfvIJOAA6D1Ou1b1TT9Enr0dO0+Jfq3ShIvaoRBe0XP/vqaKpKqXn
P7VLT3lbs/3ZMzABDnFGSUvLwFqMlfdjXdDEExTOhDbpnoYSaleCiPtFqn4a1yye
1yHG0aZ6zGO4Tyarc9v/ftAXiTMDfVDDUyj1sNwGeeqdlOpVT3nL7tP/8U1IFNOs
5OKk4KGV3AkP/teJOjiw7DJeQpfEItcXvcmSTZx+MzHUUwqdk0my72MUIfDLA8SB
8aB67dBJSUTExjlxSLGU+8GLVKvWrCr0cUMtU018rNNRrMAoazNfOmlU6i02eGx9
ZfzTFqCdKShZVKBJkOHEaxLXt7DJuiwQ1aYzHNVlyNMQpmA8adEfmZPFWWuYLohd
Hn8mGk0ORL5p2sdw5xXwJMOlsE+p5rKiwOsrxI4sCQ4q5stu29jiv7zruPE5pX1n
Utq15ldrFwf1VNjrbm9K0XgAfEVbrBVc7Kg2xMeMCbpTRwuRQk22jF/aaCBUjGue
h4brYNuSOCLxUdcs2NuE6suO+rYJgtT5nehI9PJXG/Y66zyc2Uxui3VqIyZdybID
ZkeKK5fxiMuuyeYPzMU/sAA37rG5xZsOKVVyOOkJOlipMX2gioo=
=a8kD
-----END PGP SIGNATURE-----

--8bBEDOJVaa9YlTAt--