Received-SPF: pass (google.com: domain of linux-kernel+bounces-105443-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Date: Sun, 17 Mar 2024 15:23:35 +0000
From: Conor Dooley <conor@kernel.org>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: Sylwester Nawrocki <s.nawrocki@samsung.com>,
	Chanwoo Choi <cw00.choi@samsung.com>,
	Alim Akhtar <alim.akhtar@samsung.com>,
	Michael Turquette <mturquette@baylibre.com>,
	Stephen Boyd <sboyd@kernel.org>, Rob Herring <robh@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	linux-samsung-soc@vger.kernel.org, linux-clk@vger.kernel.org,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH] dt-bindings: clock: samsung,s3c6400-clock: convert to DT
 Schema
Message-ID: <20240317-curator-smoky-99568f9308bc@spud>
References: <20240312185035.720491-1-krzysztof.kozlowski@linaro.org>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
	protocol="application/pgp-signature"; boundary="avAJPhhF1TQgfXmM"
Content-Disposition: inline
In-Reply-To: <20240312185035.720491-1-krzysztof.kozlowski@linaro.org>


--avAJPhhF1TQgfXmM
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Tue, Mar 12, 2024 at 07:50:35PM +0100, Krzysztof Kozlowski wrote:
> Convert Samsung S3C6400/S3C6410 SoC clock controller bindings to DT
> schema.

> +description: |
> +  There are several clocks that are generated outside the SoC. It is expected
> +  that they are defined using standard clock bindings with following
> +  clock-output-names:
> +   - "fin_pll" - PLL input clock (xtal/extclk) - required,
> +   - "xusbxti" - USB xtal - required,
> +   - "iiscdclk0" - I2S0 codec clock - optional,
> +   - "iiscdclk1" - I2S1 codec clock - optional,
> +   - "iiscdclk2" - I2S2 codec clock - optional,
> +   - "pcmcdclk0" - PCM0 codec clock - optional,
> +   - "pcmcdclk1" - PCM1 codec clock - optional, only S3C6410.

I know you've only transfered this from the text binding, but what is
the relevance of this to the binding for this clock controller? This
seems to be describing some ?fixed? clocks that must be provided in
addition to this controller. I guess there's probably no other suitable
place to mention these?

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

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

iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZfcK9wAKCRB4tDGHoIJi
0n1jAP4uBRwE/QjP+OzagdtaN0k4KA+oynX+Mt38hB7fNOfFLQEAst51oJuYomCe
qSldXRE+01t3wKQnk9claOuGWyjV2g0=
=ujPR
-----END PGP SIGNATURE-----

--avAJPhhF1TQgfXmM--