[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 12:

(4 comments)

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

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

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

https://review.coreboot.org/#/c/31474/12/src/include/memlayout.h@167 
PS12, Line 167: 			VB2_FIRMWARE_WORKBUF_RECOMMENDED_SIZE! (sz)));
space required before that '!' (ctx:VxW)

https://review.coreboot.org/#/c/31474/12/src/include/memlayout.h@167 
PS12, Line 167: 			VB2_FIRMWARE_WORKBUF_RECOMMENDED_SIZE! (sz)));
space prohibited after that '!' (ctx:VxW)