Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/gerrit-avatars/+/42273 )
Change subject: My avatar. ......................................................................
Patch Set 2: Code-Review+1
(3 comments)
https://review.coreboot.org/c/gerrit-avatars/+/42273/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/gerrit-avatars/+/42273/2//COMMIT_MSG@7 PS2, Line 7: My avatar. nit: commit summaries have a verb in imperative mood, and do not end with a period:
Update my avatar
https://review.coreboot.org/c/gerrit-avatars/+/42273/2//COMMIT_MSG@10 PS2, Line 10: Signed-off-by: Zheng Bao zheng.bao@amd.com There are two `Signed-off-by` lines. Since both are you, maybe keep only one?
https://review.coreboot.org/c/gerrit-avatars/+/42273/2/1000034.jpg File 1000034.jpg:
PS2: Jenkins says: image not square