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
next prev 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: linkBe 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.