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 8A56DCEFC2E for ; Tue, 8 Oct 2024 16:34:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: 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=hHt3JYUZBYz/0Fp5XNwlOF4V3R9H9TlZQyYPHiMqL64=; b=OHvVVLEsK32g4RNKEU5PgW8brM 9DErJnApad/6R29L2ZMbwtbHFUgLSqrmoiPI5+48R38h4rCDbaIAvvP/qI1C5QTBalLzcw9o02CSA H2H67uAtlgh2btz0oyjWeE91jYGQp+0vDiHGwnEL1bm7HhT+x7TpiWOhA/AtHmrdVdnc9YAQlPeC6 CJxByjEskk/REPnA2m9Qai8Ji7+zY3J4Htm/m7DljX/Ty0iFA059Jhqq0k7imxnKwdoBXFRlhOfeo xXoWJBN5y29DMexPPq3C9eeXkXWI/uOya5+E4tKWWhISniCyii0G1pVLgYcJhSSPdcVvvdIv3hHck pbn/W+nA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1syD9t-00000006VZt-1ufN; Tue, 08 Oct 2024 16:34:01 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1syD7H-00000006V5p-3Z6D; Tue, 08 Oct 2024 16:31:21 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id B791D5C4F4D; Tue, 8 Oct 2024 16:31:14 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5CDD2C4CEC7; Tue, 8 Oct 2024 16:31:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1728405078; bh=0lgV2rFoiw5wtsRBRpfDph44UOojRfjp31BzeyWcZYc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=kCJ0G0KQB3IkegGfi7xpcFnytubZPjBoE9f+MGXwi81fbYaacAABX1ePDBohhRTUI 2owuVDaqFcWK/DcJAqQQVlqMCfMvUCAZmbzaoDGpn6Q4uSg19YsI23o/PMcP4F6xYk pKDmaT1Wt3/gJDY2UjIXBtu8Ek2HKwamzbfyYq5moxFEh+/ta+TsNFPmtWabP4PgCQ 54ItjmfveVxPymysSocAKJ02/NMv+cTF+tuHAlHqdvI0wVGJeL6Lp5Bx90UAQXZPRN lNH/E77GhvHD+ep4gJMlH1gfJOp6dnTNWlfOnw9PgLOxOMiAhgx2UQeUNsx0G/j9/H +cdsvIqCUoJcQ== Date: Tue, 8 Oct 2024 17:31:12 +0100 From: Conor Dooley To: Macpaul Lin Cc: Jason-ch Chen , AngeloGioacchino Del Regno , Chunfeng Yun , Vinod Koul , Kishon Vijay Abraham I , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Matthias Brugger , linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-phy@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Alexandre Mergnat , Bear Wang , Pablo Sun , Macpaul Lin , Sen Chu , Chris-qj chen , MediaTek Chromebook Upstream , Chen-Yu Tsai Subject: Re: [PATCH] dt-bindings: phy: mediatek: tphy: add a property for power-domains Message-ID: <20241008-disorder-slacking-d8196ceb68f7@spud> References: <20240926101804.22471-1-macpaul.lin@mediatek.com> <20240926-treadmill-purr-b2e3279a14a4@spud> <3a970560-2458-f5f3-87c5-925079fa12a4@mediatek.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="63wFwyn02Gwtymi/" Content-Disposition: inline In-Reply-To: <3a970560-2458-f5f3-87c5-925079fa12a4@mediatek.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241008_093120_000585_2BCA0674 X-CRM114-Status: GOOD ( 26.46 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --63wFwyn02Gwtymi/ Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Oct 08, 2024 at 12:03:37PM +0800, Macpaul Lin wrote: >=20 >=20 > On 9/27/24 00:25, Conor Dooley wrote: > > On Thu, Sep 26, 2024 at 06:18:04PM +0800, Macpaul Lin wrote: > > > Some platforms requires a dependency for power-domains. > >=20 > > Some, so not all? Why isn't this restricted on a per compatible basis? >=20 > After discussion with Chunfeng and double check tphy design in detail. > Chunfeng commented that tphy dose not need to add mtcmos. > It is not necessary to add it, if the power of the phy is turned off, > it will affect other functions. >=20 > From the current USB hardware design perspective, even if mtcmos > is added to the phy, it is always on. Firstly, I have no idea what "mtcmos" means, sorry. I am a dt-bindings guy, not someone familiar with mediatek hardware. Secondly, it sounds like this /does/ have a power domain, so it should be in the binding. That it is turned on by something else and must remain on doesn't mean it shouldn't be documented here. What if those things try to turn it off while the tphy is using it? >=20 > > > So we add property 'power-domains' and set 'maxItems: 1' in the > > > DT Schema. > > >=20 > > > Signed-off-by: Macpaul Lin > > > --- > > > Documentation/devicetree/bindings/phy/mediatek,tphy.yaml | 3 +++ > > > 1 file changed, 3 insertions(+) > > >=20 > > > diff --git a/Documentation/devicetree/bindings/phy/mediatek,tphy.yaml= b/Documentation/devicetree/bindings/phy/mediatek,tphy.yaml > > > index 423b7c4e62f2..c77fe43c224a 100644 > > > --- a/Documentation/devicetree/bindings/phy/mediatek,tphy.yaml > > > +++ b/Documentation/devicetree/bindings/phy/mediatek,tphy.yaml > > > @@ -125,6 +125,9 @@ properties: > > > $ref: /schemas/types.yaml#/definitions/uint32 > > > default: 28 > > > + power-domains: > > > + maxItems: 1 > > > + > > > # Required child node: > > > patternProperties: > > > "^(usb|pcie|sata)-phy@[0-9a-f]+$": > > > --=20 > > > 2.45.2 > > >=20 >=20 > Please drop this patch and I'll send a new fix to mt8195.dtsi. >=20 > Thanks > Macpaul Lin --63wFwyn02Gwtymi/ Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZwVeUAAKCRB4tDGHoIJi 0qBJAP942MK2haivS9lUOXeS3zszXJHYx8a0MTJissHiNv9Y6AEAlULCD/bffod/ Ajhj7718HDTdpLU80QNMjPu2uFTR0Qg= =MNh6 -----END PGP SIGNATURE----- --63wFwyn02Gwtymi/--