[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/dmi: Add DMI driver support

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47988 )

Change subject: soc/intel/common/dmi: Add DMI driver support
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/47988/1/src/soc/intel/common/block/... 
File src/soc/intel/common/block/dmi/dmi.c:

https://review.coreboot.org/c/coreboot/+/47988/1/src/soc/intel/common/block/... 
PS1, Line 8: 2
> I was using it as 0-2 (3). I can change it to 3 and do 1-3, let me know. […]

Index can be 0-2, but MAX_ always refers to the actual count. So, it would be better to set this to 3 and use i = 0; i < MAX_GPMR_REGS; i++ below.