Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1323587 > unrolled thread

[PATCH v6 1/2] dt/bindings: Add bindings for PIC32 pin control and GPIO

Started byJoshua Henderson <joshua.henderson@microchip.com>
First post2016-02-01 23:50 +0100
Last post2016-02-08 17:50 +0100
Articles 4 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH v6 1/2] dt/bindings: Add bindings for PIC32 pin control and GPIO Joshua Henderson <joshua.henderson@microchip.com> - 2016-02-01 23:50 +0100
    Re: [PATCH v6 1/2] dt/bindings: Add bindings for PIC32 pin control  and GPIO Linus Walleij <linus.walleij@linaro.org> - 2016-02-06 00:00 +0100
    Re: [PATCH v6 2/2] pinctrl: pinctrl-pic32: Add PIC32 pin control driver Linus Walleij <linus.walleij@linaro.org> - 2016-02-06 00:00 +0100
      Re: [PATCH v6 2/2] pinctrl: pinctrl-pic32: Add PIC32 pin control  driver Joshua Henderson <joshua.henderson@microchip.com> - 2016-02-08 17:50 +0100

#1323587 — [PATCH v6 1/2] dt/bindings: Add bindings for PIC32 pin control and GPIO

FromJoshua Henderson <joshua.henderson@microchip.com>
Date2016-02-01 23:50 +0100
Subject[PATCH v6 1/2] dt/bindings: Add bindings for PIC32 pin control and GPIO
Message-ID<qXvsd-1cZ-3@gated-at.bofh.it>
Document the devicetree bindings for PINCTRL and GPIO found on Microchip
PIC32 class devices.

Signed-off-by: Joshua Henderson <joshua.henderson@microchip.com>
Cc: Ralf Baechle <ralf@linux-mips.org>
Acked-by: Rob Herring <robh@kernel.org>
Cc: Linus Walleij <linus.walleij@linaro.org>
---
Changes since v5: None
Changes since v4: None
Changes since v3: None
Changes since v2: None
Changes since v1:
	- Complete bindings API change to use standard bindings
---
 .../bindings/gpio/microchip,pic32-gpio.txt         |   49 ++++++++++++++++
 .../bindings/pinctrl/microchip,pic32-pinctrl.txt   |   60 ++++++++++++++++++++
 2 files changed, 109 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/gpio/microchip,pic32-gpio.txt
 create mode 100644 Documentation/devicetree/bindings/pinctrl/microchip,pic32-pinctrl.txt

diff --git a/Documentation/devicetree/bindings/gpio/microchip,pic32-gpio.txt b/Documentation/devicetree/bindings/gpio/microchip,pic32-gpio.txt
new file mode 100644
index 0000000..ef37528
--- /dev/null
+++ b/Documentation/devicetree/bindings/gpio/microchip,pic32-gpio.txt
@@ -0,0 +1,49 @@
+* Microchip PIC32 GPIO devices (PIO).
+
+Required properties:
+ - compatible: "microchip,pic32mzda-gpio"
+ - reg: Base address and length for the device.
+ - interrupts: The port interrupt shared by all pins.
+ - gpio-controller: Marks the port as GPIO controller.
+ - #gpio-cells: Two. The first cell is the pin number and
+   the second cell is used to specify the gpio polarity as defined in
+   defined in <dt-bindings/gpio/gpio.h>:
+      0 = GPIO_ACTIVE_HIGH
+      1 = GPIO_ACTIVE_LOW
+      2 = GPIO_OPEN_DRAIN
+ - interrupt-controller: Marks the device node as an interrupt controller.
+ - #interrupt-cells: Two. The first cell is the GPIO number and second cell
+   is used to specify the trigger type as defined in
+   <dt-bindings/interrupt-controller/irq.h>:
+      IRQ_TYPE_EDGE_RISING
+      IRQ_TYPE_EDGE_FALLING
+      IRQ_TYPE_EDGE_BOTH
+ - clocks: Clock specifier (see clock bindings for details).
+ - microchip,gpio-bank: Specifies which bank a controller owns.
+ - gpio-ranges: Interaction with the PINCTRL subsystem.
+
+Example:
+
+/* PORTA */
+gpio0: gpio0@1f860000 {
+	compatible = "microchip,pic32mzda-gpio";
+	reg = <0x1f860000 0x100>;
+	interrupts = <118 IRQ_TYPE_LEVEL_HIGH>;
+	#gpio-cells = <2>;
+	gpio-controller;
+	interrupt-controller;
+	#interrupt-cells = <2>;
+	clocks = <&PBCLK4>;
+	microchip,gpio-bank = <0>;
+	gpio-ranges = <&pic32_pinctrl 0 0 16>;
+};
+
+keys {
+	...
+
+	button@sw1 {
+		label = "ESC";
+		linux,code = <1>;
+		gpios = <&gpio0 12 0>;
+	};
+};
diff --git a/Documentation/devicetree/bindings/pinctrl/microchip,pic32-pinctrl.txt b/Documentation/devicetree/bindings/pinctrl/microchip,pic32-pinctrl.txt
new file mode 100644
index 0000000..4b5efa5
--- /dev/null
+++ b/Documentation/devicetree/bindings/pinctrl/microchip,pic32-pinctrl.txt
@@ -0,0 +1,60 @@
+* Microchip PIC32 Pin Controller
+
+Please refer to pinctrl-bindings.txt, ../gpio/gpio.txt, and
+../interrupt-controller/interrupts.txt for generic information regarding
+pin controller, GPIO, and interrupt bindings.
+
+PIC32 'pin configuration node' is a node of a group of pins which can be
+used for a specific device or function. This node represents configuraions of
+pins, optional function, and optional mux related configuration.
+
+Required properties for pin controller node:
+ - compatible: "microchip,pic32mada-pinctrl"
+ - reg: Address range of the pinctrl registers.
+ - clocks: Clock specifier (see clock bindings for details)
+
+Required properties for pin configuration sub-nodes:
+ - pins: List of pins to which the configuration applies.
+
+Optional properties for pin configuration sub-nodes:
+----------------------------------------------------
+ - function: Mux function for the specified pins.
+ - bias-pull-up: Enable weak pull-up.
+ - bias-pull-down: Enable weak pull-down.
+ - input-enable: Set the pin as an input.
+ - output-low: Set the pin as an output level low.
+ - output-high: Set the pin as an output level high.
+ - microchip,digital: Enable digital I/O.
+ - microchip,analog: Enable analog I/O.
+
+Example:
+
+pic32_pinctrl: pinctrl@1f801400{
+	#address-cells = <1>;
+	#size-cells = <1>;
+	compatible = "microchip,pic32mzda-pinctrl";
+	reg = <0x1f801400 0x400>;
+	clocks = <&PBCLK1>;
+
+	pinctrl_uart2: pinctrl_uart2 {
+		uart2-tx {
+			pins = "G9";
+			function = "U2TX";
+			microchip,digital;
+			output-low;
+		};
+		uart2-rx {
+			pins = "B0";
+			function = "U2RX";
+			microchip,digital;
+			input-enable;
+		};
+	};
+};
+
+uart2: serial@1f822200 {
+	compatible = "microchip,pic32mzda-uart";
+	reg = <0x1f822200 0x50>;
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_uart2>;
+};
-- 
1.7.9.5

