build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63026 )
Change subject: soc/qualcomm/common: verify size of memchipinfo structure ......................................................................
Patch Set 1:
(2 comments)
File src/commonlib/bsd/include/commonlib/bsd/mem_chip_info.h:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-144565): https://review.coreboot.org/c/coreboot/+/63026/comment/5f54e761_24e2d17b PS1, Line 28: static inline size_t mem_chip_info_size(struct mem_chip_info *info) { open brace '{' following function definitions go on the next line
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-144565): https://review.coreboot.org/c/coreboot/+/63026/comment/2c900641_97d69f3b PS1, Line 29: return sizeof(*info) + sizeof(info->channel[0]) * info->num_channels; please, no spaces at the start of a line