* [PATCH linux dev-5.4 v2] ARM: dts: aspeed: witherspoon: Add gpio line names
[not found] <FMfcgxwGDDpcbrVbWGfkMRbZCLSZqpMM>
@ 2020-02-05 21:55 ` Andrew Geissler
2020-02-06 6:45 ` Joel Stanley
2020-02-06 17:02 ` [PATCH v3] " Andrew Geissler
1 sibling, 1 reply; 5+ messages in thread
From: Andrew Geissler @ 2020-02-05 21:55 UTC (permalink / raw)
To: openbmc, joel, devicetree, linux-arm-kernel, linux-aspeed, linux-kernel
Cc: Andrew Geissler, linux-gpio
From: Andrew Geissler <geissonator@yahoo.com>
Name the gpios so libgiod will work with them
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
---
v2: added upstream to patch
prepended fsi- on fsi related gpios
prepended led- on led related gpios
prepended presence on presence related gpios
dropped pin_gpio_a1 definition
---
.../boot/dts/aspeed-bmc-opp-witherspoon.dts | 36 +++++++++++++++++++
1 file changed, 36 insertions(+)
diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts b/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
index 515f0f208ee6..2519722928a4 100644
--- a/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
+++ b/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
@@ -193,6 +193,42 @@
};
+&gpio {
+ status = "okay";
+ gpio-line-names =
+ /*A0-A7*/ "","cfam-reset","","","","","fsi-mux","",
+ /*B0-B7*/ "","","","","","air-water","","",
+ /*C0-C7*/ "","","","","","","","",
+ /*D0-D7*/ "fsi-enable","","","","","","","",
+ /*E0-E7*/ "fsi-data","","","","","","","",
+ /*F0-F7*/ "","","","","","","","",
+ /*G0-G7*/ "","","","","","","","",
+ /*H0-H7*/ "","","","","","","","",
+ /*I0-I7*/ "","","","","","","","",
+ /*J0-J7*/ "","","checkstop","","","","","",
+ /*K0-K7*/ "","","","","","","","",
+ /*L0-L7*/ "","","","","","","","",
+ /*M0-M7*/ "","","","","","","","",
+ /*N0-N7*/ "presence-ps1","","led-rear-fault","led-rear-power",
+ "led-rear-id","","","",
+ /*O0-O7*/ "","","","","","","","",
+ /*P0-P7*/ "","","","","","","","presence-ps0",
+ /*Q0-Q7*/ "","","","","","","","",
+ /*R0-R7*/ "","","fsi-trans","","","power-button","","",
+ /*S0-S7*/ "","","","","","","","",
+ /*T0-T7*/ "","","","","","","","",
+ /*U0-U7*/ "","","","","","","","",
+ /*V0-V7*/ "","","","","","","","",
+ /*W0-W7*/ "","","","","","","","",
+ /*X0-X7*/ "","","","","","","","",
+ /*Y0-Y7*/ "","","","","","","","",
+ /*Z0-Z7*/ "","","","","","","","",
+ /*AA0-AA7*/ "fsi-clock","","","","","","","",
+ /*AB0-AB7*/ "","","","","","","","",
+ /*AC0-AC7*/ "","","","","","","","";
+
+};
+
&fmc {
status = "okay";
--
2.21.0 (Apple Git-122)
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH linux dev-5.4 v2] ARM: dts: aspeed: witherspoon: Add gpio line names
2020-02-05 21:55 ` [PATCH linux dev-5.4 v2] ARM: dts: aspeed: witherspoon: Add gpio line names Andrew Geissler
@ 2020-02-06 6:45 ` Joel Stanley
0 siblings, 0 replies; 5+ messages in thread
From: Joel Stanley @ 2020-02-06 6:45 UTC (permalink / raw)
To: Andrew Geissler
Cc: OpenBMC Maillist, devicetree, Linux ARM, linux-aspeed,
Linux Kernel Mailing List, Andrew Geissler,
open list:GPIO SUBSYSTEM
On Wed, 5 Feb 2020 at 21:55, Andrew Geissler <geissonator@gmail.com> wrote:
>
> From: Andrew Geissler <geissonator@yahoo.com>
>
> Name the gpios so libgiod will work with them
Perhaps: "Name the GPIOs to help userspace work with them."
You could also mention that the names are describe the functionality
that the lines provide, and not a net or ball name.
A heads up: when you send patches upstream to the kernel mailing list,
we don't include the "linux dev-X.y" stuff in the subject.
> Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
> ---
> v2: added upstream to patch
> prepended fsi- on fsi related gpios
> prepended led- on led related gpios
> prepended presence on presence related gpios
> dropped pin_gpio_a1 definition
> ---
> .../boot/dts/aspeed-bmc-opp-witherspoon.dts | 36 +++++++++++++++++++
> 1 file changed, 36 insertions(+)
>
> diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts b/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
> index 515f0f208ee6..2519722928a4 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
> @@ -193,6 +193,42 @@
>
> };
>
> +&gpio {
> + status = "okay";
The status=okay is harmless but redundant, as this node is already enabled.
> + gpio-line-names =
> + /*A0-A7*/ "","cfam-reset","","","","","fsi-mux","",
> + /*B0-B7*/ "","","","","","air-water","","",
> + /*C0-C7*/ "","","","","","","","",
> + /*D0-D7*/ "fsi-enable","","","","","","","",
> + /*E0-E7*/ "fsi-data","","","","","","","",
> + /*F0-F7*/ "","","","","","","","",
> + /*G0-G7*/ "","","","","","","","",
> + /*H0-H7*/ "","","","","","","","",
> + /*I0-I7*/ "","","","","","","","",
> + /*J0-J7*/ "","","checkstop","","","","","",
> + /*K0-K7*/ "","","","","","","","",
> + /*L0-L7*/ "","","","","","","","",
> + /*M0-M7*/ "","","","","","","","",
> + /*N0-N7*/ "presence-ps1","","led-rear-fault","led-rear-power",
> + "led-rear-id","","","",
> + /*O0-O7*/ "","","","","","","","",
> + /*P0-P7*/ "","","","","","","","presence-ps0",
> + /*Q0-Q7*/ "","","","","","","","",
> + /*R0-R7*/ "","","fsi-trans","","","power-button","","",
> + /*S0-S7*/ "","","","","","","","",
> + /*T0-T7*/ "","","","","","","","",
> + /*U0-U7*/ "","","","","","","","",
> + /*V0-V7*/ "","","","","","","","",
> + /*W0-W7*/ "","","","","","","","",
> + /*X0-X7*/ "","","","","","","","",
> + /*Y0-Y7*/ "","","","","","","","",
> + /*Z0-Z7*/ "","","","","","","","",
> + /*AA0-AA7*/ "fsi-clock","","","","","","","",
> + /*AB0-AB7*/ "","","","","","","","",
> + /*AC0-AC7*/ "","","","","","","","";
> +
Cean up the line of whitespace here.
> +};
> +
> &fmc {
> status = "okay";
>
> --
> 2.21.0 (Apple Git-122)
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v3] ARM: dts: aspeed: witherspoon: Add gpio line names
[not found] <FMfcgxwGDDpcbrVbWGfkMRbZCLSZqpMM>
2020-02-05 21:55 ` [PATCH linux dev-5.4 v2] ARM: dts: aspeed: witherspoon: Add gpio line names Andrew Geissler
@ 2020-02-06 17:02 ` Andrew Geissler
2020-02-10 2:29 ` Joel Stanley
1 sibling, 1 reply; 5+ messages in thread
From: Andrew Geissler @ 2020-02-06 17:02 UTC (permalink / raw)
To: openbmc, joel, devicetree, linux-arm-kernel, linux-aspeed, linux-kernel
Cc: Andrew Geissler, linux-gpio
From: Andrew Geissler <geissonator@yahoo.com>
Name the GPIOs to help userspace work with them. The names describe the
functionality the lines provide, not the net or ball name. This makes it
easier to share userspace code across different systems and makes the
use of the lines more obvious.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
---
v3: added more detail to commit message
removed linux dev-5.4 from PATCH header
removed redundant status from gpio
removed blank line
v2: added upstream to patch
prepended fsi- on fsi related gpios
prepended led- on led related gpios
prepended presence on presence related gpios
dropped pin_gpio_a1 definition
---
.../boot/dts/aspeed-bmc-opp-witherspoon.dts | 34 +++++++++++++++++++
1 file changed, 34 insertions(+)
diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts b/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
index 515f0f208ee6..2269c73b8987 100644
--- a/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
+++ b/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
@@ -193,6 +193,40 @@
};
+&gpio {
+ gpio-line-names =
+ /*A0-A7*/ "","cfam-reset","","","","","fsi-mux","",
+ /*B0-B7*/ "","","","","","air-water","","",
+ /*C0-C7*/ "","","","","","","","",
+ /*D0-D7*/ "fsi-enable","","","","","","","",
+ /*E0-E7*/ "fsi-data","","","","","","","",
+ /*F0-F7*/ "","","","","","","","",
+ /*G0-G7*/ "","","","","","","","",
+ /*H0-H7*/ "","","","","","","","",
+ /*I0-I7*/ "","","","","","","","",
+ /*J0-J7*/ "","","checkstop","","","","","",
+ /*K0-K7*/ "","","","","","","","",
+ /*L0-L7*/ "","","","","","","","",
+ /*M0-M7*/ "","","","","","","","",
+ /*N0-N7*/ "presence-ps1","","led-rear-fault","led-rear-power",
+ "led-rear-id","","","",
+ /*O0-O7*/ "","","","","","","","",
+ /*P0-P7*/ "","","","","","","","presence-ps0",
+ /*Q0-Q7*/ "","","","","","","","",
+ /*R0-R7*/ "","","fsi-trans","","","power-button","","",
+ /*S0-S7*/ "","","","","","","","",
+ /*T0-T7*/ "","","","","","","","",
+ /*U0-U7*/ "","","","","","","","",
+ /*V0-V7*/ "","","","","","","","",
+ /*W0-W7*/ "","","","","","","","",
+ /*X0-X7*/ "","","","","","","","",
+ /*Y0-Y7*/ "","","","","","","","",
+ /*Z0-Z7*/ "","","","","","","","",
+ /*AA0-AA7*/ "fsi-clock","","","","","","","",
+ /*AB0-AB7*/ "","","","","","","","",
+ /*AC0-AC7*/ "","","","","","","","";
+};
+
&fmc {
status = "okay";
--
2.21.0 (Apple Git-122)
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v3] ARM: dts: aspeed: witherspoon: Add gpio line names
2020-02-06 17:02 ` [PATCH v3] " Andrew Geissler
@ 2020-02-10 2:29 ` Joel Stanley
2020-02-13 0:54 ` Andrew Jeffery
0 siblings, 1 reply; 5+ messages in thread
From: Joel Stanley @ 2020-02-10 2:29 UTC (permalink / raw)
To: Andrew Geissler
Cc: OpenBMC Maillist, devicetree, Linux ARM, linux-aspeed,
Linux Kernel Mailing List, Andrew Geissler,
open list:GPIO SUBSYSTEM
On Thu, 6 Feb 2020 at 17:03, Andrew Geissler <geissonator@gmail.com> wrote:
>
> From: Andrew Geissler <geissonator@yahoo.com>
>
> Name the GPIOs to help userspace work with them. The names describe the
> functionality the lines provide, not the net or ball name. This makes it
> easier to share userspace code across different systems and makes the
> use of the lines more obvious.
>
> Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Reviewed-by: Joel Stanley <joel@jms.id.au>
I will wait for Andrew before merging, but this looks good to me.
> ---
> v3: added more detail to commit message
> removed linux dev-5.4 from PATCH header
> removed redundant status from gpio
> removed blank line
> v2: added upstream to patch
> prepended fsi- on fsi related gpios
> prepended led- on led related gpios
> prepended presence on presence related gpios
> dropped pin_gpio_a1 definition
> ---
> .../boot/dts/aspeed-bmc-opp-witherspoon.dts | 34 +++++++++++++++++++
> 1 file changed, 34 insertions(+)
>
> diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts b/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
> index 515f0f208ee6..2269c73b8987 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
> @@ -193,6 +193,40 @@
>
> };
>
> +&gpio {
> + gpio-line-names =
> + /*A0-A7*/ "","cfam-reset","","","","","fsi-mux","",
> + /*B0-B7*/ "","","","","","air-water","","",
> + /*C0-C7*/ "","","","","","","","",
> + /*D0-D7*/ "fsi-enable","","","","","","","",
> + /*E0-E7*/ "fsi-data","","","","","","","",
> + /*F0-F7*/ "","","","","","","","",
> + /*G0-G7*/ "","","","","","","","",
> + /*H0-H7*/ "","","","","","","","",
> + /*I0-I7*/ "","","","","","","","",
> + /*J0-J7*/ "","","checkstop","","","","","",
> + /*K0-K7*/ "","","","","","","","",
> + /*L0-L7*/ "","","","","","","","",
> + /*M0-M7*/ "","","","","","","","",
> + /*N0-N7*/ "presence-ps1","","led-rear-fault","led-rear-power",
> + "led-rear-id","","","",
> + /*O0-O7*/ "","","","","","","","",
> + /*P0-P7*/ "","","","","","","","presence-ps0",
> + /*Q0-Q7*/ "","","","","","","","",
> + /*R0-R7*/ "","","fsi-trans","","","power-button","","",
> + /*S0-S7*/ "","","","","","","","",
> + /*T0-T7*/ "","","","","","","","",
> + /*U0-U7*/ "","","","","","","","",
> + /*V0-V7*/ "","","","","","","","",
> + /*W0-W7*/ "","","","","","","","",
> + /*X0-X7*/ "","","","","","","","",
> + /*Y0-Y7*/ "","","","","","","","",
> + /*Z0-Z7*/ "","","","","","","","",
> + /*AA0-AA7*/ "fsi-clock","","","","","","","",
> + /*AB0-AB7*/ "","","","","","","","",
> + /*AC0-AC7*/ "","","","","","","","";
> +};
> +
> &fmc {
> status = "okay";
>
> --
> 2.21.0 (Apple Git-122)
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v3] ARM: dts: aspeed: witherspoon: Add gpio line names
2020-02-10 2:29 ` Joel Stanley
@ 2020-02-13 0:54 ` Andrew Jeffery
0 siblings, 0 replies; 5+ messages in thread
From: Andrew Jeffery @ 2020-02-13 0:54 UTC (permalink / raw)
To: Joel Stanley, Andrew Geissler
Cc: devicetree, linux-aspeed, OpenBMC Maillist,
Linux Kernel Mailing List, open list:GPIO SUBSYSTEM,
Andrew Geissler, Linux ARM
On Mon, 10 Feb 2020, at 12:59, Joel Stanley wrote:
> On Thu, 6 Feb 2020 at 17:03, Andrew Geissler <geissonator@gmail.com> wrote:
> >
> > From: Andrew Geissler <geissonator@yahoo.com>
> >
> > Name the GPIOs to help userspace work with them. The names describe the
> > functionality the lines provide, not the net or ball name. This makes it
> > easier to share userspace code across different systems and makes the
> > use of the lines more obvious.
> >
> > Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
>
> Reviewed-by: Joel Stanley <joel@jms.id.au>
>
> I will wait for Andrew before merging, but this looks good to me.
To the best of my knowledge this is the only supported approach.
I haven't checked the mapping of gpio lines to functions, but for
the concept:
Acked-by: Andrew Jeffery <andrew@aj.id.au>
Andrew
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-02-13 0:54 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <FMfcgxwGDDpcbrVbWGfkMRbZCLSZqpMM>
2020-02-05 21:55 ` [PATCH linux dev-5.4 v2] ARM: dts: aspeed: witherspoon: Add gpio line names Andrew Geissler
2020-02-06 6:45 ` Joel Stanley
2020-02-06 17:02 ` [PATCH v3] " Andrew Geissler
2020-02-10 2:29 ` Joel Stanley
2020-02-13 0:54 ` Andrew Jeffery
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).