Date:   Mon, 27 Feb 2023 11:56:48 +0000
From:   Conor Dooley <conor.dooley@microchip.com>
To:     Song Shuai <suagrfillet@gmail.com>
CC:     <corbet@lwn.net>, <alexs@kernel.org>, <siyanteng@loongson.cn>,
        <robh@kernel.org>, <palmer@rivosinc.com>,
        <leyfoon.tan@starfivetech.com>, <linux-doc@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>, <linux-riscv@lists.infradead.org>
Subject: Re: [PATCH] sched/doc: supplement CPU capacity with RISC-V
Message-ID: <Y/yagE7jfRGo1FgY@wendy>
References: <20230227105941.2749193-1-suagrfillet@gmail.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="Qu+vqTjLVyvhigH7"
Content-Disposition: inline
In-Reply-To: <20230227105941.2749193-1-suagrfillet@gmail.com>
Precedence: bulk

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

On Mon, Feb 27, 2023 at 06:59:41PM +0800, Song Shuai wrote:
> This commit 7d2078310cbf ("dt-bindings: arm: move cpu-capacity to a
> shared loation") updates some references about capacity-dmips-mhz

Not requesting a respin for this, but mentioning commit 991994509ee9
("dt-bindings: riscv: add a capacity-dmips-mhz cpu property") is
probably more relevant as a justification for this change.

> property in this document.
>=20
> The list of architectures using capacity-dmips-mhz omits RISC-V, so
> supplements it here.
>=20
> Signed-off-by: Song Shuai <suagrfillet@gmail.com>
> ---
>  Documentation/scheduler/sched-capacity.rst                    | 2 +-
>  Documentation/translations/zh_CN/scheduler/sched-capacity.rst | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>=20
> diff --git a/Documentation/scheduler/sched-capacity.rst b/Documentation/s=
cheduler/sched-capacity.rst
> index 8e2b8538bc2b..e2c1cf743158 100644
> --- a/Documentation/scheduler/sched-capacity.rst
> +++ b/Documentation/scheduler/sched-capacity.rst
> @@ -258,7 +258,7 @@ Linux cannot currently figure out CPU capacity on its=
 own, this information thus
>  needs to be handed to it. Architectures must define arch_scale_cpu_capac=
ity()
>  for that purpose.
> =20
> -The arm and arm64 architectures directly map this to the arch_topology d=
river
> +The arm, arm64, and RISC-V architectures directly map this to the arch_t=
opology driver

My grep clearly missed this change, thanks.
For the only part of the patch I understand:
Reviewed-by: Conor Dooley <conor.dooley@microchip.com>

Thanks,
Conor.


--Qu+vqTjLVyvhigH7
Content-Type: application/pgp-signature; name="signature.asc"

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

iHQEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCY/yacQAKCRB4tDGHoIJi
0jMuAPUS3U8wKt0dDvUC5JFcU2VrD5RRUuYkkJESmDvd+oMvAPsE9NZjwV40Eqvv
MRChuLnLfK6qyIngBouG73xf0TMeBQ==
=v5xR
-----END PGP SIGNATURE-----

--Qu+vqTjLVyvhigH7--