* [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-05 21:55 ` Andrew Geissler 1 sibling, 0 replies; 17+ 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] 17+ messages in thread
* [PATCH linux dev-5.4 v2] ARM: dts: aspeed: witherspoon: Add gpio line names @ 2020-02-05 21:55 ` Andrew Geissler 0 siblings, 0 replies; 17+ messages in thread From: Andrew Geissler @ 2020-02-05 21:55 UTC (permalink / raw) To: linux-aspeed 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] 17+ messages in thread
* [PATCH linux dev-5.4 v2] ARM: dts: aspeed: witherspoon: Add gpio line names @ 2020-02-05 21:55 ` Andrew Geissler 0 siblings, 0 replies; 17+ 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: linux-gpio, Andrew Geissler 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) _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH linux dev-5.4 v2] ARM: dts: aspeed: witherspoon: Add gpio line names 2020-02-05 21:55 ` Andrew Geissler (?) @ 2020-02-06 6:45 ` Joel Stanley -1 siblings, 0 replies; 17+ messages in thread From: Joel Stanley @ 2020-02-06 6:45 UTC (permalink / raw) To: linux-aspeed 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] 17+ messages in thread
* Re: [PATCH linux dev-5.4 v2] ARM: dts: aspeed: witherspoon: Add gpio line names @ 2020-02-06 6:45 ` Joel Stanley 0 siblings, 0 replies; 17+ messages in thread From: Joel Stanley @ 2020-02-06 6:45 UTC (permalink / raw) To: Andrew Geissler Cc: devicetree, linux-aspeed, OpenBMC Maillist, Linux Kernel Mailing List, open list:GPIO SUBSYSTEM, Andrew Geissler, Linux ARM 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) > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH linux dev-5.4 v2] ARM: dts: aspeed: witherspoon: Add gpio line names @ 2020-02-06 6:45 ` Joel Stanley 0 siblings, 0 replies; 17+ 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] 17+ messages in thread
* Re: [PATCH linux dev-5.4 v2] ARM: dts: aspeed: witherspoon: Add gpio line names 2020-02-06 6:45 ` Joel Stanley @ 2020-02-06 16:48 ` Andrew Geissler -1 siblings, 0 replies; 17+ messages in thread From: Andrew Geissler @ 2020-02-06 16:48 UTC (permalink / raw) To: Joel Stanley Cc: OpenBMC Maillist, devicetree, Linux ARM, linux-aspeed, Linux Kernel Mailing List, open list:GPIO SUBSYSTEM [-- Attachment #1: Type: text/plain, Size: 3479 bytes --] > On Feb 6, 2020, at 12:45 AM, Joel Stanley <joel@jms.id.au> wrote: > > On Wed, 5 Feb 2020 at 21:55, Andrew Geissler <geissonator@gmail.com <mailto:geissonator@gmail.com>> wrote: >> >> From: Andrew Geissler <geissonator@yahoo.com <mailto:geissonator@yahoo.com>> >> >> Name the gpios so libgiod will work with them > > Perhaps: "Name the GPIOs to help userspace work with them." ack > > You could also mention that the names are describe the functionality > that the lines provide, and not a net or ball name. ack > > 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. thanks, will fix in next patch > >> 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. ack > >> + 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. ack > >> +}; >> + >> &fmc { >> status = "okay"; >> >> -- >> 2.21.0 (Apple Git-122) [-- Attachment #2: Type: text/html, Size: 17727 bytes --] ^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH linux dev-5.4 v2] ARM: dts: aspeed: witherspoon: Add gpio line names @ 2020-02-06 16:48 ` Andrew Geissler 0 siblings, 0 replies; 17+ messages in thread From: Andrew Geissler @ 2020-02-06 16:48 UTC (permalink / raw) To: linux-aspeed > On Feb 6, 2020, at 12:45 AM, Joel Stanley <joel@jms.id.au> wrote: > > On Wed, 5 Feb 2020 at 21:55, Andrew Geissler <geissonator at gmail.com <mailto:geissonator@gmail.com>> wrote: >> >> From: Andrew Geissler <geissonator at yahoo.com <mailto:geissonator@yahoo.com>> >> >> Name the gpios so libgiod will work with them > > Perhaps: "Name the GPIOs to help userspace work with them." ack > > You could also mention that the names are describe the functionality > that the lines provide, and not a net or ball name. ack > > 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. thanks, will fix in next patch > >> 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. ack > >> + 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. ack > >> +}; >> + >> &fmc { >> status = "okay"; >> >> -- >> 2.21.0 (Apple Git-122) -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.ozlabs.org/pipermail/linux-aspeed/attachments/20200206/bccd1fcd/attachment-0001.htm> ^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH v3] ARM: dts: aspeed: witherspoon: Add gpio line names [not found] <FMfcgxwGDDpcbrVbWGfkMRbZCLSZqpMM> 2020-02-05 21:55 ` Andrew Geissler @ 2020-02-06 17:02 ` Andrew Geissler 1 sibling, 0 replies; 17+ 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] 17+ messages in thread
* [PATCH v3] ARM: dts: aspeed: witherspoon: Add gpio line names @ 2020-02-06 17:02 ` Andrew Geissler 0 siblings, 0 replies; 17+ messages in thread From: Andrew Geissler @ 2020-02-06 17:03 UTC (permalink / raw) To: linux-aspeed 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] 17+ messages in thread
* [PATCH v3] ARM: dts: aspeed: witherspoon: Add gpio line names @ 2020-02-06 17:02 ` Andrew Geissler 0 siblings, 0 replies; 17+ 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: linux-gpio, Andrew Geissler 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) _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH v3] ARM: dts: aspeed: witherspoon: Add gpio line names 2020-02-06 17:02 ` Andrew Geissler (?) @ 2020-02-10 2:29 ` Joel Stanley -1 siblings, 0 replies; 17+ messages in thread From: Joel Stanley @ 2020-02-10 2:29 UTC (permalink / raw) To: linux-aspeed 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] 17+ messages in thread
* Re: [PATCH v3] ARM: dts: aspeed: witherspoon: Add gpio line names @ 2020-02-10 2:29 ` Joel Stanley 0 siblings, 0 replies; 17+ messages in thread From: Joel Stanley @ 2020-02-10 2:29 UTC (permalink / raw) To: Andrew Geissler Cc: devicetree, linux-aspeed, OpenBMC Maillist, Linux Kernel Mailing List, open list:GPIO SUBSYSTEM, Andrew Geissler, Linux ARM 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) > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH v3] ARM: dts: aspeed: witherspoon: Add gpio line names @ 2020-02-10 2:29 ` Joel Stanley 0 siblings, 0 replies; 17+ 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] 17+ messages in thread
* [PATCH v3] ARM: dts: aspeed: witherspoon: Add gpio line names 2020-02-10 2:29 ` Joel Stanley (?) @ 2020-02-13 0:54 ` Andrew Jeffery -1 siblings, 0 replies; 17+ messages in thread From: Andrew Jeffery @ 2020-02-13 0:54 UTC (permalink / raw) To: linux-aspeed 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] 17+ messages in thread
* Re: [PATCH v3] ARM: dts: aspeed: witherspoon: Add gpio line names @ 2020-02-13 0:54 ` Andrew Jeffery 0 siblings, 0 replies; 17+ 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 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH v3] ARM: dts: aspeed: witherspoon: Add gpio line names @ 2020-02-13 0:54 ` Andrew Jeffery 0 siblings, 0 replies; 17+ 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] 17+ messages in thread
end of thread, other threads:[~2020-02-13 0:55 UTC | newest]
Thread overview: 17+ 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-05 21:55 ` Andrew Geissler
2020-02-05 21:55 ` Andrew Geissler
2020-02-06 6:45 ` Joel Stanley
2020-02-06 6:45 ` Joel Stanley
2020-02-06 6:45 ` Joel Stanley
2020-02-06 16:48 ` Andrew Geissler
2020-02-06 16:48 ` Andrew Geissler
2020-02-06 17:02 ` [PATCH v3] " Andrew Geissler
2020-02-06 17:03 ` Andrew Geissler
2020-02-06 17:02 ` Andrew Geissler
2020-02-10 2:29 ` Joel Stanley
2020-02-10 2:29 ` Joel Stanley
2020-02-10 2:29 ` Joel Stanley
2020-02-13 0:54 ` Andrew Jeffery
2020-02-13 0:54 ` Andrew Jeffery
2020-02-13 0:54 ` Andrew Jeffery
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.