Skip to contentSkip to conversation

ARM: dts: n900: set charge current limit to 950mA

PATCH In review Sicelo A. Mhlongo 2 participants 4 messages Feb 28, 2024 – Apr 3, 2024

Subsystems (2)
Device tree bindings OMAP device tree support
lore
Share & tools
lore

Sicelo A. Mhlongo

author
#0 lore raw

PATCH 1/1 ARM: dts: n900: set charge current limit to 950mA

From: Arthur Demchenkov <[email protected]>

The vendor kernel used 950mA as the default. The same value works fine on the mainline Linux kernel, and has been tested extensively under Maemo Leste [1] and postmarketOS, who have been using it for a number of years.

[1] https://github.com/maemo-leste/n9xx-linux/commit/fbc4ce7a84e59215914a8981afe918002b191493

Signed-off-by Arthur Demchenkov <[email protected]>
Signed-off-by Sicelo A. Mhlongo <[email protected]>
---
 arch/arm/boot/dts/ti/omap/omap3-n900.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
arch/arm/boot/dts/ti/omap/omap3-n900.dts +1 -1
index d33485341251..07c5b963af78 100644
@@ -754,7 +754,7 @@ bq24150a: bq24150a@6b {
 		ti,current-limit = <100>;
 		ti,weak-battery-voltage = <3400>;
 		ti,battery-regulation-voltage = <4200>;
-		ti,charge-current = <650>;
+		ti,charge-current = <950>;
 		ti,termination-current = <100>;
 		ti,resistor-sense = <68>;
Signature · 1 line
2.43.0
maintainer
#1 lore raw
Quoted 5 lines · Sicelo A. Mhlongo

From: Arthur Demchenkov <[email protected]>

The vendor kernel used 950mA as the default. The same value works fine on the mainline Linux kernel, and has been tested extensively under Maemo Leste [1] and postmarketOS, who have been using it for a number of years.

Makes sense to me. At -rc6 I'll likely wait until v6.9-rc1 before applying new patches. If this is needed as a fix before that, then please let me know.

Regards,

Tony

author
#2 lore raw

Hi Tony

Quoted 10 lines · Tony Lindgren
Quoted 5 lines · Sicelo A. Mhlongo

From: Arthur Demchenkov <[email protected]>

The vendor kernel used 950mA as the default. The same value works fine on the mainline Linux kernel, and has been tested extensively under Maemo Leste [1] and postmarketOS, who have been using it for a number of years.

Makes sense to me. At -rc6 I'll likely wait until v6.9-rc1 before applying new patches. If this is needed as a fix before that, then please let me know.

Thank you. It can wait for v6.9-rc1, since using 650mA does not cause a major usability issue.

Regards
Sicelo A. Mhlongo

maintainer
#3 lore raw
In reply to Sicelo A. Mhlongo
Quoted 5 lines · Sicelo A. Mhlongo

From: Arthur Demchenkov <[email protected]>

The vendor kernel used 950mA as the default. The same value works fine on the mainline Linux kernel, and has been tested extensively under Maemo Leste [1] and postmarketOS, who have been using it for a number of years.

Thanks applying into omap-for-v6.10/dt.

Tony

Quoted 1 line

[1] https://github.com/maemo-leste/n9xx-linux/commit/fbc4ce7a84e59215914a8981afe918002b191493