Attention is currently required from: Derek Huang, Eric Lai, Nick Vaccaro, Robert Chen, Serin Yeh, Subrata Banik.
Martin L Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/78170?usp=email )
Change subject: mb/google/brya/var/yavilla: Add VCM power control sequence ......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1: Hi Serin, It looks like you haven't contributed to the coreboot project before.
Welcome, and thank you for the patch. We hope that this is just the first of many.
Please let us know if there's anything we can do to help get your first sets patches merged as you get used to the contribution process.
The coreboot project has a hands-off policy regarding other people's patches so nobody here is going to update the contents without your permission. If you would you like someone to take over your patches at any point, please just post a comment to that effect on the specific patch.
You might find the coding style guide and the gerrit guidelines useful to read.
https://doc.coreboot.org/contributing/coding_style.html https://doc.coreboot.org/contributing/gerrit_guidelines.html
As noted in the gerrit guidelines document, we don't typically review our own patches. It's not a big deal, but people reviewing their own patches isn't generally helpful. There is an exception when two people have written a patch together, and each is approving the other's work.
If you want to talk with anyone, you can talk to developers on one of the many options we have:
https://doc.coreboot.org/community/forums.html Again, please let us know if you have any questions, or if there's anything we can do to help.