From: krzk@kernel.org
To: Jonathan Bakker <xc-racer2@live.ca>, alim.akhtar@samsung.com
Cc: robh+dt@kernel.org, devicetree@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 7/7] ARM: dts: s5pv210: Add charger support in Aries
Date: Wed, 23 Mar 2022 16:31:48 +0100 [thread overview]
Message-ID: <2eee2611-d618-3fe2-4315-c57a26de6b21@kernel.org> (raw)
In-Reply-To: <CY4PR04MB05671BD0A7FF349E8B04EE84CB189@CY4PR04MB0567.namprd04.prod.outlook.com>
On 23/03/2022 16:03, Jonathan Bakker wrote:
> Add charger-manager support to Aries boards to allow safe
> charging of the battery without the need for userspace control.
>
> Signed-off-by: Jonathan Bakker <xc-racer2@live.ca>
> ---
> arch/arm/boot/dts/s5pv210-fascinate4g.dts | 162 ++++++++++++++++++++++
> arch/arm/boot/dts/s5pv210-galaxys.dts | 144 +++++++++++++++++++
> 2 files changed, 306 insertions(+)
>
> diff --git a/arch/arm/boot/dts/s5pv210-fascinate4g.dts b/arch/arm/boot/dts/s5pv210-fascinate4g.dts
> index 7427c84f1126..9530231b7a70 100644
> --- a/arch/arm/boot/dts/s5pv210-fascinate4g.dts
> +++ b/arch/arm/boot/dts/s5pv210-fascinate4g.dts
> @@ -57,6 +57,168 @@
> pinctrl-0 = <&main_micbias_ena>;
> };
>
> + thermal-zones {
> + batt_thermal: batt-thermal {
> + polling-delay-passive = <60000>; /* 60 seconds */
There is no passive cooling device, so why do you need it?
> + polling-delay = <600000>; /* 600 seconds */
> +
> + thermal-sensors = <&batt_thermistor>;
> + };
> + };
> +
> + batt_thermistor: thermal-sensor-0 {
> + compatible = "generic-adc-thermal";
> + #thermal-sensor-cells = <0>;
> + io-channels = <&adc 6>;
> + io-channel-names = "sensor-channel";
> +
> + temperature-lookup-table = <
> + (-20000) 1859
> + (-19000) 1846
> + (-18000) 1832
> + (-17000) 1818
> + (-16000) 1804
> + (-15000) 1790
> + (-14000) 1773
> + (-13000) 1756
> + (-12000) 1739
> + (-11000) 1722
> + (-10000) 1705
> + (-9000) 1691
> + (-8000) 1677
> + (-7000) 1663
> + (-6000) 1649
> + (-5000) 1635
> + (-4000) 1550
> + (-3000) 1510
> + (-2000) 1500
> + (-1000) 1490
> + 0 1480
> + 1000 1470
> + 2000 1460
> + 3000 1450
> + 4000 1430
> + 5000 1420
> + 6000 1406
> + 7000 1386
> + 8000 1366
> + 9000 1346
> + 10000 1326
> + 11000 1302
> + 12000 1278
> + 13000 1254
> + 14000 1230
> + 15000 1206
> + 16000 1182
> + 17000 1158
> + 18000 1134
> + 19000 1110
> + 20000 1086
> + 21000 1059
> + 22000 1035
> + 23000 1011
> + 24000 987
> + 25000 963
> + 26000 937
> + 27000 913
> + 28000 889
> + 29000 865
> + 30000 841
> + 31000 816
> + 32000 794
> + 33000 772
> + 34000 750
> + 35000 728
> + 36000 708
> + 37000 690
> + 38000 672
> + 39000 654
> + 40000 636
> + 41000 616
> + 42000 599
> + 43000 580
> + 44000 565
> + 45000 548
> + 46000 529
> + 47000 512
> + 48000 495
> + 49000 478
> + 50000 461
> + 51000 440
> + 52000 431
> + 53000 416
> + 54000 405
> + 55000 396
> + 56000 375
> + 57000 360
> + 58000 347
> + 59000 334
> + 60000 325
> + 61000 311
> + 62000 303
> + 63000 296
> + 64000 290
> + 65000 279
> + 66000 265
> + 67000 254
> + 68000 240
> + 69000 220
> + 70000 206>;
> + };
> +
> + charger_manager: charger-manager-0 {
> + compatible = "charger-manager";
Sorry, this is not a hardware. It's a hack to configure kernel charging
driver via DT which was made deprecated.
Best regards,
Krzysztof
next prev parent reply other threads:[~2022-03-23 15:31 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20220322201144.20320-1-xc-racer2@live.ca>
2022-03-22 20:11 ` [PATCH 1/7] ARM: dts: s5pv210: Split memory nodes to match spec Jonathan Bakker
2022-03-23 10:54 ` Krzysztof Kozlowski
2022-03-23 14:59 ` Jonathan Bakker
2022-03-23 15:06 ` Krzysztof Kozlowski
2022-03-23 17:05 ` Jonathan Bakker
2022-03-24 11:23 ` Krzysztof Kozlowski
2022-03-23 15:03 ` [PATCH 2/7] ARM: dts: s5pv210: Adjust I2S entries " Jonathan Bakker
2022-03-23 15:14 ` Krzysztof Kozlowski
2022-03-23 15:15 ` Krzysztof Kozlowski
2022-03-23 17:24 ` Jonathan Bakker
2022-03-24 11:49 ` Krzysztof Kozlowski
2023-04-21 9:44 ` Krzysztof Kozlowski
2023-05-03 2:33 ` Jonathan Bakker
[not found] ` <20220323150311.26699-1-xc-racer2@live.ca>
2022-03-23 15:03 ` [PATCH 3/7] ARM: dts: s5pv210: Adjust DMA node names " Jonathan Bakker
2022-03-23 15:03 ` [PATCH 4/7] ARM: dts: s5pv210: Remove spi-cs-high on panel in Aries Jonathan Bakker
2022-03-23 15:22 ` Krzysztof Kozlowski
2022-03-23 15:03 ` [PATCH 5/7] ARM: dts: s5pv210: Correct interrupt name for bluetooth " Jonathan Bakker
2022-03-23 15:23 ` krzk
2022-03-23 15:03 ` [PATCH 6/7] ARM: dts: s5pv210: Add charger regulator to max8998 " Jonathan Bakker
2022-03-23 15:03 ` [PATCH 7/7] ARM: dts: s5pv210: Add charger support " Jonathan Bakker
2022-03-23 15:31 ` krzk [this message]
2022-03-23 17:20 ` Jonathan Bakker
2022-03-24 11:47 ` Krzysztof Kozlowski
2022-03-25 2:23 ` Jonathan Bakker
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2eee2611-d618-3fe2-4315-c57a26de6b21@kernel.org \
--to=krzk@kernel.org \
--cc=alim.akhtar@samsung.com \
--cc=devicetree@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=robh+dt@kernel.org \
--cc=xc-racer2@live.ca \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).