* [PATCH V4 1/8] arm64: dts: qcom: sm8350: Add label for thermal-zones node
2021-05-05 6:55 [PATCH V4 0/8] Add PMIC DT files for sc7280 satya priya
@ 2021-05-05 6:55 ` satya priya
2021-05-05 13:33 ` Matthias Kaehlcke
2021-05-05 6:55 ` [PATCH V4 2/8] arm64: dts: qcom: sc7280: Add " satya priya
` (6 subsequent siblings)
7 siblings, 1 reply; 16+ messages in thread
From: satya priya @ 2021-05-05 6:55 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Rob Herring
Cc: mka, linux-arm-msm, linux-kernel, devicetree, kgunda, satya priya
Add label "thermal_zones" for thermal-zones node.
Signed-off-by: satya priya <skakit@codeaurora.org>
---
arch/arm64/boot/dts/qcom/sm8350.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/sm8350.dtsi b/arch/arm64/boot/dts/qcom/sm8350.dtsi
index ed0b51b..47c6c0b 100644
--- a/arch/arm64/boot/dts/qcom/sm8350.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8350.dtsi
@@ -1317,7 +1317,7 @@
};
};
- thermal-zones {
+ thermal_zones: thermal-zones {
cpu0-thermal {
polling-delay-passive = <250>;
polling-delay = <1000>;
--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH V4 1/8] arm64: dts: qcom: sm8350: Add label for thermal-zones node
2021-05-05 6:55 ` [PATCH V4 1/8] arm64: dts: qcom: sm8350: Add label for thermal-zones node satya priya
@ 2021-05-05 13:33 ` Matthias Kaehlcke
0 siblings, 0 replies; 16+ messages in thread
From: Matthias Kaehlcke @ 2021-05-05 13:33 UTC (permalink / raw)
To: satya priya
Cc: Andy Gross, Bjorn Andersson, Rob Herring, linux-arm-msm,
linux-kernel, devicetree, kgunda
On Wed, May 05, 2021 at 12:25:19PM +0530, satya priya wrote:
> Add label "thermal_zones" for thermal-zones node.
>
> Signed-off-by: satya priya <skakit@codeaurora.org>
Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH V4 2/8] arm64: dts: qcom: sc7280: Add thermal-zones node
2021-05-05 6:55 [PATCH V4 0/8] Add PMIC DT files for sc7280 satya priya
2021-05-05 6:55 ` [PATCH V4 1/8] arm64: dts: qcom: sm8350: Add label for thermal-zones node satya priya
@ 2021-05-05 6:55 ` satya priya
2021-05-05 13:41 ` Matthias Kaehlcke
2021-05-05 6:55 ` [PATCH V4 3/8] arm64: dts: qcom: pm7325: Add pm7325 base dts file satya priya
` (5 subsequent siblings)
7 siblings, 1 reply; 16+ messages in thread
From: satya priya @ 2021-05-05 6:55 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Rob Herring
Cc: mka, linux-arm-msm, linux-kernel, devicetree, kgunda, satya priya
Add thermal-zones node for SC7280 SoC.
Signed-off-by: satya priya <skakit@codeaurora.org>
---
arch/arm64/boot/dts/qcom/sc7280.dtsi | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi
index 2cc4785..2a7d488 100644
--- a/arch/arm64/boot/dts/qcom/sc7280.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi
@@ -1125,4 +1125,7 @@
<GIC_PPI 11 IRQ_TYPE_LEVEL_LOW>,
<GIC_PPI 10 IRQ_TYPE_LEVEL_LOW>;
};
+
+ thermal_zones: thermal-zones {
+ };
};
--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH V4 2/8] arm64: dts: qcom: sc7280: Add thermal-zones node
2021-05-05 6:55 ` [PATCH V4 2/8] arm64: dts: qcom: sc7280: Add " satya priya
@ 2021-05-05 13:41 ` Matthias Kaehlcke
0 siblings, 0 replies; 16+ messages in thread
From: Matthias Kaehlcke @ 2021-05-05 13:41 UTC (permalink / raw)
To: satya priya
Cc: Andy Gross, Bjorn Andersson, Rob Herring, linux-arm-msm,
linux-kernel, devicetree, kgunda
On Wed, May 05, 2021 at 12:25:20PM +0530, satya priya wrote:
> Add thermal-zones node for SC7280 SoC.
>
> Signed-off-by: satya priya <skakit@codeaurora.org>
The patch won't be needed after 'arm64: dts: qcom: SC7280: Add thermal zone
support' (https://patchwork.kernel.org/project/linux-arm-msm/patch/1619778592-8112-4-git-send-email-rkambl@codeaurora.org/)
lands, but looks good for now.
Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH V4 3/8] arm64: dts: qcom: pm7325: Add pm7325 base dts file
2021-05-05 6:55 [PATCH V4 0/8] Add PMIC DT files for sc7280 satya priya
2021-05-05 6:55 ` [PATCH V4 1/8] arm64: dts: qcom: sm8350: Add label for thermal-zones node satya priya
2021-05-05 6:55 ` [PATCH V4 2/8] arm64: dts: qcom: sc7280: Add " satya priya
@ 2021-05-05 6:55 ` satya priya
2021-05-05 6:55 ` [PATCH V4 4/8] arm64: dts: qcom: pm8350c: Add temp-alarm support satya priya
` (4 subsequent siblings)
7 siblings, 0 replies; 16+ messages in thread
From: satya priya @ 2021-05-05 6:55 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Rob Herring
Cc: mka, linux-arm-msm, linux-kernel, devicetree, kgunda, satya priya
Add base DTS file for pm7325 along with GPIOs and temp-alarm nodes.
Signed-off-by: satya priya <skakit@codeaurora.org>
Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
---
arch/arm64/boot/dts/qcom/pm7325.dtsi | 53 ++++++++++++++++++++++++++++++++++++
1 file changed, 53 insertions(+)
create mode 100644 arch/arm64/boot/dts/qcom/pm7325.dtsi
diff --git a/arch/arm64/boot/dts/qcom/pm7325.dtsi b/arch/arm64/boot/dts/qcom/pm7325.dtsi
new file mode 100644
index 0000000..e7f64a9
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/pm7325.dtsi
@@ -0,0 +1,53 @@
+// SPDX-License-Identifier: BSD-3-Clause
+// Copyright (c) 2021, The Linux Foundation. All rights reserved.
+
+#include <dt-bindings/interrupt-controller/irq.h>
+#include <dt-bindings/spmi/spmi.h>
+
+&spmi_bus {
+ pm7325: pmic@1 {
+ compatible = "qcom,pm7325", "qcom,spmi-pmic";
+ reg = <0x1 SPMI_USID>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ pm7325_temp_alarm: temp-alarm@a00 {
+ compatible = "qcom,spmi-temp-alarm";
+ reg = <0xa00>;
+ interrupts = <0x1 0xa 0x0 IRQ_TYPE_EDGE_BOTH>;
+ #thermal-sensor-cells = <0>;
+ };
+
+ pm7325_gpios: gpios@8800 {
+ compatible = "qcom,pm7325-gpio", "qcom,spmi-gpio";
+ reg = <0x8800>;
+ gpio-controller;
+ gpio-ranges = <&pm7325_gpios 0 0 10>;
+ #gpio-cells = <2>;
+ interrupt-controller;
+ #interrupt-cells = <2>;
+ };
+ };
+};
+
+&thermal_zones {
+ pm7325_thermal: pm7325-thermal {
+ polling-delay-passive = <100>;
+ polling-delay = <0>;
+ thermal-sensors = <&pm7325_temp_alarm>;
+
+ trips {
+ pm7325_trip0: trip0 {
+ temperature = <95000>;
+ hysteresis = <0>;
+ type = "passive";
+ };
+
+ pm7325_crit: pm7325-crit {
+ temperature = <115000>;
+ hysteresis = <0>;
+ type = "critical";
+ };
+ };
+ };
+};
--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH V4 4/8] arm64: dts: qcom: pm8350c: Add temp-alarm support
2021-05-05 6:55 [PATCH V4 0/8] Add PMIC DT files for sc7280 satya priya
` (2 preceding siblings ...)
2021-05-05 6:55 ` [PATCH V4 3/8] arm64: dts: qcom: pm7325: Add pm7325 base dts file satya priya
@ 2021-05-05 6:55 ` satya priya
2021-05-05 6:55 ` [PATCH V4 5/8] arm64: dts: qcom: pmr735a: " satya priya
` (3 subsequent siblings)
7 siblings, 0 replies; 16+ messages in thread
From: satya priya @ 2021-05-05 6:55 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Rob Herring
Cc: mka, linux-arm-msm, linux-kernel, devicetree, kgunda, satya priya
Add temp-alarm node for PM8350C pmic and also modify gpio
node to add gpio ranges and "qcom,spmi-gpio" compatible.
Signed-off-by: satya priya <skakit@codeaurora.org>
Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
---
arch/arm64/boot/dts/qcom/pm8350c.dtsi | 32 +++++++++++++++++++++++++++++++-
1 file changed, 31 insertions(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/pm8350c.dtsi b/arch/arm64/boot/dts/qcom/pm8350c.dtsi
index 2b9b75e..e1b75ae 100644
--- a/arch/arm64/boot/dts/qcom/pm8350c.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm8350c.dtsi
@@ -13,13 +13,43 @@
#address-cells = <1>;
#size-cells = <0>;
+ pm8350c_temp_alarm: temp-alarm@a00 {
+ compatible = "qcom,spmi-temp-alarm";
+ reg = <0xa00>;
+ interrupts = <0x2 0xa 0x0 IRQ_TYPE_EDGE_BOTH>;
+ #thermal-sensor-cells = <0>;
+ };
+
pm8350c_gpios: gpio@8800 {
- compatible = "qcom,pm8350c-gpio";
+ compatible = "qcom,pm8350c-gpio", "qcom,spmi-gpio";
reg = <0x8800>;
gpio-controller;
+ gpio-ranges = <&pm8350c_gpios 0 0 9>;
#gpio-cells = <2>;
interrupt-controller;
#interrupt-cells = <2>;
};
};
};
+
+&thermal_zones {
+ pm8350c_thermal: pm8350c-thermal {
+ polling-delay-passive = <100>;
+ polling-delay = <0>;
+ thermal-sensors = <&pm8350c_temp_alarm>;
+
+ trips {
+ pm8350c_trip0: trip0 {
+ temperature = <95000>;
+ hysteresis = <0>;
+ type = "passive";
+ };
+
+ pm8350c_crit: pm8350c-crit {
+ temperature = <115000>;
+ hysteresis = <0>;
+ type = "critical";
+ };
+ };
+ };
+};
--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH V4 5/8] arm64: dts: qcom: pmr735a: Add temp-alarm support
2021-05-05 6:55 [PATCH V4 0/8] Add PMIC DT files for sc7280 satya priya
` (3 preceding siblings ...)
2021-05-05 6:55 ` [PATCH V4 4/8] arm64: dts: qcom: pm8350c: Add temp-alarm support satya priya
@ 2021-05-05 6:55 ` satya priya
2021-05-05 6:55 ` [PATCH V4 6/8] arm64: dts: qcom: pmk8350: Add peripherals for pmk8350 satya priya
` (2 subsequent siblings)
7 siblings, 0 replies; 16+ messages in thread
From: satya priya @ 2021-05-05 6:55 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Rob Herring
Cc: mka, linux-arm-msm, linux-kernel, devicetree, kgunda, satya priya
Add temp-alarm node for PMR735A pmic and also modify gpio
node to add gpio ranges and "qcom,spmi-gpio" compatible.
Signed-off-by: satya priya <skakit@codeaurora.org>
Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
---
arch/arm64/boot/dts/qcom/pmr735a.dtsi | 32 +++++++++++++++++++++++++++++++-
1 file changed, 31 insertions(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/pmr735a.dtsi b/arch/arm64/boot/dts/qcom/pmr735a.dtsi
index 1c675af1..b4b6ba24 100644
--- a/arch/arm64/boot/dts/qcom/pmr735a.dtsi
+++ b/arch/arm64/boot/dts/qcom/pmr735a.dtsi
@@ -13,13 +13,43 @@
#address-cells = <1>;
#size-cells = <0>;
+ pmr735a_temp_alarm: temp-alarm@a00 {
+ compatible = "qcom,spmi-temp-alarm";
+ reg = <0xa00>;
+ interrupts = <0x4 0xa 0x0 IRQ_TYPE_EDGE_BOTH>;
+ #thermal-sensor-cells = <0>;
+ };
+
pmr735a_gpios: gpio@8800 {
- compatible = "qcom,pmr735a-gpio";
+ compatible = "qcom,pmr735a-gpio", "qcom,spmi-gpio";
reg = <0x8800>;
gpio-controller;
+ gpio-ranges = <&pmr735a_gpios 0 0 4>;
#gpio-cells = <2>;
interrupt-controller;
#interrupt-cells = <2>;
};
};
};
+
+&thermal_zones {
+ pmr735a_thermal: pmr735a-thermal {
+ polling-delay-passive = <100>;
+ polling-delay = <0>;
+ thermal-sensors = <&pmr735a_temp_alarm>;
+
+ trips {
+ pmr735a_trip0: trip0 {
+ temperature = <95000>;
+ hysteresis = <0>;
+ type = "passive";
+ };
+
+ pmr735a_crit: pmr735a-crit {
+ temperature = <115000>;
+ hysteresis = <0>;
+ type = "critical";
+ };
+ };
+ };
+};
--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH V4 6/8] arm64: dts: qcom: pmk8350: Add peripherals for pmk8350
2021-05-05 6:55 [PATCH V4 0/8] Add PMIC DT files for sc7280 satya priya
` (4 preceding siblings ...)
2021-05-05 6:55 ` [PATCH V4 5/8] arm64: dts: qcom: pmr735a: " satya priya
@ 2021-05-05 6:55 ` satya priya
2021-05-05 14:35 ` Matthias Kaehlcke
2021-05-05 6:55 ` [PATCH V4 7/8] arm64: dts: qcom: sc7280: Add channel nodes for sc7280-idp satya priya
2021-05-05 6:55 ` [PATCH V4 8/8] arm64: dts: qcom: sc7280: Include PMIC DT files " satya priya
7 siblings, 1 reply; 16+ messages in thread
From: satya priya @ 2021-05-05 6:55 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Rob Herring
Cc: mka, linux-arm-msm, linux-kernel, devicetree, kgunda, satya priya
Add PON, RTC, VADC and ACD_TM support for PMK8350.
Signed-off-by: satya priya <skakit@codeaurora.org>
---
arch/arm64/boot/dts/qcom/pmk8350.dtsi | 51 ++++++++++++++++++++++++++++++++++-
1 file changed, 50 insertions(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/pmk8350.dtsi b/arch/arm64/boot/dts/qcom/pmk8350.dtsi
index 1530b8f..04fc263 100644
--- a/arch/arm64/boot/dts/qcom/pmk8350.dtsi
+++ b/arch/arm64/boot/dts/qcom/pmk8350.dtsi
@@ -3,6 +3,8 @@
* Copyright (c) 2021, Linaro Limited
*/
+#include <dt-bindings/input/input.h>
+#include <dt-bindings/input/linux-event-codes.h>
#include <dt-bindings/interrupt-controller/irq.h>
#include <dt-bindings/spmi/spmi.h>
@@ -13,10 +15,57 @@
#address-cells = <1>;
#size-cells = <0>;
+ pmk8350_pon: pon@1300 {
+ compatible = "qcom,pm8998-pon";
+ reg = <0x1300>;
+
+ pwrkey {
+ compatible = "qcom,pmk8350-pwrkey";
+ interrupts = <0x0 0x13 0x7 IRQ_TYPE_EDGE_BOTH>;
+ linux,code = <KEY_POWER>;
+ };
+
+ resin {
+ compatible = "qcom,pmk8350-resin";
+ interrupts = <0x0 0x13 0x6 IRQ_TYPE_EDGE_BOTH>;
+ linux,code = <KEY_VOLUMEDOWN>;
+ };
+ };
+
+ pmk8350_vadc: adc@3100 {
+ compatible = "qcom,spmi-adc7";
+ reg = <0x3100>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ interrupts = <0x0 0x31 0x0 IRQ_TYPE_EDGE_RISING>;
+ interrupt-names = "eoc-int-en-set";
+ #io-channel-cells = <1>;
+ io-channel-ranges;
+ };
+
+ pmk8350_adc_tm: adc-tm@3400 {
+ compatible = "qcom,adc-tm7";
+ reg = <0x3400>;
+ interrupts = <0x0 0x34 0x0 IRQ_TYPE_EDGE_RISING>;
+ interrupt-names = "threshold";
+ #address-cells = <1>;
+ #size-cells = <0>;
+ #thermal-sensor-cells = <1>;
+ status = "disabled";
+ };
+
+ pmk8350_rtc: rtc@6100 {
+ compatible = "qcom,pmk8350-rtc";
+ reg = <0x6100>, <0x6200>;
+ reg-names = "rtc", "alarm";
+ interrupts = <0x0 0x62 0x1 IRQ_TYPE_EDGE_RISING>;
+ };
+
pmk8350_gpios: gpio@b000 {
- compatible = "qcom,pmk8350-gpio";
+ compatible = "qcom,pmk8350-gpio", "qcom,spmi-gpio";
reg = <0xb000>;
gpio-controller;
+ gpio-ranges = <&pmk8350_gpios 0 0 4>;
#gpio-cells = <2>;
interrupt-controller;
#interrupt-cells = <2>;
--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH V4 6/8] arm64: dts: qcom: pmk8350: Add peripherals for pmk8350
2021-05-05 6:55 ` [PATCH V4 6/8] arm64: dts: qcom: pmk8350: Add peripherals for pmk8350 satya priya
@ 2021-05-05 14:35 ` Matthias Kaehlcke
0 siblings, 0 replies; 16+ messages in thread
From: Matthias Kaehlcke @ 2021-05-05 14:35 UTC (permalink / raw)
To: satya priya
Cc: Andy Gross, Bjorn Andersson, Rob Herring, linux-arm-msm,
linux-kernel, devicetree, kgunda
On Wed, May 05, 2021 at 12:25:24PM +0530, satya priya wrote:
> Add PON, RTC, VADC and ACD_TM support for PMK8350.
>
> Signed-off-by: satya priya <skakit@codeaurora.org>
Please make sure to include per patch change logs for v > 1. It's
generally useful for following the evolution of a patch and can
save reviewers time by focussing their attention on the parts that
have changed with respect to the previous version, rather than
having to review the entire patch with possibly only minor (or no)
changes over and over again (though multiple passes are often
useful).
Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH V4 7/8] arm64: dts: qcom: sc7280: Add channel nodes for sc7280-idp
2021-05-05 6:55 [PATCH V4 0/8] Add PMIC DT files for sc7280 satya priya
` (5 preceding siblings ...)
2021-05-05 6:55 ` [PATCH V4 6/8] arm64: dts: qcom: pmk8350: Add peripherals for pmk8350 satya priya
@ 2021-05-05 6:55 ` satya priya
2021-05-05 14:46 ` Matthias Kaehlcke
2021-05-05 6:55 ` [PATCH V4 8/8] arm64: dts: qcom: sc7280: Include PMIC DT files " satya priya
7 siblings, 1 reply; 16+ messages in thread
From: satya priya @ 2021-05-05 6:55 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Rob Herring
Cc: mka, linux-arm-msm, linux-kernel, devicetree, kgunda, satya priya
Add channel nodes for the on die temperatures of PMICS
pmk8350, pm8350, pmr735a and pmr735b.
Signed-off-by: satya priya <skakit@codeaurora.org>
---
arch/arm64/boot/dts/qcom/sc7280-idp.dts | 30 ++++++++++++++++++++++++++++++
1 file changed, 30 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/sc7280-idp.dts b/arch/arm64/boot/dts/qcom/sc7280-idp.dts
index 54d2cb3..02a14fc 100644
--- a/arch/arm64/boot/dts/qcom/sc7280-idp.dts
+++ b/arch/arm64/boot/dts/qcom/sc7280-idp.dts
@@ -7,6 +7,10 @@
/dts-v1/;
+#include <dt-bindings/iio/qcom,spmi-adc7-pmr735a.h>
+#include <dt-bindings/iio/qcom,spmi-adc7-pmr735b.h>
+#include <dt-bindings/iio/qcom,spmi-adc7-pm8350.h>
+#include <dt-bindings/iio/qcom,spmi-adc7-pmk8350.h>
#include "sc7280.dtsi"
/ {
@@ -234,6 +238,32 @@
};
};
+&pmk8350_vadc {
+ pm8350_die_temp {
+ reg = <PM8350_ADC7_DIE_TEMP>;
+ label = "pm8350_die_temp";
+ qcom,pre-scaling = <1 1>;
+ };
+
+ pmk8350_die_temp {
+ reg = <PMK8350_ADC7_DIE_TEMP>;
+ label = "pmk8350_die_temp";
+ qcom,pre-scaling = <1 1>;
+ };
+
+ pmr735a_die_temp {
+ reg = <PMR735A_ADC7_DIE_TEMP>;
+ label = "pmr735a_die_temp";
+ qcom,pre-scaling = <1 1>;
+ };
+
+ pmr735b_die_temp {
+ reg = <PMR735B_ADC7_DIE_TEMP>;
+ label = "pmr735b_die_temp";
+ qcom,pre-scaling = <1 1>;
+ };
+};
+
&qupv3_id_0 {
status = "okay";
};
--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH V4 7/8] arm64: dts: qcom: sc7280: Add channel nodes for sc7280-idp
2021-05-05 6:55 ` [PATCH V4 7/8] arm64: dts: qcom: sc7280: Add channel nodes for sc7280-idp satya priya
@ 2021-05-05 14:46 ` Matthias Kaehlcke
2021-05-17 5:28 ` skakit
0 siblings, 1 reply; 16+ messages in thread
From: Matthias Kaehlcke @ 2021-05-05 14:46 UTC (permalink / raw)
To: satya priya
Cc: Andy Gross, Bjorn Andersson, Rob Herring, linux-arm-msm,
linux-kernel, devicetree, kgunda
On Wed, May 05, 2021 at 12:25:25PM +0530, satya priya wrote:
> Subject: arm64: dts: qcom: sc7280: Add channel nodes for sc7280-idp
nit: just 'channel nodes' is a bit vague. In case you respin maybe
change it to something like 'Add ADC channel nodes for PMIC
temperatures to sc7280-idp'
In any case it doesn't seem worth to respin just for this.
>
> Add channel nodes for the on die temperatures of PMICS
> pmk8350, pm8350, pmr735a and pmr735b.
>
> Signed-off-by: satya priya <skakit@codeaurora.org>
Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH V4 7/8] arm64: dts: qcom: sc7280: Add channel nodes for sc7280-idp
2021-05-05 14:46 ` Matthias Kaehlcke
@ 2021-05-17 5:28 ` skakit
0 siblings, 0 replies; 16+ messages in thread
From: skakit @ 2021-05-17 5:28 UTC (permalink / raw)
To: Matthias Kaehlcke
Cc: Andy Gross, Bjorn Andersson, Rob Herring, linux-arm-msm,
linux-kernel, devicetree, kgunda
On 2021-05-05 20:16, Matthias Kaehlcke wrote:
> On Wed, May 05, 2021 at 12:25:25PM +0530, satya priya wrote:
>
>> Subject: arm64: dts: qcom: sc7280: Add channel nodes for sc7280-idp
>
> nit: just 'channel nodes' is a bit vague. In case you respin maybe
> change it to something like 'Add ADC channel nodes for PMIC
> temperatures to sc7280-idp'
>
Okay will change it and resend v4.
> In any case it doesn't seem worth to respin just for this.
>
>>
>> Add channel nodes for the on die temperatures of PMICS
>> pmk8350, pm8350, pmr735a and pmr735b.
>>
>> Signed-off-by: satya priya <skakit@codeaurora.org>
>
> Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH V4 8/8] arm64: dts: qcom: sc7280: Include PMIC DT files for sc7280-idp
2021-05-05 6:55 [PATCH V4 0/8] Add PMIC DT files for sc7280 satya priya
` (6 preceding siblings ...)
2021-05-05 6:55 ` [PATCH V4 7/8] arm64: dts: qcom: sc7280: Add channel nodes for sc7280-idp satya priya
@ 2021-05-05 6:55 ` satya priya
2021-05-05 14:57 ` Matthias Kaehlcke
7 siblings, 1 reply; 16+ messages in thread
From: satya priya @ 2021-05-05 6:55 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Rob Herring
Cc: mka, linux-arm-msm, linux-kernel, devicetree, kgunda, satya priya
The sc7280-idp has four PMICs, include their .dtsi files.
Signed-off-by: satya priya <skakit@codeaurora.org>
---
arch/arm64/boot/dts/qcom/sc7280-idp.dts | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/sc7280-idp.dts b/arch/arm64/boot/dts/qcom/sc7280-idp.dts
index 02a14fc..704fb9a 100644
--- a/arch/arm64/boot/dts/qcom/sc7280-idp.dts
+++ b/arch/arm64/boot/dts/qcom/sc7280-idp.dts
@@ -12,6 +12,10 @@
#include <dt-bindings/iio/qcom,spmi-adc7-pm8350.h>
#include <dt-bindings/iio/qcom,spmi-adc7-pmk8350.h>
#include "sc7280.dtsi"
+#include "pm7325.dtsi"
+#include "pmr735a.dtsi"
+#include "pm8350c.dtsi"
+#include "pmk8350.dtsi"
/ {
model = "Qualcomm Technologies, Inc. sc7280 IDP platform";
--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH V4 8/8] arm64: dts: qcom: sc7280: Include PMIC DT files for sc7280-idp
2021-05-05 6:55 ` [PATCH V4 8/8] arm64: dts: qcom: sc7280: Include PMIC DT files " satya priya
@ 2021-05-05 14:57 ` Matthias Kaehlcke
2021-05-17 5:27 ` skakit
0 siblings, 1 reply; 16+ messages in thread
From: Matthias Kaehlcke @ 2021-05-05 14:57 UTC (permalink / raw)
To: satya priya
Cc: Andy Gross, Bjorn Andersson, Rob Herring, linux-arm-msm,
linux-kernel, devicetree, kgunda
On Wed, May 05, 2021 at 12:25:26PM +0530, satya priya wrote:
> The sc7280-idp has four PMICs, include their .dtsi files.
>
> Signed-off-by: satya priya <skakit@codeaurora.org>
> ---
> arch/arm64/boot/dts/qcom/sc7280-idp.dts | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/sc7280-idp.dts b/arch/arm64/boot/dts/qcom/sc7280-idp.dts
> index 02a14fc..704fb9a 100644
> --- a/arch/arm64/boot/dts/qcom/sc7280-idp.dts
> +++ b/arch/arm64/boot/dts/qcom/sc7280-idp.dts
> @@ -12,6 +12,10 @@
> #include <dt-bindings/iio/qcom,spmi-adc7-pm8350.h>
> #include <dt-bindings/iio/qcom,spmi-adc7-pmk8350.h>
> #include "sc7280.dtsi"
> +#include "pm7325.dtsi"
> +#include "pmr735a.dtsi"
> +#include "pm8350c.dtsi"
> +#include "pmk8350.dtsi"
Patch '[7/8] arm64: dts: qcom: sc7280: Add channel nodes for sc7280-idp'
of this series makes use of the label 'pmk8350_vadc', which doesn't exist
at that point.
A series should be organized in a way that applying only a
subset of the series (in order, starting with patch 1) doesn't
cause any build or runtime issues. To achieve this patch [7/8]
and [8/8] of this series need to be swapped.
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH V4 8/8] arm64: dts: qcom: sc7280: Include PMIC DT files for sc7280-idp
2021-05-05 14:57 ` Matthias Kaehlcke
@ 2021-05-17 5:27 ` skakit
0 siblings, 0 replies; 16+ messages in thread
From: skakit @ 2021-05-17 5:27 UTC (permalink / raw)
To: Matthias Kaehlcke
Cc: Andy Gross, Bjorn Andersson, Rob Herring, linux-arm-msm,
linux-kernel, devicetree, kgunda
On 2021-05-05 20:27, Matthias Kaehlcke wrote:
> On Wed, May 05, 2021 at 12:25:26PM +0530, satya priya wrote:
>> The sc7280-idp has four PMICs, include their .dtsi files.
>>
>> Signed-off-by: satya priya <skakit@codeaurora.org>
>> ---
>> arch/arm64/boot/dts/qcom/sc7280-idp.dts | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/sc7280-idp.dts
>> b/arch/arm64/boot/dts/qcom/sc7280-idp.dts
>> index 02a14fc..704fb9a 100644
>> --- a/arch/arm64/boot/dts/qcom/sc7280-idp.dts
>> +++ b/arch/arm64/boot/dts/qcom/sc7280-idp.dts
>> @@ -12,6 +12,10 @@
>> #include <dt-bindings/iio/qcom,spmi-adc7-pm8350.h>
>> #include <dt-bindings/iio/qcom,spmi-adc7-pmk8350.h>
>> #include "sc7280.dtsi"
>> +#include "pm7325.dtsi"
>> +#include "pmr735a.dtsi"
>> +#include "pm8350c.dtsi"
>> +#include "pmk8350.dtsi"
>
> Patch '[7/8] arm64: dts: qcom: sc7280: Add channel nodes for
> sc7280-idp'
> of this series makes use of the label 'pmk8350_vadc', which doesn't
> exist
> at that point.
>
> A series should be organized in a way that applying only a
> subset of the series (in order, starting with patch 1) doesn't
> cause any build or runtime issues. To achieve this patch [7/8]
> and [8/8] of this series need to be swapped.
Okay, I will re-arrange the series and resend v4.
Thanks,
Satya Priya
^ permalink raw reply [flat|nested] 16+ messages in thread