On Tue, Aug 13, 2024 at 4:51 PM Jon Mason <[email protected]> wrote: > > On Tue, Aug 13, 2024 at 03:57:52PM -0400, [email protected] wrote: > > From: Bruce Ashfield <[email protected]> > > > > Integrating the following commit(s) to linux-yocto/.: > > > > 1/1 [ > > Author: Bruce Ashfield > > Email: [email protected] > > Subject: genericarm64: drop CONFIG_MTD_NAND_FSL_IFC > > Date: Tue, 13 Aug 2024 15:17:10 -0400 > > > > We are getting the following warning in recent 6.6+ kernels: > > > > WARNING: linux-yocto-6.6.45+git-r0 do_kernel_configcheck: [kernel > > config]: specified values did not make it into the kernel's final > > configuration: > > > > [NOTE]: 'CONFIG_MTD_NAND_FSL_IFC' last val (y) and .config val (n) > > do not match > > [INFO]: CONFIG_MTD_NAND_FSL_IFC : n > > [INFO]: raw config text: > > > > config MTD_NAND_FSL_IFC > > tristate "Freescale IFC NAND controller" > > depends on (FSL_SOC || ARCH_LAYERSCAPE || SOC_LS1021A > > || COMPILE_TEST) && HAS_IOMEM && FSL_IFC && MTD_RAW_NAND && MTD > > help > > Various Freescale chips e.g P1010, include a NAND > > Flash machine > > with built-in hardware ECC capabilities. > > Enabling this option will enable you to use this to > > control > > external NAND devices. > > > > Config 'MTD_NAND_FSL_IFC' has the following Direct dependencies > > (MTD_NAND_FSL_IFC=n): > > FSL_SOC(undefined/n) || ARCH_LAYERSCAPE(=y) || > > SOC_LS1021A(undefined/n) || COMPILE_TEST(=n) (=y) && HAS_IOMEM(=y) && > > FSL_IFC(=n) && MTD_RAW_NAND(=y) && MTD(=y) > > Parent dependencies are: > > SOC_LS1021A [SOC_LS1021A] MTD [y] ARCH_LAYERSCAPE [y] > > HAS_IOMEM [y] FSL_SOC [FSL_SOC] COMPILE_TEST [n] MTD_RAW_NAND [y] FSL_IFC > > [n] > > [INFO]: config 'CONFIG_MTD_NAND_FSL_IFC' was set, but it wasn't > > assignable, check (parent) dependencies > > > > The most significant missing option that is disabling this config value > > is FSL_SOC, which is only defined in arch/powerpc. > > > > As such, there's no sense keeping this option in the genericarm64 config > > as it is only enabled via arch/powerpc. > > > > Signed-off-by: Bruce Ashfield <[email protected]> > > ] > > > > Signed-off-by: Bruce Ashfield <[email protected]> > > --- > > > > Jon, > > > > Let me know if these seems ok to you. > > Yes, I just started seeing this in my CI today, but hadn't gotten to looking > at it. > A very quick look at the kernel, I see arm64 has ARCH_LAYERSCAPE, > which references MTD_NAND_FSL_IFC. I think this is the actual cause of the > desire > to have this code. So, we probably want to keep this in the generic > image (as layerscape is what the lx2k/honeycomb platform(s) use). > Is there a reason not to enable FSL_IFC in the config? >
It depends on FSL_SOC .. which is only enabled via arch/powerpc. > > > FYI, I'm seeing a config fragment warning (which I've also not gotten > around to fixing) with Xen and qemuarm/qemuarm64. > > WARNING: linux-yocto-6.10.3+git-r0 do_kernel_configcheck: [kernel config]: > This BSP contains fragments with warnings: > [INFO]: the following symbols were not found in the active configuration: > - CONFIG_FB_SYS_FOPS > > See https://gitlab.com/jonmason00/meta-arm/-/jobs/7574214060 > https://gitlab.com/jonmason00/meta-arm/-/jobs/7574213986 > > I think this can just be removed, as I don't see it in the kernel > anymore. That''s already fixed in the recent bumps that I've been sending Richard. https://git.yoctoproject.org/yocto-kernel-cache/commit/?h=yocto-6.10&id=107c15b3ee3f07842384208ea021588d589e51ef Maybe your meta SRCREVS are different ? Bruce > > Thanks, > Jon > > > > > Bruce > > > > meta/recipes-kernel/linux/linux-yocto-rt_6.6.bb | 2 +- > > meta/recipes-kernel/linux/linux-yocto-tiny_6.6.bb | 2 +- > > meta/recipes-kernel/linux/linux-yocto_6.6.bb | 2 +- > > 3 files changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_6.6.bb > > b/meta/recipes-kernel/linux/linux-yocto-rt_6.6.bb > > index ae56e8a129..c3b4fc1e9b 100644 > > --- a/meta/recipes-kernel/linux/linux-yocto-rt_6.6.bb > > +++ b/meta/recipes-kernel/linux/linux-yocto-rt_6.6.bb > > @@ -15,7 +15,7 @@ python () { > > } > > > > SRCREV_machine ?= "9a1d7ebfd75c70dcec660e18124a37de469e68b4" > > -SRCREV_meta ?= "e55e4407942bae43ef8ba463b38e48995b023187" > > +SRCREV_meta ?= "9901a21dd1f99c833d8e1a58b3fc057e57bda598" > > > > SRC_URI = > > "git://git.yoctoproject.org/linux-yocto.git;branch=${KBRANCH};name=machine;protocol=https > > \ > > > > git://git.yoctoproject.org/yocto-kernel-cache;type=kmeta;name=meta;branch=yocto-6.6;destsuffix=${KMETA};protocol=https" > > diff --git a/meta/recipes-kernel/linux/linux-yocto-tiny_6.6.bb > > b/meta/recipes-kernel/linux/linux-yocto-tiny_6.6.bb > > index b0e4ab71ae..1c8cfd7d74 100644 > > --- a/meta/recipes-kernel/linux/linux-yocto-tiny_6.6.bb > > +++ b/meta/recipes-kernel/linux/linux-yocto-tiny_6.6.bb > > @@ -18,7 +18,7 @@ KMETA = "kernel-meta" > > KCONF_BSP_AUDIT_LEVEL = "2" > > > > SRCREV_machine ?= "e402da9835bfae267f409a48c3a2cb6e231f9d94" > > -SRCREV_meta ?= "e55e4407942bae43ef8ba463b38e48995b023187" > > +SRCREV_meta ?= "9901a21dd1f99c833d8e1a58b3fc057e57bda598" > > > > PV = "${LINUX_VERSION}+git" > > > > diff --git a/meta/recipes-kernel/linux/linux-yocto_6.6.bb > > b/meta/recipes-kernel/linux/linux-yocto_6.6.bb > > index 5688a4b109..948f46621b 100644 > > --- a/meta/recipes-kernel/linux/linux-yocto_6.6.bb > > +++ b/meta/recipes-kernel/linux/linux-yocto_6.6.bb > > @@ -32,7 +32,7 @@ SRCREV_machine:qemux86 ?= > > "e402da9835bfae267f409a48c3a2cb6e231f9d94" > > SRCREV_machine:qemux86-64 ?= "e402da9835bfae267f409a48c3a2cb6e231f9d94" > > SRCREV_machine:qemumips64 ?= "97d2336ea9ce4eac6b30a75b26f33b83827d38aa" > > SRCREV_machine ?= "e402da9835bfae267f409a48c3a2cb6e231f9d94" > > -SRCREV_meta ?= "e55e4407942bae43ef8ba463b38e48995b023187" > > +SRCREV_meta ?= "9901a21dd1f99c833d8e1a58b3fc057e57bda598" > > > > # set your preferred provider of linux-yocto to 'linux-yocto-upstream', > > and you'll > > # get the <version>/base branch, which is pure upstream -stable, and the > > same > > -- > > 2.39.2 > > > > -- - Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end - "Use the force Harry" - Gandalf, Star Trek II
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#203293): https://lists.openembedded.org/g/openembedded-core/message/203293 Mute This Topic: https://lists.openembedded.org/mt/107883009/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
