2020-09-20 19:59:20

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH v3 1/3] arm64: dts: imx8mq-librem5: correct GPIO hog property

Correct the name of property for GPIO specifier in GPIO hog.

Signed-off-by: Krzysztof Kozlowski <[email protected]>

---

Changes since v2:
1. None, split from previous patchset using common GPIO schema
---
arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi b/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi
index 56295dd2fa8f..e4dedcb58f76 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi
@@ -251,7 +251,7 @@

pmic-5v {
gpio-hog;
- gpio = <&gpio1 1 GPIO_ACTIVE_HIGH>;
+ gpios = <&gpio1 1 GPIO_ACTIVE_HIGH>;
input;
};
};
--
2.17.1


2020-09-20 19:59:28

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH v3 2/3] arm64: dts: imx8mq-librem5: align GPIO hog names with dtschema

dtschema expects GPIO hogs to end with 'hog' suffix.

Signed-off-by: Krzysztof Kozlowski <[email protected]>

---

Changes since v2:
1. None, split from previous patchset using common GPIO schema
---
arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi b/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi
index e4dedcb58f76..6cbcee2fb938 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi
@@ -249,7 +249,7 @@
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_pmic_5v>;

- pmic-5v {
+ pmic-5v-hog {
gpio-hog;
gpios = <&gpio1 1 GPIO_ACTIVE_HIGH>;
input;
--
2.17.1

2020-09-20 20:00:14

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH v3 3/3] ARM: dts: imx: align GPIO hog names with dtschema

dtschema for GPIO controllers expects GPIO hogs to end with 'hog'
suffix.

Signed-off-by: Krzysztof Kozlowski <[email protected]>

---

Changes since v2:
1. None, split from previous patchset using common GPIO schema
---
arch/arm/boot/dts/imx51-zii-rdu1.dts | 2 +-
arch/arm/boot/dts/imx6qdl-zii-rdu2.dtsi | 8 ++++----
arch/arm/boot/dts/imx6ul-ccimx6ulsbcpro.dts | 2 +-
3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/arch/arm/boot/dts/imx51-zii-rdu1.dts b/arch/arm/boot/dts/imx51-zii-rdu1.dts
index e559ab0c3645..ec8ca3ac2c1c 100644
--- a/arch/arm/boot/dts/imx51-zii-rdu1.dts
+++ b/arch/arm/boot/dts/imx51-zii-rdu1.dts
@@ -451,7 +451,7 @@
"", "", "", "",
"", "", "", "";

- unused-sd3-wp-gpio {
+ unused-sd3-wp-hog {
/*
* See pinctrl_esdhc1 below for more details on this
*/
diff --git a/arch/arm/boot/dts/imx6qdl-zii-rdu2.dtsi b/arch/arm/boot/dts/imx6qdl-zii-rdu2.dtsi
index 66b15748e287..c0a76202e16b 100644
--- a/arch/arm/boot/dts/imx6qdl-zii-rdu2.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-zii-rdu2.dtsi
@@ -330,28 +330,28 @@
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_gpio3_hog>;

- usb-emulation {
+ usb-emulation-hog {
gpio-hog;
gpios = <19 GPIO_ACTIVE_HIGH>;
output-low;
line-name = "usb-emulation";
};

- usb-mode1 {
+ usb-mode1-hog {
gpio-hog;
gpios = <20 GPIO_ACTIVE_HIGH>;
output-high;
line-name = "usb-mode1";
};

- usb-pwr {
+ usb-pwr-hog {
gpio-hog;
gpios = <22 GPIO_ACTIVE_LOW>;
output-high;
line-name = "usb-pwr-ctrl-en-n";
};

- usb-mode2 {
+ usb-mode2-hog {
gpio-hog;
gpios = <23 GPIO_ACTIVE_HIGH>;
output-high;
diff --git a/arch/arm/boot/dts/imx6ul-ccimx6ulsbcpro.dts b/arch/arm/boot/dts/imx6ul-ccimx6ulsbcpro.dts
index a0bbec57ddc7..3ec042bfccba 100644
--- a/arch/arm/boot/dts/imx6ul-ccimx6ulsbcpro.dts
+++ b/arch/arm/boot/dts/imx6ul-ccimx6ulsbcpro.dts
@@ -110,7 +110,7 @@
};

&gpio5 {
- emmc-usd-mux {
+ emmc-usd-mux-hog {
gpio-hog;
gpios = <1 GPIO_ACTIVE_LOW>;
output-high;
--
2.17.1

2020-09-22 08:30:44

by Shawn Guo

[permalink] [raw]
Subject: Re: [PATCH v3 2/3] arm64: dts: imx8mq-librem5: align GPIO hog names with dtschema

On Sun, Sep 20, 2020 at 09:57:48PM +0200, Krzysztof Kozlowski wrote:
> dtschema expects GPIO hogs to end with 'hog' suffix.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

Ping me after dtschema gets accepted.

Shawn

>
> ---
>
> Changes since v2:
> 1. None, split from previous patchset using common GPIO schema
> ---
> arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi b/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi
> index e4dedcb58f76..6cbcee2fb938 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mq-librem5.dtsi
> @@ -249,7 +249,7 @@
> pinctrl-names = "default";
> pinctrl-0 = <&pinctrl_pmic_5v>;
>
> - pmic-5v {
> + pmic-5v-hog {
> gpio-hog;
> gpios = <&gpio1 1 GPIO_ACTIVE_HIGH>;
> input;
> --
> 2.17.1
>

2020-09-22 08:32:17

by Shawn Guo

[permalink] [raw]
Subject: Re: [PATCH v3 1/3] arm64: dts: imx8mq-librem5: correct GPIO hog property

On Sun, Sep 20, 2020 at 09:57:47PM +0200, Krzysztof Kozlowski wrote:
> Correct the name of property for GPIO specifier in GPIO hog.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

Applied, thanks.

2020-09-22 14:26:56

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH v3 2/3] arm64: dts: imx8mq-librem5: align GPIO hog names with dtschema

On Tue, 22 Sep 2020 at 08:52, Shawn Guo <[email protected]> wrote:
>
> On Sun, Sep 20, 2020 at 09:57:48PM +0200, Krzysztof Kozlowski wrote:
> > dtschema expects GPIO hogs to end with 'hog' suffix.
> >
> > Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
> Ping me after dtschema gets accepted.

Hi Shawn,

Happened just now, in dt-schema repo:
https://github.com/devicetree-org/dt-schema/pull/47

Best regards,
Krzysztof

2020-10-14 02:14:32

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH v3 2/3] arm64: dts: imx8mq-librem5: align GPIO hog names with dtschema

On Tue, 22 Sep 2020 at 16:24, Krzysztof Kozlowski <[email protected]> wrote:
>
> On Tue, 22 Sep 2020 at 08:52, Shawn Guo <[email protected]> wrote:
> >
> > On Sun, Sep 20, 2020 at 09:57:48PM +0200, Krzysztof Kozlowski wrote:
> > > dtschema expects GPIO hogs to end with 'hog' suffix.
> > >
> > > Signed-off-by: Krzysztof Kozlowski <[email protected]>
> >
> > Ping me after dtschema gets accepted.
>
> Hi Shawn,
>
> Happened just now, in dt-schema repo:
> https://github.com/devicetree-org/dt-schema/pull/47

The dtschema was accepted in September. Can you pick up this patch?

Best regards,
Krzysztof

2020-10-26 08:23:29

by Shawn Guo

[permalink] [raw]
Subject: Re: [PATCH v3 2/3] arm64: dts: imx8mq-librem5: align GPIO hog names with dtschema

On Sun, Sep 20, 2020 at 09:57:48PM +0200, Krzysztof Kozlowski wrote:
> dtschema expects GPIO hogs to end with 'hog' suffix.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

Applied, thanks.

2020-10-26 08:41:06

by Shawn Guo

[permalink] [raw]
Subject: Re: [PATCH v3 3/3] ARM: dts: imx: align GPIO hog names with dtschema

On Sun, Sep 20, 2020 at 09:57:49PM +0200, Krzysztof Kozlowski wrote:
> dtschema for GPIO controllers expects GPIO hogs to end with 'hog'
> suffix.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

Applied, thanks.