From: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
To: Michael Turquette <mturquette@baylibre.com>,
Stephen Boyd <sboyd@kernel.org>, Rob Herring <robh+dt@kernel.org>,
Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>,
Sylwester Nawrocki <s.nawrocki@samsung.com>,
Tomasz Figa <tomasz.figa@gmail.com>,
Chanwoo Choi <cw00.choi@samsung.com>,
Lee Jones <lee.jones@linaro.org>,
Liam Girdwood <lgirdwood@gmail.com>,
Mark Brown <broonie@kernel.org>,
linux-clk@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-samsung-soc@vger.kernel.org
Subject: Re: [PATCH v2 09/10] dt-bindings: mfd: samsung,s5m8767: convert to dtschema
Date: Fri, 1 Oct 2021 13:18:27 +0200 [thread overview]
Message-ID: <bf26dd28-cf5e-bc09-0b99-6666f9d73d69@canonical.com> (raw)
In-Reply-To: <20211001094106.52412-10-krzysztof.kozlowski@canonical.com>
On 01/10/2021 11:41, Krzysztof Kozlowski wrote:
> Convert the MFD part of Samsung S5M8767 PMIC to DT schema format.
> Previously the bindings were mostly in mfd/samsung,sec-core.txt.
>
> Since all of bindings for Samsung S2M and S5M family of PMICs were
> converted from mfd/samsung,sec-core.txt to respective dtschema file,
> remove the former one.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> ---
> .../bindings/mfd/samsung,s5m8767.yaml | 269 ++++++++++++++++++
> .../bindings/mfd/samsung,sec-core.txt | 86 ------
> MAINTAINERS | 2 +-
> 3 files changed, 270 insertions(+), 87 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/mfd/samsung,s5m8767.yaml
> delete mode 100644 Documentation/devicetree/bindings/mfd/samsung,sec-core.txt
>
> diff --git a/Documentation/devicetree/bindings/mfd/samsung,s5m8767.yaml b/Documentation/devicetree/bindings/mfd/samsung,s5m8767.yaml
> new file mode 100644
> index 000000000000..b2529a48c890
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/mfd/samsung,s5m8767.yaml
> @@ -0,0 +1,269 @@
> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/mfd/samsung,s5m8767.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Samsung S5M8767 Power Management IC
> +
> +maintainers:
> + - Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> +
> +description: |
> + This is a part of device tree bindings for S2M and S5M family of Power
> + Management IC (PMIC).
> +
> + The Samsung S5M8767 is a Power Management IC which includes voltage
> + and current regulators, RTC, clock outputs and other sub-blocks.
> +
> +properties:
> + compatible:
> + const: samsung,s5m8767-pmic
> +
> + clocks:
> + $ref: ../clock/samsung,s2mps11.yaml
> + description:
> + Child node describing clock provider.
> +
> + interrupts:
> + maxItems: 1
> +
> + reg:
> + maxItems: 1
> +
> + regulators:
> + $ref: ../regulator/samsung,s5m8767.yaml
> + description:
> + List of child nodes that specify the regulators.
> +
> + s5m8767,pmic-buck2-dvs-voltage:
> + $ref: /schemas/types.yaml#/definitions/uint32-array
> + minItems: 8
> + maxItems: 8
> + description: |
> + A set of 8 voltage values in micro-volt (uV) units for buck2 when
> + changing voltage using gpio dvs.
> +
> + s5m8767,pmic-buck3-dvs-voltage:
> + $ref: /schemas/types.yaml#/definitions/uint32-array
> + minItems: 8
> + maxItems: 8
> + description: |
> + A set of 8 voltage values in micro-volt (uV) units for buck3 when
> + changing voltage using gpio dvs.
> +
> + s5m8767,pmic-buck4-dvs-voltage:
> + $ref: /schemas/types.yaml#/definitions/uint32-array
> + minItems: 8
> + maxItems: 8
> + description: |
> + A set of 8 voltage values in micro-volt (uV) units for buck4 when
> + changing voltage using gpio dvs.
> +
> + s5m8767,pmic-buck-ds-gpios:
> + minItems: 3
> + maxItems: 3
> + description: |
> + GPIO specifiers for three host gpio's used for selecting GPIO DVS lines.
> + It is one-to-one mapped to dvs gpio lines.
> +
> + s5m8767,pmic-buck2-uses-gpio-dvs:
> + type: boolean
> + description: buck2 can be controlled by gpio dvs.
> +
> + s5m8767,pmic-buck3-uses-gpio-dvs:
> + type: boolean
> + description: buck3 can be controlled by gpio dvs.
> +
> + s5m8767,pmic-buck4-uses-gpio-dvs:
> + type: boolean
> + description: buck4 can be controlled by gpio dvs.
> +
> + s5m8767,pmic-buck-default-dvs-idx:
> + $ref: /schemas/types.yaml#/definitions/uint32-array
There is a bug here - it's just uint32.
I will send a v3 later (feel free to review earlier, so I can accumulate
tags/comments).
> + minimum: 0
> + maximum: 7
> + default: 0
> + description: |
> + Default voltage setting selected from the possible 8 options selectable
> + by the dvs gpios. The value of this property should be between 0 and 7.
> + If not specified or if out of range, the default value of this property
> + is set to 0.
> +
Best regards,
Krzysztof
next prev parent reply other threads:[~2021-10-01 11:18 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-01 9:40 [PATCH v2 00/10] regulator/mfd/clock: dt-bindings: Samsung S2M and S5M to dtschema Krzysztof Kozlowski
2021-10-01 9:40 ` [PATCH v2 01/10] regulator: s5m8767: do not use reset value as DVS voltage if GPIO DVS is disabled Krzysztof Kozlowski
2021-10-04 14:30 ` Rob Herring
2021-10-01 9:40 ` [PATCH v2 02/10] regulator: dt-bindings: samsung,s5m8767: correct s5m8767,pmic-buck-default-dvs-idx property Krzysztof Kozlowski
2021-10-04 14:30 ` Rob Herring
2021-10-01 9:40 ` [PATCH v2 03/10] dt-bindings: clock: samsung,s2mps11: convert to dtschema Krzysztof Kozlowski
2021-10-04 14:32 ` Rob Herring
2021-10-08 1:00 ` Stephen Boyd
2021-10-01 9:41 ` [PATCH v2 04/10] regulator: dt-bindings: samsung,s2m: " Krzysztof Kozlowski
2021-10-04 14:34 ` Rob Herring
2021-10-01 9:41 ` [PATCH v2 05/10] regulator: dt-bindings: samsung,s2mpa01: " Krzysztof Kozlowski
2021-10-04 14:35 ` Rob Herring
2021-10-01 9:41 ` [PATCH v2 06/10] regulator: dt-bindings: samsung,s5m8767: " Krzysztof Kozlowski
2021-10-04 14:41 ` Rob Herring
2021-10-04 14:48 ` Krzysztof Kozlowski
2021-10-01 9:41 ` [PATCH v2 07/10] dt-bindings: mfd: samsung,s2mps11: " Krzysztof Kozlowski
2021-10-04 14:42 ` Rob Herring
2021-10-01 9:41 ` [PATCH v2 08/10] dt-bindings: mfd: samsung,s2mpa01: " Krzysztof Kozlowski
2021-10-04 14:43 ` Rob Herring
2021-10-01 9:41 ` [PATCH v2 09/10] dt-bindings: mfd: samsung,s5m8767: " Krzysztof Kozlowski
2021-10-01 11:18 ` Krzysztof Kozlowski [this message]
2021-10-04 14:49 ` Rob Herring
2021-10-01 9:41 ` [PATCH v2 10/10] dt-bindings: mfd: samsung,s5m8767: document buck and LDO supplies Krzysztof Kozlowski
2021-10-04 14:50 ` Rob Herring
2021-10-05 12:14 ` [PATCH v2 00/10] regulator/mfd/clock: dt-bindings: Samsung S2M and S5M to dtschema Mark Brown
2021-10-05 13:14 ` Lee Jones
2021-10-06 12:55 ` Krzysztof Kozlowski
2021-10-06 13:20 ` Lee Jones
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=bf26dd28-cf5e-bc09-0b99-6666f9d73d69@canonical.com \
--to=krzysztof.kozlowski@canonical.com \
--cc=b.zolnierkie@samsung.com \
--cc=broonie@kernel.org \
--cc=cw00.choi@samsung.com \
--cc=devicetree@vger.kernel.org \
--cc=lee.jones@linaro.org \
--cc=lgirdwood@gmail.com \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=mturquette@baylibre.com \
--cc=robh+dt@kernel.org \
--cc=s.nawrocki@samsung.com \
--cc=sboyd@kernel.org \
--cc=tomasz.figa@gmail.com \
/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).