Re: [PATCH v2 01/10] dt-bindings: hwmon: Describe Qualcomm BCL hardware

From: Manaf Meethalavalappu Pallikunhi

Date: Thu Jul 30 2026 - 12:57:40 EST


Hi Daniel,

On 7/30/2026 5:05 PM, Daniel Lezcano wrote:

Hi Krzystof,


On 7/23/26 10:11, Krzysztof Kozlowski wrote:
On Wed, Jul 22, 2026 at 12:00:06AM +0530, Manaf Meethalavalappu Pallikunhi wrote:
+properties:
+  compatible:
+    enum:
+      - qcom,pm7250b-bcl
+      - qcom,pm8350c-bcl
+      - qcom,pm8550-bcl
+      - qcom,pmh0101-bcl
+      - qcom,pmih0108-bcl
+      - qcom,smb2360-bcl
+      - qcom,smb2370-bcl
+
+  reg:
+    maxItems: 1
+
+  interrupts:
+    maxItems: 2
+
+  interrupt-names:
+    items:
+      - const: max-min
+      - const: critical
+

Binding looks in general ok, but feels incomplete. If this is battery
limiting hardware, does it actually need any battery data through
monitored-battery property?

I'm not sure it is relevant here. The BCL is not a power supply neither a charger. It monitors a power supply output which can be a battery or another source of power AFAIU.

The monitored-battery property is used by chargers or fuel gauges if I refer to the documentation.

The BCL monitors the output of the power supply while the chargers monitors the input for the battery.

Manaf, does it sound correct ?

Yes, that is correct. BCL monitors the current drawn by the system from the battery, as well as low VBAT or low VPH conditions. It is not involved in monitoring the charging path. PMICs such as PM8350C, PM8550, and PMH0101 do not even have charger or fuel-gauge peripherals where BCL is still supported to monitor low VPH condition.

Thanks,
Manaf