[S] Change in coreboot[main]: soc/intel/{adl,mtl}: Don't set up SPD on LPDDRx

[Jayvik Desai (Code Review)]

Attention is currently required from: Arthur Heymans, Dinesh Gehlot, Eran Mitrani, Jakub Czapiga, Kapil Porwal, Nick Vaccaro, Rishika Raj, Subrata Banik, Tarun.

Jayvik Desai has posted comments on this change by Arthur Heymans. ( https://review.coreboot.org/c/coreboot/+/84205?usp=email )

Change subject: soc/intel/{adl,mtl}: Don't set up SPD on LPDDRx
......................................................................

Patch Set 3:

(1 comment)

File src/soc/intel/alderlake/meminit.c:

https://review.coreboot.org/c/coreboot/+/84205/comment/903d52a0_68d72262?usp... :
PS3, Line 239:  = {};
Maybe use a ```memset``` instead of ```{}``` for default value initialization ?

- I don't see anywhere else in the code initializing structs default like this.
- since you have now kept the func ```mem_populate_channel_data``` under a condition ( the function was defaulting this variable to zero ), a memset here might be better IMO.

this is just a suggestion, ignore it if you think it will break the logic.