On 05/06/2022 15:32, mail@xxxxxxxxxxx wrote:
> From: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
>
> Convert the dt binding for the da9063/da9063l to yaml.
>
> Signed-off-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
> ---
> .../devicetree/bindings/mfd/da9063.txt | 111 ----------------
> .../devicetree/bindings/mfd/da9063.yaml | 123 ++++++++++++++++++
> .../devicetree/bindings/trivial-devices.yaml | 2 -
> 3 files changed, 123 insertions(+), 113 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/mfd/da9063.txt
> create mode 100644 Documentation/devicetree/bindings/mfd/da9063.yaml
>
> diff --git a/Documentation/devicetree/bindings/mfd/da9063.txt b/Documentation/devicetree/bindings/mfd/da9063.txt
> deleted file mode 100644
> index 91b79a21d403..000000000000
> --- a/Documentation/devicetree/bindings/mfd/da9063.txt
> +++ /dev/null
> @@ -1,111 +0,0 @@
> -* Dialog DA9063/DA9063L Power Management Integrated Circuit (PMIC)
> -
> -DA9063 consists of a large and varied group of sub-devices (I2C Only):
> -
> -Device Supply Names Description
> ------- ------------ -----------
> -da9063-regulator : : LDOs & BUCKs
> -da9063-onkey : : On Key
> -da9063-rtc : : Real-Time Clock (DA9063 only)
> -da9063-watchdog : : Watchdog
> -
> -======
> -
> -Required properties:
> -
> -- compatible : Should be "dlg,da9063" or "dlg,da9063l"
> -- reg : Specifies the I2C slave address (this defaults to 0x58 but it can be
> - modified to match the chip's OTP settings).
> -- interrupts : IRQ line information.
> -- interrupt-controller
> -
> -Sub-nodes:
> -
> -- regulators : This node defines the settings for the LDOs and BUCKs.
> - The DA9063(L) regulators are bound using their names listed below:
> -
> - bcore1 : BUCK CORE1
> - bcore2 : BUCK CORE2
> - bpro : BUCK PRO
> - bmem : BUCK MEM
> - bio : BUCK IO
> - bperi : BUCK PERI
> - ldo1 : LDO_1 (DA9063 only)
> - ldo2 : LDO_2 (DA9063 only)
> - ldo3 : LDO_3
> - ldo4 : LDO_4 (DA9063 only)
> - ldo5 : LDO_5 (DA9063 only)
> - ldo6 : LDO_6 (DA9063 only)
> - ldo7 : LDO_7
> - ldo8 : LDO_8
> - ldo9 : LDO_9
> - ldo10 : LDO_10 (DA9063 only)
> - ldo11 : LDO_11
> -
> - The component follows the standard regulator framework and the bindings
> - details of individual regulator device can be found in:
> - Documentation/devicetree/bindings/regulator/regulator.txt
> -
> -- rtc : This node defines settings for the Real-Time Clock associated with
> - the DA9063 only. The RTC is not present in DA9063L. There are currently
> - no entries in this binding, however compatible = "dlg,da9063-rtc" should
> - be added if a node is created.
> -
> -- onkey : This node defines the OnKey settings for controlling the key
> - functionality of the device. The node should contain the compatible property
> - with the value "dlg,da9063-onkey".
> -
> - Optional onkey properties:
> -
> - - dlg,disable-key-power : Disable power-down using a long key-press. If this
> - entry exists the OnKey driver will remove support for the KEY_POWER key
> - press. If this entry does not exist then by default the key-press
> - triggered power down is enabled and the OnKey will support both KEY_POWER
> - and KEY_SLEEP.
> -
> -- watchdog : This node defines settings for the Watchdog timer associated
> - with the DA9063 and DA9063L. There are currently no entries in this
> - binding, however compatible = "dlg,da9063-watchdog" should be added
> - if a node is created.
> -
> -
> -Example:
> -
> - pmic0: da9063@58 {
> - compatible = "dlg,da9063"
> - reg = <0x58>;
> - interrupt-parent = <&gpio6>;
> - interrupts = <11 IRQ_TYPE_LEVEL_LOW>;
> - interrupt-controller;
> -
> - rtc {
> - compatible = "dlg,da9063-rtc";
> - };
> -
> - wdt {
> - compatible = "dlg,da9063-watchdog";
> - };
> -
> - onkey {
> - compatible = "dlg,da9063-onkey";
> - dlg,disable-key-power;
> - };
> -
> - regulators {
> - DA9063_BCORE1: bcore1 {
> - regulator-name = "BCORE1";
> - regulator-min-microvolt = <300000>;
> - regulator-max-microvolt = <1570000>;
> - regulator-min-microamp = <500000>;
> - regulator-max-microamp = <2000000>;
> - regulator-boot-on;
> - };
> - DA9063_LDO11: ldo11 {
> - regulator-name = "LDO_11";
> - regulator-min-microvolt = <900000>;
> - regulator-max-microvolt = <3600000>;
> - regulator-boot-on;
> - };
> - };
> - };
> -
> diff --git a/Documentation/devicetree/bindings/mfd/da9063.yaml b/Documentation/devicetree/bindings/mfd/da9063.yaml
> new file mode 100644
> index 000000000000..376880061f42
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/mfd/da9063.yaml
dlg,da9063.yaml
> @@ -0,0 +1,123 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/mfd/da9063.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Dialog DA9063/DA9063L Power Management Integrated Circuit (PMIC)
> +
> +maintainers:
> + - Steve Twiss <stwiss.opensource@xxxxxxxxxxx>
> +
> +description: |
> + For device-tree bindings of other sub-modules refer to the binding documents
> + under the respective sub-system directories.
> +
> +properties:
> + compatible:
> + enum:
> + - dlg,da9063
> + - dlg,da9063l
> +
> + reg:
> + description:
> + I2C device address.
Skip description.
> + maxItems: 1
> +
> + interrupts:
> + maxItems: 1
> +
> + interrupt-controller: true
> +
> + "#interrupt-cells":
> + const: 2
> +
> + watchdog:
> + type: object
> + $ref: ../watchdog/watchdog.yaml#
1. full reference, so /schemas/watchdog/....
2. unevaluatedProperties:false
> + properties:
> + compatible:
> + const: dlg,da9063-watchdog
> +
> + rtc:
> + type: object
> + $ref: ../rtc/rtc.yaml#
Ditto
> + properties:
> + compatible:
> + const: dlg,da9063-rtc
> +
> + onkey:
> + type: object
> + $ref: ../input/input.yaml#
Ditto
> + properties:
> + compatible:
> + const: dlg,da9063-onkey
> +
> + dlg,disable-key-power:
> + type: boolean
> + description: |
> + Disable power-down using a long key-press.
> + If this entry does not exist then by default the key-press triggered
> + power down is enabled and the OnKey will support both KEY_POWER and
> + KEY_SLEEP.
> +
> + regulators:
> + type: object
> + patternProperties:
> + "^(ldo[1-11]|bcore[1-2]|bpro|bmem|bio|bperi)$":
> + $ref: ../regulator/regulator.yaml
Ditto
> +
> +required:
> + - compatible
> + - reg
> + - interrupts
> + - interrupt-controller
> +
Best regards,
Krzysztof
