[coreboot-gerrit] Change in coreboot[master]: mainboard/asus: Add ASUS H110M-E/M.2 mainboard

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34603 )

Change subject: mainboard/asus: Add ASUS H110M-E/M.2 mainboard
......................................................................

Patch Set 52:

(2 comments)

https://review.coreboot.org/c/coreboot/+/34603/50/src/mainboard/asus/h110m-e... 
File src/mainboard/asus/h110m-e_m2/romstage.c:

https://review.coreboot.org/c/coreboot/+/34603/50/src/mainboard/asus/h110m-e... 
PS50, Line 33: 100, 40, 40, 23, 40
> same here, closest match: 60, 26, 20, 20, 26 (for 2x SO-DIMM)

sorry, 2x SO-DIMM was 2x per channel. but I guess your board has only 1 per
channel? then it should be 50, 26, 20, 20, 26

https://review.coreboot.org/c/coreboot/+/34603/50/src/mainboard/asus/h110m-e... 
PS50, Line 45: 				0x53,
If your board has only 2 DIMMs, you can probably scratch 0x51 and 0x53.