Stefan Reinauer <[email protected]> writes: > * Sven Schnelle <[email protected]> [110312 01:18]: >> Signed-off-by: Sven Schnelle <[email protected]> >> --- >> src/mainboard/lenovo/x60/mainboard.c | 5 +++++ >> 1 files changed, 5 insertions(+), 0 deletions(-) >> >> diff --git a/src/mainboard/lenovo/x60/mainboard.c >> b/src/mainboard/lenovo/x60/mainboard.c >> index b8e6a49..79e4a83 100644 >> --- a/src/mainboard/lenovo/x60/mainboard.c >> +++ b/src/mainboard/lenovo/x60/mainboard.c >> @@ -50,6 +50,11 @@ static void wlan_enable(void) >> >> static void mainboard_enable(device_t dev) >> { >> + /* Enable 1600/1604 register pair */ >> + ec_set_bit(0x00, 0x05); >> + /* switch to just enabled registers for ACPI */ >> + ec_set_ports(0x1604, 0x1600); >> + > > I think you only need to use the high ports in SMM code to avoid race > conditions.
Yes. But the original BIOS switches to those ports at the same time, and i want to do it the same way. Makes comparing bus cycles easier, and has no disadvantages IMHO. -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
