From: Rob Herring <robh@kernel.org>
To: Chris Morgan <macroalpha82@gmail.com>
Cc: linux-pm@vger.kernel.org, linux-rockchip@lists.infradead.org,
devicetree@vger.kernel.org, maccraft123mc@gmail.com,
sre@kernel.org, heiko@sntech.de, lee.jones@linaro.org,
pgwipeout@gmail.com, Chris Morgan <macromorgan@hotmail.com>
Subject: Re: [PATCH v3 1/4] dt-bindings: Add Rockchip rk817 battery charger support
Date: Tue, 17 Aug 2021 14:11:58 -0500 [thread overview]
Message-ID: <YRwJ/riSIGfNu64B@robh.at.kernel.org> (raw)
In-Reply-To: <20210810025436.23367-2-macroalpha82@gmail.com>
On Mon, Aug 09, 2021 at 09:54:33PM -0500, Chris Morgan wrote:
> From: Chris Morgan <macromorgan@hotmail.com>
>
> Create dt-binding documentation to document rk817 battery and charger
> usage. New device-tree properties have been added.
>
> - rockchip,resistor-sense-micro-ohms: The value in microohms of the
> sample resistor.
> - rockchip,sleep-enter-current-microamp: The value in microamps of the
> sleep enter current.
> - rockchip,sleep-filter-current: The value in microamps of the sleep
> filter current.
>
> Signed-off-by: Chris Morgan <macromorgan@hotmail.com>
> Signed-off-by: Maya Matuszczyk <maccraft123mc@gmail.com>
> ---
> .../devicetree/bindings/mfd/rk808.txt | 36 ++++++++++++++++++-
> 1 file changed, 35 insertions(+), 1 deletion(-)
Kind of a big change without converting this binding to DT schema first.
>
> diff --git a/Documentation/devicetree/bindings/mfd/rk808.txt b/Documentation/devicetree/bindings/mfd/rk808.txt
> index 23a17a6663ec..89b9fa7e10b5 100644
> --- a/Documentation/devicetree/bindings/mfd/rk808.txt
> +++ b/Documentation/devicetree/bindings/mfd/rk808.txt
> @@ -70,13 +70,40 @@ Optional RK817 properties:
>
> - vcc8-supply: The input supply for BOOST
> - vcc9-supply: The input supply for OTG_SWITCH
> +
> - codec: The child node for the codec to hold additional properties.
> If no additional properties are required for the codec, this
> node can be omitted.
> -
Unrelated whitespace changes.
> - rockchip,mic-in-differential: Telling if the microphone uses differential
> mode. Should be under the codec child node.
>
> +- battery: The child node for the charger to hold additional properties.
> + If a battery is not in use, this node can be omitted. If a
> + battery node is used, the following values are required:
> + rockchip,resistor-sense, rockchip,sleep-enter-current,
> + rockchip,sleep-filter-current, and a phandle to a
Property names are out of date...
> + monitored-battery node that contains a valid value for
> + charge-full-design-microamp-hours,
> + charge-term-current-microamp,
> + constant-charge-current-max-microamp,
> + constant-charge-voltage-max-microvolt,
> + voltage-max-design-microvolt, voltage-min-design-microvolt,
> + and a valid ocv-capacity table.
Please list these 1 per line and make it clearer this is the list of
properties for the battery node.
> +- rockchip,resistor-sense-micro-ohms: Value in microohms of the battery sense
> + resistor. The PMIC only supports values
> + of either 10000 or 20000. This value is
> + used by the driver to set the correct
> + divisor value to translate ADC readings
> + into the proper units of measure.
> +- rockchip,sleep-enter-current-microamp: Value in microamps of the sleep enter
> + current for the charger. Value is used
> + by the driver to calibrate the relax
> + threshold.
> +- rockchip,sleep-filter-current-microamp: Value in microamps of the sleep
> + filter current for the charger.
> + Value is used by the driver to derive
> + the sleep sample current.
> +
> Optional RK818 properties:
> - vcc1-supply: The input supply for DCDC_REG1
> - vcc2-supply: The input supply for DCDC_REG2
> @@ -459,6 +486,13 @@ Example:
> };
> };
>
> + rk817_battery: battery {
> + monitored-battery = <&battery_cell>;
> + rockchip,resistor-sense-micro-ohms = <10000>;
> + rockchip,sleep-enter-current-microamp = <300000>;
> + rockchip,sleep-filter-current-microamp = <100000>;
> + };
> +
> rk817_codec: codec {
> rockchip,mic-in-differential;
> };
> --
> 2.25.1
>
>
next prev parent reply other threads:[~2021-08-17 19:12 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-10 2:54 [PATCH v3 0/4] power: supply: Add Support for RK817 Charger Chris Morgan
2021-08-10 2:54 ` [PATCH v3 1/4] dt-bindings: Add Rockchip rk817 battery charger support Chris Morgan
2021-08-17 19:11 ` Rob Herring [this message]
2021-08-10 2:54 ` [PATCH v3 2/4] mfd: " Chris Morgan
2021-08-10 2:54 ` [PATCH v3 3/4] power: supply: Add charger driver for Rockchip RK817 Chris Morgan
2021-08-10 2:54 ` [PATCH v3 4/4] arm64: dts: rockchip: add rk817 charger to Odroid Go Advance Chris Morgan
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=YRwJ/riSIGfNu64B@robh.at.kernel.org \
--to=robh@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=heiko@sntech.de \
--cc=lee.jones@linaro.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=maccraft123mc@gmail.com \
--cc=macroalpha82@gmail.com \
--cc=macromorgan@hotmail.com \
--cc=pgwipeout@gmail.com \
--cc=sre@kernel.org \
/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).