Message ID | 1541603580-17448-2-git-send-email-andrei.stefanescu@microchip.com |
---|---|
State | New |
Headers | show |
Series | add sama5d2 PIOBU GPIO driver | expand |
On Wed, Nov 7, 2018 at 4:12 PM <Andrei.Stefanescu@microchip.com> wrote: > +Required properties: > +- compatible: "syscon", "microchip,sama5d2-piobu" Looks like "syscon" should be dropped, it is using a syscon, but it is no syscon. > + secumod@fc040000 { > + compatible = "syscon", "microchip,sama5d2-piobu"; > + reg = <0xfc040000 0x100>; > + > + gpio-controller; > + #gpio-cells = <2>; > + }; Isn't it easiest to just make this driver be a subnode of the syscon so it can just refer to its parent to find the syscon it needs? Just state that in the bindings and make an example where it is inside the syscon. Yours, Linus Walleij
diff --git a/Documentation/devicetree/bindings/gpio/gpio-sama5d2-piobu.txt b/Documentation/devicetree/bindings/gpio/gpio-sama5d2-piobu.txt new file mode 100644 index 0000000..791ac51 --- /dev/null +++ b/Documentation/devicetree/bindings/gpio/gpio-sama5d2-piobu.txt @@ -0,0 +1,23 @@ +GPIO controller for SAMA5D2 PIOBU pins. + +This pins have the property of not losing their voltage +during Backup/Self-refresh mode. + +These bindings should be set to a node in the dtsi file. + +Required properties: +- compatible: "syscon", "microchip,sama5d2-piobu" +- #gpio-cells: There are 2. The pin number is the + first, the second represents additional + parameters such as GPIO_ACTIVE_HIGH/LOW. +- gpio-controller: Marks the port as GPIO controller. + +Example: + + secumod@fc040000 { + compatible = "syscon", "microchip,sama5d2-piobu"; + reg = <0xfc040000 0x100>; + + gpio-controller; + #gpio-cells = <2>; + };
This patch describes the compatible and the device tree bindings necessary for the sama5d2 PIOBU GPIO controller driver. Signed-off-by: Andrei Stefanescu <andrei.stefanescu@microchip.com> --- .../bindings/gpio/gpio-sama5d2-piobu.txt | 23 ++++++++++++++++++++++ 1 file changed, 23 insertions(+) create mode 100644 Documentation/devicetree/bindings/gpio/gpio-sama5d2-piobu.txt