diff mbox series

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

Message ID 20240228083846.2401108-2-absicsz@gmail.com (mailing list archive)
State New, archived
Headers show
Series ARM: dts: n900: set charge current limit to 950mA | expand

Commit Message

Sicelo Feb. 28, 2024, 8:38 a.m. UTC 
From: Arthur Demchenkov <spinal.by@gmail.com>

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 <spinal.by@gmail.com>
Signed-off-by: Sicelo A. Mhlongo <absicsz@gmail.com>
---
 arch/arm/boot/dts/ti/omap/omap3-n900.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Tony Lindgren Feb. 29, 2024, 8:19 a.m. UTC | #1 
* Sicelo A. Mhlongo <absicsz@gmail.com> [240228 10:40]:
> From: Arthur Demchenkov <spinal.by@gmail.com>
> 
> 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
Sicelo Feb. 29, 2024, 8:35 a.m. UTC | #2 
Hi Tony
On Thu, Feb 29, 2024 at 10:19:23AM +0200, Tony Lindgren wrote:
> * Sicelo A. Mhlongo <absicsz@gmail.com> [240228 10:40]:
> > From: Arthur Demchenkov <spinal.by@gmail.com>
> > 
> > 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
Tony Lindgren April 3, 2024, 9:03 a.m. UTC | #3 
* Sicelo A. Mhlongo <absicsz@gmail.com> [240228 10:40]:
> From: Arthur Demchenkov <spinal.by@gmail.com>
> 
> 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

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

Patch

diff --git a/arch/arm/boot/dts/ti/omap/omap3-n900.dts b/arch/arm/boot/dts/ti/omap/omap3-n900.dts
index d33485341251..07c5b963af78 100644
--- a/arch/arm/boot/dts/ti/omap/omap3-n900.dts
+++ b/arch/arm/boot/dts/ti/omap/omap3-n900.dts
@@ -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>;