[coreboot-gerrit] Change in ...coreboot[master]: vboot: standardize on working data size

[build bot (Jenkins) (Code Review)]

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31474 )

Change subject: vboot: standardize on working data size
......................................................................

Patch Set 3:

(4 comments)

https://review.coreboot.org/#/c/31474/3/src/include/memlayout.h 
File src/include/memlayout.h:

https://review.coreboot.org/#/c/31474/3/src/include/memlayout.h@164 
PS3, Line 164: #define VBOOT2_WORK(addr, sz) \
Macros with multiple statements should be enclosed in a do - while loop

https://review.coreboot.org/#/c/31474/3/src/include/memlayout.h@164 
PS3, Line 164: #define VBOOT2_WORK(addr, sz) \
macros should not use a trailing semicolon

https://review.coreboot.org/#/c/31474/3/src/include/memlayout.h@168 
PS3, Line 168: 			VB2_WORKBUF_RECOMMENDED_SIZE! (sz)));
space required before that '!' (ctx:VxW)

https://review.coreboot.org/#/c/31474/3/src/include/memlayout.h@168 
PS3, Line 168: 			VB2_WORKBUF_RECOMMENDED_SIZE! (sz)));
space prohibited after that '!' (ctx:VxW)