Re: [PATCH 0/6] fsl-mc: Move fsl_over to device MSI

[Christophe Leroy (CS GROUP)]

Hi Marc,

Le 23/02/2026 à 14:37, Marc Zyngier a écrit :
On Fri, 20 Feb 2026 15:41:03 +0000,
Ioana Ciornei <ioana.cior...@nxp.com> wrote:

On Wed, Feb 18, 2026 at 01:51:57PM +0000, Marc Zyngier wrote:
Over the past few years, I have become increasingly annoyed by this
sort of messages in my boot log:

[    0.067861] fsl-mc MSI: ITS@0x100100040000 domain created
[    0.073352] fsl-mc MSI: ITS@0x100100060000 domain created
[    0.078841] fsl-mc MSI: ITS@0x100100080000 domain created
[    0.084328] fsl-mc MSI: ITS@0x1001000a0000 domain created
[    0.089815] fsl-mc MSI: ITS@0x1001000c0000 domain created
[    0.095303] fsl-mc MSI: ITS@0x1001000e0000 domain created
[    0.100792] fsl-mc MSI: ITS@0x100100100000 domain created
[    0.106281] fsl-mc MSI: ITS@0x100100120000 domain created

While this is useful on fsl-mc systems, this is completely irrelevant
on 99.99999% of the arm64 machines, which know nothing about the
Freescale stuff. Including all of my machine -- bar *one*.

Global MSI domains such as the above have been obsoleted for the past
two years, but nobody at NXP seems to have got the message.

I do admit that I wasn't aware that this neeeded work. In the future,
please let me know of this kind of things and I will do the work.

I guess Thomas did too good a job not breaking fsl-mc when both PCI
and platform MSIs were rewritten, hence you missing on the good stuff!



The obvious solution is to delete some code! While my first port of
call would be to just 'git rm -r drivers/bus/fsl-mc' (only kidding!),
a less invasive solution is to drag that code into the present times.
Which is what this series is doing by converting the whole thing to
device MSI, reusing the platform MSI infrastructure instead of
duplicating it.

This results in the expected cleanup, and kills the last user of the
non-device-MSI stuff on arm64. You're welcome.

Thanks!

Tested-by: Ioana Ciornei <ioana.cior...@nxp.com> # LX2160ARDB, LS2088ARDB
Reviewed-by: Ioana Ciornei <ioana.cior...@nxp.com>

Thanks for giving it a go.

Don't know what is your merge plan. I wanted to push it to soc fsl tree 
but it doesn't apply on top of v7.0-rc1:

$ LANG= b4 shazam -l 20260218135203.2267907-1-...@kernel.org
Grabbing thread from 
lore.kernel.org/all/20260218135203.2267907-1-...@kernel.org/t.mbox.gz
Checking for newer revisions
Grabbing search results from lore.kernel.org
Analyzing 14 messages in the thread
Analyzing 0 code-review messages
Checking attestation on all messages, may take a moment...
---
  ✓ [PATCH 1/6] fsl-mc: Remove MSI domain propagation to sub-devices
    + Tested-by: Ioana Ciornei <ioana.cior...@nxp.com> (✓ DKIM/nxp.com)
    + Reviewed-by: Ioana Ciornei <ioana.cior...@nxp.com> (✓ DKIM/nxp.com)
    + Tested-by: Sascha Bischoff <sascha.bisch...@arm.com> (✓ DKIM/arm.com)
    + Link: 
https://lore.kernel.org/r/20260218135203.2267907-2-...@kernel.org
  ✓ [PATCH 2/6] fsl_mc: Add minimal infrastructure to use platform MSI
    + Tested-by: Ioana Ciornei <ioana.cior...@nxp.com> (✓ DKIM/nxp.com)
    + Reviewed-by: Ioana Ciornei <ioana.cior...@nxp.com> (✓ DKIM/nxp.com)
    + Tested-by: Sascha Bischoff <sascha.bisch...@arm.com> (✓ DKIM/arm.com)
    + Link: 
https://lore.kernel.org/r/20260218135203.2267907-3-...@kernel.org
  ✓ [PATCH 3/6] irqchip/gic-v3-its: Add fsl_mc device plumbing to the 
msi-parent handling
    + Tested-by: Ioana Ciornei <ioana.cior...@nxp.com> (✓ DKIM/nxp.com)
    + Reviewed-by: Ioana Ciornei <ioana.cior...@nxp.com> (✓ DKIM/nxp.com)
    + Tested-by: Sascha Bischoff <sascha.bisch...@arm.com> (✓ DKIM/arm.com)
    + Link: 
https://lore.kernel.org/r/20260218135203.2267907-4-...@kernel.org
  ✓ [PATCH 4/6] fsl_mc: Switch over to per-device platform MSI
    + Tested-by: Ioana Ciornei <ioana.cior...@nxp.com> (✓ DKIM/nxp.com)
    + Reviewed-by: Ioana Ciornei <ioana.cior...@nxp.com> (✓ DKIM/nxp.com)
    + Tested-by: Sascha Bischoff <sascha.bisch...@arm.com> (✓ DKIM/arm.com)
    + Link: 
https://lore.kernel.org/r/20260218135203.2267907-5-...@kernel.org
  ✓ [PATCH 5/6] fsl_mc: Remove legacy MSI implementation
    + Tested-by: Ioana Ciornei <ioana.cior...@nxp.com> (✓ DKIM/nxp.com)
    + Reviewed-by: Ioana Ciornei <ioana.cior...@nxp.com> (✓ DKIM/nxp.com)
    + Tested-by: Sascha Bischoff <sascha.bisch...@arm.com> (✓ DKIM/arm.com)
    + Link: 
https://lore.kernel.org/r/20260218135203.2267907-6-...@kernel.org
  ✓ [PATCH 6/6] platform-msi: Remove stale comment
    + Tested-by: Ioana Ciornei <ioana.cior...@nxp.com> (✓ DKIM/nxp.com)
    + Reviewed-by: Ioana Ciornei <ioana.cior...@nxp.com> (✓ DKIM/nxp.com)
    + Tested-by: Sascha Bischoff <sascha.bisch...@arm.com> (✓ DKIM/arm.com)
    + Link: 
https://lore.kernel.org/r/20260218135203.2267907-7-...@kernel.org
  ---
  ✓ Signed: DKIM/kernel.org
---
Total patches: 6
---
Applying: fsl-mc: Remove MSI domain propagation to sub-devices
Applying: fsl_mc: Add minimal infrastructure to use platform MSI
Applying: irqchip/gic-v3-its: Add fsl_mc device plumbing to the 
msi-parent handling
Patch failed at 0003 irqchip/gic-v3-its: Add fsl_mc device plumbing to 
the msi-parent handling
error: patch failed: drivers/irqchip/irq-gic-its-msi-parent.c:186
error: drivers/irqchip/irq-gic-its-msi-parent.c: patch does not apply
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am 
--abort".
hint: Disable this message with "git config set advice.mergeConflict false"


Let me know how you want to proceed.

Thanks
Christophe