[coreboot-gerrit] Change in coreboot[master]: soc/intel/icelake: Make use of "all-y"

[Subrata Banik (Code Review)]

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36546 )

Change subject: soc/intel/icelake: Make use of "all-y"
......................................................................

Patch Set 2:

(2 comments)

https://review.coreboot.org/c/coreboot/+/36546/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/36546/1//COMMIT_MSG@7 
PS1, Line 7: to avoid exclusive stage files in Makefile.inc
> Minor: I would remove this last part to make the commit summary shorter

Done

https://review.coreboot.org/c/coreboot/+/36546/1/src/soc/intel/icelake/Makef... 
File src/soc/intel/icelake/Makefile.inc:

https://review.coreboot.org/c/coreboot/+/36546/1/src/soc/intel/icelake/Makef... 
PS1, Line 22: espi
> > Is it bad to add some more files like espi.c, gpio.c memmap.c p2sb. […]

also shouldn't we try to limit the files inclusion in stages unless required ?