diff mbox series

[v3,bus+gpio,5/5] dt-bindings: gpio: Document GPIOs via Moxtet bus

Message ID 20190301035852.26780-6-marek.behun@nic.cz
State New
Headers show
Series Add Moxtet bus and GPIO over Moxtet bus | expand

Commit Message

Marek Behún March 1, 2019, 3:58 a.m. UTC 
This patch adds documentation of the device tree bindings for GPIOs
on the devices connected via Moxtet bus.

Signed-off-by: Marek Behún <marek.behun@nic.cz>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: devicetree@vger.kernel.org
---
 .../devicetree/bindings/gpio/gpio-moxtet.txt   | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/gpio/gpio-moxtet.txt

Comments

Linus Walleij March 1, 2019, 2:36 p.m. UTC | #1 
On Fri, Mar 1, 2019 at 4:59 AM Marek Behún <marek.behun@nic.cz> wrote:

> This patch adds documentation of the device tree bindings for GPIOs
> on the devices connected via Moxtet bus.
>
> Signed-off-by: Marek Behún <marek.behun@nic.cz>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: devicetree@vger.kernel.org

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

> +               compatible = "cznic,moxtet-gpio";

Is cznic registered in
Documentation/devicetree/bindings/vendor-prefixes.txt?

Yours,
Linus Walleij
Marek Behún March 1, 2019, 3:16 p.m. UTC | #2 
On Fri, 1 Mar 2019 15:36:09 +0100
Linus Walleij <linus.walleij@linaro.org> wrote:

> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
> 
> > +               compatible = "cznic,moxtet-gpio";  
> 
> Is cznic registered in
> Documentation/devicetree/bindings/vendor-prefixes.txt?

Yes, it is registered.
Thank you for the review.
Marek
Marek Behún March 3, 2019, 11:17 p.m. UTC | #3 
Hi Linus,

do you know who is responsible for merging into drivers/bus?

I have received reviews for this and for previous versions only from
you and Rob.

Also, please do not merge yet. I have prepared a fourth version with
debugging over debugfs, fixed some checkpatch issues, and most
importantnly added interrupt reading support from moxtet.

I will send after proper testing is done for this new version.

Marek

On Fri, 1 Mar 2019 15:36:09 +0100
Linus Walleij <linus.walleij@linaro.org> wrote:

> On Fri, Mar 1, 2019 at 4:59 AM Marek Behún <marek.behun@nic.cz> wrote:
> 
> > This patch adds documentation of the device tree bindings for GPIOs
> > on the devices connected via Moxtet bus.
> >
> > Signed-off-by: Marek Behún <marek.behun@nic.cz>
> > Cc: Rob Herring <robh+dt@kernel.org>
> > Cc: devicetree@vger.kernel.org  
> 
> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
> 
> > +               compatible = "cznic,moxtet-gpio";  
> 
> Is cznic registered in
> Documentation/devicetree/bindings/vendor-prefixes.txt?
> 
> Yours,
> Linus Walleij
Linus Walleij March 4, 2019, 9:53 a.m. UTC | #4 
On Mon, Mar 4, 2019 at 12:17 AM Marek Behun <marek.behun@nic.cz> wrote:

> do you know who is responsible for merging into drivers/bus?

Usually the ARM SoC maintainers arm@kernel.org merge drivers
there. You should just send them a pull request and explain the
situation I think. Else they know who should deal with it.

Yours,
Linus Walleij
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/gpio/gpio-moxtet.txt b/Documentation/devicetree/bindings/gpio/gpio-moxtet.txt
new file mode 100644
index 000000000000..410759de9f09
--- /dev/null
+++ b/Documentation/devicetree/bindings/gpio/gpio-moxtet.txt
@@ -0,0 +1,18 @@ 
+Turris Mox Moxtet GPIO expander via Moxtet bus
+
+Required properties:
+ - compatible		: Should be "cznic,moxtet-gpio".
+ - gpio-controller	: Marks the device node as a GPIO controller.
+ - #gpio-cells		: Should be two. For consumer use see gpio.txt.
+
+Other properties are required for a Moxtet bus device, please refer to
+Documentation/devicetree/bindings/bus/moxtet.txt.
+
+Example:
+
+	moxtet_sfp: gpio@0 {
+		compatible = "cznic,moxtet-gpio";
+		gpio-controller;
+		#gpio-cells = <2>;
+		reg = <0>;
+	}