Re: [PATCH 1/5 v2] pinctrl: Add DT bindings for Cortina Gemini

[Rob Herring]

On Mon, Jul 31, 2017 at 11:30:50PM +0200, Linus Walleij wrote:
> The Cortina Gemini pin controller uses the standard pin control
> bindings for muxing functions with groups so these bindings
> should be entirely uncontroversial.
> 
> Cc: devicet...@vger.kernel.org
> Signed-off-by: Linus Walleij 
> ---
> ChangeLog v1->v2:
> - State that the pin controller must be a subnode of the
>   system controller.
> - Drop the phandle to the system controller: we can simply
>   look up the parent.
> ---
>  .../bindings/pinctrl/cortina,gemini-pinctrl.txt| 59 
> ++
>  1 file changed, 59 insertions(+)
>  create mode 100644 
> Documentation/devicetree/bindings/pinctrl/cortina,gemini-pinctrl.txt

Acked-by: Rob Herring 

Re: [PATCH 1/5 v2] pinctrl: Add DT bindings for Cortina Gemini

[Rob Herring]

On Mon, Jul 31, 2017 at 11:30:50PM +0200, Linus Walleij wrote:
> The Cortina Gemini pin controller uses the standard pin control
> bindings for muxing functions with groups so these bindings
> should be entirely uncontroversial.
> 
> Cc: devicet...@vger.kernel.org
> Signed-off-by: Linus Walleij 
> ---
> ChangeLog v1->v2:
> - State that the pin controller must be a subnode of the
>   system controller.
> - Drop the phandle to the system controller: we can simply
>   look up the parent.
> ---
>  .../bindings/pinctrl/cortina,gemini-pinctrl.txt| 59 
> ++
>  1 file changed, 59 insertions(+)
>  create mode 100644 
> Documentation/devicetree/bindings/pinctrl/cortina,gemini-pinctrl.txt

Acked-by: Rob Herring 

[PATCH 1/5 v2] pinctrl: Add DT bindings for Cortina Gemini

[Linus Walleij]

The Cortina Gemini pin controller uses the standard pin control
bindings for muxing functions with groups so these bindings
should be entirely uncontroversial.

Cc: devicet...@vger.kernel.org
Signed-off-by: Linus Walleij 
---
ChangeLog v1->v2:
- State that the pin controller must be a subnode of the
  system controller.
- Drop the phandle to the system controller: we can simply
  look up the parent.
---
 .../bindings/pinctrl/cortina,gemini-pinctrl.txt| 59 ++
 1 file changed, 59 insertions(+)
 create mode 100644 
Documentation/devicetree/bindings/pinctrl/cortina,gemini-pinctrl.txt

diff --git 
a/Documentation/devicetree/bindings/pinctrl/cortina,gemini-pinctrl.txt 
b/Documentation/devicetree/bindings/pinctrl/cortina,gemini-pinctrl.txt
new file mode 100644
index ..61466c58faae
--- /dev/null
+++ b/Documentation/devicetree/bindings/pinctrl/cortina,gemini-pinctrl.txt
@@ -0,0 +1,59 @@
+Cortina Systems Gemini pin controller
+
+This pin controller is found in the Cortina Systems Gemini SoC family,
+see further arm/gemini.txt. It is a purely group-based multiplexing pin
+controller.
+
+The pin controller node must be a subnode of the system controller node.
+
+Required properties:
+- compatible: "cortina,gemini-pinctrl"
+
+Subnodes of the pin controller contain pin control multiplexing set-up.
+Please refer to pinctrl-bindings.txt for generic pin multiplexing nodes.
+
+Example:
+
+
+syscon {
+   compatible = "cortina,gemini-syscon";
+   ...
+   pinctrl {
+   compatible = "cortina,gemini-pinctrl";
+   pinctrl-names = "default";
+   pinctrl-0 = <_default_pins>, <_default_pins>,
+   <_default_pins>;
+
+   dram_default_pins: pinctrl-dram {
+   mux {
+   function = "dram";
+   groups = "dramgrp";
+   };
+   };
+   rtc_default_pins: pinctrl-rtc {
+   mux {
+   function = "rtc";
+   groups = "rtcgrp";
+   };
+   };
+   power_default_pins: pinctrl-power {
+   mux {
+   function = "power";
+   groups = "powergrp";
+   };
+   };
+   system_default_pins: pinctrl-system {
+   mux {
+   function = "system";
+   groups = "systemgrp";
+   };
+   };
+   (...)
+   uart_default_pins: pinctrl-uart {
+   mux {
+   function = "uart";
+   groups = "uartrxtxgrp";
+   };
+   };
+   };
+};
-- 
2.9.4

[PATCH 1/5 v2] pinctrl: Add DT bindings for Cortina Gemini

[Linus Walleij]

The Cortina Gemini pin controller uses the standard pin control
bindings for muxing functions with groups so these bindings
should be entirely uncontroversial.

Cc: devicet...@vger.kernel.org
Signed-off-by: Linus Walleij 
---
ChangeLog v1->v2:
- State that the pin controller must be a subnode of the
  system controller.
- Drop the phandle to the system controller: we can simply
  look up the parent.
---
 .../bindings/pinctrl/cortina,gemini-pinctrl.txt| 59 ++
 1 file changed, 59 insertions(+)
 create mode 100644 
Documentation/devicetree/bindings/pinctrl/cortina,gemini-pinctrl.txt

diff --git 
a/Documentation/devicetree/bindings/pinctrl/cortina,gemini-pinctrl.txt 
b/Documentation/devicetree/bindings/pinctrl/cortina,gemini-pinctrl.txt
new file mode 100644
index ..61466c58faae
--- /dev/null
+++ b/Documentation/devicetree/bindings/pinctrl/cortina,gemini-pinctrl.txt
@@ -0,0 +1,59 @@
+Cortina Systems Gemini pin controller
+
+This pin controller is found in the Cortina Systems Gemini SoC family,
+see further arm/gemini.txt. It is a purely group-based multiplexing pin
+controller.
+
+The pin controller node must be a subnode of the system controller node.
+
+Required properties:
+- compatible: "cortina,gemini-pinctrl"
+
+Subnodes of the pin controller contain pin control multiplexing set-up.
+Please refer to pinctrl-bindings.txt for generic pin multiplexing nodes.
+
+Example:
+
+
+syscon {
+   compatible = "cortina,gemini-syscon";
+   ...
+   pinctrl {
+   compatible = "cortina,gemini-pinctrl";
+   pinctrl-names = "default";
+   pinctrl-0 = <_default_pins>, <_default_pins>,
+   <_default_pins>;
+
+   dram_default_pins: pinctrl-dram {
+   mux {
+   function = "dram";
+   groups = "dramgrp";
+   };
+   };
+   rtc_default_pins: pinctrl-rtc {
+   mux {
+   function = "rtc";
+   groups = "rtcgrp";
+   };
+   };
+   power_default_pins: pinctrl-power {
+   mux {
+   function = "power";
+   groups = "powergrp";
+   };
+   };
+   system_default_pins: pinctrl-system {
+   mux {
+   function = "system";
+   groups = "systemgrp";
+   };
+   };
+   (...)
+   uart_default_pins: pinctrl-uart {
+   mux {
+   function = "uart";
+   groups = "uartrxtxgrp";
+   };
+   };
+   };
+};
-- 
2.9.4