[coreboot-gerrit] Change in coreboot[master]: mainboard(s): drop redundant entries from Makefiles

[Wim Vervoorn (Code Review)]

Wim Vervoorn has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41107 )

Change subject: mainboard(s): drop redundant entries from Makefiles
......................................................................

Patch Set 10:

(2 comments)

https://review.coreboot.org/c/coreboot/+/41107/10/src/mainboard/facebook/fbg... 
File src/mainboard/facebook/fbg1701/Makefile.inc:

https://review.coreboot.org/c/coreboot/+/41107/10/src/mainboard/facebook/fbg... 
PS10, Line 14: ramstage-y += hda_verb.c
This removes the hds_verb.c from the build. It isn't included in another way. AZALIA_PLUGIN_SUPPORT is not set and the Braswell support doesn't pull it in either.

https://review.coreboot.org/c/coreboot/+/41107/10/src/mainboard/portwell/m10... 
File src/mainboard/portwell/m107/Makefile.inc:

https://review.coreboot.org/c/coreboot/+/41107/10/src/mainboard/portwell/m10... 
PS10, Line 6: ramstage-y += hda_verb.c
This removes the hds_verb.c from the build. It isn't included in another way. AZALIA_PLUGIN_SUPPORT is not set and the Braswell support doesn't pull it in either.

As a side item, wouldn't it be good to somehow document which files will be included implicitely from the mainboard directory? This makes creating a new mainboard directory a bit easier and prevents the inclusion of files that are not required.