* [PATCH] arm64: dts: renesas: ulcb-kf: document a problemw with old firmware
@ 2024-01-30 9:52 Wolfram Sang
2024-02-01 9:21 ` Geert Uytterhoeven
0 siblings, 1 reply; 3+ messages in thread
From: Wolfram Sang @ 2024-01-30 9:52 UTC (permalink / raw)
To: linux-renesas-soc
Cc: Wolfram Sang, Geert Uytterhoeven, Magnus Damm, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, devicetree, linux-kernel
Old firmware set the GPIO to output, so no interrupts could be read.
Newer firmware does this correctly. The exact version could not be
determined, sadly.
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---
arch/arm64/boot/dts/renesas/ulcb-kf.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/renesas/ulcb-kf.dtsi b/arch/arm64/boot/dts/renesas/ulcb-kf.dtsi
index 84b0976503b7..45e55ce760d1 100644
--- a/arch/arm64/boot/dts/renesas/ulcb-kf.dtsi
+++ b/arch/arm64/boot/dts/renesas/ulcb-kf.dtsi
@@ -324,6 +324,7 @@ gpio_exp_77: gpio@77 {
reg = <0x77>;
gpio-controller;
#gpio-cells = <2>;
+ /* If these interrupts don't work, please update your firmware */
interrupt-controller;
interrupt-parent = <&gpio5>;
interrupts = <9 IRQ_TYPE_EDGE_FALLING>;
--
2.39.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] arm64: dts: renesas: ulcb-kf: document a problemw with old firmware
2024-01-30 9:52 [PATCH] arm64: dts: renesas: ulcb-kf: document a problemw with old firmware Wolfram Sang
@ 2024-02-01 9:21 ` Geert Uytterhoeven
2024-02-01 9:32 ` Wolfram Sang
0 siblings, 1 reply; 3+ messages in thread
From: Geert Uytterhoeven @ 2024-02-01 9:21 UTC (permalink / raw)
To: Wolfram Sang
Cc: linux-renesas-soc, Magnus Damm, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, devicetree, linux-kernel
Hi Wolfram,
On Tue, Jan 30, 2024 at 10:52 AM Wolfram Sang
<wsa+renesas@sang-engineering.com> wrote:
> Old firmware set the GPIO to output, so no interrupts could be read.
> Newer firmware does this correctly. The exact version could not be
> determined, sadly.
>
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Thanks for your patch!
> --- a/arch/arm64/boot/dts/renesas/ulcb-kf.dtsi
> +++ b/arch/arm64/boot/dts/renesas/ulcb-kf.dtsi
> @@ -324,6 +324,7 @@ gpio_exp_77: gpio@77 {
> reg = <0x77>;
> gpio-controller;
> #gpio-cells = <2>;
> + /* If these interrupts don't work, please update your firmware */
> interrupt-controller;
> interrupt-parent = <&gpio5>;
> interrupts = <9 IRQ_TYPE_EDGE_FALLING>;
I'd rather not add such comments (especially +6y after the fact), as
this is the standard operating procedure in case something doesn't work.
Do you think we should sprinkle the following, too??
/* If secondary CPUs don't work, please update your firmware */
/* If cpuidle crashes, please update your firmware */
/* If watchdog restart doesn't work, please update your firmware */
/* If PSCI reboot doesn't work, please update your firmware */
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] arm64: dts: renesas: ulcb-kf: document a problemw with old firmware
2024-02-01 9:21 ` Geert Uytterhoeven
@ 2024-02-01 9:32 ` Wolfram Sang
0 siblings, 0 replies; 3+ messages in thread
From: Wolfram Sang @ 2024-02-01 9:32 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: linux-renesas-soc, Magnus Damm, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, devicetree, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 637 bytes --]
Hi Geert,
> I'd rather not add such comments (especially +6y after the fact), as
> this is the standard operating procedure in case something doesn't work.
Okay, can be argued.
> /* If secondary CPUs don't work, please update your firmware */
> /* If cpuidle crashes, please update your firmware */
> /* If watchdog restart doesn't work, please update your firmware */
> /* If PSCI reboot doesn't work, please update your firmware */
Actually, I wouldn't mind. It documents that we *know* updating firmware
helps in this particular case. But as said, I can see your point as
well. Let's drop this patch.
Happy hacking,
Wolfram
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-02-01 9:32 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-01-30 9:52 [PATCH] arm64: dts: renesas: ulcb-kf: document a problemw with old firmware Wolfram Sang
2024-02-01 9:21 ` Geert Uytterhoeven
2024-02-01 9:32 ` Wolfram Sang
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).