Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35895 )
Change subject: SMBIOS (Type 17): Update the string for Intel persistent memory ......................................................................
Patch Set 1:
Wasn't this long enough already? that it still compiles even means that it's unused?
That is the string defined in SMBIOS specification v3.3.0 ( https://www.dmtf.org/standards/smbios )
note that I'm fine to drop this :) find it too long also
Alternatively you can ask others for their opinion. Maybe somebody thinks we should keep the names aligned as close as possible. However, I wonder now: Did the name change in SMBIOS? maybe the author of that line shortened it already on purpose??