mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Christophe Leroy (CS GROUP)" <chleroy@kernel.org>
To: Arnd Bergmann <arnd@arndb.de>, soc@kernel.org
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Ioana Ciornei <ioana.ciornei@nxp.com>
Subject: Re: [GIT PULL] SOC FSL for 6.20
Date: Tue, 14 Apr 2026 17:13:05 +0200	[thread overview]
Message-ID: <2755af3f-d7d6-49b8-88f5-66352c237c35@kernel.org> (raw)
In-Reply-To: <a7d13f9e-7a95-4464-abe8-c095354f9908@kernel.org>



Le 14/04/2026 à 17:02, Christophe Leroy (CS GROUP) a écrit :
> 
> 
> Le 14/04/2026 à 16:55, Arnd Bergmann a écrit :
>> On Tue, Apr 14, 2026, at 16:08, Christophe Leroy (CS GROUP) wrote:
>>> Hi Arnd,
>>>
>>> Please pull the following Freescale Soc Drivers changes for 7.1
>>>
>>> Changes to fsl-mc bus are reviewed or acked by Ioana Ciornei.
>>
>> Hi Christophe,
>>
>> This is really too late for 7.1: the merge window is
>> already open, and I see that all the commits have a committer
>> date of today, though some appear to be freshly rebased
>> and had an earlier version in linux-next.
>>
>> I don't think any of these are urgent bugfixes, so I'd
>> suggest you rebase onto 7.1-rc1 and send these again for
>> 7.2.
> 
> Uhm .... Maybe I made it wrong. All those changes have spent quite some 
> time in linux-next but the branch was based on the tag soc_fsl-7.0-2 
> that you merged into 7.0-rc5.
> This morning I fetched soc tree and saw that the soc/driver branch was 
> based on rc1, so to avoid pulling 7.0-rc5 I rebased the changes on 7.0-rc1.
> 
> Should I restore the original branch as it was in Linux-next until this 
> morning and send a new pull request ?
> 

FWIW, here is the log of the original branch as it was in linux next 
until this morning before the rebase, with commit dates:

commit 208858b1b48eba83d073542372329cf8ed606526
Author:     Ioana Ciornei <ioana.ciornei@nxp.com>
AuthorDate: Wed Apr 1 17:45:08 2026 +0300
Commit:     Christophe Leroy (CS GROUP) <chleroy@kernel.org>
CommitDate: Sat Apr 4 18:54:05 2026 +0200

     bus: fsl-mc: wait for the MC firmware to complete its boot

     There are use cases in which the Management Complex firmware boot
     process is started by the bootloader which does not wait for the 
boot to
     complete. This is mainly done in order to reduce the overall boot time
     of a DPAA2 based SoC.

     In this kind of circumstance, the fsl-mc bus driver needs to make sure
     that the MC firmware boot process is finished before proceeding to the
     usual operations such as interrogating the firmware to gather all
     existent DPAA2 objects, creating the fsl-mc devices on the bus etc.

     Add this kind of check early in the boot process of the fsl-mc bus and
     defer the probe in case the firmware is still in its boot process.

     Signed-off-by: Ioana Ciornei <ioana.ciornei@nxp.com>
     Link: 
https://lore.kernel.org/r/20260401144508.3062019-1-ioana.ciornei@nxp.com
     Signed-off-by: Christophe Leroy (CS GROUP) <chleroy@kernel.org>

