Re: [PATCH 2/3] dt-bindings: pinctrl: qcom-pmic-gpio: Convert pmic gpio bindings to YAML

2021-03-05 Thread Rob Herring
On Fri, 05 Mar 2021 11:59:58 +0530, satya priya wrote:
> Convert Qualcomm PMIC GPIO bindings from .txt to .yaml format.
> 
> Signed-off-by: satya priya 
> ---
>  .../devicetree/bindings/pinctrl/qcom,pmic-gpio.txt | 272 
>  .../bindings/pinctrl/qcom,pmic-gpio.yaml   | 273 
> +
>  2 files changed, 273 insertions(+), 272 deletions(-)
>  delete mode 100644 
> Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.txt
>  create mode 100644 
> Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml
> 

My bot found errors running 'make dt_binding_check' on your patch:

yamllint warnings/errors:
./Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml:10:2: [warning] 
wrong indentation: expected 2 but found 1 (indentation)
./Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml:131:17: 
[warning] wrong indentation: expected 14 but found 16 (indentation)
./Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml:136:17: 
[warning] wrong indentation: expected 14 but found 16 (indentation)
./Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml:141:17: 
[warning] wrong indentation: expected 14 but found 16 (indentation)
./Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml:155:17: 
[warning] wrong indentation: expected 14 but found 16 (indentation)
./Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml:190:17: 
[warning] wrong indentation: expected 14 but found 16 (indentation)
./Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml:195:17: 
[warning] wrong indentation: expected 14 but found 16 (indentation)
./Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml:198:12: 
[warning] wrong indentation: expected 12 but found 11 (indentation)
./Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml:200:16: 
[warning] wrong indentation: expected 13 but found 15 (indentation)

dtschema/dtc warnings/errors:

See https://patchwork.ozlabs.org/patch/1447657

This check can fail if there are any dependencies. The base for a patch
series is generally the most recent rc1.

If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:

pip3 install dtschema --upgrade

Please check and re-submit.



[PATCH 2/3] dt-bindings: pinctrl: qcom-pmic-gpio: Convert pmic gpio bindings to YAML

2021-03-04 Thread satya priya
Convert Qualcomm PMIC GPIO bindings from .txt to .yaml format.

Signed-off-by: satya priya 
---
 .../devicetree/bindings/pinctrl/qcom,pmic-gpio.txt | 272 
 .../bindings/pinctrl/qcom,pmic-gpio.yaml   | 273 +
 2 files changed, 273 insertions(+), 272 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.txt
 create mode 100644 
Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml

diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.txt 
b/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.txt
deleted file mode 100644
index 7648ab0..000
--- a/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.txt
+++ /dev/null
@@ -1,272 +0,0 @@
-Qualcomm PMIC GPIO block
-
-This binding describes the GPIO block(s) found in the 8xxx series of
-PMIC's from Qualcomm.
-
-- compatible:
-   Usage: required
-   Value type: 
-   Definition: must be one of:
-   "qcom,pm8005-gpio"
-   "qcom,pm8018-gpio"
-   "qcom,pm8038-gpio"
-   "qcom,pm8058-gpio"
-   "qcom,pm8916-gpio"
-   "qcom,pm8917-gpio"
-   "qcom,pm8921-gpio"
-   "qcom,pm8941-gpio"
-   "qcom,pm8950-gpio"
-   "qcom,pm8994-gpio"
-   "qcom,pm8998-gpio"
-   "qcom,pma8084-gpio"
-   "qcom,pmi8950-gpio"
-   "qcom,pmi8994-gpio"
-   "qcom,pmi8998-gpio"
-   "qcom,pms405-gpio"
-   "qcom,pm660-gpio"
-   "qcom,pm660l-gpio"
-   "qcom,pm8150-gpio"
-   "qcom,pm8150b-gpio"
-   "qcom,pm6150-gpio"
-   "qcom,pm6150l-gpio"
-   "qcom,pmx55-gpio"
-
-   And must contain either "qcom,spmi-gpio" or "qcom,ssbi-gpio"
-   if the device is on an spmi bus or an ssbi bus respectively
-
-- reg:
-   Usage: required
-   Value type: 
-   Definition: Register base of the GPIO block and length.
-
-- interrupts:
-   Usage: required
-   Value type: 
-   Definition: Must contain an array of encoded interrupt specifiers for
-   each available GPIO
-
-- gpio-controller:
-   Usage: required
-   Value type: 
-   Definition: Mark the device node as a GPIO controller
-
-- #gpio-cells:
-   Usage: required
-   Value type: 
-   Definition: Must be 2;
-   the first cell will be used to define gpio number and the
-   second denotes the flags for this gpio
-
-Please refer to ../gpio/gpio.txt and ../interrupt-controller/interrupts.txt for
-a general description of GPIO and interrupt bindings.
-
-Please refer to pinctrl-bindings.txt in this directory for details of the
-common pinctrl bindings used by client devices, including the meaning of the
-phrase "pin configuration node".
-
-The pin configuration nodes act as a container for an arbitrary number of
-subnodes. Each of these subnodes represents some desired configuration for a
-pin or a list of pins. This configuration can include the
-mux function to select on those pin(s), and various pin configuration
-parameters, as listed below.
-
-
-SUBNODES:
-
-The name of each subnode is not important; all subnodes should be enumerated
-and processed purely based on their content.
-
-Each subnode only affects those parameters that are explicitly listed. In
-other words, a subnode that lists a mux function but no pin configuration
-parameters implies no information about any pin configuration parameters.
-Similarly, a pin subnode that describes a pullup parameter implies no
-information about e.g. the mux function.
-
-The following generic properties as defined in pinctrl-bindings.txt are valid
-to specify in a pin configuration subnode:
-
-- pins:
-   Usage: required
-   Value type: 
-   Definition: List of gpio pins affected by the properties specified in
-   this subnode.  Valid pins are:
-   gpio1-gpio4 for pm8005
-   gpio1-gpio6 for pm8018
-   gpio1-gpio12 for pm8038
-   gpio1-gpio40 for pm8058
-   gpio1-gpio4 for pm8916
-   gpio1-gpio38 for pm8917
-   gpio1-gpio44 for pm8921
-   gpio1-gpio36 for pm8941
-   gpio1-gpio8 for pm8950 (hole on gpio3)
-   gpio1-gpio22 for pm8994
-   gpio1-gpio26 for pm8998
-   gpio1-gpio22 for pma8084
-   gpio1-gpio2 for pmi8950
-   gpio1-gpio10 for pmi8994
-   gpio1-gpio12 for pms405 (holes on gpio1, gpio9 and gpio10)
-   gpio1-gpio10 for pm8150 (holes on gpio2, gpio5, gpio7
-and gpio8)
-