RE: nPM1300 BCHGISETDISCHARGE

runsiv — Tue, 10 Dec 2024 12:45:57 GMT

Update:

The battery current calculation uses the old definition of the battery current discharge limit, and the PS has not been updated yet to reflect this. As I understand it, the update is coming but I'm not sure about the timeline

NCS 2.5.0 is not updated to reflect this.

The Zephyr main driver was only very recently updated, and this change has not made its way into NCS yet:

https://github.com/zephyrproject-rtos/zephyr/commit/ab6bca66ac9c3352559fdbe786cb38a34c8fa29c

So PowerUP and NCS is at the moment using the old definition of the discharge limit configuration. After the next upmerge, NCS will start using the new definition, and the PowerUP overlay export functionality will be updated accordingly. At that point, you will get a compile error if you try to use an overlay with a discharge current limit other than 200 mA or 1000 mA.

1. The table only shows the high and low value, is is possible to select values between min and high value

3. The PS at the moment states that you can calculable the battery current from the ADC measurement value

A value of 01 means the battery is discharging. During a discharge, the full scale current is the weighted sum of registers BCHGISETDISCHARGEMSB and BCHGISETDISCHARGELSB multiplied by 0.836.

But the PS does not mention that the fullscale is actually calculated based on 270 mA or 1340 mA values instead of 200 mA and 1000 mA as mentioned here:

So to properly calculate the battery current, you need use an ADC fullscale value of 270 mA or 1340 mA, multiplied by 0.836.

Regards

Runar

RE: nPM1300 BCHGISETDISCHARGE

runsiv — Tue, 10 Dec 2024 12:06:52 GMT

Hi Kevin

I'm looking into your issue

Regarding question 2. What is the value you are seeing?

Regards

Runar