Shelley Chen has submitted this change. ( https://review.coreboot.org/c/coreboot/+/63407 )
Change subject: commonlib/bsd: Add mem_chip_info_size() function ......................................................................
commonlib/bsd: Add mem_chip_info_size() function
Add a helper function mem_chip_info_size() as the size of mem_chip_info structure is used in multiple places.
BUG=b:182963902,b:177917361 TEST=Validated on qualcomm sc7280 development board
Signed-off-by: Ravi Kumar Bokka rbokka@codeaurora.org Change-Id: Iaada45d63b82c28495166024a9655d871ba65b20 Reviewed-on: https://review.coreboot.org/c/coreboot/+/63407 Reviewed-by: Julius Werner jwerner@chromium.org Reviewed-by: Yu-Ping Wu yupingso@google.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/commonlib/bsd/include/commonlib/bsd/mem_chip_info.h 1 file changed, 7 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Julius Werner: Looks good to me, approved Yu-Ping Wu: Looks good to me, approved
diff --git a/src/commonlib/bsd/include/commonlib/bsd/mem_chip_info.h b/src/commonlib/bsd/include/commonlib/bsd/mem_chip_info.h index 8dc6db3..d07f4d6 100644 --- a/src/commonlib/bsd/include/commonlib/bsd/mem_chip_info.h +++ b/src/commonlib/bsd/include/commonlib/bsd/mem_chip_info.h @@ -3,6 +3,8 @@ #ifndef _COMMONLIB_BSD_MEM_CHIP_INFO_H_ #define _COMMONLIB_BSD_MEM_CHIP_INFO_H_
+#include <stddef.h> + enum mem_chip_type { MEM_CHIP_DDR3 = 0x30, MEM_CHIP_LPDDR3 = 0x38, @@ -25,4 +27,9 @@ } channel[0]; };
+static inline size_t mem_chip_info_size(struct mem_chip_info *info) +{ + return sizeof(*info) + sizeof(info->channel[0]) * info->num_channels; +}; + #endif /* _COMMONLIB_BSD_MEM_CHIP_INFO_H_ */