[coreboot-gerrit] Change in coreboot[master]: mb/google/zork: keep the c-state IO base address alignment

[chris wang (Code Review)]

chris wang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44135 )

Change subject: mb/google/zork: keep the c-state IO base address alignment
......................................................................

Patch Set 6:

(3 comments)

https://review.coreboot.org/c/coreboot/+/44135/3//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/44135/3//COMMIT_MSG@7 
PS3, Line 7: mb/google/zork: C-state IO base address alignment
> Please make this a statement by adding a verb (in imperative mood).

Done

https://review.coreboot.org/c/coreboot/+/44135/4/src/soc/amd/picasso/cpu.c 
File src/soc/amd/picasso/cpu.c:

https://review.coreboot.org/c/coreboot/+/44135/4/src/soc/amd/picasso/cpu.c@5... 
PS4, Line 56:
> Remove blank line

Done

https://review.coreboot.org/c/coreboot/+/44135/4/src/soc/amd/picasso/include... 
File src/soc/amd/picasso/include/soc/iomap.h:

https://review.coreboot.org/c/coreboot/+/44135/4/src/soc/amd/picasso/include... 
PS4, Line 72: 0x0c
> to my other comment: change this to 0x13 to align things with the AGESA side and don't introduce CON […]

Done