[PATCH 5/5] dt-bindings: pinctrl: dw: move sps,dwapb-gpio.txt to pinctrl

* [PATCH 5/5] dt-bindings: pinctrl: dw: move sps,dwapb-gpio.txt to pinctrl
@ 2019-12-04 10:10 Clement Leger
  2019-12-04 12:45 ` Andy Shevchenko
  0 siblings, 1 reply; 5+ messages in thread
From: Clement Leger @ 2019-12-04 10:10 UTC (permalink / raw)
  To: linux-kernel
  Cc: Clement Leger, Linus Walleij, Bartosz Golaszewski, Rob Herring,
	Mark Rutland, Hoan Tran, linux-gpio, devicetree

Since the driver has been moved to pinctrl and now supports it, move the
documentation into pinctrl folder. In the same time, add documentation
for pinctrl properties such has snps,has-pinctrl and description of pin
alternate functions.

Signed-off-by: Clement Leger <cleger@kalray.eu>
---
 .../bindings/{gpio => pinctrl}/snps-dwapb-gpio.txt  | 21 ++++++++++++++++++++-
 MAINTAINERS                                         |  2 +-
 2 files changed, 21 insertions(+), 2 deletions(-)
 rename Documentation/devicetree/bindings/{gpio => pinctrl}/snps-dwapb-gpio.txt (76%)

diff --git a/Documentation/devicetree/bindings/gpio/snps-dwapb-gpio.txt b/Documentation/devicetree/bindings/pinctrl/snps-dwapb-gpio.txt
similarity index 76%
rename from Documentation/devicetree/bindings/gpio/snps-dwapb-gpio.txt
rename to Documentation/devicetree/bindings/pinctrl/snps-dwapb-gpio.txt
index 839dd32ffe11..4fb1b754a2bd 100644
--- a/Documentation/devicetree/bindings/gpio/snps-dwapb-gpio.txt
+++ b/Documentation/devicetree/bindings/pinctrl/snps-dwapb-gpio.txt
@@ -1,4 +1,4 @@
-* Synopsys DesignWare APB GPIO controller
+* Synopsys DesignWare APB GPIO & pinmux controller
 
 Required properties:
 - compatible : Should contain "snps,dw-apb-gpio"
@@ -33,8 +33,16 @@ controller.
   use the interrupts-extended property to specify the interrupts and set the
   interrupt controller handle for unused interrupts to 0.
 - snps,nr-gpios : The number of pins in the port, a single cell.
+- snps,has-pinctrl : If present, register the pinctrl controller.
 - resets : Reset line for the controller.
 
+Required properties for pin configuration node:
+- function: string representing a function to mux for pins
+    dw apb controller only has two functions for each pins (sw and hw). When set
+    in software mode, the gpio controller controls the pin output. When
+    configured in hardware mode, an external peripheral controls the pin signal.
+- pins: string array of pins to be muxed.
+
 Example:
 
 gpio: gpio@20000 {
@@ -60,6 +68,17 @@ gpio: gpio@20000 {
 		gpio-controller;
 		#gpio-cells = <2>;
 		snps,nr-gpios = <8>;
+		snps,has-pinctrl;
 		reg = <1>;
+
+		uart0_pins: pinmux_uart0_pins {
+			function = "hw";
+			pins = "pin0", "pin1";
+		};
+
+		uart1_pins: pinmux_uart1_pins {
+			function = "hw";
+			pins = "pin2", "pin3";
+		};
 	};
 };
diff --git a/MAINTAINERS b/MAINTAINERS
index 35b90ad9f594..e8f9e827d687 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -15873,7 +15873,7 @@ M:	Hoan Tran <hoan@os.amperecomputing.com>
 L:	linux-gpio@vger.kernel.org
 S:	Maintained
 F:	drivers/pinctrl/dw/pinctrl-dwapb.c
-F:	Documentation/devicetree/bindings/gpio/snps-dwapb-gpio.txt
+F:	Documentation/devicetree/bindings/pinctrl/snps-dwapb-gpio.txt
 
 SYNOPSYS DESIGNWARE AXI DMAC DRIVER
 M:	Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com>
-- 
2.15.0.276.g89ea799


^ permalink raw reply related	[flat|nested] 5+ messages in thread