[coreboot-gerrit] Change in coreboot[master]: mb/ocp/deltalake: Override smbios_fill_dimm_locator for type 17

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45799 )

Change subject: mb/ocp/deltalake: Override smbios_fill_dimm_locator for type 17
......................................................................

Patch Set 8:

(1 comment)

https://review.coreboot.org/c/coreboot/+/45799/5/src/mainboard/ocp/deltalake... 
File src/mainboard/ocp/deltalake/ramstage.c:

https://review.coreboot.org/c/coreboot/+/45799/5/src/mainboard/ocp/deltalake... 
PS5, Line 246: dimm_locator[dimm->channel_num]
> Do you mean like A0, A1 ..,etc? I use the same location string definitions in UEFI BIOS (A0, B0 .. […]

No, what I'd like to avoid is the character array. The alphabet letters have consecutive numeric values in the ASCII table: http://www.asciitable.com/

So, (dimm->channel_num + 'A') is valid C. If `dimm->channel_num` is 1, the result will be 'B', and so on. Doing this eliminates the need for the `dimm_locator` array.