Received-SPF: pass (google.com: domain of linux-kernel+bounces-182988-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Date: Sat, 18 May 2024 23:49:36 +0200
From: Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= <u.kleine-koenig@pengutronix.de>
To: Conor Dooley <conor@kernel.org>
Cc: linux-pwm@vger.kernel.org, Conor Dooley <conor.dooley@microchip.com>, 
	Chris Packham <Chris.Packham@alliedtelesis.co.nz>, Guenter Roeck <linux@roeck-us.net>, 
	Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= <ukleinek@kernel.org>, Rob Herring <robh@kernel.org>, 
	Krzysztof Kozlowski <krzk+dt@kernel.org>, Conor Dooley <conor+dt@kernel.org>, devicetree@vger.kernel.org, 
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH v1] dt-bindings: pwm: describe the cells in #pwm-cells in
 pwm.yaml
Message-ID: <oan6mlql2co4jsmnh3ghbomakdphab352fyq3plupellevkvly@f3jhiwidtkv3>
References: <20240517-patient-stingily-30611f73e792@spud>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="k5mzdz6sjp46o5pa"
Content-Disposition: inline
In-Reply-To: <20240517-patient-stingily-30611f73e792@spud>


--k5mzdz6sjp46o5pa
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Fri, May 17, 2024 at 06:38:46PM +0100, Conor Dooley wrote:
> From: Conor Dooley <conor.dooley@microchip.com>
>=20
> When commit 89650a1e3b6f ("dt-bindings: pwm: Convert PWM bindings
> to json-schema") converted the pwm provider section of the text binding
> to dt-schema it also updated all references to pwm.txt in pwm provider
> bindings to pwm.yaml.
>=20
> Most pwm provider bindings had a reference to pwm.txt as it contains a
> description of what the cells in #pwm-cells are, albeit in the consumer
> section of the document. Only information in the provider section of the
> document was moved to the yaml binding, and it contains no information
> about the cell format, making all references to it for the cell format
> unhelpful.
>=20
> Fixes: 89650a1e3b6f ("dt-bindings: pwm: Convert PWM bindings to json-sche=
ma")
> Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
> ---
>=20
> I yoinked the wording from pwm.txt. Having "Typically" feels a bit
> clumsy, but a given provider is free to override that order AFAIU.

All drivers that have 3 cells use that order, some drivers use less and
skip different values.

The patch looks fine, thanks. Will apply it after the merge window.

Thanks
Uwe

--=20
Pengutronix e.K.                           | Uwe Kleine-K=F6nig            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |

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

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

iQEzBAABCgAdFiEEP4GsaTp6HlmJrf7Tj4D7WH0S/k4FAmZJIm8ACgkQj4D7WH0S
/k7DTwgAraN3fJOaIOLC5VKHDnwJQlMIaZjEL5SGJiFHDYb/4B17nm6oWl2fOpLb
w/2Fky3ybzg8/wqbDWQOOm9/3qK7MD5cF+pZWr3P9DH+tXVh4u/719TJ9YkaSKJ1
KW/gRT1PaDyibyJ7Qp4og+DNfpK0wg9W3dd3JE/Ggczi+jDf2+L0tNlj5Gmf7NuE
f767PhkURNijgxwZagcwgYlO0oHahLW6ORnt4k+48sYLdmTBVGwzj92jIVpd2K6d
LqAd3iHHHdneIXwX7BX1F3k7Zfjg1Rg1umaJ5I+roEmcxYsynn3ULCjPeVhN7CKV
WAcG4VDTeF8RaqPeQ60EuPQeJoVnog==
=RAwU
-----END PGP SIGNATURE-----

--k5mzdz6sjp46o5pa--