commit a0fe29d20e7822182e12324905af5115c1b3aed3
Author:     Wang Jun <1742789905@qq.com>
AuthorDate: Fri Mar 27 08:12:25 2026 +0800
Commit:     Christophe Leroy (CS GROUP) <chleroy@kernel.org>
CommitDate: Sat Apr 4 18:52:09 2026 +0200

     soc: fsl: qe: panic on ioremap() failure in qe_reset()

     When ioremap() fails in qe_reset(), the global pointer qe_immr remains
     NULL, leading to a subsequent NULL pointer dereference when the pointer
     is accessed. Since this happens early in the boot process, a failure to
     map a few bytes of I/O memory indicates a fatal error from which the
     system cannot recover.

     Follow the same pattern as qe_sdma_init() and panic immediately when
     ioremap() fails. This avoids a silent NULL pointer dereference later
     and makes the error explicit.

     Fixes: 986585385131 ("[POWERPC] Add QUICC Engine (QE) infrastructure")
     Cc: stable@vger.kernel.org
     Signed-off-by: Wang Jun <1742789905@qq.com>
     Link: 
https://lore.kernel.org/r/tencent_FED49CF5331CC0C7910618883332A08E2606@qq.com
     [chleroy: Rearranged change to reduce churn]
     Signed-off-by: Christophe Leroy (CS GROUP) <chleroy@kernel.org>

