* [PATCH v4 0/2] Add WiFi/BT node for qcs615 ride board
@ 2025-07-07 2:51 Yu Zhang(Yuriy)
2025-07-07 2:51 ` [PATCH v4 1/2] arm64: dts: qcom: qcs615: add a PCIe port for WLAN Yu Zhang(Yuriy)
2025-07-07 2:51 ` [PATCH v4 2/2] arm64: dts: qcom: qcs615-ride: add WiFi/BT nodes Yu Zhang(Yuriy)
0 siblings, 2 replies; 11+ messages in thread
From: Yu Zhang(Yuriy) @ 2025-07-07 2:51 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley
Cc: linux-arm-msm, devicetree, linux-kernel, Yu Zhang (Yuriy),
Konrad Dybcio
This patch series depends on:
- PCIe
https://lore.kernel.org/all/20250703095630.669044-1-ziyue.zhang@oss.qualcomm.com/
Changes in v4:
- Fix node placement issue.
- Link to v3: https://lore.kernel.org/r/20250704-615-v3-0-6c384e0470f2@oss.qualcomm.com
Changes in v3:
- Formatting: adjust layout based on reviewer feedback.
- Fix DT checking warnings.
- Link to v2:
https://lore.kernel.org/r/20250625-qcs615-v2-0-0f1899647b36@oss.qualcomm.com
Changes in V2:
- Correct the version metadata of the previous patch.
- Separate the modifications to the DTS and DTSI files into different
commits.
- Formatting: adjust layout based on reviewer feedback.
- Updated the email address.
- Link to v1:
https://lore.kernel.org/all/20241203060318.1750927-1-quic_yuzha@quicinc.com/
Signed-off-by: Yu Zhang (Yuriy) <yu.zhang@oss.qualcomm.com>
---
Yu Zhang(Yuriy) (2):
arm64: dts: qcom: qcs615: add a PCIe port for WLAN
arm64: dts: qcom: qcs615-ride: add WiFi/BT nodes
arch/arm64/boot/dts/qcom/qcs615-ride.dts | 135 +++++++++++++++++++++++++++++++
arch/arm64/boot/dts/qcom/qcs615.dtsi | 9 +++
2 files changed, 144 insertions(+)
---
base-commit: 8d6c58332c7a8ba025fcfa76888b6c37dbce9633
change-id: 20250704-615-21c25118e919
prerequisite-message-id: <20250703095630.669044-1-ziyue.zhang@oss.qualcomm.com>
prerequisite-patch-id: 37d5ab98a6918dba98ef8fd7e6c91ab98aace170
prerequisite-patch-id: 1693faa1e23bbb0a37d4fb9cbe13e5dd43148ce8
prerequisite-patch-id: efa688b2ddd5744df7aef40182787ee4ceb90c98
Best regards,
--
Yu Zhang(Yuriy) <yu.zhang@oss.qualcomm.com>
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v4 1/2] arm64: dts: qcom: qcs615: add a PCIe port for WLAN
2025-07-07 2:51 [PATCH v4 0/2] Add WiFi/BT node for qcs615 ride board Yu Zhang(Yuriy)
@ 2025-07-07 2:51 ` Yu Zhang(Yuriy)
2025-07-07 2:51 ` [PATCH v4 2/2] arm64: dts: qcom: qcs615-ride: add WiFi/BT nodes Yu Zhang(Yuriy)
1 sibling, 0 replies; 11+ messages in thread
From: Yu Zhang(Yuriy) @ 2025-07-07 2:51 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley
Cc: linux-arm-msm, devicetree, linux-kernel, Yu Zhang (Yuriy),
Konrad Dybcio
Add an original PCIe port for WLAN. This port will be referenced and
supplemented by specific WLAN devices.
Signed-off-by: Yu Zhang (Yuriy) <yu.zhang@oss.qualcomm.com>
Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
---
arch/arm64/boot/dts/qcom/qcs615.dtsi | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/qcs615.dtsi b/arch/arm64/boot/dts/qcom/qcs615.dtsi
index f4b51cca17de5518aad4332330340a59edfdc7ec..b4fbed517cde18e088096d182fe6b482363e403d 100644
--- a/arch/arm64/boot/dts/qcom/qcs615.dtsi
+++ b/arch/arm64/boot/dts/qcom/qcs615.dtsi
@@ -1173,6 +1173,15 @@ opp-5000000 {
opp-peak-kBps = <500000 1>;
};
};
+
+ pcie_port0: pcie@0 {
+ device_type = "pci";
+ reg = <0x0 0x0 0x0 0x0 0x0>;
+ #address-cells = <3>;
+ #size-cells = <2>;
+ ranges;
+ bus-range = <0x01 0xff>;
+ };
};
pcie_phy: phy@1c0e000 {
--
2.34.1
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH v4 2/2] arm64: dts: qcom: qcs615-ride: add WiFi/BT nodes
2025-07-07 2:51 [PATCH v4 0/2] Add WiFi/BT node for qcs615 ride board Yu Zhang(Yuriy)
2025-07-07 2:51 ` [PATCH v4 1/2] arm64: dts: qcom: qcs615: add a PCIe port for WLAN Yu Zhang(Yuriy)
@ 2025-07-07 2:51 ` Yu Zhang(Yuriy)
2025-07-19 10:13 ` Dmitry Baryshkov
2025-07-23 12:12 ` Krzysztof Kozlowski
1 sibling, 2 replies; 11+ messages in thread
From: Yu Zhang(Yuriy) @ 2025-07-07 2:51 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley
Cc: linux-arm-msm, devicetree, linux-kernel, Yu Zhang (Yuriy)
Add a node for the PMU module of the WCN6855 present on the qcs615 ride
board. Assign its LDO power outputs to the existing WiFi/BT module.
Signed-off-by: Yu Zhang(Yuriy) <yu.zhang@oss.qualcomm.com>
---
arch/arm64/boot/dts/qcom/qcs615-ride.dts | 135 +++++++++++++++++++++++++++++++
1 file changed, 135 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/qcs615-ride.dts b/arch/arm64/boot/dts/qcom/qcs615-ride.dts
index 011f8ae077c256f079ce1b07720374a9bf721488..2df8e7e3c1d3b6d6353a6753b8387c7411edd927 100644
--- a/arch/arm64/boot/dts/qcom/qcs615-ride.dts
+++ b/arch/arm64/boot/dts/qcom/qcs615-ride.dts
@@ -18,6 +18,7 @@ aliases {
mmc0 = &sdhc_1;
mmc1 = &sdhc_2;
serial0 = &uart0;
+ serial1 = &uart7;
};
chosen {
@@ -47,6 +48,85 @@ regulator-usb2-vbus {
enable-active-high;
regulator-always-on;
};
+
+ vreg_conn_1p8: vreg_conn_1p8 {
+ compatible = "regulator-fixed";
+ regulator-name = "vreg_conn_1p8";
+ startup-delay-us = <4000>;
+ enable-active-high;
+ gpio = <&pm8150_gpios 1 GPIO_ACTIVE_HIGH>;
+ };
+
+ vreg_conn_pa: vreg_conn_pa {
+ compatible = "regulator-fixed";
+ regulator-name = "vreg_conn_pa";
+ startup-delay-us = <4000>;
+ enable-active-high;
+ gpio = <&pm8150_gpios 6 GPIO_ACTIVE_HIGH>;
+ };
+
+ wcn6855-pmu {
+ compatible = "qcom,wcn6855-pmu";
+
+ pinctrl-0 = <&bt_en_state>, <&wlan_en_state>;
+ pinctrl-names = "default";
+
+ bt-enable-gpios = <&tlmm 85 GPIO_ACTIVE_HIGH>;
+ wlan-enable-gpios = <&tlmm 98 GPIO_ACTIVE_HIGH>;
+
+ vddio-supply = <&vreg_conn_pa>;
+ vddaon-supply = <&vreg_s5a>;
+ vddpmu-supply = <&vreg_conn_1p8>;
+ vddpmumx-supply = <&vreg_conn_1p8>;
+ vddpmucx-supply = <&vreg_conn_pa>;
+ vddrfa0p95-supply = <&vreg_s5a>;
+ vddrfa1p3-supply = <&vreg_s6a>;
+ vddrfa1p9-supply = <&vreg_l15a>;
+ vddpcie1p3-supply = <&vreg_s6a>;
+ vddpcie1p9-supply = <&vreg_l15a>;
+
+ regulators {
+ vreg_pmu_rfa_cmn: ldo0 {
+ regulator-name = "vreg_pmu_rfa_cmn";
+ };
+
+ vreg_pmu_aon_0p59: ldo1 {
+ regulator-name = "vreg_pmu_aon_0p59";
+ };
+
+ vreg_pmu_wlcx_0p8: ldo2 {
+ regulator-name = "vreg_pmu_wlcx_0p8";
+ };
+
+ vreg_pmu_wlmx_0p85: ldo3 {
+ regulator-name = "vreg_pmu_wlmx_0p85";
+ };
+
+ vreg_pmu_btcmx_0p85: ldo4 {
+ regulator-name = "vreg_pmu_btcmx_0p85";
+ };
+
+ vreg_pmu_rfa_0p8: ldo5 {
+ regulator-name = "vreg_pmu_rfa_0p8";
+ };
+
+ vreg_pmu_rfa_1p2: ldo6 {
+ regulator-name = "vreg_pmu_rfa_1p2";
+ };
+
+ vreg_pmu_rfa_1p7: ldo7 {
+ regulator-name = "vreg_pmu_rfa_1p7";
+ };
+
+ vreg_pmu_pcie_0p9: ldo8 {
+ regulator-name = "vreg_pmu_pcie_0p9";
+ };
+
+ vreg_pmu_pcie_1p8: ldo9 {
+ regulator-name = "vreg_pmu_pcie_1p8";
+ };
+ };
+ };
};
&apps_rsc {
@@ -234,6 +314,25 @@ &pcie_phy {
status = "okay";
};
+&pcie_port0 {
+ wifi@0 {
+ compatible = "pci17cb,1103";
+ reg = <0x10000 0x0 0x0 0x0 0x0>;
+
+ qcom,calibration-variant = "QC_QCS615_Ride";
+
+ vddrfacmn-supply = <&vreg_pmu_rfa_cmn>;
+ vddaon-supply = <&vreg_pmu_aon_0p59>;
+ vddwlcx-supply = <&vreg_pmu_wlcx_0p8>;
+ vddwlmx-supply = <&vreg_pmu_wlmx_0p85>;
+ vddrfa0p8-supply = <&vreg_pmu_rfa_0p8>;
+ vddrfa1p2-supply = <&vreg_pmu_rfa_1p2>;
+ vddrfa1p8-supply = <&vreg_pmu_rfa_1p7>;
+ vddpcie0p9-supply = <&vreg_pmu_pcie_0p9>;
+ vddpcie1p8-supply = <&vreg_pmu_pcie_1p8>;
+ };
+};
+
&pm8150_gpios {
usb2_en: usb2-en-state {
pins = "gpio10";
@@ -257,6 +356,10 @@ &qupv3_id_0 {
status = "okay";
};
+&qupv3_id_1 {
+ status = "okay";
+};
+
&remoteproc_adsp {
firmware-name = "qcom/qcs615/adsp.mbn";
@@ -274,6 +377,13 @@ &rpmhcc {
};
&tlmm {
+ bt_en_state: bt-en-state {
+ pins = "gpio85";
+ function = "gpio";
+ bias-pull-down;
+ output-low;
+ };
+
pcie_default_state: pcie-default-state {
clkreq-pins {
pins = "gpio90";
@@ -296,6 +406,13 @@ wake-pins {
bias-pull-up;
};
};
+
+ wlan_en_state: wlan-en-state {
+ pins = "gpio98";
+ function = "gpio";
+ bias-pull-down;
+ output-low;
+ };
};
&sdhc_1 {
@@ -336,6 +453,24 @@ &uart0 {
status = "okay";
};
+&uart7 {
+ status = "okay";
+
+ bluetooth {
+ compatible = "qcom,wcn6855-bt";
+ firmware-name = "QCA6698/hpnv21", "QCA6698/hpbtfw21.tlv";
+
+ vddrfacmn-supply = <&vreg_pmu_rfa_cmn>;
+ vddaon-supply = <&vreg_pmu_aon_0p59>;
+ vddwlcx-supply = <&vreg_pmu_wlcx_0p8>;
+ vddwlmx-supply = <&vreg_pmu_wlmx_0p85>;
+ vddbtcmx-supply = <&vreg_pmu_btcmx_0p85>;
+ vddrfa0p8-supply = <&vreg_pmu_rfa_0p8>;
+ vddrfa1p2-supply = <&vreg_pmu_rfa_1p2>;
+ vddrfa1p8-supply = <&vreg_pmu_rfa_1p7>;
+ };
+};
+
&usb_1_hsphy {
vdd-supply = <&vreg_l5a>;
vdda-pll-supply = <&vreg_l12a>;
--
2.34.1
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH v4 2/2] arm64: dts: qcom: qcs615-ride: add WiFi/BT nodes
2025-07-07 2:51 ` [PATCH v4 2/2] arm64: dts: qcom: qcs615-ride: add WiFi/BT nodes Yu Zhang(Yuriy)
@ 2025-07-19 10:13 ` Dmitry Baryshkov
2025-07-22 15:58 ` Yu Zhang(Yuriy)
2025-07-23 12:12 ` Krzysztof Kozlowski
1 sibling, 1 reply; 11+ messages in thread
From: Dmitry Baryshkov @ 2025-07-19 10:13 UTC (permalink / raw)
To: Yu Zhang(Yuriy)
Cc: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, linux-arm-msm, devicetree, linux-kernel
On Mon, Jul 07, 2025 at 10:51:06AM +0800, Yu Zhang(Yuriy) wrote:
> Add a node for the PMU module of the WCN6855 present on the qcs615 ride
> board. Assign its LDO power outputs to the existing WiFi/BT module.
What is "existing WiFI/BT module"? There is no module in the DT. Not to
mention that PMU is a part of the WCN6855.
>
> Signed-off-by: Yu Zhang(Yuriy) <yu.zhang@oss.qualcomm.com>
> ---
> arch/arm64/boot/dts/qcom/qcs615-ride.dts | 135 +++++++++++++++++++++++++++++++
> 1 file changed, 135 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/qcs615-ride.dts b/arch/arm64/boot/dts/qcom/qcs615-ride.dts
> index 011f8ae077c256f079ce1b07720374a9bf721488..2df8e7e3c1d3b6d6353a6753b8387c7411edd927 100644
> --- a/arch/arm64/boot/dts/qcom/qcs615-ride.dts
> +++ b/arch/arm64/boot/dts/qcom/qcs615-ride.dts
> @@ -18,6 +18,7 @@ aliases {
> mmc0 = &sdhc_1;
> mmc1 = &sdhc_2;
> serial0 = &uart0;
> + serial1 = &uart7;
> };
>
> chosen {
> @@ -47,6 +48,85 @@ regulator-usb2-vbus {
> enable-active-high;
> regulator-always-on;
> };
> +
> + vreg_conn_1p8: vreg_conn_1p8 {
No improvement. Please try again.
> + compatible = "regulator-fixed";
> + regulator-name = "vreg_conn_1p8";
> + startup-delay-us = <4000>;
> + enable-active-high;
> + gpio = <&pm8150_gpios 1 GPIO_ACTIVE_HIGH>;
> + };
> +
> + vreg_conn_pa: vreg_conn_pa {
> + compatible = "regulator-fixed";
> + regulator-name = "vreg_conn_pa";
> + startup-delay-us = <4000>;
> + enable-active-high;
> + gpio = <&pm8150_gpios 6 GPIO_ACTIVE_HIGH>;
> + };
> +
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v4 2/2] arm64: dts: qcom: qcs615-ride: add WiFi/BT nodes
2025-07-19 10:13 ` Dmitry Baryshkov
@ 2025-07-22 15:58 ` Yu Zhang(Yuriy)
2025-07-22 17:36 ` Dmitry Baryshkov
0 siblings, 1 reply; 11+ messages in thread
From: Yu Zhang(Yuriy) @ 2025-07-22 15:58 UTC (permalink / raw)
To: Dmitry Baryshkov
Cc: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, linux-arm-msm, devicetree, linux-kernel
On 7/19/2025 6:13 PM, Dmitry Baryshkov wrote:
> On Mon, Jul 07, 2025 at 10:51:06AM +0800, Yu Zhang(Yuriy) wrote:
>> Add a node for the PMU module of the WCN6855 present on the qcs615 ride
>> board. Assign its LDO power outputs to the existing WiFi/BT module.
>
>
> What is "existing WiFI/BT module"? There is no module in the DT. Not to
> mention that PMU is a part of the WCN6855.
>
>>
>> Signed-off-by: Yu Zhang(Yuriy) <yu.zhang@oss.qualcomm.com>
>> ---
>> arch/arm64/boot/dts/qcom/qcs615-ride.dts | 135 +++++++++++++++++++++++++++++++
>> 1 file changed, 135 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/qcs615-ride.dts b/arch/arm64/boot/dts/qcom/qcs615-ride.dts
>> index 011f8ae077c256f079ce1b07720374a9bf721488..2df8e7e3c1d3b6d6353a6753b8387c7411edd927 100644
>> --- a/arch/arm64/boot/dts/qcom/qcs615-ride.dts
>> +++ b/arch/arm64/boot/dts/qcom/qcs615-ride.dts
>> @@ -18,6 +18,7 @@ aliases {
>> mmc0 = &sdhc_1;
>> mmc1 = &sdhc_2;
>> serial0 = &uart0;
>> + serial1 = &uart7;
>> };
>>
>> chosen {
>> @@ -47,6 +48,85 @@ regulator-usb2-vbus {
>> enable-active-high;
>> regulator-always-on;
>> };
>> +
>> + vreg_conn_1p8: vreg_conn_1p8 {
>
> No improvement. Please try again.
>
you mean should be "vreg_conn_1p8: vreg-conn-1p8",right? I'll update it,
thanks.>> + compatible = "regulator-fixed";
>> + regulator-name = "vreg_conn_1p8";
>> + startup-delay-us = <4000>;
>> + enable-active-high;
>> + gpio = <&pm8150_gpios 1 GPIO_ACTIVE_HIGH>;
>> + };
>> +
>> + vreg_conn_pa: vreg_conn_pa {
>> + compatible = "regulator-fixed";
>> + regulator-name = "vreg_conn_pa";
>> + startup-delay-us = <4000>;
>> + enable-active-high;
>> + gpio = <&pm8150_gpios 6 GPIO_ACTIVE_HIGH>;
>> + };
>> +
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v4 2/2] arm64: dts: qcom: qcs615-ride: add WiFi/BT nodes
2025-07-22 15:58 ` Yu Zhang(Yuriy)
@ 2025-07-22 17:36 ` Dmitry Baryshkov
2025-07-23 2:02 ` Yu Zhang(Yuriy)
0 siblings, 1 reply; 11+ messages in thread
From: Dmitry Baryshkov @ 2025-07-22 17:36 UTC (permalink / raw)
To: Yu Zhang(Yuriy)
Cc: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, linux-arm-msm, devicetree, linux-kernel
On 22/07/2025 18:58, Yu Zhang(Yuriy) wrote:
>
>
> On 7/19/2025 6:13 PM, Dmitry Baryshkov wrote:
>> On Mon, Jul 07, 2025 at 10:51:06AM +0800, Yu Zhang(Yuriy) wrote:
>>> Add a node for the PMU module of the WCN6855 present on the qcs615 ride
>>> board. Assign its LDO power outputs to the existing WiFi/BT module.
>>
>>
>> What is "existing WiFI/BT module"? There is no module in the DT. Not to
>> mention that PMU is a part of the WCN6855.
>>
>>>
>>> Signed-off-by: Yu Zhang(Yuriy) <yu.zhang@oss.qualcomm.com>
>>> ---
>>> arch/arm64/boot/dts/qcom/qcs615-ride.dts | 135 ++++++++++++++++++++
>>> +++++++++++
>>> 1 file changed, 135 insertions(+)
>>>
>>> diff --git a/arch/arm64/boot/dts/qcom/qcs615-ride.dts b/arch/arm64/
>>> boot/dts/qcom/qcs615-ride.dts
>>> index
>>> 011f8ae077c256f079ce1b07720374a9bf721488..2df8e7e3c1d3b6d6353a6753b8387c7411edd927 100644
>>> --- a/arch/arm64/boot/dts/qcom/qcs615-ride.dts
>>> +++ b/arch/arm64/boot/dts/qcom/qcs615-ride.dts
>>> @@ -18,6 +18,7 @@ aliases {
>>> mmc0 = &sdhc_1;
>>> mmc1 = &sdhc_2;
>>> serial0 = &uart0;
>>> + serial1 = &uart7;
>>> };
>>> chosen {
>>> @@ -47,6 +48,85 @@ regulator-usb2-vbus {
>>> enable-active-high;
>>> regulator-always-on;
>>> };
>>> +
>>> + vreg_conn_1p8: vreg_conn_1p8 {
>>
>> No improvement. Please try again.
>>
> you mean should be "vreg_conn_1p8: vreg-conn-1p8",right? I'll update it,
What is the node name for the previous regulator device? Anything
preventing you from following the pattern?
> thanks.>> + compatible = "regulator-fixed";
>>> + regulator-name = "vreg_conn_1p8";
>>> + startup-delay-us = <4000>;
>>> + enable-active-high;
>>> + gpio = <&pm8150_gpios 1 GPIO_ACTIVE_HIGH>;
>>> + };
>>> +
>>> + vreg_conn_pa: vreg_conn_pa {
>>> + compatible = "regulator-fixed";
>>> + regulator-name = "vreg_conn_pa";
>>> + startup-delay-us = <4000>;
>>> + enable-active-high;
>>> + gpio = <&pm8150_gpios 6 GPIO_ACTIVE_HIGH>;
>>> + };
>>> +
>>
>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v4 2/2] arm64: dts: qcom: qcs615-ride: add WiFi/BT nodes
2025-07-22 17:36 ` Dmitry Baryshkov
@ 2025-07-23 2:02 ` Yu Zhang(Yuriy)
2025-07-23 11:31 ` Dmitry Baryshkov
0 siblings, 1 reply; 11+ messages in thread
From: Yu Zhang(Yuriy) @ 2025-07-23 2:02 UTC (permalink / raw)
To: Dmitry Baryshkov
Cc: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, linux-arm-msm, devicetree, linux-kernel
On 7/23/2025 1:36 AM, Dmitry Baryshkov wrote:
> On 22/07/2025 18:58, Yu Zhang(Yuriy) wrote:
>>
>>
>> On 7/19/2025 6:13 PM, Dmitry Baryshkov wrote:
>>> On Mon, Jul 07, 2025 at 10:51:06AM +0800, Yu Zhang(Yuriy) wrote:
>>>> Add a node for the PMU module of the WCN6855 present on the qcs615 ride
>>>> board. Assign its LDO power outputs to the existing WiFi/BT module.
>>>
>>>
>>> What is "existing WiFI/BT module"? There is no module in the DT. Not to
>>> mention that PMU is a part of the WCN6855.
>>>
>>>>
>>>> Signed-off-by: Yu Zhang(Yuriy) <yu.zhang@oss.qualcomm.com>
>>>> ---
>>>> arch/arm64/boot/dts/qcom/qcs615-ride.dts | 135 +++++++++++++++++++
>>>> + +++++++++++
>>>> 1 file changed, 135 insertions(+)
>>>>
>>>> diff --git a/arch/arm64/boot/dts/qcom/qcs615-ride.dts b/arch/arm64/
>>>> boot/dts/qcom/qcs615-ride.dts
>>>> index
>>>> 011f8ae077c256f079ce1b07720374a9bf721488..2df8e7e3c1d3b6d6353a6753b8387c7411edd927 100644
>>>> --- a/arch/arm64/boot/dts/qcom/qcs615-ride.dts
>>>> +++ b/arch/arm64/boot/dts/qcom/qcs615-ride.dts
>>>> @@ -18,6 +18,7 @@ aliases {
>>>> mmc0 = &sdhc_1;
>>>> mmc1 = &sdhc_2;
>>>> serial0 = &uart0;
>>>> + serial1 = &uart7;
>>>> };
>>>> chosen {
>>>> @@ -47,6 +48,85 @@ regulator-usb2-vbus {
>>>> enable-active-high;
>>>> regulator-always-on;
>>>> };
>>>> +
>>>> + vreg_conn_1p8: vreg_conn_1p8 {
>>>
>>> No improvement. Please try again.
>>>
>> you mean should be "vreg_conn_1p8: vreg-conn-1p8",right? I'll update it,
>
> What is the node name for the previous regulator device? Anything
> preventing you from following the pattern?
>
Do you mean the prefix 'regulator'?
I just want to keep 'vreg_conn_1p8' and 'vreg_conn_pa' the same as in
sa8775p-ride.dtsi, because they're using the same module.
>> thanks.>> + compatible = "regulator-fixed";
>>>> + regulator-name = "vreg_conn_1p8";
>>>> + startup-delay-us = <4000>;
>>>> + enable-active-high;
>>>> + gpio = <&pm8150_gpios 1 GPIO_ACTIVE_HIGH>;
>>>> + };
>>>> +
>>>> + vreg_conn_pa: vreg_conn_pa {
>>>> + compatible = "regulator-fixed";
>>>> + regulator-name = "vreg_conn_pa";
>>>> + startup-delay-us = <4000>;
>>>> + enable-active-high;
>>>> + gpio = <&pm8150_gpios 6 GPIO_ACTIVE_HIGH>;
>>>> + };
>>>> +
>>>
>>
>
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v4 2/2] arm64: dts: qcom: qcs615-ride: add WiFi/BT nodes
2025-07-23 2:02 ` Yu Zhang(Yuriy)
@ 2025-07-23 11:31 ` Dmitry Baryshkov
2025-07-24 6:58 ` Yu Zhang(Yuriy)
0 siblings, 1 reply; 11+ messages in thread
From: Dmitry Baryshkov @ 2025-07-23 11:31 UTC (permalink / raw)
To: Yu Zhang(Yuriy)
Cc: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, linux-arm-msm, devicetree, linux-kernel
On Wed, Jul 23, 2025 at 10:02:36AM +0800, Yu Zhang(Yuriy) wrote:
>
>
> On 7/23/2025 1:36 AM, Dmitry Baryshkov wrote:
> > On 22/07/2025 18:58, Yu Zhang(Yuriy) wrote:
> > >
> > >
> > > On 7/19/2025 6:13 PM, Dmitry Baryshkov wrote:
> > > > On Mon, Jul 07, 2025 at 10:51:06AM +0800, Yu Zhang(Yuriy) wrote:
> > > > > Add a node for the PMU module of the WCN6855 present on the qcs615 ride
> > > > > board. Assign its LDO power outputs to the existing WiFi/BT module.
> > > >
> > > >
> > > > What is "existing WiFI/BT module"? There is no module in the DT. Not to
> > > > mention that PMU is a part of the WCN6855.
> > > >
> > > > >
> > > > > Signed-off-by: Yu Zhang(Yuriy) <yu.zhang@oss.qualcomm.com>
> > > > > ---
> > > > > arch/arm64/boot/dts/qcom/qcs615-ride.dts | 135
> > > > > +++++++++++++++++++ + +++++++++++
> > > > > 1 file changed, 135 insertions(+)
> > > > >
> > > > > diff --git a/arch/arm64/boot/dts/qcom/qcs615-ride.dts
> > > > > b/arch/arm64/ boot/dts/qcom/qcs615-ride.dts
> > > > > index 011f8ae077c256f079ce1b07720374a9bf721488..2df8e7e3c1d3b6d6353a6753b8387c7411edd927
> > > > > 100644
> > > > > --- a/arch/arm64/boot/dts/qcom/qcs615-ride.dts
> > > > > +++ b/arch/arm64/boot/dts/qcom/qcs615-ride.dts
> > > > > @@ -18,6 +18,7 @@ aliases {
> > > > > mmc0 = &sdhc_1;
> > > > > mmc1 = &sdhc_2;
> > > > > serial0 = &uart0;
> > > > > + serial1 = &uart7;
> > > > > };
> > > > > chosen {
> > > > > @@ -47,6 +48,85 @@ regulator-usb2-vbus {
> > > > > enable-active-high;
> > > > > regulator-always-on;
> > > > > };
> > > > > +
> > > > > + vreg_conn_1p8: vreg_conn_1p8 {
> > > >
> > > > No improvement. Please try again.
> > > >
> > > you mean should be "vreg_conn_1p8: vreg-conn-1p8",right? I'll update
> > > it,
> >
> > What is the node name for the previous regulator device? Anything
> > preventing you from following the pattern?
> >
> Do you mean the prefix 'regulator'?
> I just want to keep 'vreg_conn_1p8' and 'vreg_conn_pa' the same as in
> sa8775p-ride.dtsi, because they're using the same module.
That's a different device. Please use the style of the file that you are
patching.
>
> > > thanks.>> + compatible = "regulator-fixed";
> > > > > + regulator-name = "vreg_conn_1p8";
> > > > > + startup-delay-us = <4000>;
> > > > > + enable-active-high;
> > > > > + gpio = <&pm8150_gpios 1 GPIO_ACTIVE_HIGH>;
> > > > > + };
> > > > > +
> > > > > + vreg_conn_pa: vreg_conn_pa {
> > > > > + compatible = "regulator-fixed";
> > > > > + regulator-name = "vreg_conn_pa";
> > > > > + startup-delay-us = <4000>;
> > > > > + enable-active-high;
> > > > > + gpio = <&pm8150_gpios 6 GPIO_ACTIVE_HIGH>;
> > > > > + };
> > > > > +
> > > >
> > >
> >
> >
>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v4 2/2] arm64: dts: qcom: qcs615-ride: add WiFi/BT nodes
2025-07-07 2:51 ` [PATCH v4 2/2] arm64: dts: qcom: qcs615-ride: add WiFi/BT nodes Yu Zhang(Yuriy)
2025-07-19 10:13 ` Dmitry Baryshkov
@ 2025-07-23 12:12 ` Krzysztof Kozlowski
2025-07-24 7:06 ` Yu Zhang(Yuriy)
1 sibling, 1 reply; 11+ messages in thread
From: Krzysztof Kozlowski @ 2025-07-23 12:12 UTC (permalink / raw)
To: Yu Zhang(Yuriy), Bjorn Andersson, Konrad Dybcio, Rob Herring,
Krzysztof Kozlowski, Conor Dooley
Cc: linux-arm-msm, devicetree, linux-kernel
On 07/07/2025 04:51, Yu Zhang(Yuriy) wrote:
> Add a node for the PMU module of the WCN6855 present on the qcs615 ride
> board. Assign its LDO power outputs to the existing WiFi/BT module.
>
> Signed-off-by: Yu Zhang(Yuriy) <yu.zhang@oss.qualcomm.com>
> ---
> arch/arm64/boot/dts/qcom/qcs615-ride.dts | 135 +++++++++++++++++++++++++++++++
> 1 file changed, 135 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/qcs615-ride.dts b/arch/arm64/boot/dts/qcom/qcs615-ride.dts
> index 011f8ae077c256f079ce1b07720374a9bf721488..2df8e7e3c1d3b6d6353a6753b8387c7411edd927 100644
> --- a/arch/arm64/boot/dts/qcom/qcs615-ride.dts
> +++ b/arch/arm64/boot/dts/qcom/qcs615-ride.dts
> @@ -18,6 +18,7 @@ aliases {
> mmc0 = &sdhc_1;
> mmc1 = &sdhc_2;
> serial0 = &uart0;
> + serial1 = &uart7;
> };
>
> chosen {
> @@ -47,6 +48,85 @@ regulator-usb2-vbus {
> enable-active-high;
> regulator-always-on;
> };
> +
> + vreg_conn_1p8: vreg_conn_1p8 {
Are you going to keep sending the same? Third or fourth time, even
though there were comments? Please take comments seriously and implement
them or at least RESPOND.
NAK
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v4 2/2] arm64: dts: qcom: qcs615-ride: add WiFi/BT nodes
2025-07-23 11:31 ` Dmitry Baryshkov
@ 2025-07-24 6:58 ` Yu Zhang(Yuriy)
0 siblings, 0 replies; 11+ messages in thread
From: Yu Zhang(Yuriy) @ 2025-07-24 6:58 UTC (permalink / raw)
To: Dmitry Baryshkov
Cc: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, linux-arm-msm, devicetree, linux-kernel
On 7/23/2025 7:31 PM, Dmitry Baryshkov wrote:
> On Wed, Jul 23, 2025 at 10:02:36AM +0800, Yu Zhang(Yuriy) wrote:
>>
>>
>> On 7/23/2025 1:36 AM, Dmitry Baryshkov wrote:
>>> On 22/07/2025 18:58, Yu Zhang(Yuriy) wrote:
>>>>
>>>>
>>>> On 7/19/2025 6:13 PM, Dmitry Baryshkov wrote:
>>>>> On Mon, Jul 07, 2025 at 10:51:06AM +0800, Yu Zhang(Yuriy) wrote:
>>>>>> Add a node for the PMU module of the WCN6855 present on the qcs615 ride
>>>>>> board. Assign its LDO power outputs to the existing WiFi/BT module.
>>>>>
>>>>>
>>>>> What is "existing WiFI/BT module"? There is no module in the DT. Not to
>>>>> mention that PMU is a part of the WCN6855.
>>>>>
>>>>>>
>>>>>> Signed-off-by: Yu Zhang(Yuriy) <yu.zhang@oss.qualcomm.com>
>>>>>> ---
>>>>>> arch/arm64/boot/dts/qcom/qcs615-ride.dts | 135
>>>>>> +++++++++++++++++++ + +++++++++++
>>>>>> 1 file changed, 135 insertions(+)
>>>>>>
>>>>>> diff --git a/arch/arm64/boot/dts/qcom/qcs615-ride.dts
>>>>>> b/arch/arm64/ boot/dts/qcom/qcs615-ride.dts
>>>>>> index 011f8ae077c256f079ce1b07720374a9bf721488..2df8e7e3c1d3b6d6353a6753b8387c7411edd927
>>>>>> 100644
>>>>>> --- a/arch/arm64/boot/dts/qcom/qcs615-ride.dts
>>>>>> +++ b/arch/arm64/boot/dts/qcom/qcs615-ride.dts
>>>>>> @@ -18,6 +18,7 @@ aliases {
>>>>>> mmc0 = &sdhc_1;
>>>>>> mmc1 = &sdhc_2;
>>>>>> serial0 = &uart0;
>>>>>> + serial1 = &uart7;
>>>>>> };
>>>>>> chosen {
>>>>>> @@ -47,6 +48,85 @@ regulator-usb2-vbus {
>>>>>> enable-active-high;
>>>>>> regulator-always-on;
>>>>>> };
>>>>>> +
>>>>>> + vreg_conn_1p8: vreg_conn_1p8 {
>>>>>
>>>>> No improvement. Please try again.
>>>>>
>>>> you mean should be "vreg_conn_1p8: vreg-conn-1p8",right? I'll update
>>>> it,
>>>
>>> What is the node name for the previous regulator device? Anything
>>> preventing you from following the pattern?
>>>
>> Do you mean the prefix 'regulator'?
>> I just want to keep 'vreg_conn_1p8' and 'vreg_conn_pa' the same as in
>> sa8775p-ride.dtsi, because they're using the same module.
>
> That's a different device. Please use the style of the file that you are
> patching.
>
Got it, thanks.>>
>>>> thanks.>> + compatible = "regulator-fixed";
>>>>>> + regulator-name = "vreg_conn_1p8";
>>>>>> + startup-delay-us = <4000>;
>>>>>> + enable-active-high;
>>>>>> + gpio = <&pm8150_gpios 1 GPIO_ACTIVE_HIGH>;
>>>>>> + };
>>>>>> +
>>>>>> + vreg_conn_pa: vreg_conn_pa {
>>>>>> + compatible = "regulator-fixed";
>>>>>> + regulator-name = "vreg_conn_pa";
>>>>>> + startup-delay-us = <4000>;
>>>>>> + enable-active-high;
>>>>>> + gpio = <&pm8150_gpios 6 GPIO_ACTIVE_HIGH>;
>>>>>> + };
>>>>>> +
>>>>>
>>>>
>>>
>>>
>>
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v4 2/2] arm64: dts: qcom: qcs615-ride: add WiFi/BT nodes
2025-07-23 12:12 ` Krzysztof Kozlowski
@ 2025-07-24 7:06 ` Yu Zhang(Yuriy)
0 siblings, 0 replies; 11+ messages in thread
From: Yu Zhang(Yuriy) @ 2025-07-24 7:06 UTC (permalink / raw)
To: Krzysztof Kozlowski, Bjorn Andersson, Konrad Dybcio, Rob Herring,
Krzysztof Kozlowski, Conor Dooley
Cc: linux-arm-msm, devicetree, linux-kernel
On 7/23/2025 8:12 PM, Krzysztof Kozlowski wrote:
> On 07/07/2025 04:51, Yu Zhang(Yuriy) wrote:
>> Add a node for the PMU module of the WCN6855 present on the qcs615 ride
>> board. Assign its LDO power outputs to the existing WiFi/BT module.
>>
>> Signed-off-by: Yu Zhang(Yuriy) <yu.zhang@oss.qualcomm.com>
>> ---
>> arch/arm64/boot/dts/qcom/qcs615-ride.dts | 135 +++++++++++++++++++++++++++++++
>> 1 file changed, 135 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/qcs615-ride.dts b/arch/arm64/boot/dts/qcom/qcs615-ride.dts
>> index 011f8ae077c256f079ce1b07720374a9bf721488..2df8e7e3c1d3b6d6353a6753b8387c7411edd927 100644
>> --- a/arch/arm64/boot/dts/qcom/qcs615-ride.dts
>> +++ b/arch/arm64/boot/dts/qcom/qcs615-ride.dts
>> @@ -18,6 +18,7 @@ aliases {
>> mmc0 = &sdhc_1;
>> mmc1 = &sdhc_2;
>> serial0 = &uart0;
>> + serial1 = &uart7;
>> };
>>
>> chosen {
>> @@ -47,6 +48,85 @@ regulator-usb2-vbus {
>> enable-active-high;
>> regulator-always-on;
>> };
>> +
>> + vreg_conn_1p8: vreg_conn_1p8 {
>
>
> Are you going to keep sending the same? Third or fourth time, even
> though there were comments? Please take comments seriously and implement
> them or at least RESPOND.
>
I appreciate your comments and apologize for the repeated submissions.
Due to a possible issue with my Thunderbird client, I did not receive
earlier replies and was unaware of additional feedback.
The v3 and v4 revisions were based solely on Rob Herring’s suggestions
and included some dependency adjustments and rebasing.
I will ensure to properly address and respond to all comments in future
revisions.
Thanks a lot.
> NAK
>
> Best regards,
> Krzysztof
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2025-07-24 7:06 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-07 2:51 [PATCH v4 0/2] Add WiFi/BT node for qcs615 ride board Yu Zhang(Yuriy)
2025-07-07 2:51 ` [PATCH v4 1/2] arm64: dts: qcom: qcs615: add a PCIe port for WLAN Yu Zhang(Yuriy)
2025-07-07 2:51 ` [PATCH v4 2/2] arm64: dts: qcom: qcs615-ride: add WiFi/BT nodes Yu Zhang(Yuriy)
2025-07-19 10:13 ` Dmitry Baryshkov
2025-07-22 15:58 ` Yu Zhang(Yuriy)
2025-07-22 17:36 ` Dmitry Baryshkov
2025-07-23 2:02 ` Yu Zhang(Yuriy)
2025-07-23 11:31 ` Dmitry Baryshkov
2025-07-24 6:58 ` Yu Zhang(Yuriy)
2025-07-23 12:12 ` Krzysztof Kozlowski
2025-07-24 7:06 ` Yu Zhang(Yuriy)
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).