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?



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.

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
> 
> 
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#203292): 
https://lists.openembedded.org/g/openembedded-core/message/203292
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]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to