* [PATCH] ARM: dts: n900: set charge current limit to 950mA
@ 2024-02-28 8:38 Sicelo A. Mhlongo
2024-02-29 8:19 ` Tony Lindgren
2024-04-03 9:03 ` Tony Lindgren
0 siblings, 2 replies; 4+ messages in thread
From: Sicelo A. Mhlongo @ 2024-02-28 8:38 UTC (permalink / raw)
To: devicetree, Benoît Cousson, Tony Lindgren,
Krzysztof Kozlowski, Conor Dooley
Cc: linux-pm, pali, sre, spinal.by, maemo-leste, linux-omap,
Sicelo A . Mhlongo
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(-)
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>;
--
2.43.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] ARM: dts: n900: set charge current limit to 950mA
2024-02-28 8:38 [PATCH] ARM: dts: n900: set charge current limit to 950mA Sicelo A. Mhlongo
@ 2024-02-29 8:19 ` Tony Lindgren
2024-02-29 8:35 ` Sicelo
2024-04-03 9:03 ` Tony Lindgren
1 sibling, 1 reply; 4+ messages in thread
From: Tony Lindgren @ 2024-02-29 8:19 UTC (permalink / raw)
To: Sicelo A. Mhlongo
Cc: devicetree, Benoît Cousson, Krzysztof Kozlowski,
Conor Dooley, linux-pm, pali, sre, spinal.by, maemo-leste,
linux-omap
* 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
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ARM: dts: n900: set charge current limit to 950mA
2024-02-29 8:19 ` Tony Lindgren
@ 2024-02-29 8:35 ` Sicelo
0 siblings, 0 replies; 4+ messages in thread
From: Sicelo @ 2024-02-29 8:35 UTC (permalink / raw)
To: Tony Lindgren
Cc: devicetree, Benoît Cousson, Krzysztof Kozlowski,
Conor Dooley, linux-pm, pali, sre, spinal.by, maemo-leste,
linux-omap
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
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ARM: dts: n900: set charge current limit to 950mA
2024-02-28 8:38 [PATCH] ARM: dts: n900: set charge current limit to 950mA Sicelo A. Mhlongo
2024-02-29 8:19 ` Tony Lindgren
@ 2024-04-03 9:03 ` Tony Lindgren
1 sibling, 0 replies; 4+ messages in thread
From: Tony Lindgren @ 2024-04-03 9:03 UTC (permalink / raw)
To: Sicelo A. Mhlongo
Cc: devicetree, Benoît Cousson, Krzysztof Kozlowski,
Conor Dooley, linux-pm, pali, sre, spinal.by, maemo-leste,
linux-omap
* 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
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2024-04-03 9:04 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-02-28 8:38 [PATCH] ARM: dts: n900: set charge current limit to 950mA Sicelo A. Mhlongo
2024-02-29 8:19 ` Tony Lindgren
2024-02-29 8:35 ` Sicelo
2024-04-03 9:03 ` Tony Lindgren
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).