6 comments:
You can dump the DSDT of the vendor firmware. […]
I've redone the ACPI table since the first, broken one, but haven't tested it yet. I had planned to test all this at once, but the display (mainly dGPU at this time) is still giving me trouble.
Patch Set #45, Line 28: - Remaining RAM slots (need other SPD addresses)
How many RAM slots does this thing have?
The vendor firmware and dmidecode list 4, but the internet says that there are only 2.
I checked the datasheet just now, which also only lists 2. I'll remove this, but will fix it if I find evidence to the contrary.
(I've also realised now that the vendor firmware would have been written for many devices, so hidden information in the advanced settings might even be a lie, after a fashion. dmidecode cannot help with confirmation on this, because it receives information from the vendor firmware)
File src/mainboard/acer/aspire_vn7_572g/devicetree.cb:
Patch Set #13, Line 150: # register "PcieRpEnable[6]" = "1"
NVMe should have x4 lanes. […]
I enabled it in the list of PCI devices and it probably doesn't hurt to enable it in general. I'm just concerned that these ports need special clock values, etc, that I don't know yet.
File src/mainboard/acer/aspire_vn7_572g/devicetree.cb:
Patch Set #40, Line 71: register "SaGv" = "3"
It does, so you should use `SaGv_Enabled`: […]
Done
File src/mainboard/acer/aspire_vn7_572g/hda_verb.c:
Patch Set #45, Line 29: 0x10251037
No need to repeat the literal hex value in the comment
Done
nit: use lowercase for these hex constants
Done
To view, visit change 35523. To unsubscribe, or for help writing mail filters, visit settings.