All of lore.kernel.org
 help / color / mirror / Atom feed
From: "A.s. Dong" <aisheng.dong@nxp.com>
To: "linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>
Cc: "A.s. Dong" <aisheng.dong@nxp.com>,
	Mark Rutland <mark.rutland@arm.com>,
	"dongas86@gmail.com" <dongas86@gmail.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	Linus Walleij <linus.walleij@linaro.org>,
	"linux@armlinux.org.uk" <linux@armlinux.org.uk>,
	Stefan Agner <stefan@agner.ch>,
	"linux-gpio@vger.kernel.org" <linux-gpio@vger.kernel.org>,
	"robh+dt@kernel.org" <robh+dt@kernel.org>,
	dl-linux-imx <linux-imx@nxp.com>,
	"kernel@pengutronix.de" <kernel@pengutronix.de>,
	Fabio Estevam <fabio.estevam@nxp.com>,
	"shawnguo@kernel.org" <shawnguo@kernel.org>
Subject: [PATCH V4 03/10] dt-bindings: gpio: vf610: add optional clocks property
Date: Fri, 2 Nov 2018 09:12:48 +0000	[thread overview]
Message-ID: <1541149669-10857-4-git-send-email-aisheng.dong@nxp.com> (raw)
In-Reply-To: <1541149669-10857-1-git-send-email-aisheng.dong@nxp.com>

On some SoCs(e.g. MX7ULP), GPIO clock is gatable and maybe
disabled by default. Users have to make sure it's enabled before
being able to access controller registers, otherwise an external
abort error may occur. Let's add the optional clocks property to
handle this case.

For ULP GPIO clock, it includes two separate clocks: one is for
GPIO controller Input/Output function clock while another is
GPIO port control clock for interrupt function.

Cc: Linus Walleij <linus.walleij@linaro.org>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Stefan Agner <stefan@agner.ch>
Cc: linux-gpio@vger.kernel.org
Cc: devicetree@vger.kernel.org
Reviewed-by: Rob Herring <robh@kernel.org>
Signed-off-by: Dong Aisheng <aisheng.dong@nxp.com>
---
v2->v3:
 * no changes
v1->v2:
 * new patch
---
 Documentation/devicetree/bindings/gpio/gpio-vf610.txt | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/Documentation/devicetree/bindings/gpio/gpio-vf610.txt b/Documentation/devicetree/bindings/gpio/gpio-vf610.txt
index 0ccbae4..ae254aa 100644
--- a/Documentation/devicetree/bindings/gpio/gpio-vf610.txt
+++ b/Documentation/devicetree/bindings/gpio/gpio-vf610.txt
@@ -24,6 +24,12 @@ Required properties for GPIO node:
       4 = active high level-sensitive.
       8 = active low level-sensitive.
 
+Optional properties:
+-clocks:	Must contain an entry for each entry in clock-names.
+		See common clock-bindings.txt for details.
+-clock-names:	A list of clock names. For imx7ulp, it must contain
+		"gpio", "port".
+
 Note: Each GPIO port should have an alias correctly numbered in "aliases"
 node.
 
-- 
2.7.4

WARNING: multiple messages have this Message-ID (diff)
From: aisheng.dong@nxp.com (A.s. Dong)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH V4 03/10] dt-bindings: gpio: vf610: add optional clocks property
Date: Fri, 2 Nov 2018 09:12:48 +0000	[thread overview]
Message-ID: <1541149669-10857-4-git-send-email-aisheng.dong@nxp.com> (raw)
In-Reply-To: <1541149669-10857-1-git-send-email-aisheng.dong@nxp.com>

On some SoCs(e.g. MX7ULP), GPIO clock is gatable and maybe
disabled by default. Users have to make sure it's enabled before
being able to access controller registers, otherwise an external
abort error may occur. Let's add the optional clocks property to
handle this case.

For ULP GPIO clock, it includes two separate clocks: one is for
GPIO controller Input/Output function clock while another is
GPIO port control clock for interrupt function.

Cc: Linus Walleij <linus.walleij@linaro.org>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Stefan Agner <stefan@agner.ch>
Cc: linux-gpio at vger.kernel.org
Cc: devicetree at vger.kernel.org
Reviewed-by: Rob Herring <robh@kernel.org>
Signed-off-by: Dong Aisheng <aisheng.dong@nxp.com>
---
v2->v3:
 * no changes
v1->v2:
 * new patch
---
 Documentation/devicetree/bindings/gpio/gpio-vf610.txt | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/Documentation/devicetree/bindings/gpio/gpio-vf610.txt b/Documentation/devicetree/bindings/gpio/gpio-vf610.txt
