Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/62850 )
Change subject: commonlib/bsd: Add struct name "mem_chip_channel" for external access ......................................................................
commonlib/bsd: Add struct name "mem_chip_channel" for external access
struct mem_chip_info { ... struct { --> If no struct name, can't access the channel structure ... } channel[0]; };
BUG=b:182963902,b:177917361 TEST=Build pass on Kingler
Signed-off-by: Xi Chen xixi.chen@mediatek.corp-partner.google.com Change-Id: I8dcd3b52f33f80afb7885ffdcad826d86b54b543 Reviewed-on: https://review.coreboot.org/c/coreboot/+/62850 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Yu-Ping Wu yupingso@google.com --- M src/commonlib/bsd/include/commonlib/bsd/mem_chip_info.h 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified 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 7194f70..8dc6db3 100644 --- a/src/commonlib/bsd/include/commonlib/bsd/mem_chip_info.h +++ b/src/commonlib/bsd/include/commonlib/bsd/mem_chip_info.h @@ -15,7 +15,7 @@ uint8_t type; /* enum mem_chip_type */ uint8_t num_channels; uint8_t reserved[6]; - struct { + struct mem_chip_channel { uint64_t density; uint8_t io_width; uint8_t manufacturer_id;
3 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one.