Change in coreboot[master]: soc/intel/alderlake: Update meminit code due to upd changes FSP 2147 ...

[Bora Guvendik (Code Review)]

Attention is currently required from: Furquan Shaikh, Selma Bensaid, Maulik V Vaghela, Tim Wawrzynczak, Patrick Rudolph.
Bora Guvendik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/54036 )

Change subject: soc/intel/alderlake: Update meminit code due to upd changes FSP 2147 onwards
......................................................................

Patch Set 2:

(5 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/54036/comment/06a9e5ff_ac294d0f 
PS1, Line 7: WIP
> Thanks Furquan, I will fix these.

Done

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

https://review.coreboot.org/c/coreboot/+/54036/comment/041774c9_50bf6172 
PS1, Line 135: disable_dimm_upds
> This should be renamed to `disable_channel_upds` and the pointers need update too.

Done

https://review.coreboot.org/c/coreboot/+/54036/comment/b9a5b3fb_98259325 
PS1, Line 136: DisableDimmMc0Ch0
> Should be replaced by DisableMc0Ch0, right?

I was expecting another patch to take care of that. Done now.

https://review.coreboot.org/c/coreboot/+/54036/comment/82bea405_e8b0abdd 
PS1, Line 164:
> nit: Instead of disabling all channels individually above, you can use a flag here: […]

Done

https://review.coreboot.org/c/coreboot/+/54036/comment/75fe72e8_fbff309e 
PS1, Line 168: break;
> This is wrong. […]

Done