Re: [PATCH 1/5] arm64: dts: qcom: pmi8998: add rradc node

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 17/10/2022 02:10, Krzysztof Kozlowski wrote:
On 16/10/2022 14:03, Caleb Connolly wrote:
Add a DT node for the Round Robin ADC found in the PMI8998 PMIC.

Signed-off-by: Caleb Connolly <caleb.connolly@xxxxxxxxxx>
---
  arch/arm64/boot/dts/qcom/pmi8998.dtsi | 8 ++++++++
  1 file changed, 8 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/pmi8998.dtsi b/arch/arm64/boot/dts/qcom/pmi8998.dtsi
index 6d3d212560c1..5a479259c041 100644
--- a/arch/arm64/boot/dts/qcom/pmi8998.dtsi
+++ b/arch/arm64/boot/dts/qcom/pmi8998.dtsi
@@ -18,6 +18,14 @@ pmi8998_gpio: gpios@c000 {
  			interrupt-controller;
  			#interrupt-cells = <2>;
  		};
+
+		pmi8998_rradc: adc@4500 {
+			compatible = "qcom,pmi8998-rradc";
+			reg = <0x4500>;
+			#io-channel-cells = <1>;
+
+			status = "disabled";

Why disabling it? It does not need any external/board resources, so
maybe it should be just like other adcs - enabled by default? What does
it measure? What is its input?

The RRADC mostly reports values which only make sense on mobile devices, battery ID and temperature, USB and DC input voltage/current as well as a (duplicate?) die temperature of the PMIC - I guess closer to the SMB/FG block.

When I last tested the DC input readings didn't work on db845c, as it just produces the 4.2v you'd get from a battery.

Enabling it by default should be fine (and would certainly simplify this series :P).

Best regards,
Krzysztof


--
Kind Regards,
Caleb (they/them)



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux