[coreboot-gerrit] Change in coreboot[master]: mb/{google,intel}/*: select SOC_INTEL_CSE_LITE_SKU only if CHROMEOS

[Matt DeVillier (Code Review)]

Matt DeVillier has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/49059 )

Change subject: mb/{google,intel}/*: select SOC_INTEL_CSE_LITE_SKU only if CHROMEOS
......................................................................

Patch Set 1:
> Patch Set 1:
>
> I'm not convinced that the dependency on CHROMEOS is correct. Mapped
> to reality that would imply that one switches the CSE SKU when switching
> CHROMEOS off?

I'm inclined to agree. Looking at cse_lite.c, it would seem there's an implicit dependency on vboot, not necessarily ChromeOS. Dropping 'depends on CHROMEOS' from SOC_INTEL_CSE_LITE_SKU would allow certain boards to build w/o CHROMEOS selected, but not sure that would result in a bootable image