diff options
author | Vladimir Serbinenko <phcoder@gmail.com> | 2015-05-27 08:30:47 +0200 |
---|---|---|
committer | Vladimir Serbinenko <phcoder@gmail.com> | 2015-05-29 07:45:55 +0200 |
commit | 852014cf003efa142e1992b45b2a4a627426a5df (patch) | |
tree | 8228c62dde92183866fbaaf9c00613ac1bdc0792 | |
parent | faa46e0cb697371535fef1b806101f1fd57c354e (diff) | |
download | coreboot-852014cf003efa142e1992b45b2a4a627426a5df.tar.xz |
lenovo: Move pc_keyboard_init to h8 init.
PS/2 emulation is part of H8, so should be inited in relevant files.
Change-Id: Ie873ea7f6f88f68f622351799462d0b000d17585
Signed-off-by: Vladimir Serbinenko <phcoder@gmail.com>
Reviewed-on: http://review.coreboot.org/10348
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Edward O'Callaghan <edward.ocallaghan@koparo.com>
Reviewed-by: Alexander Couzens <lynxis@fe80.eu>
-rw-r--r-- | src/ec/lenovo/h8/h8.c | 9 | ||||
-rw-r--r-- | src/mainboard/lenovo/t420s/mainboard.c | 2 | ||||
-rw-r--r-- | src/mainboard/lenovo/t430s/mainboard.c | 2 | ||||
-rw-r--r-- | src/mainboard/lenovo/t520/mainboard.c | 2 | ||||
-rw-r--r-- | src/mainboard/lenovo/t530/mainboard.c | 5 | ||||
-rw-r--r-- | src/mainboard/lenovo/x200/mainboard.c | 9 | ||||
-rw-r--r-- | src/mainboard/lenovo/x201/mainboard.c | 4 | ||||
-rw-r--r-- | src/mainboard/lenovo/x220/mainboard.c | 5 | ||||
-rw-r--r-- | src/mainboard/lenovo/x230/mainboard.c | 5 |
9 files changed, 8 insertions, 35 deletions
diff --git a/src/ec/lenovo/h8/h8.c b/src/ec/lenovo/h8/h8.c index 70424a1f41..46480f4174 100644 --- a/src/ec/lenovo/h8/h8.c +++ b/src/ec/lenovo/h8/h8.c @@ -27,6 +27,7 @@ #include <string.h> #include <smbios.h> #include <pc80/mc146818rtc.h> +#include <pc80/keyboard.h> #include "h8.h" #include "chip.h" @@ -176,8 +177,14 @@ static void h8_smbios_strings(struct device *dev, struct smbios_type11 *t) t->count = smbios_add_string(t->eos, tpec); } +static void h8_init(device_t dev) +{ + pc_keyboard_init(); +} + struct device_operations h8_dev_ops = { - .get_smbios_strings = h8_smbios_strings + .get_smbios_strings = h8_smbios_strings, + .init = h8_init, }; static void h8_enable(struct device *dev) diff --git a/src/mainboard/lenovo/t420s/mainboard.c b/src/mainboard/lenovo/t420s/mainboard.c index e9a400bec2..acebcae7f2 100644 --- a/src/mainboard/lenovo/t420s/mainboard.c +++ b/src/mainboard/lenovo/t420s/mainboard.c @@ -45,8 +45,6 @@ static void mainboard_init(device_t dev) RCBA32(0x38c8) = 0x00002005; RCBA32(0x38c4) = 0x00802005; RCBA32(0x38c0) = 0x00000007; - - pc_keyboard_init(); } static void mainboard_enable(device_t dev) diff --git a/src/mainboard/lenovo/t430s/mainboard.c b/src/mainboard/lenovo/t430s/mainboard.c index 49e828e088..2bbb8bdf43 100644 --- a/src/mainboard/lenovo/t430s/mainboard.c +++ b/src/mainboard/lenovo/t430s/mainboard.c @@ -45,8 +45,6 @@ static void mainboard_init(device_t dev) RCBA32(0x38c8) = 0x00002005; RCBA32(0x38c4) = 0x00802005; RCBA32(0x38c0) = 0x00000007; - - pc_keyboard_init(); } static void mainboard_enable(device_t dev) diff --git a/src/mainboard/lenovo/t520/mainboard.c b/src/mainboard/lenovo/t520/mainboard.c index 609d616c0a..2bfe5430af 100644 --- a/src/mainboard/lenovo/t520/mainboard.c +++ b/src/mainboard/lenovo/t520/mainboard.c @@ -49,8 +49,6 @@ static void mainboard_init(device_t dev) RCBA32(0x38c8) = 0x00002005; RCBA32(0x38c4) = 0x00802005; RCBA32(0x38c0) = 0x00000007; - - pc_keyboard_init(); } /* mainboard_enable is executed as first thing after diff --git a/src/mainboard/lenovo/t530/mainboard.c b/src/mainboard/lenovo/t530/mainboard.c index a81b5b2e80..fe99afad87 100644 --- a/src/mainboard/lenovo/t530/mainboard.c +++ b/src/mainboard/lenovo/t530/mainboard.c @@ -49,11 +49,6 @@ static void mainboard_init(device_t dev) RCBA32(0x38c8) = 0x00002005; RCBA32(0x38c4) = 0x00802005; RCBA32(0x38c0) = 0x00000007; - - /* This sneaked in here, because X201 SuperIO chip isn't really - connected to anything and hence we don't init it. - */ - pc_keyboard_init(); } // mainboard_enable is executed as first thing after diff --git a/src/mainboard/lenovo/x200/mainboard.c b/src/mainboard/lenovo/x200/mainboard.c index 518b51a9dd..3ec615fe00 100644 --- a/src/mainboard/lenovo/x200/mainboard.c +++ b/src/mainboard/lenovo/x200/mainboard.c @@ -37,19 +37,10 @@ static void fill_ssdt(void) drivers_lenovo_serial_ports_ssdt_generate("\\_SB.PCI0.LPCB", 0); } -static void mainboard_init(device_t dev) -{ - /* This sneaked in here, because X200 SuperIO chip isn't really - connected to anything and hence we don't init it. - */ - pc_keyboard_init(); -} - static void mainboard_enable(device_t dev) { install_intel_vga_int15_handler(GMA_INT15_ACTIVE_LFP_INT_LVDS, GMA_INT15_PANEL_FIT_CENTERING, GMA_INT15_BOOT_DISPLAY_DEFAULT, 2); - dev->ops->init = mainboard_init; dev->ops->acpi_fill_ssdt_generator = fill_ssdt; } diff --git a/src/mainboard/lenovo/x201/mainboard.c b/src/mainboard/lenovo/x201/mainboard.c index 85c27b2088..a4da528a93 100644 --- a/src/mainboard/lenovo/x201/mainboard.c +++ b/src/mainboard/lenovo/x201/mainboard.c @@ -85,10 +85,6 @@ static void mainboard_init(device_t dev) RCBA32(0x3804) = 0x3f04e008; printk(BIOS_SPEW, "SPI configured\n"); - /* This sneaked in here, because X201 SuperIO chip isn't really - connected to anything and hence we don't init it. - */ - pc_keyboard_init(); } static void fill_ssdt(void) diff --git a/src/mainboard/lenovo/x220/mainboard.c b/src/mainboard/lenovo/x220/mainboard.c index 570e74a872..04b8e7fe9f 100644 --- a/src/mainboard/lenovo/x220/mainboard.c +++ b/src/mainboard/lenovo/x220/mainboard.c @@ -44,11 +44,6 @@ static void mainboard_init(device_t dev) RCBA32(0x38c8) = 0x00002005; RCBA32(0x38c4) = 0x00802005; RCBA32(0x38c0) = 0x00000007; - - /* This sneaked in here, because X201 SuperIO chip isn't really - connected to anything and hence we don't init it. - */ - pc_keyboard_init(); } // mainboard_enable is executed as first thing after diff --git a/src/mainboard/lenovo/x230/mainboard.c b/src/mainboard/lenovo/x230/mainboard.c index 8ea18dea04..09b8d69334 100644 --- a/src/mainboard/lenovo/x230/mainboard.c +++ b/src/mainboard/lenovo/x230/mainboard.c @@ -45,11 +45,6 @@ static void mainboard_init(device_t dev) RCBA32(0x38c8) = 0x00002005; RCBA32(0x38c4) = 0x00802005; RCBA32(0x38c0) = 0x00000007; - - /* This sneaked in here, because X201 SuperIO chip isn't really - connected to anything and hence we don't init it. - */ - pc_keyboard_init(); } // mainboard_enable is executed as first thing after |