Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Mon, 16 Jan 2023 17:30:24 +0100
From:   Thierry Reding <thierry.reding@gmail.com>
To:     Sumit Gupta <sumitg@nvidia.com>
Cc:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        treding@nvidia.com, dmitry.osipenko@collabora.com,
        viresh.kumar@linaro.org, rafael@kernel.org, jonathanh@nvidia.com,
        robh+dt@kernel.org, linux-kernel@vger.kernel.org,
        linux-tegra@vger.kernel.org, linux-pm@vger.kernel.org,
        devicetree@vger.kernel.org, sanjayc@nvidia.com,
        ksitaraman@nvidia.com, ishah@nvidia.com, bbasu@nvidia.com
Subject: Re: [Patch v1 09/10] memory: tegra: get number of enabled mc channels
Message-ID: <Y8V7oIzVPffETqtO@orome>
References: <20221220160240.27494-1-sumitg@nvidia.com>
 <20221220160240.27494-10-sumitg@nvidia.com>
 <db223161-a424-c4cf-09a8-ff2241fda71a@linaro.org>
 <86f94266-d88e-af82-0352-876bc369f6a2@nvidia.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="g8Q09nNnddE8uScx"
Content-Disposition: inline
In-Reply-To: <86f94266-d88e-af82-0352-876bc369f6a2@nvidia.com>
User-Agent: Mutt/2.2.9 (2022-11-12)
Precedence: bulk


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

On Fri, Jan 13, 2023 at 08:34:18PM +0530, Sumit Gupta wrote:
>=20
>=20
> On 22/12/22 17:07, Krzysztof Kozlowski wrote:
> > External email: Use caution opening links or attachments
> >=20
> >=20
> > On 20/12/2022 17:02, Sumit Gupta wrote:
> > > Get number of MC channels which are actually enabled
> > > in current boot configuration.
> >=20
> > Why? You don't do anything with it. Commit msg should give the reason of
> > changes.
> >=20
> >=20
> > Best regards,
> > Krzysztof
> >=20
>=20
> CPU OPP tables have per channel bandwidth info. The "mc->num_channels" is
> used in [1] (Patch v1 10/10) to make the per MC channel bandwidth request=
ed
> by the CPU cluster as a multiple of number of the enabled mc channels.
>=20
> Will update the commit description with this info.
>=20
> [1] https://lore.kernel.org/lkml/20221220160240.27494-1-sumitg@nvidia.com=
/T/#m3ac150a86977e89b97c5d19c60384f29d7a01d21

Both patch 9 and 10 are reasonably small, so it would be okay to merge
the two patches and avoid any need for an extra explanation.

Thierry

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

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

iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAmPFe6AACgkQ3SOs138+
s6E/2BAAiVcPFgJQq47pbkFQvbySJLa/pcKt6BI8lTuPJwZsRJ13RxfCp2Y+KkE0
xSYrAmtvD+IDy2s67qTGm6UZjT9KqGmV9zMaZ6ooTylKYtOW4N/Ln4///T9jwRGM
oTG6NN2DEPVKkV96gTtLzYAAM0cNx7lx7pNEHV7uXDUr2ieKHUhEkEeliDK73xBN
6so7/FzLBCf/O7mitcLOud1qC1pHSXwULHMxDFZyU3VinsqIJBBc4Ua5IB0dkc8U
Cy8E04CrxU7H49ylIl4cpO/bq0PuazW6PlIemqoheJyWKyr0foyRpjvJI6nsy1CT
9Mb+ocRAB+Cu2IILTFl4IaW09982TBJOyIYJ5qRi0DvMhFAb64dUWT/rdMPvQ4q9
P6RLArDLfcksDnYnhFwhFESdEPOa+YccCYwfFyeYcN2/XzE9AMGW9iNYKPw4fnpM
fDmU33Q/0ze7q4BDcEDfLFwiRmoHpXZv2QYIi/U2fhN6cuux9QEMWEX4Lksxvqst
uH/9FFgHHlViYPkb/iW1qZjuOsG4Z5A6xNVCMxbKYt89W7800FlI0tWZGQ/LYT2R
JdorAB6uZnW0MkXm5N70rnCJn75+2xJmrRLnnIfmAivLrRE6uo8WwXOTVYgTZVAN
8WONvNE9WgSPFSw2nJ8Rr+UE/pektA2R8QfNrVgyF5w9txkkCDA=
=jOs7
-----END PGP SIGNATURE-----

--g8Q09nNnddE8uScx--