linux-renesas-soc.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2] ARM: dts: renesas: kzm9g: name interrupts for accelerometer
@ 2025-10-11 21:06 Wolfram Sang
  2025-10-23  7:07 ` Wolfram Sang
  0 siblings, 1 reply; 3+ messages in thread
From: Wolfram Sang @ 2025-10-11 21:06 UTC (permalink / raw)
  To: linux-renesas-soc
  Cc: Wolfram Sang, Geert Uytterhoeven, Magnus Damm, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, devicetree

Name the interrupts to make them descriptive.

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---

Changes since v1 RFC:
* reworded commit message
* added tag from Geert, thanks!

I'd think we can apply this before my binding update patch goes in.
Because this DT already causes a warning before the update (2 interrupts
not supported). And it will cause a wanring after the binding update
(interrupt-names are required). Or?

 arch/arm/boot/dts/renesas/sh73a0-kzm9g.dts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/renesas/sh73a0-kzm9g.dts b/arch/arm/boot/dts/renesas/sh73a0-kzm9g.dts
index 1ce07d0878dc..0a9cd61bcb5f 100644
--- a/arch/arm/boot/dts/renesas/sh73a0-kzm9g.dts
+++ b/arch/arm/boot/dts/renesas/sh73a0-kzm9g.dts
@@ -209,6 +209,7 @@ accelerometer@1d {
 		reg = <0x1d>;
 		interrupts-extended = <&irqpin3 2 IRQ_TYPE_LEVEL_HIGH>,
 				      <&irqpin3 3 IRQ_TYPE_LEVEL_HIGH>;
+		interrupt-names = "INT1", "INT2";
 	};
 
 	rtc@32 {
-- 
2.47.2


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] ARM: dts: renesas: kzm9g: name interrupts for accelerometer
  2025-10-11 21:06 [PATCH v2] ARM: dts: renesas: kzm9g: name interrupts for accelerometer Wolfram Sang
@ 2025-10-23  7:07 ` Wolfram Sang
  2025-10-23  7:41   ` Geert Uytterhoeven
  0 siblings, 1 reply; 3+ messages in thread
From: Wolfram Sang @ 2025-10-23  7:07 UTC (permalink / raw)
  To: linux-renesas-soc
  Cc: Geert Uytterhoeven, Magnus Damm, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, devicetree

[-- Attachment #1: Type: text/plain, Size: 105 bytes --]


> I'd think we can apply this before my binding update patch goes in.

Binding update is in -next now.


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] ARM: dts: renesas: kzm9g: name interrupts for accelerometer
  2025-10-23  7:07 ` Wolfram Sang
@ 2025-10-23  7:41   ` Geert Uytterhoeven
  0 siblings, 0 replies; 3+ messages in thread
From: Geert Uytterhoeven @ 2025-10-23  7:41 UTC (permalink / raw)
  To: Wolfram Sang
  Cc: linux-renesas-soc, Magnus Damm, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, devicetree

Hi Wolfram,

On Thu, 23 Oct 2025 at 09:07, Wolfram Sang
<wsa+renesas@sang-engineering.com> wrote:
> > I'd think we can apply this before my binding update patch goes in.
>
> Binding update is in -next now.

Thanks for notifying me, I had missed that, as adxl345 is not one of
the patterns I keep an eye on.
Will queue in renesas-devel for v6.19.

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

end of thread, other threads:[~2025-10-23  7:41 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-10-11 21:06 [PATCH v2] ARM: dts: renesas: kzm9g: name interrupts for accelerometer Wolfram Sang
2025-10-23  7:07 ` Wolfram Sang
2025-10-23  7:41   ` Geert Uytterhoeven

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).