From patchwork Fri Dec 6 16:31:02 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Andr=C3=A9_Draszik?= X-Patchwork-Id: 13897439 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 2F15FE77173 for ; Fri, 6 Dec 2024 16:35:16 +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-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:Cc:To:In-Reply-To:References:Message-Id :MIME-Version:Subject:Date:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=VugotS20M+jRfHmakgF9cuBe+j99kM2ZlUJygH6H75U=; b=kWJ8/DyBygsY2S 3kasxhwLX1eNiwQatStKF3aIfkdqLBXv6JqI0MdxAl06EA65bzPWXAyWgjpBLXm8UpWgBqJClPlPW YYPwEds9z5j79ig/0g5YJ4jPBK+YxgIS3klRTjYXoFoZebtNYrjvFXC0cnLrb2OIu6KpziPsNr7QR Xf9/kb4s63izF8ys86FqqKgBeLQPOry0HxsvhpXWJ1YTTuQ5KmUpunCq53JWCP3U4cS9deM7AFi3/ zoa74gLrqOozHsHtqPUfaAJcYxpq7kJLJIxlSkuKRusy+QKaOI+jRbHX0ZCl2uptKlyMSfXw5xcOS eG5CpyG8fqb0SbIM6h6A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tJbIR-00000002BTP-3zTe; Fri, 06 Dec 2024 16:35:15 +0000 Received: from mail-ej1-x631.google.com ([2a00:1450:4864:20::631]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tJbEO-00000002AD2-2MUt for linux-phy@lists.infradead.org; Fri, 06 Dec 2024 16:31:07 +0000 Received: by mail-ej1-x631.google.com with SMTP id a640c23a62f3a-a9f1d76dab1so68311166b.0 for ; Fri, 06 Dec 2024 08:31:03 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1733502663; x=1734107463; darn=lists.infradead.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=PKgkddTbsb36SqzZl1dMALspeUepmvmFxGiaTIa2E3A=; b=f1/Gb1LVDJ2gygESXZ1YGtEpkj2zaIWqbsfaspr3BGU9ZFUPnjEyFS2o4giyvSsqMF Ocjd7zFlzjd4Inr4yC3PQ61oCV1JR2airmMq4W5MlsO0DYOSOI12+KecZLCoddUjfijl VSzGQ9JK9oc2GvfNrBx30xjQ23CZBD33gNJ+OLzwBTQS5fiZE8AuATupbczD1k3gb+J0 mP9BUNGDpNi4lnVol/VkfIIGAOHQPg4ZVBKUrqUYCsreeFgvjDnxH7g8rdou3AJy/fmi xfSEfjBKYzOxSrGzaWJpmifjqK0SAwE/rHsNbC7SjkpwqFIUZSY9a6an5de3K3ZBlclN XWSQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1733502663; x=1734107463; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=PKgkddTbsb36SqzZl1dMALspeUepmvmFxGiaTIa2E3A=; b=n66iZwIbHskqty283U5itTEysF2inyuLRDzLIAF7qA5rzahlPbyb3DaVBJGFIs79zH 0R1VnKlrwAthzvp1im+hgJe4pyAFSlYsWvgwYA5iBqPikTMfC4zhRf0V8bwd6Rrm8e+4 Ved0sxMTlqzCtrzrzV7PVhy2b3AOdpHY91Uzk+1lXFdV7JSxLBL5Tw29Zk5eFwNAtS+b 3ORJWapyxP7MMQn7q0DEjh0arSgXcacz04UmZFuwUB/ybghrfiBgqYRnN78jPT2uADMb ZyHxgo5MsRWiGJDrfrybEGefFtqeIa18L1dhk4sGZh3gbVJ1LcKk1gM8jJaeA+ccSUZK SysQ== X-Forwarded-Encrypted: i=1; AJvYcCXhAs949wdpx12Dnd9G2af5+OH6/8S7jYHerUKpgeMyCjpKj68h+8CPHSt0XdS47nP0eAng40gaw0Y=@lists.infradead.org X-Gm-Message-State: AOJu0YysT/PKWmN3MKQrb8YkumMARwHtTkMrqeZ865L4CgDuPIaoHtoE wsqrfHRArsQni3adile4My6n+HJQWcu8BbcWWpRBdjPJ7wwe4RhnydR/QOVDCrE= X-Gm-Gg: ASbGncuJG1RTAFu17lT/wMdMC8jGn5edCuJEwKK7GdGxVmvFe8cEBaB79UBpd0a/Er0 jtOUBx8TZYz60opGXa91eATLr9Mg/SYOoA2G3vvsY8b3zUnnhdFa9+XqeS0D8Ll3Ku87hW50Y7S JlFiL+mtltV5D+3cb9GOGnmNBXZOAL8QCGrK2UkqwIYFyEQtI4mZQGsTTPG+9F1r/qMgbXXEDma xmGSop5BY1XP+hRifNry4X4LsROOaf0uGnkrnmyw5pFRvQ28dRadKwEajFMRXZeFgTGm8pMz9nx 5tvpYPdVU1VZpC2EqV0oQBTpDA+XcbTXCg== X-Google-Smtp-Source: AGHT+IGRlX7BIbgKcq4E8LlKztNa00vBgQAERS4YEdJ5MYW+4Wqp/mGqY/4IqsX/I0DaInCg04SuNg== X-Received: by 2002:a17:907:7841:b0:aa6:32c0:5d05 with SMTP id a640c23a62f3a-aa639febf00mr320394766b.13.1733502662697; Fri, 06 Dec 2024 08:31:02 -0800 (PST) Received: from puffmais.c.googlers.com (64.227.90.34.bc.googleusercontent.com. [34.90.227.64]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-aa6260e8af8sm257710266b.191.2024.12.06.08.31.02 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 06 Dec 2024 08:31:02 -0800 (PST) From: =?utf-8?q?Andr=C3=A9_Draszik?= Date: Fri, 06 Dec 2024 16:31:02 +0000 Subject: [PATCH v4 2/7] dt-bindings: phy: samsung,usb3-drd-phy: gs101: require Type-C properties MIME-Version: 1.0 Message-Id: <20241206-gs101-phy-lanes-orientation-phy-v4-2-f5961268b149@linaro.org> References: <20241206-gs101-phy-lanes-orientation-phy-v4-0-f5961268b149@linaro.org> In-Reply-To: <20241206-gs101-phy-lanes-orientation-phy-v4-0-f5961268b149@linaro.org> To: Vinod Koul , Kishon Vijay Abraham I , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Marek Szyprowski , Sylwester Nawrocki , Alim Akhtar Cc: Peter Griffin , Tudor Ambarus , Sam Protsenko , Will McVicker , Roy Luo , kernel-team@android.com, linux-phy@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, =?utf-8?q?Andr=C3=A9_Draszik?= X-Mailer: b4 0.13.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241206_083104_605488_624E092B X-CRM114-Status: GOOD ( 10.89 ) X-BeenThere: linux-phy@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux Phy Mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-phy" Errors-To: linux-phy-bounces+linux-phy=archiver.kernel.org@lists.infradead.org orientation-switch is the standard declaration to inform the Type-C mux layer that a remote-endpoint is capable of processing orientation change messages. The USB PHY on gs101 needs to be configured based on the orientation of the connector. For that the DTS needs a link between the phy's port and a TCPCi, and we'll need to inform the phy driver that it should handle the orientation (register a handler). Update the schema to enforce that by requiring the orientation-switch and port properties on gs101 (only). We disallow orientation-switch on all other supported platforms, since other versions of this phy (or its system integration) don't currently support or even need it. Even though this new required gs101 property is an ABI break, the intention for the driver is to behave as before if it's missing (meaning for gs101 it will work in SS mode in one orientation only). Other platforms are not affected. Reviewed-by: Peter Griffin Signed-off-by: André Draszik Reviewed-by: Rob Herring (Arm) --- v3: * update as per Rob's suggestion (I hope :-) v2: * squash original patches #2 and #3 * actually disallow orientation-switch on !gs101 (not just optional) (Conor) * update commit message to clarify that the intention for the driver is to work with old and new DTS (Conor) * collect tags foo --- Documentation/devicetree/bindings/phy/samsung,usb3-drd-phy.yaml | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/phy/samsung,usb3-drd-phy.yaml b/Documentation/devicetree/bindings/phy/samsung,usb3-drd-phy.yaml index 1f8b35917b11..27295acbba76 100644 --- a/Documentation/devicetree/bindings/phy/samsung,usb3-drd-phy.yaml +++ b/Documentation/devicetree/bindings/phy/samsung,usb3-drd-phy.yaml @@ -114,6 +114,8 @@ allOf: contains: const: google,gs101-usb31drd-phy then: + $ref: /schemas/usb/usb-switch.yaml# + properties: clocks: items: @@ -139,6 +141,8 @@ allOf: required: - reg-names + - orientation-switch + - port - pll-supply - dvdd-usb20-supply - vddh-usb20-supply @@ -198,7 +202,7 @@ allOf: reg-names: maxItems: 1 -additionalProperties: false +unevaluatedProperties: false examples: - |