[coreboot-gerrit] [M] Change in coreboot[master]: soc/qualcomm/sc7280: Add socinfo_pro_part() function in coreboot

[Taniya Das (Code Review)]

Attention is currently required from: Shelley Chen, Taniya Das, Venkat Thogaru, Julius Werner, Sudheer Amrabadi.

Taniya Das has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68384 )

Change subject: soc/qualcomm/sc7280: Add  socinfo_pro_part() function in coreboot
......................................................................

Patch Set 6:

(2 comments)

File src/soc/qualcomm/sc7280/socinfo.c:

https://review.coreboot.org/c/coreboot/+/68384/comment/247bf8c3_5b82adc7 
PS4, Line 12: static struct chipinfo chipinfolut[] = {
> Not related directly to this patch, but just some general observations about this code: each entry h […]

Changed structure members range uint32_t to uint16_t, in structure initialization instead of macros used designated initializers, here we are keeping chip_id, because we need it for future purpose use.

https://review.coreboot.org/c/coreboot/+/68384/comment/0d4ac63b_5e4c4968 
PS4, Line 57: 	return false;
> Done. […]

Done