diff mbox series

[v5] dt-bindings: gpio: Convert TI TPIC2810 GPIO Controller bindings to YAML

Message ID 20220223174215.17838-1-a-m1@ti.com
State Accepted
Commit 6b7e8c796de0a0f1089c818097b9cd65817a2834
Headers show
Series [v5] dt-bindings: gpio: Convert TI TPIC2810 GPIO Controller bindings to YAML | expand

Commit Message

M, Aparna Feb. 23, 2022, 5:42 p.m. UTC
Convert gpio-tpic2810 bindings to yaml format and remove outdated
bindings in .txt format.

Signed-off-by: Aparna M <a-m1@ti.com>
---

v4 -> v5: Fix indentation issues
v3 -> v4: Add gpio-line-names property
v2 -> v3: Remove redundant descriptions and make minor change in example
v1 -> v2: Fix identation issues and update commit message

 .../bindings/gpio/gpio-tpic2810.txt           | 16 ------
 .../bindings/gpio/gpio-tpic2810.yaml          | 51 +++++++++++++++++++
 2 files changed, 51 insertions(+), 16 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/gpio/gpio-tpic2810.txt
 create mode 100644 Documentation/devicetree/bindings/gpio/gpio-tpic2810.yaml

Comments

Rob Herring Feb. 24, 2022, 5:35 p.m. UTC | #1
On Wed, 23 Feb 2022 23:12:15 +0530, Aparna M wrote:
> Convert gpio-tpic2810 bindings to yaml format and remove outdated
> bindings in .txt format.
> 
> Signed-off-by: Aparna M <a-m1@ti.com>
> ---
> 
> v4 -> v5: Fix indentation issues
> v3 -> v4: Add gpio-line-names property
> v2 -> v3: Remove redundant descriptions and make minor change in example
> v1 -> v2: Fix identation issues and update commit message
> 
>  .../bindings/gpio/gpio-tpic2810.txt           | 16 ------
>  .../bindings/gpio/gpio-tpic2810.yaml          | 51 +++++++++++++++++++
>  2 files changed, 51 insertions(+), 16 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/gpio/gpio-tpic2810.txt
>  create mode 100644 Documentation/devicetree/bindings/gpio/gpio-tpic2810.yaml
> 

Reviewed-by: Rob Herring <robh@kernel.org>
Linus Walleij March 15, 2022, 1:01 a.m. UTC | #2
On Wed, Feb 23, 2022 at 6:42 PM Aparna M <a-m1@ti.com> wrote:

> Convert gpio-tpic2810 bindings to yaml format and remove outdated
> bindings in .txt format.
>
> Signed-off-by: Aparna M <a-m1@ti.com>

Reviewed-by: Linus Walleij <linus.walleij@linaro.org>

Yours,
Linus Walleij
M, Aparna May 5, 2022, 8:58 a.m. UTC | #3
+Vignesh

Ping. Can this be picked up for next merge cycle?

On 15/03/22 06:31, Linus Walleij wrote:
> On Wed, Feb 23, 2022 at 6:42 PM Aparna M <a-m1@ti.com> wrote:
>
>> Convert gpio-tpic2810 bindings to yaml format and remove outdated
>> bindings in .txt format.
>>
>> Signed-off-by: Aparna M <a-m1@ti.com>
> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
>
> Yours,
> Linus Walleij
>
Thanks,
Vignesh Raghavendra July 7, 2022, 5:19 a.m. UTC | #4
Hi Bartosz,

On 05/05/22 14:28, M, Aparna wrote:
> +Vignesh
> 
> Ping. Can this be picked up for next merge cycle?
> 
> On 15/03/22 06:31, Linus Walleij wrote:
>> On Wed, Feb 23, 2022 at 6:42 PM Aparna M <a-m1@ti.com> wrote:
>>
>>> Convert gpio-tpic2810 bindings to yaml format and remove outdated
>>> bindings in .txt format.
>>>
>>> Signed-off-by: Aparna M <a-m1@ti.com>
>> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
>>
>> Yours,
>> Linus Walleij
>>

Sorry, this patch has been around for sometime w/o any comments and has
relevant R-bys. Wondering if this fell off the radar

I believe patch would go via GPIO tree?
Linus Walleij July 7, 2022, 11:19 a.m. UTC | #5
On Thu, Jul 7, 2022 at 7:19 AM Vignesh Raghavendra <vigneshr@ti.com> wrote:

> Hi Bartosz,
>
> On 05/05/22 14:28, M, Aparna wrote:
> > +Vignesh
> >
> > Ping. Can this be picked up for next merge cycle?
> >
> > On 15/03/22 06:31, Linus Walleij wrote:
> >> On Wed, Feb 23, 2022 at 6:42 PM Aparna M <a-m1@ti.com> wrote:
> >>
> >>> Convert gpio-tpic2810 bindings to yaml format and remove outdated
> >>> bindings in .txt format.
> >>>
> >>> Signed-off-by: Aparna M <a-m1@ti.com>
> >> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
> >>
> >> Yours,
> >> Linus Walleij
> >>
>
> Sorry, this patch has been around for sometime w/o any comments and has
> relevant R-bys. Wondering if this fell off the radar
>
> I believe patch would go via GPIO tree?

You need to use the right mail address to Bartosz so he can pick it up.

Bartosz: this seems to have fallen between the mail address cracks,
check it out.

