Attention is currently required from: Hung-Te Lin, Jakub Czapiga, Julius Werner, Yu-Ping Wu. Jianjun Wang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63494 )
Change subject: coreboot_tables: Replace 'struct lb_uint64' with lb_uint64_t ......................................................................
Patch Set 4:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/63494/comment/c088f663_c0c1ab83 PS3, Line 7: Convert 'struct lb_uint64' to
Replace '... […]
Done
File src/commonlib/include/commonlib/coreboot_tables.h:
https://review.coreboot.org/c/coreboot/+/63494/comment/10838431_fc20e124 PS3, Line 97: /* Since coreboot is usually compiled 32bit, gcc will align 64bit : * types to 32bit boundaries. If the coreboot table is dumped on a : * 64bit system, a uint64_t would be aligned to 64bit boundaries, : * breaking the table format. : * : * lb_uint64 will keep 64bit coreboot table values aligned to 32bit : * to ensure compatibility. They can be accessed with the two functions : * below: unpack_lb64() and pack_lb64()
It's still nice to have this comment, with some modification.
Done