Attention is currently required from: Jayvik Desai, Subrata Banik.
Paul Menzel has posted comments on this change by Jayvik Desai. ( https://review.coreboot.org/c/coreboot/+/85221?usp=email )
Change subject: mb/google/fatcat: Adjust EC host command range for fatcat_ish variant ......................................................................
Patch Set 1:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/85221/comment/726f9956_0083cd1a?usp... : PS1, Line 7: for fatcat_ish variant Make this part of the prefix?
https://review.coreboot.org/c/coreboot/+/85221/comment/72a5e308_b62af714?usp... : PS1, Line 9: This commit adjusts the EC host command range for the fatcat_ish variant “This commit” is redundant. Maybe just imperative mood: Adjust …
https://review.coreboot.org/c/coreboot/+/85221/comment/f4f016b7_1f591713?usp... : PS1, Line 9: This commit adjusts the EC host command range for the fatcat_ish variant : to 0x800-0x807 & 0x200-0x20f. According to what source?