Received-SPF: pass (google.com: domain of linux-kernel+bounces-183992-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Date: Mon, 20 May 2024 18:14:58 +0100
From: Conor Dooley <conor@kernel.org>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: Rob Herring <robh@kernel.org>, Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Kishon Vijay Abraham I <kishon@kernel.org>,
	Roger Quadros <rogerq@kernel.org>, Nishanth Menon <nm@ti.com>,
	Vignesh Raghavendra <vigneshr@ti.com>,
	Tero Kristo <kristo@kernel.org>, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 2/5] dt-bindings: soc: ti: am645-system-controller: add
 AM654 syscon
Message-ID: <20240520-deskwork-unpinned-fb6b87cc7ffa@spud>
References: <20240518-dt-bindings-ti-soc-mfd-v1-0-b3952f104c9a@linaro.org>
 <20240518-dt-bindings-ti-soc-mfd-v1-2-b3952f104c9a@linaro.org>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
	protocol="application/pgp-signature"; boundary="0al9K0Y79EnPUy/+"
Content-Disposition: inline
In-Reply-To: <20240518-dt-bindings-ti-soc-mfd-v1-2-b3952f104c9a@linaro.org>


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

On Sat, May 18, 2024 at 10:07:17PM +0200, Krzysztof Kozlowski wrote:
> Add dedicated binding for the AM654 MCU SCM system controller registers,
> already used in the DTS to properly describe its children.
>=20
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  .../soc/ti/ti,am654-system-controller.yaml         | 60 ++++++++++++++++=
++++++
>  1 file changed, 60 insertions(+)
>=20
> diff --git a/Documentation/devicetree/bindings/soc/ti/ti,am654-system-con=
troller.yaml b/Documentation/devicetree/bindings/soc/ti/ti,am654-system-con=
troller.yaml
> new file mode 100644
> index 000000000000..e79803e586ca
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/soc/ti/ti,am654-system-controller=
=2Eyaml
> @@ -0,0 +1,60 @@
> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/soc/ti/ti,am654-system-controller.yam=
l#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: TI AM654 System Controller Registers R/W

"R/W"? Is that a copy-paste mistake from the register map for this SoC?

Otherwise,
Reviewed-by: Conor Dooley <conor.dooley@microchip.com>

Cheers,
Conor.

--0al9K0Y79EnPUy/+
Content-Type: application/pgp-signature; name="signature.asc"

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

iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZkuFEgAKCRB4tDGHoIJi
0m6dAP93xYp8ADQ/ZttCjnuveWAIRutjhPRI5gZc1jAbhGcspgD/WN2Lb9H7PkgN
XKd0YnLVM+On2SryiO2Ibp2Q/xvshgI=
=iMTG
-----END PGP SIGNATURE-----

--0al9K0Y79EnPUy/+--