[coreboot-gerrit] [L] Change in coreboot[master]: mb/google/skyrim: Add mainboard chip.h

[Karthik Ramasubramanian (Code Review)]

Attention is currently required from: Raul Rangel, Jason Nien, Jon Murphy, Martin Roth, Tim Van Patten, Mark Hasemeyer.

Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74609 )

Change subject: mb/google/skyrim: Add mainboard chip.h
......................................................................

Patch Set 8:

(1 comment)

File src/mainboard/google/skyrim/variants/baseboard/devicetree.cb:

https://review.coreboot.org/c/coreboot/+/74609/comment/a2036857_028060c1 
PS8, Line 2: chip mainboard/google/skyrim/variants/baseboard
> +1 to Arthur and Raul's comments. […]

If you are concerned about updating all the mainboards with devicetree.h file, how about adding include/import keyword in devicetree parser? That way only the boards that need to include a mainboard or variant specific header can include them.