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 B83BCC83F17 for ; Wed, 16 Jul 2025 01:21:54 +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:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=y9UGnpqAWUlGLKMTmekl851slDMZjV5j64meupx2I6w=; b=y5ScqtpPhJZ8l7dKvaKVebz22f j33e6xNhtrCWBjLFZOk0FsH7ZZPhcUEuUOha3a4POKLfs6dY0CBFXBX9oHJqb/XL3OjUOYLiO048E tLvDSc+Ur3rafXMbztxBFVvlApUI53YmmAgWS8mt811+xEsdwq3tcCYgDC8zn5n+kUkQOeB98x6hi cOh74eJZkS0mwZF8b0E7bMHlopyufMnvYof2dSGMZHH7+6Q7T6BXzFLyZHWtTUgquZN6Sab8X/QDu P6FHUxYJVXHxgYXSV1SDyJwoD/UcyE+AX8qT3p3mwutW+0yo6h+4uEJcUARoMzR3OekZeXAj1pAxC J2ktB4pg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1ubqqC-00000006XLL-2fF4; Wed, 16 Jul 2025 01:21:48 +0000 Received: from mail-m1973191.qiye.163.com ([220.197.31.91]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1ubqne-00000006X6S-41xJ; Wed, 16 Jul 2025 01:19:12 +0000 Received: from [127.0.0.1] (gy-adaptive-ssl-proxy-3-entmail-virt135.gy.ntes [58.22.7.114]) by smtp.qiye.163.com (Hmail) with ESMTP id 1c20f26cc; Wed, 16 Jul 2025 09:18:56 +0800 (GMT+08:00) Message-ID: <9e6c432a-9bed-474c-afa9-fd6ada8bb8f8@rock-chips.com> Date: Wed, 16 Jul 2025 09:18:55 +0800 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 2/5] dt-bindings: phy: rockchip: rk3399-typec-phy: Support mode/orientation switch To: Krzysztof Kozlowski , Chaoyi Chen , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Vinod Koul , Kishon Vijay Abraham I , Heiko Stuebner , Sandy Huang , Andy Yan , Yubing Zhang , Frank Wang , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Amit Sunil Dhamne , Greg Kroah-Hartman , Dragan Simic , Johan Jonker , Diederik de Haas Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-phy@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, dri-devel@lists.freedesktop.org References: <20250715112456.101-1-kernel@airkyi.com> <20250715112456.101-3-kernel@airkyi.com> <4dfed94c-665d-4e04-b527-ddd34fd3db8f@kernel.org> From: Chaoyi Chen In-Reply-To: <4dfed94c-665d-4e04-b527-ddd34fd3db8f@kernel.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-HM-Spam-Status: e1kfGhgUHx5ZQUpXWQgPGg8OCBgUHx5ZQUlOS1dZFg8aDwILHllBWSg2Ly tZV1koWUFDSUNOT01LS0k3V1ktWUFJV1kPCRoVCBIfWUFZGh0eSVZKSEpKS0pIQh5KHR1WFRQJFh oXVRMBExYaEhckFA4PWVdZGBILWUFZTkNVSUlVTFVKSk9ZV1kWGg8SFR0UWUFZT0tIVUpLSEpPSE xVSktLVUpCS0tZBg++ X-HM-Tid: 0a9810d0310c03abkunmd19fa1411423d24 X-HM-MType: 1 X-HM-Sender-Digest: e1kMHhlZQR0aFwgeV1kSHx4VD1lBWUc6NBg6LTo*FDE0NwlMMVFCFTc8 KTUKCSlVSlVKTE5JTUlDTE9NS01JVTMWGhIXVRgTGhQCElUYEx4VOwkUGBBWGBMSCwhVGBQWRVlX WRILWUFZTkNVSUlVTFVKSk9ZV1kIAVlBSExITTcG DKIM-Signature: a=rsa-sha256; b=WzL1xQNdru/zKe4DMlBTqwjgA2IwGOyI3/txUXdcBAVE1KQqUbQT9JBMVDdMm8ZIMgaP0HN8jb4iceyiqmOwPb0XX7j6gmpr27mF8e13nD8gV8pFdhA9hTR5AW1FxVdoVWvT2k+f5qv827ZzKc29N2+W2JtkLU7e4ajVmc3kr0g=; c=relaxed/relaxed; s=default; d=rock-chips.com; v=1; bh=y9UGnpqAWUlGLKMTmekl851slDMZjV5j64meupx2I6w=; h=date:mime-version:subject:message-id:from; X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250715_181911_162473_4953EA24 X-CRM114-Status: GOOD ( 17.92 ) 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 Hi Krzysztof, On 2025/7/15 19:34, Krzysztof Kozlowski wrote: > On 15/07/2025 13:24, Chaoyi Chen wrote: >> From: Chaoyi Chen >> >> Add support for Type-C orientation and altmode switch. >> The Type-C controller can be specified to handling switching. > Please describe the hardware in details. Above sentences don't help me > to understand this. I will add more description in v2. > > >> Signed-off-by: Chaoyi Chen >> --- >> .../bindings/phy/rockchip,rk3399-typec-phy.yaml | 14 ++++++++++++++ >> 1 file changed, 14 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/phy/rockchip,rk3399-typec-phy.yaml b/Documentation/devicetree/bindings/phy/rockchip,rk3399-typec-phy.yaml >> index 91c011f68cd0..a885c6893a90 100644 >> --- a/Documentation/devicetree/bindings/phy/rockchip,rk3399-typec-phy.yaml >> +++ b/Documentation/devicetree/bindings/phy/rockchip,rk3399-typec-phy.yaml >> @@ -43,6 +43,20 @@ properties: >> description: >> Phandle to the syscon managing the "general register files" (GRF). >> >> + orientation-switch: >> + description: Flag the port as possible handler of orientation switching >> + type: boolean >> + >> + mode-switch: >> + description: Flag the port as possible handler of altmode switching >> + type: boolean >> + >> + port: >> + $ref: /schemas/graph.yaml#/properties/port >> + description: >> + A port node to link the PHY to a TypeC controller for the purpose of >> + handling orientation switching. > > You are using usb-switch.yaml properties in phy node, which raises > questions whether this is actually complete. It might be, but commit msg > is so vague that I have doubts. > > Also, why only one port? > > Or wait... you already have ports! two of them. This needs to stop, why > are you adding more? Oh, I will try to reuse them. Will fix in v2. > > Best regards, > Krzysztof > >