[coreboot-gerrit] Change in coreboot[master]: mb/google/zork: Create Shuboz variant

[Felix Held (Code Review)]

Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47152 )

Change subject: mb/google/zork: Create Shuboz variant
......................................................................

Patch Set 3: Code-Review+2

looks good to me.

FYI: In upstream coreboot review it's rather uncommon to +1 own patches; usually when a patch is uploaded for review and it's not marked as work in progress, not for merge or something like that, people will assume that it's working and ready to be reviewed. You probably can't +2 patches yet, but +2ing own patches is something that people shouldn't do as it basically would bypass the review. Having someone else do the review really helps spotting bugs.
No need to worry about this here, since it's only a +1, but it would be good to keep this in mind for future patches