From patchwork Sat May 11 22:04:09 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dmitry Baryshkov X-Patchwork-Id: 796261 Received: from mail-lf1-f43.google.com (mail-lf1-f43.google.com [209.85.167.43]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 65E2D2E85A for ; Sat, 11 May 2024 22:04:13 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.43 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715465055; cv=none; b=sJGlFWxf7890Bvg7h7vCgLRZC9yaTRs+SSUiasr6fPPtpefL7EbkZpBWjEkU09S/SO1Ang67obPr8VKfWi2gR4mrWMDvASMLKznuvubyxZJSG+X9EPd6sL7Zb1uCwIrUc98chJEp9k1vvUdZm6scAIfv70Liyt30PK2O1l8F6Zk= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715465055; c=relaxed/simple; bh=Xy9PldcR183DRnBL5409rKkzxlfVFfpBt9Qkn4329v8=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=Bdtw6XmyUturfqE41LiftciLHbJY5KpKOCLEJhg+ItqsL18GoxvPn3wkhQIxUiOOZp1dUdkZAmUlywlync7Y5Gt1Jjg8IeUfhAR3XTTcQkz+vGNcknMyXjDY50THVWMMhfVQwr4Ne/UncI864sxrfJBa9eN8pSw05K+26ZTx4Vc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=qQiFTy6Z; arc=none smtp.client-ip=209.85.167.43 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="qQiFTy6Z" Received: by mail-lf1-f43.google.com with SMTP id 2adb3069b0e04-51f0602bc58so3830415e87.0 for ; Sat, 11 May 2024 15:04:13 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1715465052; x=1716069852; darn=vger.kernel.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=0NiTVC0ap1sbMmX+P4azo73mVUt/soqEHYFiWewJv/E=; b=qQiFTy6Z93EMH6MuhQu1OQ9VE7kkwcd6fK8nc2CkKPkcBmge8hhBcJnNZdF1scpPcJ VaQgiVJsJVJ4v5iH/e4NAh8tJy4Ds85H86guQm1xgj7cskGkxAUT1PBfyuOlUwVSfbB1 ajJz2sSlnTlRwsNd/3aOn80Gg4IP7nun/Xk2BoTJeBnar2XpAKY0ReHcLYjnDxw8fkm6 gVuJXOAE1cGQhR5OAMR6ak7L7Ev9GqSHqd1vCoRU2XsGMTde+Rj/QTdOBviPgKAKBbvc k/BY/WcO5cEvW4N5V4Ao7K1M6ZsLH8kLPy4nU5MNIC9p5GqS14ijA6wfUf/tjGQ3nvjl Nv+g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1715465052; x=1716069852; 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=0NiTVC0ap1sbMmX+P4azo73mVUt/soqEHYFiWewJv/E=; b=KGtNPl1hj0YQq3Ce/N8MqVMC3FLSxYxnoXVnd3kZgmU5ymjJYrfvb1qEURDVUR36O2 9AXufuiyXDw0CvXQjBpEM61ws8LlBByi7nbZxyB0Erdl7tcLAxvAOW6COmFG+XE7YShA OoSXaCNCWd1WYpdnTZWHwXprCQv6g4b5Z6worE/SMZNfX17zfSipx3au0v+H/LThEH6+ 9D4xzZixfacYDyfggbSStZUCaifEFWfkXhvLO3HJKuov//ga2StF5vqJK4akeu5ZDlSi nvbaT7vW63Jj9LG3e6ERR7/0iMnWuyvbCioXWfIb1SEgsLGXrgCI7oR6WtW2UY+SKIhy 9tWg== X-Gm-Message-State: AOJu0YxZijE85rQ9OWotHJyr892lbQAaydGz8tNpOcxif3KtJOD0Bbiv 1GO6WhcvTHrjGp19Zn2DBgmppWOo+wUHWKLzL4VYqmRAIob2btV7VDyi/EpDufY= X-Google-Smtp-Source: AGHT+IEz5IPOIfCDmcunyW3gkasjS/UeOuHKyWwCEW7bEHsz6couePyR36Hvo3oJ0/6OzwNn4Jrz5w== X-Received: by 2002:a05:6512:2309:b0:51f:3e41:efd8 with SMTP id 2adb3069b0e04-5220e27a2ccmr2259688e87.1.1715465051765; Sat, 11 May 2024 15:04:11 -0700 (PDT) Received: from umbar.lan ([192.130.178.91]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-521f38d878fsm1123875e87.206.2024.05.11.15.04.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 11 May 2024 15:04:11 -0700 (PDT) From: Dmitry Baryshkov Date: Sun, 12 May 2024 01:04:09 +0300 Subject: [PATCH v4 3/9] arm64: dts: qcom: sm8250: describe HS signals properly Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Message-Id: <20240512-typec-fix-sm8250-v4-3-ad153c747a97@linaro.org> References: <20240512-typec-fix-sm8250-v4-0-ad153c747a97@linaro.org> In-Reply-To: <20240512-typec-fix-sm8250-v4-0-ad153c747a97@linaro.org> To: Bjorn Andersson , Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Bryan O'Donoghue Cc: linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Luca Weiss X-Mailer: b4 0.13.0 X-Developer-Signature: v=1; a=openpgp-sha256; l=2645; i=dmitry.baryshkov@linaro.org; h=from:subject:message-id; bh=Xy9PldcR183DRnBL5409rKkzxlfVFfpBt9Qkn4329v8=; b=owGbwMvMwMXYbdNlx6SpcZXxtFoSQ5r96/DeyuzcYMY3y9mPm9cVcx+9XqHY2r4uilGS939D9 tXa1qZORmMWBkYuBlkxRRafgpapMZuSwz7smFoPM4iVCWQKAxenAEzkqgoHwyKOrbUsTxKjfyfU ads9W3QualfKvEsnbkleMN3f3+QtLc90cR4b60HOikv71/K92Md04j3XJuEFDo9Lko4ukdD44XH k0TSBgq02UVv4513f7LXbJEheJHahe0SHifZ2rzIzQ//ft7KMf6r7zzgYVsMkUz+/JzNaW1N0Rg mXTsy9NTEcapd7g716P+qtnDa573H0lO6VLvW7G5UeVaQZJK/PuZ6QWBamldvLcqGXYylrXemGZ RdN5r3s/vPa7klK6aE9e2wyBZwuOl493PMzeQLj70tWC9T2P5RMOSr3wV6CZ4rkX4/vIWc7Dp7U f+fA61B9RP+JjdgZqcgC9eATwSu23IkJUG4zqZ0QmXEMAA== X-Developer-Key: i=dmitry.baryshkov@linaro.org; a=openpgp; fpr=8F88381DD5C873E4AE487DA5199BF1243632046A The OF graph should describe physical signals. There is no 'role switch' signal between Type-C connector and the DWC3 USB controller. Rename endpoints to mention USB HS signal instead (this follows the example lead by other plaforms, including QRB2210 RB1, QRB4210 RB2 and all PMIC GLINK platforms). Reviewed-by: Bryan O'Donoghue Signed-off-by: Dmitry Baryshkov --- arch/arm64/boot/dts/qcom/qrb5165-rb5.dts | 8 ++++---- arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-common.dtsi | 8 ++++---- arch/arm64/boot/dts/qcom/sm8250.dtsi | 2 +- 3 files changed, 9 insertions(+), 9 deletions(-) diff --git a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts index cd0db4f31d4a..44239d1e57e9 100644 --- a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts +++ b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts @@ -1356,8 +1356,8 @@ &usb_1_dwc3 { usb-role-switch; }; -&usb_1_role_switch_out { - remote-endpoint = <&pm8150b_role_switch_in>; +&usb_1_dwc3_hs_out { + remote-endpoint = <&pm8150b_hs_in>; }; &usb_1_hsphy { @@ -1465,8 +1465,8 @@ ports { port@0 { reg = <0>; - pm8150b_role_switch_in: endpoint { - remote-endpoint = <&usb_1_role_switch_out>; + pm8150b_hs_in: endpoint { + remote-endpoint = <&usb_1_dwc3_hs_out>; }; }; diff --git a/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-common.dtsi b/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-common.dtsi index 41f117474872..3596dd328c31 100644 --- a/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-common.dtsi +++ b/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-common.dtsi @@ -659,8 +659,8 @@ ports { port@0 { reg = <0>; - pm8150b_role_switch_in: endpoint { - remote-endpoint = <&usb_1_role_switch_out>; + pm8150b_hs_in: endpoint { + remote-endpoint = <&usb_1_dwc3_hs_out>; }; }; }; @@ -725,8 +725,8 @@ &usb_1_hsphy { status = "okay"; }; -&usb_1_role_switch_out { - remote-endpoint = <&pm8150b_role_switch_in>; +&usb_1_dwc3_hs_out { + remote-endpoint = <&pm8150b_hs_in>; }; &ufs_mem_hc { diff --git a/arch/arm64/boot/dts/qcom/sm8250.dtsi b/arch/arm64/boot/dts/qcom/sm8250.dtsi index 8ccade628f1f..0ec473ceb3d2 100644 --- a/arch/arm64/boot/dts/qcom/sm8250.dtsi +++ b/arch/arm64/boot/dts/qcom/sm8250.dtsi @@ -4226,7 +4226,7 @@ usb_1_dwc3: usb@a600000 { phy-names = "usb2-phy", "usb3-phy"; port { - usb_1_role_switch_out: endpoint {}; + usb_1_dwc3_hs_out: endpoint {}; }; }; };