BugFix: Issue #53 - Fix set setInputCurrentLimit() for 900mA.

[ir-mark]

Bug: Setting the input current limit to 900mA does not work
Issue #53

Summary: Setting the input current limit to 900mA does not work.

File: BQ24195.cpp
Function: bool PMICClass::setInputCurrentLimit(float current)

Problem: There is a typo preventing correct operation:

Line 350:
if (current >= CURRENT_LIM_900) { current_val = CURRENT_LIM_900; }

should be:
if (current >= 0.9) { current_val = CURRENT_LIM_900; }

[github-actions]

Memory usage change @ 6484835

Board	flash	%	RAM for global variables	%
arduino:mbed_nano:nano33ble	0 - 0	0.0 - 0.0	0 - 0	0.0 - 0.0
arduino:mbed_nano:nanorp2040connect	🔺 0 - +12	0.0 - 0.0	0 - 0	0.0 - 0.0
arduino:mbed_portenta:envie_m7	N/A	N/A	N/A	N/A
arduino:mbed_portenta:envie_m7:target_core=cm4	N/A	N/A	N/A	N/A
arduino:samd:mkrgsm1400	🔺 0 - +12	0.0 - 0.0	0 - 0	0.0 - 0.0
arduino:samd:mkrnb1500	🔺 0 - +12	0.0 - 0.0	0 - 0	0.0 - 0.0
arduino:samd:mkrvidor4000	🔺 0 - +12	0.0 - 0.0	0 - 0	0.0 - 0.0
arduino:samd:mkrwifi1010	🔺 0 - +12	0.0 - 0.0	0 - 0	0.0 - 0.0
arduino:samd:nano_33_iot	🔺 0 - +12	0.0 - 0.0	0 - 0	0.0 - 0.0

Click for full report table

Board	examples/BatteryCharger flash	%	examples/BatteryCharger RAM for global variables	%	examples/BatteryChargerInterrupt flash	%	examples/BatteryChargerInterrupt RAM for global variables	%	examples/PMICBoostMode flash	%	examples/PMICBoostMode RAM for global variables	%	examples/PMICFaultCheck flash	%	examples/PMICFaultCheck RAM for global variables	%
arduino:mbed_nano:nano33ble	0	0.0	0	0.0	0	0.0	0	0.0	0	0.0	0	0.0	0	0.0	0	0.0
arduino:mbed_nano:nanorp2040connect	12	0.0	0	0.0	12	0.0	0	0.0	0	0.0	0	0.0	0	0.0	0	0.0
arduino:mbed_portenta:envie_m7	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A
arduino:mbed_portenta:envie_m7:target_core=cm4	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A
arduino:samd:mkrgsm1400	12	0.0	0	0.0	12	0.0	0	0.0	0	0.0	0	0.0	0	0.0	0	0.0
arduino:samd:mkrnb1500	12	0.0	0	0.0	12	0.0	0	0.0	0	0.0	0	0.0	0	0.0	0	0.0
arduino:samd:mkrvidor4000	12	0.0	0	0.0	12	0.0	0	0.0	0	0.0	0	0.0	0	0.0	0	0.0
arduino:samd:mkrwifi1010	12	0.0	0	0.0	12	0.0	0	0.0	0	0.0	0	0.0	0	0.0	0	0.0
arduino:samd:nano_33_iot	12	0.0	0	0.0	12	0.0	0	0.0	0	0.0	0	0.0	0	0.0	0	0.0

Click for full report CSV

Board,examples/BatteryCharger<br>flash,%,examples/BatteryCharger<br>RAM for global variables,%,examples/BatteryChargerInterrupt<br>flash,%,examples/BatteryChargerInterrupt<br>RAM for global variables,%,examples/PMICBoostMode<br>flash,%,examples/PMICBoostMode<br>RAM for global variables,%,examples/PMICFaultCheck<br>flash,%,examples/PMICFaultCheck<br>RAM for global variables,%
arduino:mbed_nano:nano33ble,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0
arduino:mbed_nano:nanorp2040connect,12,0.0,0,0.0,12,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0
arduino:mbed_portenta:envie_m7,N/A,N/A,N/A,N/A,N/A,N/A,N/A,N/A,N/A,N/A,N/A,N/A,N/A,N/A,N/A,N/A
arduino:mbed_portenta:envie_m7:target_core=cm4,N/A,N/A,N/A,N/A,N/A,N/A,N/A,N/A,N/A,N/A,N/A,N/A,N/A,N/A,N/A,N/A
arduino:samd:mkrgsm1400,12,0.0,0,0.0,12,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0
arduino:samd:mkrnb1500,12,0.0,0,0.0,12,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0
arduino:samd:mkrvidor4000,12,0.0,0,0.0,12,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0
arduino:samd:mkrwifi1010,12,0.0,0,0.0,12,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0
arduino:samd:nano_33_iot,12,0.0,0,0.0,12,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0

[per1234]

The fix proposed was previously submitted in #34

That pull request was not merged due to uncertainty regarding additional changes it contained, but the specific parts of that patch that are contained in this PR were approved:

#34 (comment)

And also here:

#35 (comment)

So I will approve this PR based on that previous approval.

Thanks @ir-mark!

[ir-mark]

Hi Per, What is the process for creating a release of the Arduino_BQ24195 library containing this fix? How is it decided to make a release? Is creating a release something I can help with? thank you for your help, Mark