From: Jonas Gorski <jonas.gorski@gmail.com>
To: "Kamal Dasu" <kamal.dasu@broadcom.com>,
"Broadcom internal kernel review list"
<bcm-kernel-feedback-list@broadcom.com>,
"Mark Brown" <broonie@kernel.org>,
"Rob Herring" <robh@kernel.org>,
"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
"Conor Dooley" <conor+dt@kernel.org>,
"Florian Fainelli" <florian.fainelli@broadcom.com>,
"Thomas Bogendoerfer" <tsbogend@alpha.franken.de>,
"Rafał Miłecki" <rafal@milecki.pl>
Cc: Kamal Dasu <kdasu.kdev@gmail.com>,
linux-spi@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-mips@vger.kernel.org
Subject: [PATCH 3/4] mips: bmips: dts: fix qspi interrupt order
Date: Thu, 25 Dec 2025 11:25:32 +0100 [thread overview]
Message-ID: <20251225102533.30772-4-jonas.gorski@gmail.com> (raw)
In-Reply-To: <20251225102533.30772-1-jonas.gorski@gmail.com>
The binding for brcm,spi-bcm-qspi requires the interrupt-names in a
certain order, so reorder the interrupts to match it.
Fixes warnings like:
arch/mips/boot/dts/brcm/bcm97435svmb.dtb: spi@41d200 (brcm,spi-brcmstb-qspi): interrupt-names: 'oneOf' conditional failed, one must be fixed:
['spi_lr_fullness_reached', 'spi_lr_session_aborted', 'spi_lr_impatient', 'spi_lr_session_done', 'spi_lr_overread', 'mspi_done', 'mspi_halted'] is too long
'mspi_done' was expected
'spi_l1_intr' was expected
'mspi_halted' was expected
'spi_lr_fullness_reached' was expected
'spi_lr_session_aborted' was expected
'spi_lr_impatient' was expected
'spi_lr_session_done' was expected
'spi_lr_overread' was expected
from schema $id: http://devicetree.org/schemas/spi/brcm,spi-bcm-qspi.yaml
Signed-off-by: Jonas Gorski <jonas.gorski@gmail.com>
---
arch/mips/boot/dts/brcm/bcm7125.dtsi | 10 +++++-----
arch/mips/boot/dts/brcm/bcm7346.dtsi | 10 +++++-----
arch/mips/boot/dts/brcm/bcm7358.dtsi | 10 +++++-----
arch/mips/boot/dts/brcm/bcm7360.dtsi | 10 +++++-----
arch/mips/boot/dts/brcm/bcm7362.dtsi | 10 +++++-----
arch/mips/boot/dts/brcm/bcm7420.dtsi | 10 +++++-----
arch/mips/boot/dts/brcm/bcm7425.dtsi | 10 +++++-----
arch/mips/boot/dts/brcm/bcm7435.dtsi | 10 +++++-----
8 files changed, 40 insertions(+), 40 deletions(-)
diff --git a/arch/mips/boot/dts/brcm/bcm7125.dtsi b/arch/mips/boot/dts/brcm/bcm7125.dtsi
index 92963a1998b7..dd1cc3cda694 100644
--- a/arch/mips/boot/dts/brcm/bcm7125.dtsi
+++ b/arch/mips/boot/dts/brcm/bcm7125.dtsi
@@ -252,15 +252,15 @@ qspi: spi@443000 {
clocks = <&upg_clk>;
reg = <0x440920 0x4 0x443200 0x188 0x443000 0x50>;
reg-names = "cs_reg", "hif_mspi", "bspi";
- interrupts = <0x0 0x1 0x2 0x3 0x4 0x5 0x6>;
+ interrupts = <0x5 0x6 0x0 0x1 0x2 0x3 0x4>;
interrupt-parent = <&spi_l2_intc>;
- interrupt-names = "spi_lr_fullness_reached",
+ interrupt-names = "mspi_done",
+ "mspi_halted",
+ "spi_lr_fullness_reached",
"spi_lr_session_aborted",
"spi_lr_impatient",
"spi_lr_session_done",
- "spi_lr_overread",
- "mspi_done",
- "mspi_halted";
+ "spi_lr_overread";
status = "disabled";
};
diff --git a/arch/mips/boot/dts/brcm/bcm7346.dtsi b/arch/mips/boot/dts/brcm/bcm7346.dtsi
index 42125684b523..21c7418aaf0d 100644
--- a/arch/mips/boot/dts/brcm/bcm7346.dtsi
+++ b/arch/mips/boot/dts/brcm/bcm7346.dtsi
@@ -476,15 +476,15 @@ qspi: spi@413000 {
clocks = <&upg_clk>;
reg = <0x410920 0x4 0x413200 0x188 0x413000 0x50>;
reg-names = "cs_reg", "hif_mspi", "bspi";
- interrupts = <0x0 0x1 0x2 0x3 0x4 0x5 0x6>;
+ interrupts = <0x5 0x6 0x0 0x1 0x2 0x3 0x4>;
interrupt-parent = <&spi_l2_intc>;
- interrupt-names = "spi_lr_fullness_reached",
+ interrupt-names = "mspi_done",
+ "mspi_halted",
+ "spi_lr_fullness_reached",
"spi_lr_session_aborted",
"spi_lr_impatient",
"spi_lr_session_done",
- "spi_lr_overread",
- "mspi_done",
- "mspi_halted";
+ "spi_lr_overread";
status = "disabled";
};
diff --git a/arch/mips/boot/dts/brcm/bcm7358.dtsi b/arch/mips/boot/dts/brcm/bcm7358.dtsi
index fdc52acab8c8..63b3a42a8dc6 100644
--- a/arch/mips/boot/dts/brcm/bcm7358.dtsi
+++ b/arch/mips/boot/dts/brcm/bcm7358.dtsi
@@ -344,15 +344,15 @@ qspi: spi@413000 {
clocks = <&upg_clk>;
reg = <0x410920 0x4 0x413200 0x188 0x413000 0x50>;
reg-names = "cs_reg", "hif_mspi", "bspi";
- interrupts = <0x0 0x1 0x2 0x3 0x4 0x5 0x6>;
+ interrupts = <0x5 0x6 0x0 0x1 0x2 0x3 0x4>;
interrupt-parent = <&spi_l2_intc>;
- interrupt-names = "spi_lr_fullness_reached",
+ interrupt-names = "mspi_done",
+ "mspi_halted",
+ "spi_lr_fullness_reached",
"spi_lr_session_aborted",
"spi_lr_impatient",
"spi_lr_session_done",
- "spi_lr_overread",
- "mspi_done",
- "mspi_halted";
+ "spi_lr_overread";
status = "disabled";
};
diff --git a/arch/mips/boot/dts/brcm/bcm7360.dtsi b/arch/mips/boot/dts/brcm/bcm7360.dtsi
index e8b0b0ff7588..9d89da400104 100644
--- a/arch/mips/boot/dts/brcm/bcm7360.dtsi
+++ b/arch/mips/boot/dts/brcm/bcm7360.dtsi
@@ -395,15 +395,15 @@ qspi: spi@413000 {
clocks = <&upg_clk>;
reg = <0x410920 0x4 0x413200 0x188 0x413000 0x50>;
reg-names = "cs_reg", "hif_mspi", "bspi";
- interrupts = <0x0 0x1 0x2 0x3 0x4 0x5 0x6>;
+ interrupts = <0x5 0x6 0x0 0x1 0x2 0x3 0x4>;
interrupt-parent = <&spi_l2_intc>;
- interrupt-names = "spi_lr_fullness_reached",
+ interrupt-names = "mspi_done",
+ "mspi_halted",
+ "spi_lr_fullness_reached",
"spi_lr_session_aborted",
"spi_lr_impatient",
"spi_lr_session_done",
- "spi_lr_overread",
- "mspi_done",
- "mspi_halted";
+ "spi_lr_overread";
status = "disabled";
};
diff --git a/arch/mips/boot/dts/brcm/bcm7362.dtsi b/arch/mips/boot/dts/brcm/bcm7362.dtsi
index a1ed0b193389..a0d15fcd3b81 100644
--- a/arch/mips/boot/dts/brcm/bcm7362.dtsi
+++ b/arch/mips/boot/dts/brcm/bcm7362.dtsi
@@ -391,15 +391,15 @@ qspi: spi@413000 {
clocks = <&upg_clk>;
reg = <0x410920 0x4 0x413200 0x188 0x413000 0x50>;
reg-names = "cs_reg", "hif_mspi", "bspi";
- interrupts = <0x0 0x1 0x2 0x3 0x4 0x5 0x6>;
+ interrupts = <0x5 0x6 0x0 0x1 0x2 0x3 0x4>;
interrupt-parent = <&spi_l2_intc>;
- interrupt-names = "spi_lr_fullness_reached",
+ interrupt-names = "mspi_done",
+ "mspi_halted",
+ "spi_lr_fullness_reached",
"spi_lr_session_aborted",
"spi_lr_impatient",
"spi_lr_session_done",
- "spi_lr_overread",
- "mspi_done",
- "mspi_halted";
+ "spi_lr_overread";
status = "disabled";
};
diff --git a/arch/mips/boot/dts/brcm/bcm7420.dtsi b/arch/mips/boot/dts/brcm/bcm7420.dtsi
index 62e7e1602d4e..a38041451faf 100644
--- a/arch/mips/boot/dts/brcm/bcm7420.dtsi
+++ b/arch/mips/boot/dts/brcm/bcm7420.dtsi
@@ -313,15 +313,15 @@ qspi: spi@443000 {
clocks = <&upg_clk>;
reg = <0x440920 0x4 0x443200 0x188 0x443000 0x50>;
reg-names = "cs_reg", "hif_mspi", "bspi";
- interrupts = <0x0 0x1 0x2 0x3 0x4 0x5 0x6>;
+ interrupts = <0x5 0x6 0x0 0x1 0x2 0x3 0x4>;
interrupt-parent = <&spi_l2_intc>;
- interrupt-names = "spi_lr_fullness_reached",
+ interrupt-names = "mspi_done",
+ "mspi_halted",
+ "spi_lr_fullness_reached",
"spi_lr_session_aborted",
"spi_lr_impatient",
"spi_lr_session_done",
- "spi_lr_overread",
- "mspi_done",
- "mspi_halted";
+ "spi_lr_overread";
status = "disabled";
};
diff --git a/arch/mips/boot/dts/brcm/bcm7425.dtsi b/arch/mips/boot/dts/brcm/bcm7425.dtsi
index 71422fd2ecc9..c2b3ea21410d 100644
--- a/arch/mips/boot/dts/brcm/bcm7425.dtsi
+++ b/arch/mips/boot/dts/brcm/bcm7425.dtsi
@@ -487,15 +487,15 @@ qspi: spi@41c000 {
clocks = <&upg_clk>;
reg = <0x419920 0x4 0x41c200 0x188 0x41c000 0x50>;
reg-names = "cs_reg", "hif_mspi", "bspi";
- interrupts = <0x0 0x1 0x2 0x3 0x4 0x5 0x6>;
+ interrupts = <0x5 0x6 0x0 0x1 0x2 0x3 0x4>;
interrupt-parent = <&spi_l2_intc>;
- interrupt-names = "spi_lr_fullness_reached",
+ interrupt-names = "mspi_done",
+ "mspi_halted",
+ "spi_lr_fullness_reached",
"spi_lr_session_aborted",
"spi_lr_impatient",
"spi_lr_session_done",
- "spi_lr_overread",
- "mspi_done",
- "mspi_halted";
+ "spi_lr_overread";
status = "disabled";
};
diff --git a/arch/mips/boot/dts/brcm/bcm7435.dtsi b/arch/mips/boot/dts/brcm/bcm7435.dtsi
index 38eea5373b66..826731515f0b 100644
--- a/arch/mips/boot/dts/brcm/bcm7435.dtsi
+++ b/arch/mips/boot/dts/brcm/bcm7435.dtsi
@@ -503,15 +503,15 @@ qspi: spi@41d200 {
clocks = <&upg_clk>;
reg = <0x41a920 0x4 0x41d400 0x188 0x41d200 0x50>;
reg-names = "cs_reg", "hif_mspi", "bspi";
- interrupts = <0x0 0x1 0x2 0x3 0x4 0x5 0x6>;
+ interrupts = <0x5 0x6 0x0 0x1 0x2 0x3 0x4>;
interrupt-parent = <&spi_l2_intc>;
- interrupt-names = "spi_lr_fullness_reached",
+ interrupt-names = "mspi_done",
+ "mspi_halted",
+ "spi_lr_fullness_reached",
"spi_lr_session_aborted",
"spi_lr_impatient",
"spi_lr_session_done",
- "spi_lr_overread",
- "mspi_done",
- "mspi_halted";
+ "spi_lr_overread";
status = "disabled";
};
--
2.43.0
next prev parent reply other threads:[~2025-12-25 10:26 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-12-25 10:25 [PATCH 0/4] mips: bmips: fix brcm,spi-bcm-qspi devicetree usage Jonas Gorski
2025-12-25 10:25 ` [PATCH 1/4] dt-bindings: brcm,spi-bcm-qspi: allow hif_mspi as alternative for mspi Jonas Gorski
2025-12-25 12:32 ` Krzysztof Kozlowski
2025-12-25 12:34 ` Krzysztof Kozlowski
2025-12-25 10:25 ` [PATCH 2/4] mips: bmips: dts: fix {m,q}spi compatible order Jonas Gorski
2025-12-25 10:25 ` Jonas Gorski [this message]
2025-12-25 10:25 ` [PATCH 4/4] mips: bmips: dts: fix qspi register order Jonas Gorski
2025-12-25 12:35 ` Krzysztof Kozlowski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20251225102533.30772-4-jonas.gorski@gmail.com \
--to=jonas.gorski@gmail.com \
--cc=bcm-kernel-feedback-list@broadcom.com \
--cc=broonie@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=florian.fainelli@broadcom.com \
--cc=kamal.dasu@broadcom.com \
--cc=kdasu.kdev@gmail.com \
--cc=krzk+dt@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mips@vger.kernel.org \
--cc=linux-spi@vger.kernel.org \
--cc=rafal@milecki.pl \
--cc=robh@kernel.org \
--cc=tsbogend@alpha.franken.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox