Attention is currently required from: Felix Singer, Paul Menzel, Michał Kopeć, Angel Pons.
Michał Żygowski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68448 )
Change subject: mb/msi/ms7d25: Add support for DDR5 variant ......................................................................
Patch Set 7:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/68448/comment/254ac595_afb4be99 PS7, Line 10: difference is the board's DDR5 memory design.
The only difference seems to be the PCB revision. From internet pictures, the DDR4 version is rev. […]
Why not, in the dmidecode outputs, I can see that the original firmware initialized the revision to 2.0 in SMBIOS tables. I will recheck in the lab what is imprinted on the board.
Patchset:
PS7:
Do you know if GPIO settings are the same for both board revisions? (DDR4 and DDR5)
They are the same. It has been checked.
File configs/config.dell_precision_t1650:
PS7:
Oopsie?
Yeah.. `$ history`:
2052 git add configs/ configs/config.msi_ms7d25_ddr5