Hello,

Quick question for Kconfig lint checking!

For my patch https://review.coreboot.org/c/coreboot/+/71120, the Jenkins build 
fails with following error:

"Check Kconfig files for errors (lint-stable-008-kconfig): #!!!!! Error: 
'CONFIG_SOC_INTEL_COMMON_BLOCK_PRMRR_SIZE_2MB' selected at 
src/soc/intel/meteorlake/Kconfig:101.  Symbols created in a choice cannot be 
selected"

What is the rationale behind enforcing this? As of now, PRMRR Kconfig settings 
are in common code and i wanted to set the right size from SOC specific code. 
e.g. the patch i mentioned earlier.

Should we get rid of enforcing "lint-stable-008-kconfig" test? If not what is 
the best way to achieve the scenario i mentioned above.

One option i think is to simplify PRMRR Kconfig options (get rid of choice 
menu) and define PRMRR_SIZE config option in SOC specific code. But then all 
SOC specific code would have to define this Kconfig option. If some SOC does 
not need PRMRR then also the Kconfig option with default set 0 might be needed.

Thanks for suggestions in advance.


--
Regards,
Pratik

_______________________________________________
coreboot mailing list -- coreboot@coreboot.org
To unsubscribe send an email to coreboot-le...@coreboot.org

Reply via email to