Re: [PATCH 5/5] arm64: dts: Add gpio_intc node for Amlogic S7D SoCs

From: Rob Herring <robh@kernel.org>
To: Thomas Gleixner <tglx@linutronix.de>
Cc: Xianwei Zhao <xianwei.zhao@amlogic.com>,
	Neil Armstrong <neil.armstrong@linaro.org>,
	 linux-kernel@vger.kernel.org,
	 Martin Blumenstingl <martin.blumenstingl@googlemail.com>,
	linux-amlogic@lists.infradead.org,
	 Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Kevin Hilman <khilman@baylibre.com>,
	devicetree@vger.kernel.org,
	 Heiner Kallweit <hkallweit1@gmail.com>,
	linux-arm-kernel@lists.infradead.org,
	 Jerome Brunet <jbrunet@baylibre.com>,
	Conor Dooley <conor+dt@kernel.org>
Subject: Re: [PATCH 5/5] arm64: dts: Add gpio_intc node for Amlogic S7D SoCs
Date: Fri, 14 Nov 2025 12:41:01 -0600	[thread overview]
Message-ID: <CAL_Jsq+tqGrSmFiEdyrgnsJ6eQ-gt2K9GVfb2nag_fLUWFcLpw@mail.gmail.com> (raw)
In-Reply-To: <87qzu0bnmi.ffs@tglx>

On Fri, Nov 14, 2025 at 10:11 AM Thomas Gleixner <tglx@linutronix.de> wrote:
>
> On Fri, Nov 14 2025 at 09:58, Rob Herring wrote:
> > On Wed, 05 Nov 2025 17:45:36 +0800, Xianwei Zhao wrote:
>
> > New warnings running 'make CHECK_DTBS=y for arch/arm64/boot/dts/amlogic/' for 20251105-irqchip-gpio-s6-s7-s7d-v1-5-b4d1fe4781c1@amlogic.com:
> >
> > arch/arm64/boot/dts/amlogic/amlogic-s7d-s905x5m-bm202.dtb: interrupt-controller@4080 (amlogic,s7d-gpio-intc): compatible: 'oneOf' conditional failed, one must be fixed:
> >       ['amlogic,s7d-gpio-intc', 'amlogic,meson-gpio-intc'] is too long
> >       'amlogic,meson-gpio-intc' was expected
> >       'amlogic,s7d-gpio-intc' is not one of ['amlogic,meson8-gpio-intc', 'amlogic,meson8b-gpio-intc', 'amlogic,meson-gxbb-gpio-intc', 'amlogic,meson-gxl-gpio-intc', 'amlogic,meson-axg-gpio-intc', 'amlogic,meson-g12a-gpio-intc', 'amlogic,meson-sm1-gpio-intc', 'amlogic,meson-a1-gpio-intc', 'amlogic,meson-s4-gpio-intc', 'amlogic,a4-gpio-intc', 'amlogic,a4-gpio-ao-intc', 'amlogic,a5-gpio-intc', 'amlogic,c3-gpio-intc', 'amlogic,t7-gpio-intc']
> >       from schema $id: http://devicetree.org/schemas/interrupt-controller/amlogic,meson-gpio-intc.yaml
> > arch/arm64/boot/dts/amlogic/amlogic-s7d-s905x5m-bm202.dtb: /soc/bus@fe000000/interrupt-controller@4080: failed to match any schema with compatible: ['amlogic,s7d-gpio-intc', 'amlogic,meson-gpio-intc']
>
> The first patch in this series clearly adds the "missing"
> bindings.
>
> Confused....

You and b4. It looks normal reading the thread, but shazam is
thoroughly confused. I think that's because this series is missing a
'v2' tag.

$ b4 shazam -H 20251105-irqchip-gpio-s6-s7-s7d-v1-3-b4d1fe4781c1@amlogic.com
Grabbing thread from
lore.kernel.org/all/20251105-irqchip-gpio-s6-s7-s7d-v1-3-b4d1fe4781c1@amlogic.com/t.mbox.gz
Breaking thread to remove parents of
20251105-irqchip-gpio-s6-s7-s7d-v1-0-b4d1fe4781c1@amlogic.com
Checking for newer revisions
Grabbing search results from lore.kernel.org
Analyzing 18 messages in the thread
WARNING: duplicate messages found at index 1
   Subject 1: dt-bindings: interrupt-controller: Add support for
Amlogic S6 S7 and S7D SoCs
   Subject 2: dt-bindings: interrupt-controller: Add support for
Amlogic S6 S7 and S7D SoCs
  2 is not a reply... assume additional patch
WARNING: duplicate messages found at index 1
   Subject 1: irqchip/meson-gpio: Add support for Amlogic S6 S7 and S7D SoCs
   Subject 2: dt-bindings: interrupt-controller: Add support for
Amlogic S6 S7 and S7D SoCs
  2 is not a reply... assume additional patch
WARNING: duplicate messages found at index 1
   Subject 1: arm64: dts: amlogic: Add gpio_intc node for Amlogic S6
SoCs
   Subject 2: irqchip/meson-gpio: Add support for Amlogic S6 S7 and S7D SoCs
  2 is not a reply... assume additional patch
WARNING: duplicate messages found at index 1
   Subject 1: arm64: dts: amlogic: Add gpio_intc node for Amlogic S7 SoCs
   Subject 2: arm64: dts: amlogic: Add gpio_intc node for Amlogic S6 SoCs
  2 is not a reply... assume additional patch
WARNING: duplicate messages found at index 1
   Subject 1: arm64: dts: amlogic: Add gpio_intc node for Amlogic S7D SoCs
   Subject 2: arm64: dts: amlogic: Add gpio_intc node for Amlogic S7 SoCs
  2 is not a reply... assume additional patch
Looking for additional code-review trailers on lore.kernel.org
Analyzing 0 code-review messages
Checking attestation on all messages, may take a moment...
---
  ✓ [PATCH] arm64: dts: amlogic: Add gpio_intc node for Amlogic S7D SoCs
    ✓ Signed: DKIM/linutronix.de
  ✓ [PATCH] arm64: dts: amlogic: Add gpio_intc node for Amlogic S7 SoCs
    ✓ Signed: DKIM/linutronix.de
  [PATCH 3/5] arm64: dts: Add gpio_intc node for Amlogic S6 SoCs
    ✗ No key: ed25519/xianwei.zhao@amlogic.com
    + Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org> (✗
DKIM/linaro.org)
  [PATCH 4/5] arm64: dts: Add gpio_intc node for Amlogic S7 SoCs
    ✗ No key: ed25519/xianwei.zhao@amlogic.com
    + Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org> (✗
DKIM/linaro.org)
  [PATCH 5/5] arm64: dts: Add gpio_intc node for Amlogic S7D SoCs
    ✗ No key: ed25519/xianwei.zhao@amlogic.com
    + Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org> (✗
DKIM/linaro.org)
  ERROR: missing [6/5]!
  ERROR: missing [7/5]!
  ERROR: missing [8/5]!
  ERROR: missing [9/5]!
  ERROR: missing [10/5]!
---
Total patches: 5
---
WARNING: Thread incomplete!

_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic