Attention is currently required from: Julius Werner. Ravi Kumar Bokka 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 3:
(1 comment)
File src/soc/qualcomm/common/qclib.c:
https://review.coreboot.org/c/coreboot/+/63026/comment/d34ef10c_cfd6cd49 PS1, Line 31: mem_chip_info_size(&memchip)
Ravi, seriously, _please_ stop answering "Done" to things you didn't actually do! We're losing so mu […]
Sincere applogies julius.. and sorry for the inconvenience happened by me. I thought this was older comment which is already address. that's the reason i have keep without seeing code changes. I will not repeat these mistakes further.
I have addressed this comment. Please let me know if any thing missing here.
Thanks. Ravi
-
Ravi Kumar Bokka (Code Review)