From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f181.google.com (mail-pf1-f181.google.com [209.85.210.181]) (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 D3B6F2D0620 for ; Thu, 15 Jan 2026 06:00:16 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.181 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768456818; cv=none; b=uOfAIgVybMgmgSAXBZ4xhb/99y3GXE+TnU3Zi2v9KFbSGho9+//XnUwB73gvY/A5daOWqFhFUTs3fjId5QMNHV2Pf9fvqrNIE88764CrE+DYwir6G1xS3kNkZs3P6Zme2QomJUihHYUwoabAn+i44/WTlch2ljpU7fDyQQ/z2OA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768456818; c=relaxed/simple; bh=M2emRNPsd5dLS6UEkhkUcbjO9H7QF2ZNB8ZeyoJWdhM=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=VM5+iRVhizQ3OKTROdxb7bibk85ztWi8vQhv67LGlWRd2JG6ZwF7OoAHCCTsiBHOyMx8cM3+EulBdyEaoqqRvQNjGhC32p6hDVzxKLjimCjfe54JQzzqFeq13d9sEz0Z43CZz3yZCI15h8QLzRq10z+N13aSgD2MkLibe6c5Eyw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=IbWQyiY8; arc=none smtp.client-ip=209.85.210.181 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="IbWQyiY8" Received: by mail-pf1-f181.google.com with SMTP id d2e1a72fcca58-81f4e136481so246867b3a.3 for ; Wed, 14 Jan 2026 22:00:16 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1768456816; x=1769061616; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=dDgez86Z8vlifJV/rVDagwU1IW0RLqLFSFiUpTN6tUg=; b=IbWQyiY8Yjg8zvPebTf8SXW/92XlwmYeJhCsY4Z0XnnOr9rDqOGrLGqIdgdImrM6AO CWV5CDDSLRilTWMRovnEUp+bx9KYxZHC3WxlAUI8ivFMXmpo/WSPhiD3YRCJjeSw7C1L mzTdFYvBphA4J6C14Vq6eyS/CGsw5AXmBQ1enr8a5QBrdFsQgGQ8wCYLS0CnENkkbKrr scC7TSxkJc/q2SXjU0S1eW+Ua/mNaeBlzqJfCInDB3jupc/8Q0DwGOp3XRUHNZ7S+roh hqlaoZ7tkIxkY3G5tsp1gnlU0ejL4AKOdfaHPqfcLWJV6sFFSp111l+J6mK62+koHbPA 0KWA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1768456816; x=1769061616; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=dDgez86Z8vlifJV/rVDagwU1IW0RLqLFSFiUpTN6tUg=; b=MVYW077Xk6BdPJWUvxUDcHucBc+gRZJtCpi+BESBoAPDVFlmHRjYSEupRHncdXx6Is Tu6cd5XCvyA7FzzpKHY+2TFrllwjpwg0vzI1t/rkfUGCNCbraxUkDWcMWnUbmCbgpRyI 12u3p0jgLGG7v2irwrxYb5jKMPWpR3EuYQPwsOU2ikBdNnP0pxJ2vpWXetWE0c/KjP/c LSQ8pLyscAeWVJ/6gJG/P0eVIzeJZyHflKVizMf9PiQbtT7CpxZPmzkzNeTYmtw5YPo1 oiKxkve6zDTgy+P/2KsCK82m5CSBxoxYjXDV+7t5rV3j9Nh/KcckWp1iuJ2ZteCb7Q8/ TXzg== X-Forwarded-Encrypted: i=1; AJvYcCUl9iwImS3/BVJxiYzA2UIb5f4UkbdDavpIAfL3mkAOvOAuAWrAv6UhQR0N/yngo6bVgvXELfP2x+vBvRg=@vger.kernel.org X-Gm-Message-State: AOJu0YzZ7mRBDXJqPH3ZkLs6HYL4xEZ8RnJtsspbCFwRnPf3mwmK8CDx eLGKSSYgoyCz0pO4pa09dZoJ8ybPXV9M0llLGqEpk2+dVQ4BZ2W4de74 X-Gm-Gg: AY/fxX4FgbdV2YNDg4nZnIcpAyO8qgHMc9fTsmV9f8kObg8bN5pooThP3JsP2pzZha1 ot7RQ+5r/yKKXmbZhG28RgoVZkISVgqp2wovRDY1+PhiDuFrIqQ9XdzMVwl5FrQG1Aiy4ojAOWl SgVkinL6W5qxKpU78jCim1oNd2kSi9OGSM8bLSiDqjb/b1VEsWtOqhTC6HVghtq8MjeQ8/qK1av 2vQmuSGLufcWJBeBAwNDZeagJkt/kqgaJztuLYpV5Jqho5XX2nezpQkiPt1GFfdONovCspz/QSy qFpwhNEAUKbDj8gwK1BlM7vqLr7b3JudVnKHwNV37qBXTXiZnnqL35VuLwMWipNyW/t74bYyRC2 oYbRoaGD7vmToVtQhxJE8ZYjAOp3GHt/kOZw3+/Yz09GMU5min+8T1U81HGprkSAghlzf/R8Ki8 DZ1V0nF9kV2EejKeRBf/vYUg2gZnG7tx9K+w== X-Received: by 2002:a05:6a21:32a8:b0:35f:10a7:df67 with SMTP id adf61e73a8af0-38bed0d6fbcmr5294061637.17.1768456815900; Wed, 14 Jan 2026 22:00:15 -0800 (PST) Received: from [172.16.20.12] ([136.226.253.21]) by smtp.gmail.com with ESMTPSA id 41be03b00d2f7-c4cbfd27953sm23744006a12.11.2026.01.14.22.00.12 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 14 Jan 2026 22:00:15 -0800 (PST) Message-ID: Date: Thu, 15 Jan 2026 11:30:40 +0530 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v12 2/3] arm64: dts: qcom: talos/qcs615-ride: Fix inconsistent USB PHY node naming To: Konrad Dybcio , andersson@kernel.org, konradybcio@kernel.org Cc: robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Dmitry Baryshkov References: <20260114100043.1310164-1-tessolveupstream@gmail.com> <20260114100043.1310164-3-tessolveupstream@gmail.com> <444b2e30-5675-4a17-89dc-ca24ca3910d9@oss.qualcomm.com> Content-Language: en-US From: tessolveupstream@gmail.com In-Reply-To: <444b2e30-5675-4a17-89dc-ca24ca3910d9@oss.qualcomm.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 14-01-2026 15:49, Konrad Dybcio wrote: > On 1/14/26 11:00 AM, Sudarshan Shetty wrote: >> The USB PHY nodes has inconsistent labels as 'usb_1_hspy' >> and 'usb_hsphy_2' across talos.dtsi and qcs615-ride.dts. >> This patch renames them to follow a consistent naming >> scheme. > > Sorry for only noticing this now, but usb_hsphy_x is wrong.. > > *every* other dtsi uses usb_x_hsphy, please flip it around > > $ rg 'usb_1_hsphy' arch -l | wc -l > 93 > > $ rg 'usb_hsphy_1' arch -l | wc -l > 0 > I've updated the patch to use usb_x_hsphy naming convention to match the rest of the devicetree files as below: diff --git a/arch/arm64/boot/dts/qcom/qcs615-ride.dts b/arch/arm64/boot/dts/qcom/qcs615-ride.dts index 0ca6b50a6de1..a5f763cf1a55 100644 --- a/arch/arm64/boot/dts/qcom/qcs615-ride.dts +++ b/arch/arm64/boot/dts/qcom/qcs615-ride.dts @@ -609,7 +609,7 @@ bluetooth { }; }; -&usb_hsphy_1 { +&usb_1_hsphy { vdd-supply = <&vreg_l5a>; vdda-pll-supply = <&vreg_l12a>; vdda-phy-dpdm-supply = <&vreg_l13a>; @@ -632,7 +632,7 @@ &usb_1_dwc3 { dr_mode = "peripheral"; }; -&usb_hsphy_2 { +&usb_2_hsphy { vdd-supply = <&vreg_l5a>; vdda-pll-supply = <&vreg_l12a>; vdda-phy-dpdm-supply = <&vreg_l13a>; diff --git a/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi index 95ed335bcb08..a236f8a65937 100644 --- a/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi +++ b/arch/arm64/boot/dts/qcom/talos-evk-som.dtsi @@ -556,7 +556,7 @@ &usb_1_dwc3 { dr_mode = "host"; }; -&usb_hsphy_1 { +&usb_1_hsphy { vdd-supply = <&vreg_l5a>; vdda-pll-supply = <&vreg_l12a>; vdda-phy-dpdm-supply = <&vreg_l13a>; @@ -572,7 +572,7 @@ &usb_2_dwc3 { dr_mode = "host"; }; -&usb_hsphy_2 { +&usb_2_hsphy { vdd-supply = <&vreg_l5a>; vdda-pll-supply = <&vreg_l12a>; vdda-phy-dpdm-supply = <&vreg_l13a>; diff --git a/arch/arm64/boot/dts/qcom/talos.dtsi b/arch/arm64/boot/dts/qcom/talos.dtsi index c7dc507a50b5..cb32bfe732fb 100644 --- a/arch/arm64/boot/dts/qcom/talos.dtsi +++ b/arch/arm64/boot/dts/qcom/talos.dtsi @@ -4304,7 +4304,7 @@ osm_l3: interconnect@18321000 { #interconnect-cells = <1>; }; - usb_hsphy_1: phy@88e2000 { + usb_1_hsphy: phy@88e2000 { compatible = "qcom,qcs615-qusb2-phy"; reg = <0x0 0x88e2000 0x0 0x180>; @@ -4319,7 +4319,7 @@ usb_hsphy_1: phy@88e2000 { status = "disabled"; }; - usb_hsphy_2: phy@88e3000 { + usb_2_hsphy: phy@88e3000 { compatible = "qcom,qcs615-qusb2-phy"; reg = <0x0 0x088e3000 0x0 0x180>; @@ -4412,7 +4412,7 @@ usb_1_dwc3: usb@a600000 { iommus = <&apps_smmu 0x140 0x0>; interrupts = ; - phys = <&usb_hsphy_1>, <&usb_qmpphy>; + phys = <&usb_1_hsphy>, <&usb_qmpphy>; phy-names = "usb2-phy", "usb3-phy"; snps,dis-u1-entry-quirk; @@ -4476,7 +4476,7 @@ usb_2_dwc3: usb@a800000 { iommus = <&apps_smmu 0xe0 0x0>; interrupts = ; - phys = <&usb_hsphy_2>; + phys = <&usb_2_hsphy>; phy-names = "usb2-phy"; snps,dis_u2_susphy_quirk; (END) }; - usb_hsphy_1: phy@88e2000 { + usb_1_hsphy: phy@88e2000 { compatible = "qcom,qcs615-qusb2-phy"; reg = <0x0 0x88e2000 0x0 0x180>; @@ -4319,7 +4319,7 @@ usb_hsphy_1: phy@88e2000 { status = "disabled"; }; - usb_hsphy_2: phy@88e3000 { + usb_2_hsphy: phy@88e3000 { compatible = "qcom,qcs615-qusb2-phy"; reg = <0x0 0x088e3000 0x0 0x180>; @@ -4412,7 +4412,7 @@ usb_1_dwc3: usb@a600000 { iommus = <&apps_smmu 0x140 0x0>; interrupts = ; - phys = <&usb_hsphy_1>, <&usb_qmpphy>; + phys = <&usb_1_hsphy>, <&usb_qmpphy>; phy-names = "usb2-phy", "usb3-phy"; snps,dis-u1-entry-quirk; @@ -4476,7 +4476,7 @@ usb_2_dwc3: usb@a800000 { iommus = <&apps_smmu 0xe0 0x0>; interrupts = ; - phys = <&usb_hsphy_2>; + phys = <&usb_2_hsphy>; phy-names = "usb2-phy"; snps,dis_u2_susphy_quirk; Could you please confirm if this change aligns with your expectation. > Konrad