Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45469 )
Change subject: apollolake boards: Enable CSE in devicetree ......................................................................
Patch Set 4:
Patch Set 4: Code-Review+2
Patch Set 4:
Patch Set 4: Code-Review-1
I'm not sure if we really want to enable CSE/ME/... by default
To answer your question, please look at this code https://github.com/coreboot/coreboot/blob/master/src/soc/intel/apollolake/ch...
Like every platform, coreboot will ensure to make CSE bus function disable that means prior to boot to OS (after EOP is done), CSE will not visible on bus irrespective of being listen in devicetree.cb. hence your concern is already been taken care that CSE shouldn't be default enable.
Hope this help to clarify things ?
if yes, can you please reconsider your vote?
Got it, thank you!
Thanks a lot for review