[coreboot-gerrit] Change in coreboot[master]: mb/intel/coffeelake_rvp/var/cfl_u: Add empty verb table

[Michael Niewöhner (Code Review)]

Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48360 )

Change subject: mb/intel/coffeelake_rvp/var/cfl_u: Add empty verb table
......................................................................

Patch Set 2:
> Patch Set 2:
> > Empty c files instead of empty headers? How does that make a difference? The right way is to have no files when none are required -> fix the build system
>
> I don't think he meant replacing the header file by an empty C file.
>
> #include "variant/hda_verb.h"
>
> This is from mb/intel/coffeelake_rvp/hda_verb.c, which requires every variant to have such a header file. I think what he suggested was to move the hda_verb.c to the variant directories and add them to their Makefile.inc, if needed.

That's not what I understood from the commit message tbh