mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH 0/2] arm64: errata: Fix two Kconfig column entries
@ 2026-11-06 23:56 TAKANO Mitsuhiro
  2026-11-06 23:56 ` [PATCH 1/2] arm64: errata: Mark GIC-700 #2941627 N/A in silicon-errata TAKANO Mitsuhiro
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: TAKANO Mitsuhiro @ 2026-11-06 23:56 UTC (permalink / raw)
  To: Catalin Marinas, Will Deacon
  Cc: Mark Rutland, Jonathan Corbet, linux-arm-kernel,
	Lorenzo Pieralisi, linux-doc, linux-kernel, TAKANO Mitsuhiro

Two entries in the arm64 silicon-errata table name Kconfig symbols that
do not exist.  The GIC-700 #2941627 row names ARM64_ERRATUM_2941627, but
the workaround has no Kconfig option at all; it is enabled from
irq-gic-v3.c through gic_enable_quirks().  The Falkor E1041 row spells
QCOM_FALKOR_ERRATUM_E1041 without the E.

The first patch marks the GIC-700 row "N/A", matching other workarounds
that have no option.  The second corrects the Falkor symbol name.  Both
changes touch only the text of the table; no Kconfig symbol or code
changes, and no build was run.

TAKANO Mitsuhiro (2):
  arm64: errata: Mark GIC-700 #2941627 N/A in silicon-errata
  arm64: errata: Fix Falkor E1041 Kconfig name in silicon-errata

 Documentation/arch/arm64/silicon-errata.rst | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

base-commit: d48723711bae9b8cdc56719c448ba42fef28cbee
-- 
2.53.0


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

* [PATCH 1/2] arm64: errata: Mark GIC-700 #2941627 N/A in silicon-errata
  2026-11-06 23:56 [PATCH 0/2] arm64: errata: Fix two Kconfig column entries TAKANO Mitsuhiro
@ 2026-11-06 23:56 ` TAKANO Mitsuhiro
  2026-11-06 23:56 ` [PATCH 2/2] arm64: errata: Fix Falkor E1041 Kconfig name " TAKANO Mitsuhiro
  2026-11-06 23:59 ` [PATCH 0/2] arm64: errata: Fix two Kconfig column entries TAKANO Mitsuhiro
  2 siblings, 0 replies; 5+ messages in thread
From: TAKANO Mitsuhiro @ 2026-11-06 23:56 UTC (permalink / raw)
  To: Catalin Marinas, Will Deacon
  Cc: Mark Rutland, Jonathan Corbet, linux-arm-kernel,
	Lorenzo Pieralisi, linux-doc, linux-kernel, TAKANO Mitsuhiro

The GIC-700 #2941627 entry in the arm64 silicon-errata table names
ARM64_ERRATUM_2941627 in its Kconfig column, but no Kconfig file defines
that symbol and no code refers to it, so a reader who follows the name
finds nothing at the other end.

The workaround has no Kconfig option: irq-gic-v3.c passes the GICD_IIDR
value to gic_enable_quirks(), which matches it against gic_quirks[], and
gic_enable_quirk_arm64_2941627() enables a static branch.  The table
already records workarounds of that kind as "N/A", for example NVIDIA
T241-FABRIC-4, a GIC workaround like this one.

Set the Kconfig column to "N/A", as the table does for workarounds
that have no option.  The row stays because this file acts as the
registry of software workarounds in the kernel: deleting it would drop
the table's only pointer to gic_enable_quirk_arm64_2941627().

Only the text of the table changes, and no build was run.

Fixes: 6fe5c68ee6a1 ("irqchip/gic-v3: Workaround for GIC-700 erratum 2941627")
Signed-off-by: TAKANO Mitsuhiro <takano32@gmail.com>
---
 Documentation/arch/arm64/silicon-errata.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/arch/arm64/silicon-errata.rst b/Documentation/arch/arm64/silicon-errata.rst