commit 7b9233b25131fa54ef9c64ebbe95aef6ad3de47c
Author:     Christophe Leroy (CS GROUP) <chleroy@kernel.org>
AuthorDate: Tue Mar 24 23:34:16 2026 +0100
Commit:     Christophe Leroy (CS GROUP) <chleroy@kernel.org>
CommitDate: Wed Mar 25 11:24:07 2026 +0100

     soc: fsl: qe_ports_ic: switch to irq_domain_create_linear()

     irq_domain_add_linear() is about to be removed, replace by the
     more generic irq_domain_create_linear(),
     see commit 42b8b16fe56c ("irqdomain: Drop irq_domain_add_*()
     functions") for details.

     Link: 
https://lore.kernel.org/r/a9de2f351ea71e4b794baaea8d9d790fbfac8d26.1774391374.git.chleroy@kernel.org
     Signed-off-by: Christophe Leroy (CS GROUP) <chleroy@kernel.org>

commit 7dad18a179741dbad9f40799e549fa9111987c0c
Author:     Felix Gu <ustc.gu@gmail.com>
AuthorDate: Tue Mar 10 00:25:30 2026 +0800
Commit:     Christophe Leroy (CS GROUP) <chleroy@kernel.org>
CommitDate: Fri Mar 13 10:13:55 2026 +0100

     soc: fsl: qe_ports_ic: Add missing cleanup on device removal

     Add a devm action handler to properly clean up the irq_domain and
     chained handler when the device is removed.

     Fixes: f0bcd784e1b7 ("soc: fsl: qe: Add an interrupt controller for 
QUICC Engine Ports")
     Signed-off-by: Felix Gu <ustc.gu@gmail.com>
     Link: 
https://lore.kernel.org/r/20260310-qe_ports_ic-v1-1-608293026561@gmail.com
     Signed-off-by: Christophe Leroy (CS GROUP) <chleroy@kernel.org>

commit cf8e9203cc653f0a82639f7ce8089fa92afe6739
Author:     Randy Dunlap <rdunlap@infradead.org>
AuthorDate: Thu Feb 26 14:09:42 2026 -0800
Commit:     Christophe Leroy (CS GROUP) <chleroy@kernel.org>
CommitDate: Fri Feb 27 07:28:08 2026 +0100

     virt: fsl_hypervisor: fix header kernel-doc warnings

     Correct struct member names to placate kernel-doc warnings:

     Warning: include/uapi/linux/fsl_hypervisor.h:148 struct member
      'local_vaddr' not described in 'fsl_hv_ioctl_memcpy'
     Warning: include/uapi/linux/fsl_hypervisor.h:148 struct member
      'remote_paddr' not described in 'fsl_hv_ioctl_memcpy'

     Fixes: 6db7199407ca ("drivers/virt: introduce Freescale hypervisor 
management driver")
     Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
     Link: 
https://lore.kernel.org/r/20260226220942.1035295-1-rdunlap@infradead.org
     Signed-off-by: Christophe Leroy (CS GROUP) <chleroy@kernel.org>

commit f0a2eac6a597268034fd40d92c1469182438b53d
Author:     Marc Zyngier <maz@kernel.org>
AuthorDate: Tue Feb 24 10:09:36 2026 +0000
Commit:     Christophe Leroy (CS GROUP) <chleroy@kernel.org>
CommitDate: Thu Feb 26 10:48:19 2026 +0100

     platform-msi: Remove stale comment

     The backward compatibility code for the previous incarnation of
     platform MSI was removed in e9894248994ca ("genirq/msi: Remove
     platform MSI leftovers"), but the comment about that removal is
     still present.  Remove it.

     Reviewed-by: Ioana Ciornei <ioana.ciornei@nxp.com>
     Tested-by: Ioana Ciornei <ioana.ciornei@nxp.com> # LX2160ARDB, 
LS2088ARDB
     Tested-by: Sascha Bischoff <sascha.bischoff@arm.com>
     Signed-off-by: Marc Zyngier <maz@kernel.org>
     Acked-by: Thomas Gleixner <tglx@kernel.org>
     Link: https://lore.kernel.org/r/20260224100936.3752303-7-maz@kernel.org
     Signed-off-by: Christophe Leroy (CS GROUP) <chleroy@kernel.org>

commit 14b1cbcc6cec0b02298f4adf717646cd943b7ef6
Author:     Marc Zyngier <maz@kernel.org>
AuthorDate: Tue Feb 24 10:09:35 2026 +0000
Commit:     Christophe Leroy (CS GROUP) <chleroy@kernel.org>
CommitDate: Thu Feb 26 10:48:18 2026 +0100

     fsl-mc: Remove legacy MSI implementation

     Get rid of most of the fsl_mc MSI infrastructure, which is now replaced
     by common code.

     Reviewed-by: Ioana Ciornei <ioana.ciornei@nxp.com>
     Tested-by: Ioana Ciornei <ioana.ciornei@nxp.com> # LX2160ARDB, 
LS2088ARDB
     Tested-by: Sascha Bischoff <sascha.bischoff@arm.com>
     Signed-off-by: Marc Zyngier <maz@kernel.org>
     Acked-by: Thomas Gleixner <tglx@kernel.org>
     Link: https://lore.kernel.org/r/20260224100936.3752303-6-maz@kernel.org
     Signed-off-by: Christophe Leroy (CS GROUP) <chleroy@kernel.org>

commit 4a958e47c246fa3fb8954f4303e0da15ab3d026d
Author:     Marc Zyngier <maz@kernel.org>
AuthorDate: Tue Feb 24 10:09:34 2026 +0000
Commit:     Christophe Leroy (CS GROUP) <chleroy@kernel.org>
CommitDate: Thu Feb 26 10:48:18 2026 +0100

     fsl-mc: Switch over to per-device platform MSI

     Obtain the msi-parent irqdomain instead of the fsl_mc domain,
     which magically engages the per-device infrastructure.

     Additionally, simplify the overly complicated error handling.

     Reviewed-by: Ioana Ciornei <ioana.ciornei@nxp.com>
     Tested-by: Ioana Ciornei <ioana.ciornei@nxp.com> # LX2160ARDB, 
LS2088ARDB
     Tested-by: Sascha Bischoff <sascha.bischoff@arm.com>
     Signed-off-by: Marc Zyngier <maz@kernel.org>
     Acked-by: Thomas Gleixner <tglx@kernel.org>
     Link: https://lore.kernel.org/r/20260224100936.3752303-5-maz@kernel.org
     Signed-off-by: Christophe Leroy (CS GROUP) <chleroy@kernel.org>

commit cf3179b4e53f527aba9f0c6c3b921619c8adf761
Author:     Marc Zyngier <maz@kernel.org>
AuthorDate: Tue Feb 24 10:09:33 2026 +0000
Commit:     Christophe Leroy (CS GROUP) <chleroy@kernel.org>
CommitDate: Thu Feb 26 10:48:18 2026 +0100

     irqchip/gic-v3-its: Add fsl_mc device plumbing to the msi-parent 
handling

     Make the ITS code aware of fsl_mc devices by plumbing the devid
     retrieval primitive.

     Reviewed-by: Ioana Ciornei <ioana.ciornei@nxp.com>
     Tested-by: Ioana Ciornei <ioana.ciornei@nxp.com> # LX2160ARDB, 
LS2088ARDB
     Tested-by: Sascha Bischoff <sascha.bischoff@arm.com>
     Signed-off-by: Marc Zyngier <maz@kernel.org>
     Acked-by: Thomas Gleixner <tglx@kernel.org>
     Link: https://lore.kernel.org/r/20260224100936.3752303-4-maz@kernel.org
     Signed-off-by: Christophe Leroy (CS GROUP) <chleroy@kernel.org>

commit 0c9f522f2d41c7e055a602a0d2c41dc7af01010b
Author:     Marc Zyngier <maz@kernel.org>
AuthorDate: Tue Feb 24 10:09:32 2026 +0000
Commit:     Christophe Leroy (CS GROUP) <chleroy@kernel.org>
CommitDate: Thu Feb 26 10:48:18 2026 +0100

     fsl-mc: Add minimal infrastructure to use platform MSI

     Add the tiny bit of infrastructure required to use platform MSI
     instead of the current hack. This means providing a write_msi_msg
     callback, as well as irq domain and devid retrieval helpers.

     Reviewed-by: Ioana Ciornei <ioana.ciornei@nxp.com>
     Tested-by: Ioana Ciornei <ioana.ciornei@nxp.com> # LX2160ARDB, 
LS2088ARDB
     Tested-by: Sascha Bischoff <sascha.bischoff@arm.com>
     Signed-off-by: Marc Zyngier <maz@kernel.org>
     Acked-by: Thomas Gleixner <tglx@kernel.org>
     Link: https://lore.kernel.org/r/20260224100936.3752303-3-maz@kernel.org
     Signed-off-by: Christophe Leroy (CS GROUP) <chleroy@kernel.org>

commit 1fb7392ee3408494d4d62c09a8c3e5f5934caba7
Author:     Marc Zyngier <maz@kernel.org>
AuthorDate: Tue Feb 24 10:09:31 2026 +0000
Commit:     Christophe Leroy (CS GROUP) <chleroy@kernel.org>
CommitDate: Thu Feb 26 10:48:18 2026 +0100

     fsl-mc: Remove MSI domain propagation to sub-devices

     Only the root device generates MSIs (it is the only one talking to
     the ITS), so propagating the domain is pretty pointless. Remove it.

     Reviewed-by: Ioana Ciornei <ioana.ciornei@nxp.com>
     Tested-by: Ioana Ciornei <ioana.ciornei@nxp.com> # LX2160ARDB, 
LS2088ARDB
     Tested-by: Sascha Bischoff <sascha.bischoff@arm.com>
     Signed-off-by: Marc Zyngier <maz@kernel.org>
     Acked-by: Thomas Gleixner <tglx@kernel.org>
     Link: https://lore.kernel.org/r/20260224100936.3752303-2-maz@kernel.org
     Signed-off-by: Christophe Leroy (CS GROUP) <chleroy@kernel.org>

commit 3f4e403304186d79fddace860360540fc3af97f9
Author:     Chen Ni <nichen@iscas.ac.cn>
AuthorDate: Mon Feb 9 09:59:04 2026 +0800
Commit:     Christophe Leroy (CS GROUP) <chleroy@kernel.org>
CommitDate: Mon Feb 23 14:49:27 2026 +0100

     soc: fsl: cpm1: qmc: Fix error check for devm_ioremap_resource() in 
qmc_qe_init_resources()

     Fix wrong variable used for error checking after 
devm_ioremap_resource()
     call. The function checks qmc->scc_pram instead of qmc->dpram, which
     could lead to incorrect error handling.

     Fixes: eb680d563089 ("soc: fsl: cpm1: qmc: Add support for QUICC 
Engine (QE) implementation")
     Signed-off-by: Chen Ni <nichen@iscas.ac.cn>
     Acked-by: Herve Codina <herve.codina@bootlin.com>
     Link: 
https://lore.kernel.org/r/20260209015904.871269-1-nichen@iscas.ac.cn
     Signed-off-by: Christophe Leroy (CS GROUP) <chleroy@kernel.org>

commit 014077044e874e270ec480515edbc1cadb976cf2
Author:     Richard Genoud <richard.genoud@bootlin.com>
AuthorDate: Tue Dec 23 08:25:49 2025 +0100
Commit:     Christophe Leroy (CS GROUP) <chleroy@kernel.org>
CommitDate: Mon Feb 23 14:49:27 2026 +0100

     soc: fsl: qbman: fix race condition in qman_destroy_fq

     When QMAN_FQ_FLAG_DYNAMIC_FQID is set, there's a race condition between
     fq_table[fq->idx] state and freeing/allocating from the pool and
     WARN_ON(fq_table[fq->idx]) in qman_create_fq() gets triggered.

     Indeed, we can have:
              Thread A                             Thread B
         qman_destroy_fq()                    qman_create_fq()
           qman_release_fqid()
             qman_shutdown_fq()
             gen_pool_free()
                -- At this point, the fqid is available again --
                                                qman_alloc_fqid()
                -- so, we can get the just-freed fqid in thread B --
                                                fq->fqid = fqid;
                                                fq->idx = fqid * 2;
                                                WARN_ON(fq_table[fq->idx]);
                                                fq_table[fq->idx] = fq;
          fq_table[fq->idx] = NULL;

     And adding some logs between qman_release_fqid() and
     fq_table[fq->idx] = NULL makes the WARN_ON() trigger a lot more.

     To prevent that, ensure that fq_table[fq->idx] is set to NULL before
     gen_pool_free() is called by using smp_wmb().

     Fixes: c535e923bb97 ("soc/fsl: Introduce DPAA 1.x QMan device driver")
     Signed-off-by: Richard Genoud <richard.genoud@bootlin.com>
     Tested-by: CHAMPSEIX Thomas <thomas.champseix@alstomgroup.com>
     Link: 
https://lore.kernel.org/r/20251223072549.397625-1-richard.genoud@bootlin.com
     Signed-off-by: Christophe Leroy (CS GROUP) <chleroy@kernel.org>

commit 6de23f81a5e08be8fbf5e8d7e9febc72a5b5f27f
Author:     Linus Torvalds <torvalds@linux-foundation.org>
AuthorDate: Sun Feb 22 13:18:59 2026 -0800
Commit:     Linus Torvalds <torvalds@linux-foundation.org>
CommitDate: Sun Feb 22 13:18:59 2026 -0800

     Linux 7.0-rc1



  reply	other threads:[~2026-04-14 15:13 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-14 14:08 Christophe Leroy (CS GROUP)
2026-04-14 14:55 ` Arnd Bergmann
2026-04-14 15:02   ` Christophe Leroy (CS GROUP)
2026-04-14 15:13     ` Christophe Leroy (CS GROUP) [this message]
  -- strict thread matches above, loose matches on Subject: below --
2026-01-31  7:38 Christophe Leroy (CS GROUP)

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=2755af3f-d7d6-49b8-88f5-66352c237c35@kernel.org \
    --to=chleroy@kernel.org \
    --cc=arnd@arndb.de \
    --cc=ioana.ciornei@nxp.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=soc@kernel.org \
    /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

all inboxes | Powered by JetHome®