Attention is currently required from: Nico Huber, Eugene Myers, Paul Menzel, Angel Pons, Michael Niewöhner.
5 comments:
File src/mainboard/acer/aspire_vn7_572g/devicetree.cb:
Patch Set #192, Line 14: .chipset_lockdown = CHIPSET_LOCKDOWN_COREBOOT,
Should no longer be needed, it's the default now.
Done
File src/mainboard/acer/aspire_vn7_572g/romstage.c:
Patch Set #192, Line 13: /* TODO: Search vendor FW for Dq/Dqs */
What does this mean? DQ/DQS mapping is only needed for LPDDR.
The FSP binary has non-zero defaults for Dq/Dqs. Do we know if the FSP detects LPDDR and only applies Dq/Dqs if it's present? If so, I actually can drop this. Otherwise, I should find the optimum settings for this board.
File src/mainboard/acer/aspire_vn7_572g/smihandler.c:
Patch Set #192, Line 26: IDA_Disable
Where does this name come from? IDA was the name from Core 2 era processors, it's called "TURBO_MODE […]
I think that was the definition in the SDM I had? Anyways, done.
Patch Set #192, Line 32: 0x1A0
Why not use the `IA32_MISC_ENABLE` macro from src/include/cpu/x86/msr. […]
Done
Patch Set #192, Line 54: unused_was_osys
Does this still work?
Not really, but OSYS wasn't actually implemented before the recent-ish commits to drop it either. I think that this underlies whether EC supports some features, such as the touchpad toggle hotkey, so I want to get _OSI method back in coreboot.
However, the vendor implements the touchpad toggle hotkey in WMI. While coreboot obviously won't do that, perhaps simply telling the EC to support the feature so that the enable bit works isn't enough. I may need to reverse engineer the WMI blob as well.
I'm adding a TODO for this and marking it as done?
To view, visit change 35523. To unsubscribe, or for help writing mail filters, visit settings.