index afe109afe692..afea5e57ab95 100644
--- a/Documentation/arch/arm64/silicon-errata.rst
+++ b/Documentation/arch/arm64/silicon-errata.rst
@@ -279,7 +279,7 @@ stable kernels.
 | ARM            | MMU S3          | #3995052,       | N/A                         |
 |                |                 | #3673557        |                             |
 +----------------+-----------------+-----------------+-----------------------------+
-| ARM            | GIC-700         | #2941627        | ARM64_ERRATUM_2941627       |
+| ARM            | GIC-700         | #2941627        | N/A                         |
 +----------------+-----------------+-----------------+-----------------------------+
 | ARM            | SI L1           | #4311569        | ARM64_ERRATUM_4311569       |
 +----------------+-----------------+-----------------+-----------------------------+
-- 
2.53.0


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

* [PATCH 2/2] arm64: errata: Fix Falkor E1041 Kconfig name in silicon-errata
  2026-11-06 23:56 [PATCH 0/2] arm64: errata: Fix two Kconfig column entries TAKANO Mitsuhiro
  2026-11-06 23:56 ` [PATCH 1/2] arm64: errata: Mark GIC-700 #2941627 N/A in silicon-errata TAKANO Mitsuhiro
@ 2026-11-06 23:56 ` TAKANO Mitsuhiro
  2026-11-06 23:59 ` [PATCH 0/2] arm64: errata: Fix two Kconfig column entries TAKANO Mitsuhiro
  2 siblings, 0 replies; 5+ messages in thread
From: TAKANO Mitsuhiro @ 2026-11-06 23:56 UTC (permalink / raw)
  To: Catalin Marinas, Will Deacon
  Cc: Mark Rutland, Jonathan Corbet, linux-arm-kernel,
	Lorenzo Pieralisi, linux-doc, linux-kernel, TAKANO Mitsuhiro

The Falkor E1041 entry in the arm64 silicon-errata table names
QCOM_FALKOR_ERRATUM_1041, but the symbol defined in arch/arm64/Kconfig
and used in arch/arm64/include/asm/assembler.h is
QCOM_FALKOR_ERRATUM_E1041.  Correct the name in the table.

The E1003 and E1009 rows above name QCOM_FALKOR_ERRATUM_1003 and
QCOM_FALKOR_ERRATUM_1009, without the E, so a missing E in the E1041 row
looked right; only the E1041 symbol keeps it.

The Kconfig symbol stays as it is; only the table is wrong.

Only the text of the table changes, and no build was run.

Fixes: 932b50c7c1c6 ("arm64: Add software workaround for Falkor erratum 1041")
Signed-off-by: TAKANO Mitsuhiro <takano32@gmail.com>
---
 Documentation/arch/arm64/silicon-errata.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/arch/arm64/silicon-errata.rst b/Documentation/arch/arm64/silicon-errata.rst
index afea5e57ab95..f554fa04662c 100644
--- a/Documentation/arch/arm64/silicon-errata.rst
+++ b/Documentation/arch/arm64/silicon-errata.rst
@@ -360,7 +360,7 @@ stable kernels.
 +----------------+-----------------+-----------------+-----------------------------+
 | Qualcomm Tech. | QDF2400 ITS     | E0065           | QCOM_QDF2400_ERRATUM_0065   |
 +----------------+-----------------+-----------------+-----------------------------+
-| Qualcomm Tech. | Falkor v{1,2}   | E1041           | QCOM_FALKOR_ERRATUM_1041    |
+| Qualcomm Tech. | Falkor v{1,2}   | E1041           | QCOM_FALKOR_ERRATUM_E1041   |
 +----------------+-----------------+-----------------+-----------------------------+
 | Qualcomm Tech. | Kryo4xx Gold    | N/A             | ARM64_ERRATUM_1463225       |
 +----------------+-----------------+-----------------+-----------------------------+
-- 
2.53.0


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

* Re: [PATCH 0/2] arm64: errata: Fix two Kconfig column entries
  2026-11-06 23:56 [PATCH 0/2] arm64: errata: Fix two Kconfig column entries TAKANO Mitsuhiro
  2026-11-06 23:56 ` [PATCH 1/2] arm64: errata: Mark GIC-700 #2941627 N/A in silicon-errata TAKANO Mitsuhiro
  2026-11-06 23:56 ` [PATCH 2/2] arm64: errata: Fix Falkor E1041 Kconfig name " TAKANO Mitsuhiro
@ 2026-11-06 23:59 ` TAKANO Mitsuhiro
  2 siblings, 0 replies; 5+ messages in thread