Yours,
Linus Walleij
Vignesh Raghavendra July 8, 2022, 5:24 a.m. UTC | #6
On 07/07/22 16:49, Linus Walleij wrote:
> On Thu, Jul 7, 2022 at 7:19 AM Vignesh Raghavendra <vigneshr@ti.com> wrote:
> 
>> Hi Bartosz,
>>
>> On 05/05/22 14:28, M, Aparna wrote:
>>> +Vignesh
>>>
>>> Ping. Can this be picked up for next merge cycle?
>>>
>>> On 15/03/22 06:31, Linus Walleij wrote:
>>>> On Wed, Feb 23, 2022 at 6:42 PM Aparna M <a-m1@ti.com> wrote:
>>>>
>>>>> Convert gpio-tpic2810 bindings to yaml format and remove outdated
>>>>> bindings in .txt format.
>>>>>
>>>>> Signed-off-by: Aparna M <a-m1@ti.com>
>>>> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
>>>>
>>>> Yours,
>>>> Linus Walleij
>>>>
>>
>> Sorry, this patch has been around for sometime w/o any comments and has
>> relevant R-bys. Wondering if this fell off the radar
>>
>> I believe patch would go via GPIO tree?
> 
> You need to use the right mail address to Bartosz so he can pick it up.

oops, thanks! looks like original author copied wrong ID

> 
> Bartosz: this seems to have fallen between the mail address cracks,
> check it out.
> 

Bartosz,

Let me if I need to resend to right inbox.

> Yours,
> Linus Walleij
Bartosz Golaszewski July 8, 2022, 3:50 p.m. UTC | #7
On Fri, Jul 8, 2022 at 7:24 AM Vignesh Raghavendra <vigneshr@ti.com> wrote:
>
>
>
> On 07/07/22 16:49, Linus Walleij wrote:
> > On Thu, Jul 7, 2022 at 7:19 AM Vignesh Raghavendra <vigneshr@ti.com> wrote:
> >
> >> Hi Bartosz,
> >>
> >> On 05/05/22 14:28, M, Aparna wrote:
> >>> +Vignesh
> >>>
> >>> Ping. Can this be picked up for next merge cycle?
> >>>
> >>> On 15/03/22 06:31, Linus Walleij wrote:
> >>>> On Wed, Feb 23, 2022 at 6:42 PM Aparna M <a-m1@ti.com> wrote:
> >>>>
> >>>>> Convert gpio-tpic2810 bindings to yaml format and remove outdated
> >>>>> bindings in .txt format.
> >>>>>
> >>>>> Signed-off-by: Aparna M <a-m1@ti.com>
> >>>> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
> >>>>
> >>>> Yours,
> >>>> Linus Walleij
> >>>>
> >>
> >> Sorry, this patch has been around for sometime w/o any comments and has
> >> relevant R-bys. Wondering if this fell off the radar
> >>
> >> I believe patch would go via GPIO tree?
> >
> > You need to use the right mail address to Bartosz so he can pick it up.
>
> oops, thanks! looks like original author copied wrong ID
>
> >
> > Bartosz: this seems to have fallen between the mail address cracks,
> > check it out.
> >
>
> Bartosz,
>
> Let me if I need to resend to right inbox.
>

Yes please resend it, I can't find it now.

Bart
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/gpio/gpio-tpic2810.txt b/Documentation/devicetree/bindings/gpio/gpio-tpic2810.txt
deleted file mode 100644
index 1afc2de7a537..000000000000
--- a/Documentation/devicetree/bindings/gpio/gpio-tpic2810.txt
+++ /dev/null
@@ -1,16 +0,0 @@ 
-TPIC2810 GPIO controller bindings
-
-Required properties:
- - compatible		: Should be "ti,tpic2810".
- - reg			: The I2C address of the device
- - gpio-controller	: Marks the device node as a GPIO controller.
- - #gpio-cells		: Should be two. For consumer use see gpio.txt.
-
-Example:
-
-	gpio@60 {
-		compatible = "ti,tpic2810";
-		reg = <0x60>;
-		gpio-controller;
-		#gpio-cells = <2>;
-	};
diff --git a/Documentation/devicetree/bindings/gpio/gpio-tpic2810.yaml b/Documentation/devicetree/bindings/gpio/gpio-tpic2810.yaml
new file mode 100644
index 000000000000..cb8a5c376e1e
--- /dev/null
+++ b/Documentation/devicetree/bindings/gpio/gpio-tpic2810.yaml
@@ -0,0 +1,51 @@ 
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/gpio/gpio-tpic2810.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: TPIC2810 GPIO controller bindings
+
+maintainers:
+  - Aswath Govindraju <a-govindraju@ti.com>
+
+properties:
+  compatible:
+    enum:
+      - ti,tpic2810
+
+  reg:
+    maxItems: 1
+
+  gpio-controller: true
+
+  "#gpio-cells":
+    const: 2
+
+  gpio-line-names:
+    minItems: 1
+    maxItems: 32
+
+required:
+  - compatible
+  - reg
+  - gpio-controller
+  - "#gpio-cells"
+
+additionalProperties: false
+
+examples:
+  - |
+    #include <dt-bindings/gpio/gpio.h>
+
+    i2c {
+        #address-cells = <1>;
+        #size-cells = <0>;
+        gpio@60 {
+            compatible = "ti,tpic2810";
+            reg = <0x60>;
+            gpio-controller;
+            #gpio-cells = <2>;
+            gpio-line-names = "LED A", "LED B", "LED C";
+        };
+    };