* [PATCH 1/2] ARM: dts: aspeed: rainier: add reset-cause-pinhole
@ 2022-01-13 21:17 ` Andrew Geissler
0 siblings, 0 replies; 9+ messages in thread
From: Andrew Geissler @ 2022-01-13 21:17 UTC (permalink / raw)
To: joel, andrew, robh+dt, devicetree, linux-arm-kernel,
linux-aspeed, linux-kernel
Cc: Andrew Geissler, linux-gpio, openbmc
This GPIO is used on the rainier system to indicate the BMC was reset
due to a physical pinhole reset.
See the following doc for more information:
https://github.com/openbmc/docs/blob/master/designs/device-tree-gpio-naming.md
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
---
arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
index cfaa606666e2..7ea70c7d698f 100644
--- a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
+++ b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
@@ -247,7 +247,8 @@ &gpio0 {
/*C0-C7*/ "","","","","","","","",
/*D0-D7*/ "","","","","","","","",
/*E0-E7*/ "","","","","","","","",
- /*F0-F7*/ "","","battery-voltage-read-enable","","","","factory-reset-toggle","",
+ /*F0-F7*/ "","","battery-voltage-read-enable","reset-cause-pinhole","","",
+ "factory-reset-toggle","",
/*G0-G7*/ "","","","","","","","",
/*H0-H7*/ "","bmc-ingraham0","rear-enc-id0","rear-enc-fault0","","","","",
/*I0-I7*/ "","","","","","","bmc-secure-boot","",
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 1/2] ARM: dts: aspeed: rainier: add reset-cause-pinhole
@ 2022-01-13 21:17 ` Andrew Geissler
0 siblings, 0 replies; 9+ messages in thread
From: Andrew Geissler @ 2022-01-13 21:17 UTC (permalink / raw)
To: joel, andrew, robh+dt, devicetree, linux-arm-kernel,
linux-aspeed, linux-kernel
Cc: linux-gpio, openbmc, Andrew Geissler
This GPIO is used on the rainier system to indicate the BMC was reset
due to a physical pinhole reset.
See the following doc for more information:
https://github.com/openbmc/docs/blob/master/designs/device-tree-gpio-naming.md
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
---
arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
index cfaa606666e2..7ea70c7d698f 100644
--- a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
+++ b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
@@ -247,7 +247,8 @@ &gpio0 {
/*C0-C7*/ "","","","","","","","",
/*D0-D7*/ "","","","","","","","",
/*E0-E7*/ "","","","","","","","",
- /*F0-F7*/ "","","battery-voltage-read-enable","","","","factory-reset-toggle","",
+ /*F0-F7*/ "","","battery-voltage-read-enable","reset-cause-pinhole","","",
+ "factory-reset-toggle","",
/*G0-G7*/ "","","","","","","","",
/*H0-H7*/ "","bmc-ingraham0","rear-enc-id0","rear-enc-fault0","","","","",
/*I0-I7*/ "","","","","","","bmc-secure-boot","",
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 1/2] ARM: dts: aspeed: rainier: add reset-cause-pinhole
@ 2022-01-13 21:17 ` Andrew Geissler
0 siblings, 0 replies; 9+ messages in thread
From: Andrew Geissler @ 2022-01-13 21:17 UTC (permalink / raw)
To: joel, andrew, robh+dt, devicetree, linux-arm-kernel,
linux-aspeed, linux-kernel
Cc: Andrew Geissler, linux-gpio, openbmc
This GPIO is used on the rainier system to indicate the BMC was reset
due to a physical pinhole reset.
See the following doc for more information:
https://github.com/openbmc/docs/blob/master/designs/device-tree-gpio-naming.md
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
---
arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
index cfaa606666e2..7ea70c7d698f 100644
--- a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
+++ b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
@@ -247,7 +247,8 @@ &gpio0 {
/*C0-C7*/ "","","","","","","","",
/*D0-D7*/ "","","","","","","","",
/*E0-E7*/ "","","","","","","","",
- /*F0-F7*/ "","","battery-voltage-read-enable","","","","factory-reset-toggle","",
+ /*F0-F7*/ "","","battery-voltage-read-enable","reset-cause-pinhole","","",
+ "factory-reset-toggle","",
/*G0-G7*/ "","","","","","","","",
/*H0-H7*/ "","bmc-ingraham0","rear-enc-id0","rear-enc-fault0","","","","",
/*I0-I7*/ "","","","","","","bmc-secure-boot","",
--
2.25.1
_______________________________________________
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] 9+ messages in thread
* [PATCH 2/2] ARM: dts: aspeed: everest: add reset-cause-pinhole
2022-01-13 21:17 ` Andrew Geissler
(?)
@ 2022-01-13 21:17 ` Andrew Geissler
-1 siblings, 0 replies; 9+ messages in thread
From: Andrew Geissler @ 2022-01-13 21:17 UTC (permalink / raw)
To: joel, andrew, robh+dt, devicetree, linux-arm-kernel,
linux-aspeed, linux-kernel
Cc: Andrew Geissler, linux-gpio, openbmc
This GPIO is used on the everest system to indicate the BMC was reset
due to a physical pinhole reset.
It has been verified that the previous name for this pin has not been
utilized by userspace so the name change is ok.
See the following doc for more information:
https://github.com/openbmc/docs/blob/master/designs/device-tree-gpio-naming.md
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
---
arch/arm/boot/dts/aspeed-bmc-ibm-everest.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/aspeed-bmc-ibm-everest.dts b/arch/arm/boot/dts/aspeed-bmc-ibm-everest.dts
index f48b33b0c606..6b5d5c56951f 100644
--- a/arch/arm/boot/dts/aspeed-bmc-ibm-everest.dts
+++ b/arch/arm/boot/dts/aspeed-bmc-ibm-everest.dts
@@ -271,7 +271,7 @@ &gpio0 {
/*D0-D7*/ "","","","","","","","",
/*E0-E7*/ "","","","","","","","",
/*F0-F7*/ "PIN_HOLE_RESET_IN_N","","battery-voltage-read-enable",
- "PIN_HOLE_RESET_OUT_N","","",
+ "reset-cause-pinhole","","",
"factory-reset-toggle","",
/*G0-G7*/ "","","","","","","","",
/*H0-H7*/ "led-rtc-battery","led-bmc","led-rear-enc-id0","led-rear-enc-fault0","","","","",
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 2/2] ARM: dts: aspeed: everest: add reset-cause-pinhole
@ 2022-01-13 21:17 ` Andrew Geissler
0 siblings, 0 replies; 9+ messages in thread
From: Andrew Geissler @ 2022-01-13 21:17 UTC (permalink / raw)
To: joel, andrew, robh+dt, devicetree, linux-arm-kernel,
linux-aspeed, linux-kernel
Cc: linux-gpio, openbmc, Andrew Geissler
This GPIO is used on the everest system to indicate the BMC was reset
due to a physical pinhole reset.
It has been verified that the previous name for this pin has not been
utilized by userspace so the name change is ok.
See the following doc for more information:
https://github.com/openbmc/docs/blob/master/designs/device-tree-gpio-naming.md
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
---
arch/arm/boot/dts/aspeed-bmc-ibm-everest.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/aspeed-bmc-ibm-everest.dts b/arch/arm/boot/dts/aspeed-bmc-ibm-everest.dts
index f48b33b0c606..6b5d5c56951f 100644
--- a/arch/arm/boot/dts/aspeed-bmc-ibm-everest.dts
+++ b/arch/arm/boot/dts/aspeed-bmc-ibm-everest.dts
@@ -271,7 +271,7 @@ &gpio0 {
/*D0-D7*/ "","","","","","","","",
/*E0-E7*/ "","","","","","","","",
/*F0-F7*/ "PIN_HOLE_RESET_IN_N","","battery-voltage-read-enable",
- "PIN_HOLE_RESET_OUT_N","","",
+ "reset-cause-pinhole","","",
"factory-reset-toggle","",
/*G0-G7*/ "","","","","","","","",
/*H0-H7*/ "led-rtc-battery","led-bmc","led-rear-enc-id0","led-rear-enc-fault0","","","","",
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 2/2] ARM: dts: aspeed: everest: add reset-cause-pinhole
@ 2022-01-13 21:17 ` Andrew Geissler
0 siblings, 0 replies; 9+ messages in thread
From: Andrew Geissler @ 2022-01-13 21:17 UTC (permalink / raw)
To: joel, andrew, robh+dt, devicetree, linux-arm-kernel,
linux-aspeed, linux-kernel
Cc: Andrew Geissler, linux-gpio, openbmc
This GPIO is used on the everest system to indicate the BMC was reset
due to a physical pinhole reset.
It has been verified that the previous name for this pin has not been
utilized by userspace so the name change is ok.
See the following doc for more information:
https://github.com/openbmc/docs/blob/master/designs/device-tree-gpio-naming.md
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
---
arch/arm/boot/dts/aspeed-bmc-ibm-everest.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/aspeed-bmc-ibm-everest.dts b/arch/arm/boot/dts/aspeed-bmc-ibm-everest.dts
index f48b33b0c606..6b5d5c56951f 100644
--- a/arch/arm/boot/dts/aspeed-bmc-ibm-everest.dts
+++ b/arch/arm/boot/dts/aspeed-bmc-ibm-everest.dts
@@ -271,7 +271,7 @@ &gpio0 {
/*D0-D7*/ "","","","","","","","",
/*E0-E7*/ "","","","","","","","",
/*F0-F7*/ "PIN_HOLE_RESET_IN_N","","battery-voltage-read-enable",
- "PIN_HOLE_RESET_OUT_N","","",
+ "reset-cause-pinhole","","",
"factory-reset-toggle","",
/*G0-G7*/ "","","","","","","","",
/*H0-H7*/ "led-rtc-battery","led-bmc","led-rear-enc-id0","led-rear-enc-fault0","","","","",
--
2.25.1
_______________________________________________
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] 9+ messages in thread
* Re: [PATCH 1/2] ARM: dts: aspeed: rainier: add reset-cause-pinhole
2022-01-13 21:17 ` Andrew Geissler
(?)
@ 2022-01-13 22:35 ` Joel Stanley
-1 siblings, 0 replies; 9+ messages in thread
From: Joel Stanley @ 2022-01-13 22:35 UTC (permalink / raw)
To: Andrew Geissler
Cc: Andrew Jeffery, Rob Herring, devicetree, Linux ARM, linux-aspeed,
Linux Kernel Mailing List, Andrew Geissler,
open list:GPIO SUBSYSTEM, OpenBMC Maillist
Hi Andrew,
On Thu, 13 Jan 2022 at 21:18, Andrew Geissler <geissonator@gmail.com> wrote:
>
> This GPIO is used on the rainier system to indicate the BMC was reset
> due to a physical pinhole reset.
>
> See the following doc for more information:
> https://github.com/openbmc/docs/blob/master/designs/device-tree-gpio-naming.md
>
> Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
I couldn't apply these cleanly to any tree I had. I think you had a
patch to add battery-voltage-read-enable, but that's not in the tree
yet.
I fixed them up to apply.
Cheers,
Joel
> ---
> arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
> index cfaa606666e2..7ea70c7d698f 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
> @@ -247,7 +247,8 @@ &gpio0 {
> /*C0-C7*/ "","","","","","","","",
> /*D0-D7*/ "","","","","","","","",
> /*E0-E7*/ "","","","","","","","",
> - /*F0-F7*/ "","","battery-voltage-read-enable","","","","factory-reset-toggle","",
> + /*F0-F7*/ "","","battery-voltage-read-enable","reset-cause-pinhole","","",
> + "factory-reset-toggle","",
> /*G0-G7*/ "","","","","","","","",
> /*H0-H7*/ "","bmc-ingraham0","rear-enc-id0","rear-enc-fault0","","","","",
> /*I0-I7*/ "","","","","","","bmc-secure-boot","",
> --
> 2.25.1
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] ARM: dts: aspeed: rainier: add reset-cause-pinhole
@ 2022-01-13 22:35 ` Joel Stanley
0 siblings, 0 replies; 9+ messages in thread
From: Joel Stanley @ 2022-01-13 22:35 UTC (permalink / raw)
To: Andrew Geissler
Cc: devicetree, linux-aspeed, Andrew Jeffery, OpenBMC Maillist,
Linux Kernel Mailing List, open list:GPIO SUBSYSTEM, Rob Herring,
Andrew Geissler, Linux ARM
Hi Andrew,
On Thu, 13 Jan 2022 at 21:18, Andrew Geissler <geissonator@gmail.com> wrote:
>
> This GPIO is used on the rainier system to indicate the BMC was reset
> due to a physical pinhole reset.
>
> See the following doc for more information:
> https://github.com/openbmc/docs/blob/master/designs/device-tree-gpio-naming.md
>
> Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
I couldn't apply these cleanly to any tree I had. I think you had a
patch to add battery-voltage-read-enable, but that's not in the tree
yet.
I fixed them up to apply.
Cheers,
Joel
> ---
> arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
> index cfaa606666e2..7ea70c7d698f 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
> @@ -247,7 +247,8 @@ &gpio0 {
> /*C0-C7*/ "","","","","","","","",
> /*D0-D7*/ "","","","","","","","",
> /*E0-E7*/ "","","","","","","","",
> - /*F0-F7*/ "","","battery-voltage-read-enable","","","","factory-reset-toggle","",
> + /*F0-F7*/ "","","battery-voltage-read-enable","reset-cause-pinhole","","",
> + "factory-reset-toggle","",
> /*G0-G7*/ "","","","","","","","",
> /*H0-H7*/ "","bmc-ingraham0","rear-enc-id0","rear-enc-fault0","","","","",
> /*I0-I7*/ "","","","","","","bmc-secure-boot","",
> --
> 2.25.1
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] ARM: dts: aspeed: rainier: add reset-cause-pinhole
@ 2022-01-13 22:35 ` Joel Stanley
0 siblings, 0 replies; 9+ messages in thread
From: Joel Stanley @ 2022-01-13 22:35 UTC (permalink / raw)
To: Andrew Geissler
Cc: Andrew Jeffery, Rob Herring, devicetree, Linux ARM, linux-aspeed,
Linux Kernel Mailing List, Andrew Geissler,
open list:GPIO SUBSYSTEM, OpenBMC Maillist
Hi Andrew,
On Thu, 13 Jan 2022 at 21:18, Andrew Geissler <geissonator@gmail.com> wrote:
>
> This GPIO is used on the rainier system to indicate the BMC was reset
> due to a physical pinhole reset.
>
> See the following doc for more information:
> https://github.com/openbmc/docs/blob/master/designs/device-tree-gpio-naming.md
>
> Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
I couldn't apply these cleanly to any tree I had. I think you had a
patch to add battery-voltage-read-enable, but that's not in the tree
yet.
I fixed them up to apply.
Cheers,
Joel
> ---
> arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
> index cfaa606666e2..7ea70c7d698f 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
> @@ -247,7 +247,8 @@ &gpio0 {
> /*C0-C7*/ "","","","","","","","",
> /*D0-D7*/ "","","","","","","","",
> /*E0-E7*/ "","","","","","","","",
> - /*F0-F7*/ "","","battery-voltage-read-enable","","","","factory-reset-toggle","",
> + /*F0-F7*/ "","","battery-voltage-read-enable","reset-cause-pinhole","","",
> + "factory-reset-toggle","",
> /*G0-G7*/ "","","","","","","","",
> /*H0-H7*/ "","bmc-ingraham0","rear-enc-id0","rear-enc-fault0","","","","",
> /*I0-I7*/ "","","","","","","bmc-secure-boot","",
> --
> 2.25.1
>
_______________________________________________
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] 9+ messages in thread
end of thread, other threads:[~2022-01-13 22:37 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-13 21:17 [PATCH 1/2] ARM: dts: aspeed: rainier: add reset-cause-pinhole Andrew Geissler
2022-01-13 21:17 ` Andrew Geissler
2022-01-13 21:17 ` Andrew Geissler
2022-01-13 21:17 ` [PATCH 2/2] ARM: dts: aspeed: everest: " Andrew Geissler
2022-01-13 21:17 ` Andrew Geissler
2022-01-13 21:17 ` Andrew Geissler
2022-01-13 22:35 ` [PATCH 1/2] ARM: dts: aspeed: rainier: " Joel Stanley
2022-01-13 22:35 ` Joel Stanley
2022-01-13 22:35 ` Joel Stanley
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.