In message: [linux-yocto] [yocto-kernel-cache] [yocto-5.10-rt] [PATCH] Fix 
warning from do_kernel_configcheck in intel-x86 bsp
on 24/05/2021 qiang.zh...@windriver.com wrote:

> From: Zqiang <qiang.zh...@windriver.com>
> 
> WARNING: linux-yocto-rt-5.10.x+gitAUTOINC+cd049697e9_ff83ae5580-r0 
> do_kernel_configcheck:
> [kernel config]: specified values did not make it into the kernel's final 
> configuration:
> 
>     [NOTE]: 'CONFIG_CFG80211' last val (y) and .config val (m) do not match
>     [INFO]: CONFIG_CFG80211 : m ## .config: 1559 
> :configs/v5.10/standard/preempt-rt/
>     intel-x86/features/mac80211/mac80211.cfg (m)
>     configs/v5.10/standard/preempt-rt/intel-x86/features/hostapd/hostapd.cfg 
> (y)
>     [INFO]: raw config text:
> 
>         config CFG80211
>               tristate "cfg80211 - wireless configuration API"
>               select FW_LOADER
>               select CRC32
>               select CRYPTO_SHA256 if CFG80211_USE_KERNEL_REGDB_KEYS
>               depends on (RFKILL || !RFKILL) && WIRELESS && NET
>               help
>                 cfg80211 is the Linux wireless LAN (802.11) configuration API.
>                 Enable this if you have a wireless device.
> 
>                 For more information refer to documentation on the wireless 
> wiki:
> 
>                 
> https://wireless.wiki.kernel.org/en/developers/Documentation/cfg80211
> 
>                 When built as a module it will be called cfg80211.
> 
>         Config 'CFG80211' has the following Direct dependencies (CFG80211=y):
>                 RFKILL(=y) || !RFKILL(=y) (=y) && WIRELESS(=y) && NET(=y)
>         Parent dependencies are:
>              CFG80211_USE_KERNEL_REGDB_KEYS [y] WIRELESS [y] RFKILL [y] NET 
> [y]
>     [NOTE]: 'CONFIG_MAC80211' last val (y) and .config val (m) do not match
>     [INFO]: CONFIG_MAC80211 : m ## .config: 1574 
> :configs/v5.10/standard/preempt-rt/intel-x86/
>             features/mac80211/mac80211.cfg (m)
>             
> configs/v5.10/standard/preempt-rt/intel-x86/features/hostapd/hostapd.cfg (y)
>     [INFO]: raw config text:
> 
>         config MAC80211
>               tristate "Generic IEEE 802.11 Networking Stack (mac80211)"
>               select CRYPTO
>               select CRYPTO_LIB_ARC4
>               select CRYPTO_AES
>               select CRYPTO_CCM
>               select CRYPTO_GCM
>               select CRYPTO_CMAC
>               select CRC32
>               depends on CFG80211 && WIRELESS && NET
>               help
>                 This option enables the hardware independent IEEE 802.11
>                 networking stack.
> 
>         Config 'MAC80211' has the following Direct dependencies (MAC80211=m):
>                 CFG80211(=m) && WIRELESS(=y) && NET(=y)
>         Parent dependencies are:
>              WIRELESS [y] NET [y] CFG80211 [m]
>     [NOTE]: 'CONFIG_DPTF_PCH_FIVR' last val (m) and .config val (n) do not 
> match
>     [INFO]: CONFIG_DPTF_PCH_FIVR : n
>     [INFO]: raw config text:
> 
>         config DPTF_PCH_FIVR
>               tristate "PCH FIVR DPTF Participant"
>               default m
>               depends on ACPI_DPTF && ACPI
>               help
>                 This driver adds support for Dynamic Platform and Thermal 
> Framework
>                 (DPTF) PCH FIVR Participant device support. This driver 
> allows to
>                 switch the PCH FIVR (Fully Integrated Voltage Regulator) 
> frequency.
>                 This participant is responsible for exposing:
>                       freq_mhz_low_clock
>                       freq_mhz_high_clock
> 
>                 To compile this driver as a module, choose M here:
>                 the module will be called dptf_pch_fivr.
> 
>         Config 'DPTF_PCH_FIVR' has the following Direct dependencies 
> (DPTF_PCH_FIVR=n):
>                 ACPI_DPTF(=n) && ACPI(=y)
>         Parent dependencies are:
>              ACPI_DPTF [n] ACPI [y]
>     [INFO]: config 'CONFIG_DPTF_PCH_FIVR' was set, but it wasn't assignable, 
> check
> 
> Reconfigure the variable value of CFG80211, MAC80211 and ACPI_DPTF
> in intel-x86 BSP.

This is the right approach for this BSP.

We could declare these options as "non-hardware" and the warning would
go away, but then, we wouldn't be alerted to changes in the future.

With this, if someting changes the values so that they change to different
values in the final .config, the warnings will reappear and we can address
them then.

Bruce

> 
> Signed-off-by: Zqiang <qiang.zh...@windriver.com>
> ---
>  bsp/intel-x86/intel-x86-cfg80211-overrides.cfg | 3 +++
>  bsp/intel-x86/intel-x86-dptf-overrides.cfg     | 2 ++
>  bsp/intel-x86/intel-x86.scc                    | 2 ++
>  3 files changed, 7 insertions(+)
>  create mode 100644 bsp/intel-x86/intel-x86-cfg80211-overrides.cfg
>  create mode 100644 bsp/intel-x86/intel-x86-dptf-overrides.cfg
> 
> diff --git a/bsp/intel-x86/intel-x86-cfg80211-overrides.cfg 
> b/bsp/intel-x86/intel-x86-cfg80211-overrides.cfg
> new file mode 100644
> index 00000000..26957814
> --- /dev/null
> +++ b/bsp/intel-x86/intel-x86-cfg80211-overrides.cfg
> @@ -0,0 +1,3 @@
> +# SPDX-License-Identifier: MIT
> +CONFIG_CFG80211=y
> +CONFIG_MAC80211=y
> diff --git a/bsp/intel-x86/intel-x86-dptf-overrides.cfg 
> b/bsp/intel-x86/intel-x86-dptf-overrides.cfg
> new file mode 100644
> index 00000000..2c206883
> --- /dev/null
> +++ b/bsp/intel-x86/intel-x86-dptf-overrides.cfg
> @@ -0,0 +1,2 @@
> +# SPDX-License-Identifier: MIT
> +CONFIG_ACPI_DPTF=y
> diff --git a/bsp/intel-x86/intel-x86.scc b/bsp/intel-x86/intel-x86.scc
> index d837c58f..d6b2f931 100644
> --- a/bsp/intel-x86/intel-x86.scc
> +++ b/bsp/intel-x86/intel-x86.scc
> @@ -58,3 +58,5 @@ kconf hardware intel-x86-acpi.cfg
>  kconf hardware intel-x86-igb-overrides.cfg
>  kconf hardware intel-x86-ixgbe-overrides.cfg
>  kconf hardware intel-x86-i915-overrides.cfg
> +kconf hardware intel-x86-cfg80211-overrides.cfg
> +kconf hardware intel-x86-dptf-overrides.cfg
> -- 
> 2.17.1
> 
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#9918): 
https://lists.yoctoproject.org/g/linux-yocto/message/9918
Mute This Topic: https://lists.yoctoproject.org/mt/83046575/21656
Group Owner: linux-yocto+ow...@lists.yoctoproject.org
Unsubscribe: https://lists.yoctoproject.org/g/linux-yocto/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to