[coreboot-gerrit] Change in coreboot[master]: mb/google/drallion: Add two variants - arcada_cml & sarien_cml

[Thejaswani Putta (Code Review)]

Thejaswani Putta has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34887 )

Change subject: mb/google/drallion: Add two variants - arcada_cml & sarien_cml
......................................................................

Patch Set 7:

(4 comments)

https://review.coreboot.org/c/coreboot/+/34887/5//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/34887/5//COMMIT_MSG@7 
PS5, Line 7: Moffet:
> What does "moffet" mean?  Should this be "mb/google/drallion: Add arcada_cml & sarien_cml variants"?

Thanks, Done!

https://review.coreboot.org/c/coreboot/+/34887/5//COMMIT_MSG@9 
PS5, Line 9: EV
> What is EV?

Ack, Will make sure not to use such words. Thanks, modified

https://review.coreboot.org/c/coreboot/+/34887/5//COMMIT_MSG@11 
PS5, Line 11: EVT
> Please, don't *EVER* mention the phase of a project. […]

Yes, will never repeat it. Thanks for correcting, Done.

https://review.coreboot.org/c/coreboot/+/34887/5/src/mainboard/google/dralli... 
File src/mainboard/google/drallion/variants/arcada_cml/devicetree.cb:

https://review.coreboot.org/c/coreboot/+/34887/5/src/mainboard/google/dralli... 
PS5, Line 230: arcada_ish.bin
> Should this be called drallion_ish. […]

Done