[coreboot-gerrit] Change in coreboot[master]: mb/asus: Add Asus A88XM-E FM2+ with documentation

[Balázs Vinarz (Code Review)]

Balázs Vinarz has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30987 )

Change subject: mb/asus: Add Asus A88XM-E FM2+ with documentation
......................................................................

Patch Set 45:

(1 comment)

It last couple of build I just simply skipped to insert the VGA bios binary.
This time I realized it's pretty needed even for the IOMMU to work correctly, this means that IOMMU was never broken.
But can we be sure that the enabled IOMMU is usable for the Athlon 7xx and 8xx series az well?
Kyoesti, do you have some ideas about it?

https://review.coreboot.org/c/coreboot/+/30987/38/src/mainboard/asus/a88xm-e... 
File src/mainboard/asus/a88xm-e/Kconfig:

https://review.coreboot.org/c/coreboot/+/30987/38/src/mainboard/asus/a88xm-e... 
PS38, Line 39: 	default BOARD_ASUS_A88XM_E_DDR3_VOLT_150
> Should we remove it, then? Or could preserve if it doesn't do any harm, and maybe fix later?

I would say, we should left it and fix later(or never :) ), but definately mention on the Documentation.