From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id B1F22C54E5D for ; Mon, 18 Mar 2024 23:59:51 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date: Reply-To:Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=oT0LIKTHTd8jOkLm+T6xfedP8VeDl9K60TTs3j6gFE8=; b=Nz017uA24gSfG7yiaSiDO5Gep5 /n151gUQEathYt2tvapCh1Tho8QbMmtD4IvZgbfLXCx8ofaJGypOgr9Ap4BLJxAAJnFDNLUGzSz3a PDhkboH7MSz+QeqCiyDEtyROQsaRTd++QZjs1lKFEcXZuBBlA8HDKkYRqjlEJFqopBFwel5kYqsbZ yD1NiXuKyh04w6zev3ag1RcG+rt3MBqbQD/sUEPhXi7ysi2zKYkK38N8aim78a0Ea0cxdNkWyF7kF MFaGw96ktQjCGgf84i/uhNTok194jibxHA3T463QcyPA1z+AGoU6mhwoTb+P2tu6WnazrkfAf1hK8 M6RYDecQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rmMtG-0000000AcmQ-00DG; Mon, 18 Mar 2024 23:59:38 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rmMtC-0000000Acll-1yVi for linux-arm-kernel@lists.infradead.org; Mon, 18 Mar 2024 23:59:35 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 248DF60C80; Mon, 18 Mar 2024 23:59:33 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 2F906C433C7; Mon, 18 Mar 2024 23:59:30 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1710806372; bh=/cuO+2prC5+m8EiBqi7dT0cOyOlyonzfiRrZm1oPk7o=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=C1AdiFvAp5GBUxAns2H0MMAN9yTybi5z6O17Afr1xOowOuzOjbiQmOXLsX1anKBLE ekuU64TRkO34e0BQBu7aabyDQG2sab2rSYLeZaY1509JycooK2lwRbysy0jPUfvXss ZRCF8DJJX02IUQkm1/E1tAUjhprfG0K9DFFFIuIyH+8Pt2mMW0mjy6g9YOtZDzEl// vXvyhqPPectG1Zd3rdKITre0iOJNGzLglOg7oAZq6oXMXE25yfVzzwGLC+WhuuyUz3 1caAiqVqEZBiePx3A8nkfPjnmZNHqtNaEsG2uvZdBwwNRX10XivuZ6IDrtK5JMSXqO vTJEHG9cUEnpw== Date: Mon, 18 Mar 2024 23:59:28 +0000 From: Conor Dooley To: Krzysztof Kozlowski Cc: Sylwester Nawrocki , Chanwoo Choi , Alim Akhtar , Michael Turquette , Stephen Boyd , Rob Herring , Conor Dooley , 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: <20240318-mummify-helmet-91a1f970a07d@spud> References: <20240312185035.720491-1-krzysztof.kozlowski@linaro.org> <20240317-curator-smoky-99568f9308bc@spud> <60039f49-a20d-49b9-8a3d-2ded499435a4@linaro.org> <20240317-jersey-trolling-d4678546e87d@spud> MIME-Version: 1.0 In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240318_165934_651495_0CDE624B X-CRM114-Status: GOOD ( 30.16 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: multipart/mixed; boundary="===============2445064019652978290==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============2445064019652978290== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="phI+MIIf734WzXdq" Content-Disposition: inline --phI+MIIf734WzXdq Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Mar 18, 2024 at 05:20:50PM +0100, Krzysztof Kozlowski wrote: > On 17/03/2024 16:49, Conor Dooley wrote: > > On Sun, Mar 17, 2024 at 04:26:55PM +0100, Krzysztof Kozlowski wrote: > >> On 17/03/2024 16:23, Conor Dooley wrote: > >>> 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 i= s 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 suitab= le > >>> place to mention these? > >> > >> To make it correct, these should be made clock inputs to the clock > >> controller, even if the driver does not take them, however that's > >> obsolete platform which might be removed from kernel this or next year, > >> so I don't want to spend time on it. > >=20 > > I think the comment should probably mention that these are the expected > > inputs, part of me thought that that was what you were getting at but I > > wasn't sure if instead they were inputs to some other IP on the SoC. >=20 > I can change it, but just to emphasize: in half a year or next year we > will probably remove entire platform, thus also this binding. I know, I saw that. I don't really care what you do given the platform is being deleted and it is unlikely that anyone is actually going to be assembling a from-scratch dtsi for this SoC. On the other hand, if you're doing a conversion, even in this scenario, I think it should be clear.=20 I didn't ack the patch cos I figured you were taking the patch via the samsung tree (and on to Stephen) yourself, but here: Acked-by: Conor Dooley I'd rather argue about the definition of erratum instead of this :) --phI+MIIf734WzXdq Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZfjVXwAKCRB4tDGHoIJi 0oZhAQD+pmOY+P/bz4THs5Dh9jBUlSFieYOnAX0+K/xeZPaoigEAtpb48qbPfE/w ORq53TSn9+qiVqYzXqWjizxmSyWJRgw= =BACD -----END PGP SIGNATURE----- --phI+MIIf734WzXdq-- --===============2445064019652978290== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============2445064019652978290==--