[toc] | [next] | [standalone]


#1328121 — Re: [PATCH v6 1/2] dt/bindings: Add bindings for PIC32 pin control and GPIO

FromLinus Walleij <linus.walleij@linaro.org>
Date2016-02-06 00:00 +0100
SubjectRe: [PATCH v6 1/2] dt/bindings: Add bindings for PIC32 pin control and GPIO
Message-ID<qYXw6-6sJ-27@gated-at.bofh.it>
In reply to#1323587
On Mon, Feb 1, 2016 at 11:48 PM, Joshua Henderson
<joshua.henderson@microchip.com> wrote:

> Document the devicetree bindings for PINCTRL and GPIO found on Microchip
> PIC32 class devices.
>
> Signed-off-by: Joshua Henderson <joshua.henderson@microchip.com>
> Cc: Ralf Baechle <ralf@linux-mips.org>
> Acked-by: Rob Herring <robh@kernel.org>
> Cc: Linus Walleij <linus.walleij@linaro.org>
> ---
> Changes since v5: None

Patch applied.

Yours,
Linus Walleij

[toc] | [prev] | [next] | [standalone]


#1328122 — Re: [PATCH v6 2/2] pinctrl: pinctrl-pic32: Add PIC32 pin control driver

FromLinus Walleij <linus.walleij@linaro.org>
Date2016-02-06 00:00 +0100
SubjectRe: [PATCH v6 2/2] pinctrl: pinctrl-pic32: Add PIC32 pin control driver
Message-ID<qYXw6-6sJ-29@gated-at.bofh.it>
In reply to#1323587
On Mon, Feb 1, 2016 at 11:48 PM, Joshua Henderson
<joshua.henderson@microchip.com> wrote:

> Add a driver for the pin controller present on the Microchip PIC32
> including the specific variant PIC32MZDA. This driver provides pinmux
> and pinconfig operations as well as GPIO and IRQ chips for the GPIO
> banks.
>
> Signed-off-by: Joshua Henderson <joshua.henderson@microchip.com>
> Cc: Ralf Baechle <ralf@linux-mips.org>
> Cc: Linus Walleij <linus.walleij@linaro.org>
> ---
> Changes since v5:

Patch applied.

Yours,
Linus Walleij

[toc] | [prev] | [next] | [standalone]


#1329297 — Re: [PATCH v6 2/2] pinctrl: pinctrl-pic32: Add PIC32 pin control driver

FromJoshua Henderson <joshua.henderson@microchip.com>
Date2016-02-08 17:50 +0100
SubjectRe: [PATCH v6 2/2] pinctrl: pinctrl-pic32: Add PIC32 pin control driver
Message-ID<qZXaF-7Ql-1@gated-at.bofh.it>
In reply to#1328122
On 02/05/2016 03:57 PM, Linus Walleij wrote:
> On Mon, Feb 1, 2016 at 11:48 PM, Joshua Henderson
> <joshua.henderson@microchip.com> wrote:
> 
>> Add a driver for the pin controller present on the Microchip PIC32
>> including the specific variant PIC32MZDA. This driver provides pinmux
>> and pinconfig operations as well as GPIO and IRQ chips for the GPIO
>> banks.
>>
>> Signed-off-by: Joshua Henderson <joshua.henderson@microchip.com>
>> Cc: Ralf Baechle <ralf@linux-mips.org>
>> Cc: Linus Walleij <linus.walleij@linaro.org>
>> ---
>> Changes since v5:
> 
> Patch applied.

Thanks Linus.

Josh

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web