index 0ccbae4..ae254aa 100644
--- a/Documentation/devicetree/bindings/gpio/gpio-vf610.txt
+++ b/Documentation/devicetree/bindings/gpio/gpio-vf610.txt
@@ -24,6 +24,12 @@ Required properties for GPIO node:
       4 = active high level-sensitive.
       8 = active low level-sensitive.
 
+Optional properties:
+-clocks:	Must contain an entry for each entry in clock-names.
+		See common clock-bindings.txt for details.
+-clock-names:	A list of clock names. For imx7ulp, it must contain
+		"gpio", "port".
+
 Note: Each GPIO port should have an alias correctly numbered in "aliases"
 node.
 
-- 
2.7.4

  parent reply	other threads:[~2018-11-02  9:12 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-11-02  9:12 [PATCH V4 00/10] ARM: imx: add imx7ulp support A.s. Dong
2018-11-02  9:12 ` [PATCH V4 01/10] dt-bindings: fsl: add compatible for imx7ulp evk A.s. Dong
2018-11-02  9:12   ` A.s. Dong
2018-11-02  9:12 ` [PATCH V4 02/10] dt-bindings: fsl: add imx7ulp pm related components bindings A.s. Dong
2018-11-02  9:12   ` A.s. Dong
2018-11-02  9:12 ` A.s. Dong [this message]
2018-11-02  9:12   ` [PATCH V4 03/10] dt-bindings: gpio: vf610: add optional clocks property A.s. Dong
2018-11-02  9:34   ` Linus Walleij
2018-11-02  9:34     ` Linus Walleij
2018-11-02  9:12 ` [PATCH V4 04/10] gpio: vf610: add optional clock support A.s. Dong
2018-11-02  9:12   ` A.s. Dong
2018-11-02  9:36   ` Linus Walleij
2018-11-02  9:36     ` Linus Walleij
2018-11-05 14:08     ` [PATCH V5 1/1] " A.s. Dong
2018-11-05 14:08       ` A.s. Dong
2018-11-09  9:53       ` Linus Walleij
2018-11-09  9:53         ` Linus Walleij
2018-11-10 14:25         ` A.s. Dong
2018-11-10 14:25           ` A.s. Dong
2018-11-05 14:10     ` [PATCH V4 04/10] " A.s. Dong
2018-11-05 14:10       ` A.s. Dong
2018-11-02  9:12 ` [PATCH V4 05/10] dt-bindings: pinctrl: imx7ulp: back to imx legacy binding for consistency A.s. Dong
2018-11-02  9:12   ` A.s. Dong
2018-11-02  9:37   ` Linus Walleij
2018-11-02  9:37     ` Linus Walleij
2018-11-05 13:04     ` A.s. Dong
2018-11-05 13:04       ` A.s. Dong
2018-11-02 15:43   ` Fabio Estevam
2018-11-02 15:43     ` Fabio Estevam
2018-11-05 19:46   ` Rob Herring
2018-11-05 19:46     ` Rob Herring
2018-11-09  9:50   ` Linus Walleij
2018-11-09  9:50     ` Linus Walleij
2018-11-02  9:13 ` [PATCH V4 06/10] pinctrl: fsl: imx7ulp: change to use imx legacy binding A.s. Dong
2018-11-02  9:13   ` A.s. Dong
2018-11-09  9:54   ` Linus Walleij
2018-11-09  9:54     ` Linus Walleij
2018-11-02  9:13 ` [PATCH V4 07/10] ARM: imx: add initial support for imx7ulp A.s. Dong
2018-11-02  9:13 ` [PATCH V4 08/10] dts: imx: add common imx7ulp dtsi support A.s. Dong
2018-11-02  9:13   ` A.s. Dong
2018-11-02  9:13 ` [PATCH V4 09/10] dts: fsl: add imx7ulp evk support A.s. Dong
2018-11-02  9:13   ` A.s. Dong
2018-11-02  9:13 ` [PATCH V4 10/10] ARM: imx_v6_v7_defconfig: add imx7ulp support A.s. Dong

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1541149669-10857-4-git-send-email-aisheng.dong@nxp.com \
    --to=aisheng.dong@nxp.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dongas86@gmail.com \
    --cc=fabio.estevam@nxp.com \
    --cc=kernel@pengutronix.de \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-gpio@vger.kernel.org \
    --cc=linux-imx@nxp.com \
    --cc=linux@armlinux.org.uk \
    --cc=mark.rutland@arm.com \
    --cc=robh+dt@kernel.org \
    --cc=shawnguo@kernel.org \
    --cc=stefan@agner.ch \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.