From: TAKANO Mitsuhiro @ 2026-11-06 23:59 UTC (permalink / raw)
  To: Catalin Marinas, Will Deacon
  Cc: Mark Rutland, Jonathan Corbet, linux-arm-kernel,
	Lorenzo Pieralisi, linux-doc, linux-kernel, TAKANO Mitsuhiro

Please disregard this copy -- it is a duplicate of the series sent
earlier as <20261106234541.1474324-1-takano32@gmail.com>.

Sorry for the noise.

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

* [PATCH 0/2] arm64: errata: Fix two Kconfig column entries
@ 2026-11-06 23:45 TAKANO Mitsuhiro
  0 siblings, 0 replies; 5+ messages in thread
From: TAKANO Mitsuhiro @ 2026-11-06 23:45 UTC (permalink / raw)
  To: Catalin Marinas, Will Deacon
  Cc: Mark Rutland, Jonathan Corbet, linux-arm-kernel,
	Lorenzo Pieralisi, linux-doc, linux-kernel, TAKANO Mitsuhiro

Documentation/arch/arm64/silicon-errata.rst names two Kconfig symbols
that no Kconfig file defines.  I checked every Kconfig name in the
table: these are the only two.

The GIC-700 #2941627 entry names ARM64_ERRATUM_2941627; that workaround
is a GIC driver quirk, matched at runtime against the GIC's IIDR, so
there is no symbol to name.  The Falkor E1041 entry names
QCOM_FALKOR_ERRATUM_1041; the symbol defined in the tree is
QCOM_FALKOR_ERRATUM_E1041.

The two rows come from different commits, so they are kept separate and
each carries its own Fixes: tag.  Both are documentation-only changes:
the table keeps its column widths and no build was run.  I did not add
a Cc: stable tag, as nothing outside Documentation changes.

The changes and the changelog text were produced with an AI coding
assistant (Hermes Agent, model deepseek-v4.1-flash) working from the
tree and the git history.  I reviewed the diff and the reasoning, and I
am responsible for the submission.

TAKANO Mitsuhiro (2):
  arm64: errata: Set GIC-700 #2941627 Kconfig entry to N/A
  arm64: errata: Fix Falkor E1041 Kconfig name

 Documentation/arch/arm64/silicon-errata.rst | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)


base-commit: d48723711bae9b8cdc56719c448ba42fef28cbee
-- 
2.53.0


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

end of thread, other threads:[~2026-11-06 23:59 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-11-06 23:56 [PATCH 0/2] arm64: errata: Fix two Kconfig column entries TAKANO Mitsuhiro
2026-11-06 23:56 ` [PATCH 1/2] arm64: errata: Mark GIC-700 #2941627 N/A in silicon-errata TAKANO Mitsuhiro
2026-11-06 23:56 ` [PATCH 2/2] arm64: errata: Fix Falkor E1041 Kconfig name " TAKANO Mitsuhiro
2026-11-06 23:59 ` [PATCH 0/2] arm64: errata: Fix two Kconfig column entries TAKANO Mitsuhiro
  -- strict thread matches above, loose matches on Subject: below --
2026-11-06 23:45 TAKANO Mitsuhiro

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®