Attention is currently required from: Furquan Shaikh, Tim Wawrzynczak, Paul Menzel, Meera Ravindranath, EricR Lai. Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48345 )
Change subject: mb/intel/adlrvp: Drop INTEL_CAR_NEM Kconfig select on ADL-P RVP ......................................................................
Patch Set 21:
(1 comment)
File src/mainboard/intel/adlrvp/Kconfig.name:
https://review.coreboot.org/c/coreboot/+/48345/comment/ca0c2b51_7b4a0ee8 PS21, Line 20: select INTEL_CAR_NEM #TODO - Enable INTEL_CAR_NEM_ENHANCED : : config BOARD_INTEL_ADLRVP_M_EXT_EC : bool "Alderlake-M RVP with Chrome EC" : select INTEL_LPSS_UART_FOR_CONSOLE : select DRIVERS_INTEL_MIPI_CAMERA : select SOC_INTEL_COMMON_BLOCK_IPU : select INTEL_CAR_NEM #TODO - Enable INTEL_CAR_NEM_ENHANCED
These shouldn't be encessary, the BOARD_SPECIFIC_OPTIONS are covered by all 4 BOARD_INTEL_ADLRVP_* t […]
I don't know Tim, what all SoC is being used for ADL-M so, don't really want to enable eNEM for them by default as I'm yet to hear their validation feedback.
Once I would get full validation data from ADL-M, we could just enable eNEM for all ADL-RVPs and remove those from here as well. Does that make sense to you ?