coreboot-gerrit
Threads by month
- ----- 2026 -----
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2025 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2024 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2023 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2022 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2021 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2020 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2019 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2018 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2017 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2016 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2015 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2014 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2013 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
April 2014
- 1 participants
- 779 discussions
Patch set updated for coreboot: ad2aef7 mainboard/supermicro/h8qme_fam10: Provide local pnp_ sio func
by Edward O'Callaghan April 30, 2014
by Edward O'Callaghan April 30, 2014
April 30, 2014
Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5610
-gerrit
commit ad2aef76cf696346dae85afe1de6c8c56a1f9479
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Tue Apr 29 17:49:35 2014 +1000
mainboard/supermicro/h8qme_fam10: Provide local pnp_ sio func
Provide local superio pnp_ programming entry/exit functions as to avoid
making superio implementation global symbols. Although this is not the
proper/final solution, it does mitigate possibly symbol collisions and
allow for continued superio refactorisation.
Change-Id: I5d3421ff9a12413da1e570c09d507206f3f09641
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
src/mainboard/supermicro/h8qme_fam10/romstage.c | 18 ++++++++++++++++--
1 file changed, 16 insertions(+), 2 deletions(-)
diff --git a/src/mainboard/supermicro/h8qme_fam10/romstage.c b/src/mainboard/supermicro/h8qme_fam10/romstage.c
index cca464c..f3c224c 100644
--- a/src/mainboard/supermicro/h8qme_fam10/romstage.c
+++ b/src/mainboard/supermicro/h8qme_fam10/romstage.c
@@ -39,8 +39,8 @@
#include "lib/delay.c"
#include "cpu/x86/lapic.h"
#include "northbridge/amd/amdfam10/reset_test.c"
-#include "superio/winbond/w83627hf/early_serial.c"
-#include "superio/winbond/w83627hf/early_init.c"
+#include <superio/winbond/common/winbond.h>
+#include <superio/winbond/w83627hf/w83627hf.h>
#include "cpu/x86/bist.h"
#include "northbridge/amd/amdfam10/debug.c"
#include "northbridge/amd/amdfam10/setup_resource_map.c"
@@ -114,6 +114,20 @@ static const u8 spd_addr[] = {
#define GPIO2_DEV PNP_DEV(0x2e, W83627HF_GPIO2)
#define GPIO3_DEV PNP_DEV(0x2e, W83627HF_GPIO3)
+/* TODO: superio code should really not be in mainboard */
+static void pnp_enter_ext_func_mode(device_t dev)
+{
+ u16 port = dev >> 8;
+ outb(0x87, port);
+ outb(0x87, port);
+}
+
+static void pnp_exit_ext_func_mode(device_t dev)
+{
+ u16 port = dev >> 8;
+ outb(0xaa, port);
+}
+
static void write_GPIO(void)
{
pnp_enter_ext_func_mode(GPIO1_DEV);
1
0
Patch set updated for coreboot: b51c318 superio/winbond/w83627hf: Avoid .c includes in mainboards
by Edward O'Callaghan April 30, 2014
by Edward O'Callaghan April 30, 2014
April 30, 2014
Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5601
-gerrit
commit b51c31819c86a3715d808632414c131b1a62842c
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Tue Apr 29 13:09:50 2014 +1000
superio/winbond/w83627hf: Avoid .c includes in mainboards
Move towards the removal od rhw superio model specific xxx_serial_enable
implementation. Make remaining superio romstage parts link-time symbols
and fix corresponding mainboards to match.
The following mainboards remain unconverted as they are ROMCC:
- mainboard/supermicro/x6dai_g
- mainboard/supermicro/x6dhe_g
- mainboard/supermicro/x6dhr_ig
- mainboard/supermicro/x6dhr_ig2
and so block the final removal of w83627hf_serial_enable().
Change-Id: Iaefb25d77512503050cb38313ca90855ebb538ad
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
src/mainboard/advansus/a785e-i/romstage.c | 7 +++--
src/mainboard/avalue/eax-785e/romstage.c | 7 +++--
src/mainboard/msi/ms6178/romstage.c | 5 ++--
src/mainboard/pcengines/alix1c/romstage.c | 5 ++--
src/mainboard/supermicro/h8dme/romstage.c | 6 ++--
src/mainboard/supermicro/h8dmr/romstage.c | 8 +++---
src/mainboard/supermicro/h8dmr_fam10/romstage.c | 6 ++--
src/mainboard/supermicro/h8qme_fam10/romstage.c | 4 +--
src/mainboard/supermicro/x6dai_g/romstage.c | 2 +-
src/mainboard/supermicro/x6dhe_g/romstage.c | 2 +-
src/mainboard/supermicro/x6dhr_ig/romstage.c | 2 +-
src/mainboard/supermicro/x6dhr_ig2/romstage.c | 2 +-
src/mainboard/winent/pl6064/romstage.c | 5 ++--
src/superio/winbond/w83627hf/Makefile.inc | 2 +-
src/superio/winbond/w83627hf/early_init.c | 38 -------------------------
src/superio/winbond/w83627hf/early_serial.c | 10 +++++++
src/superio/winbond/w83627hf/w83627hf.h | 14 ++++-----
17 files changed, 52 insertions(+), 73 deletions(-)
diff --git a/src/mainboard/advansus/a785e-i/romstage.c b/src/mainboard/advansus/a785e-i/romstage.c
index 490d146..2402798 100644
--- a/src/mainboard/advansus/a785e-i/romstage.c
+++ b/src/mainboard/advansus/a785e-i/romstage.c
@@ -41,7 +41,8 @@
#include "northbridge/amd/amdfam10/reset_test.c"
#include <console/loglevel.h>
#include "cpu/x86/bist.h"
-#include "superio/winbond/w83627hf/early_serial.c"
+#include <superio/winbond/common/winbond.h>
+#include <superio/winbond/w83627hf/w83627hf.h>
#include <cpu/amd/mtrr.h>
#include "northbridge/amd/amdfam10/setup_resource_map.c"
#include "southbridge/amd/rs780/early_setup.c"
@@ -50,6 +51,8 @@
#include <southbridge/amd/cimx/sb800/smbus.h>
#include "northbridge/amd/amdfam10/debug.c"
+#define SERIAL_DEV PNP_DEV(0x2e, W83627HF_SP1)
+
static void activate_spd_rom(const struct mem_controller *ctrl)
{
}
@@ -100,7 +103,7 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
sb800_clk_output_48Mhz();
w83627hf_set_clksel_48(PNP_DEV(0x2e, 0));
- w83627hf_enable_serial(0, CONFIG_TTYS0_BASE);
+ winbond_enable_serial(SERIAL_DEV, CONFIG_TTYS0_BASE);
console_init();
printk(BIOS_DEBUG, "\n");
diff --git a/src/mainboard/avalue/eax-785e/romstage.c b/src/mainboard/avalue/eax-785e/romstage.c
index 69cefd9..f6a10dc 100644
--- a/src/mainboard/avalue/eax-785e/romstage.c
+++ b/src/mainboard/avalue/eax-785e/romstage.c
@@ -41,7 +41,8 @@
#include "northbridge/amd/amdfam10/reset_test.c"
#include <console/loglevel.h>
#include "cpu/x86/bist.h"
-#include "superio/winbond/w83627hf/early_serial.c"
+#include <superio/winbond/common/winbond.h>
+#include <superio/winbond/w83627hf/w83627hf.h>
#include <cpu/amd/mtrr.h>
#include "northbridge/amd/amdfam10/setup_resource_map.c"
#include "southbridge/amd/rs780/early_setup.c"
@@ -50,6 +51,8 @@
#include <southbridge/amd/cimx/sb800/smbus.h>
#include "northbridge/amd/amdfam10/debug.c"
+#define SERIAL_DEV PNP_DEV(0x2e, W83627HF_SP1)
+
static void activate_spd_rom(const struct mem_controller *ctrl)
{
}
@@ -100,7 +103,7 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
sb800_clk_output_48Mhz();
w83627hf_set_clksel_48(PNP_DEV(CONFIG_SIO_PORT, 0));
- w83627hf_enable_serial(0, CONFIG_TTYS0_BASE);
+ winbond_enable_serial(SERIAL_DEV, CONFIG_TTYS0_BASE);
console_init();
printk(BIOS_DEBUG, "\n");
diff --git a/src/mainboard/msi/ms6178/romstage.c b/src/mainboard/msi/ms6178/romstage.c
index 518944f..fa78a82 100644
--- a/src/mainboard/msi/ms6178/romstage.c
+++ b/src/mainboard/msi/ms6178/romstage.c
@@ -25,7 +25,8 @@
#include <arch/hlt.h>
#include <stdlib.h>
#include <console/console.h>
-#include "superio/winbond/w83627hf/early_serial.c"
+#include <superio/winbond/common/winbond.h>
+#include <superio/winbond/w83627hf/w83627hf.h>
#include "northbridge/intel/i82810/raminit.h"
#include "cpu/x86/bist.h"
#include "southbridge/intel/i82801ax/i82801ax.h"
@@ -38,7 +39,7 @@
void main(unsigned long bist)
{
w83627hf_set_clksel_48(DUMMY_DEV);
- w83627hf_enable_serial(SERIAL_DEV, CONFIG_TTYS0_BASE);
+ winbond_enable_serial(SERIAL_DEV, CONFIG_TTYS0_BASE);
console_init();
enable_smbus();
diff --git a/src/mainboard/pcengines/alix1c/romstage.c b/src/mainboard/pcengines/alix1c/romstage.c
index 1c4ae09..204f0bd 100644
--- a/src/mainboard/pcengines/alix1c/romstage.c
+++ b/src/mainboard/pcengines/alix1c/romstage.c
@@ -39,7 +39,8 @@
static void cs5536_enable_smbus(void) { }
#include "southbridge/amd/cs5536/early_setup.c"
-#include "superio/winbond/w83627hf/early_serial.c"
+#include <superio/winbond/common/winbond.h>
+#include <superio/winbond/w83627hf/w83627hf.h>
/* The part is a Hynix hy5du121622ctp-d43.
*
@@ -121,7 +122,7 @@ void main(unsigned long bist)
* It is counting on some early MSR setup for the CS5536.
*/
cs5536_disable_internal_uart();
- w83627hf_enable_serial(SERIAL_DEV, CONFIG_TTYS0_BASE);
+ winbond_enable_serial(SERIAL_DEV, CONFIG_TTYS0_BASE);
console_init();
/* Halt if there was a built in self test failure */
diff --git a/src/mainboard/supermicro/h8dme/romstage.c b/src/mainboard/supermicro/h8dme/romstage.c
index df5e2c8..47f5806 100644
--- a/src/mainboard/supermicro/h8dme/romstage.c
+++ b/src/mainboard/supermicro/h8dme/romstage.c
@@ -33,8 +33,8 @@
#include "lib/delay.c"
#include "cpu/x86/lapic.h"
#include "northbridge/amd/amdk8/reset_test.c"
-#include "superio/winbond/w83627hf/early_serial.c"
-#include "superio/winbond/w83627hf/early_init.c"
+#include <superio/winbond/common/winbond.h>
+#include <superio/winbond/w83627hf/w83627hf.h>
#include "cpu/x86/bist.h"
#include "northbridge/amd/amdk8/debug.c"
#include "northbridge/amd/amdk8/setup_resource_map.c"
@@ -186,7 +186,7 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
bsp_apicid = init_cpus(cpu_init_detectedx, sysinfo);
w83627hf_set_clksel_48(DUMMY_DEV);
- w83627hf_enable_serial(SERIAL_DEV, CONFIG_TTYS0_BASE);
+ winbond_enable_serial(SERIAL_DEV, CONFIG_TTYS0_BASE);
console_init();
diff --git a/src/mainboard/supermicro/h8dmr/romstage.c b/src/mainboard/supermicro/h8dmr/romstage.c
index 8ed7e6d..7d1f834 100644
--- a/src/mainboard/supermicro/h8dmr/romstage.c
+++ b/src/mainboard/supermicro/h8dmr/romstage.c
@@ -36,8 +36,8 @@
#include "lib/delay.c"
#include "cpu/x86/lapic.h"
#include "northbridge/amd/amdk8/reset_test.c"
-#include "superio/winbond/w83627hf/early_serial.c"
-#include "superio/winbond/w83627hf/early_init.c"
+#include <superio/winbond/common/winbond.h>
+#include <superio/winbond/w83627hf/w83627hf.h>
#include "cpu/x86/bist.h"
#include "northbridge/amd/amdk8/debug.c"
#include "northbridge/amd/amdk8/setup_resource_map.c"
@@ -114,8 +114,8 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
if (bist == 0)
bsp_apicid = init_cpus(cpu_init_detectedx, sysinfo);
- w83627hf_set_clksel_48(DUMMY_DEV);
- w83627hf_enable_serial(SERIAL_DEV, CONFIG_TTYS0_BASE);
+ w83627hf_set_clksel_48(DUMMY_DEV);
+ winbond_enable_serial(SERIAL_DEV, CONFIG_TTYS0_BASE);
console_init();
diff --git a/src/mainboard/supermicro/h8dmr_fam10/romstage.c b/src/mainboard/supermicro/h8dmr_fam10/romstage.c
index 3f6ea70..cd185f3 100644
--- a/src/mainboard/supermicro/h8dmr_fam10/romstage.c
+++ b/src/mainboard/supermicro/h8dmr_fam10/romstage.c
@@ -39,8 +39,8 @@
#include "lib/delay.c"
#include "cpu/x86/lapic.h"
#include "northbridge/amd/amdfam10/reset_test.c"
-#include "superio/winbond/w83627hf/early_serial.c"
-#include "superio/winbond/w83627hf/early_init.c"
+#include <superio/winbond/common/winbond.h>
+#include <superio/winbond/w83627hf/w83627hf.h>
#include "cpu/x86/bist.h"
#include "northbridge/amd/amdfam10/debug.c"
#include "northbridge/amd/amdfam10/setup_resource_map.c"
@@ -122,7 +122,7 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
post_code(0x32);
w83627hf_set_clksel_48(DUMMY_DEV);
- w83627hf_enable_serial(SERIAL_DEV, CONFIG_TTYS0_BASE);
+ winbond_enable_serial(SERIAL_DEV, CONFIG_TTYS0_BASE);
console_init();
diff --git a/src/mainboard/supermicro/h8qme_fam10/romstage.c b/src/mainboard/supermicro/h8qme_fam10/romstage.c
index f3c224c..9898a25 100644
--- a/src/mainboard/supermicro/h8qme_fam10/romstage.c
+++ b/src/mainboard/supermicro/h8qme_fam10/romstage.c
@@ -186,8 +186,8 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
post_code(0x32);
- w83627hf_set_clksel_48(DUMMY_DEV);
- w83627hf_enable_serial(SERIAL_DEV, CONFIG_TTYS0_BASE);
+ w83627hf_set_clksel_48(DUMMY_DEV);
+ winbond_enable_serial(SERIAL_DEV, CONFIG_TTYS0_BASE);
console_init();
write_GPIO();
diff --git a/src/mainboard/supermicro/x6dai_g/romstage.c b/src/mainboard/supermicro/x6dai_g/romstage.c
index f57adaf..c998466 100644
--- a/src/mainboard/supermicro/x6dai_g/romstage.c
+++ b/src/mainboard/supermicro/x6dai_g/romstage.c
@@ -9,13 +9,13 @@
#include "lib/delay.c"
#include "southbridge/intel/esb6300/early_smbus.c"
#include "northbridge/intel/e7525/raminit.h"
-#include "superio/winbond/w83627hf/w83627hf.h"
#include "cpu/x86/lapic/boot_cpu.c"
#include "cpu/x86/mtrr/earlymtrr.c"
#include "debug.c"
#include "watchdog.c"
#include "southbridge/intel/esb6300/reset.c"
#include "superio/winbond/w83627hf/early_serial.c"
+#include <superio/winbond/w83627hf/w83627hf.h>
#include "northbridge/intel/e7525/memory_initialized.c"
#include "cpu/x86/bist.h"
#include <spd.h>
diff --git a/src/mainboard/supermicro/x6dhe_g/romstage.c b/src/mainboard/supermicro/x6dhe_g/romstage.c
index 071bb35..b1c3cfa 100644
--- a/src/mainboard/supermicro/x6dhe_g/romstage.c
+++ b/src/mainboard/supermicro/x6dhe_g/romstage.c
@@ -9,13 +9,13 @@
#include "lib/delay.c"
#include "southbridge/intel/esb6300/early_smbus.c"
#include "northbridge/intel/e7520/raminit.h"
-#include "superio/winbond/w83627hf/w83627hf.h"
#include "cpu/x86/lapic/boot_cpu.c"
#include "cpu/x86/mtrr/earlymtrr.c"
#include "debug.c"
#include "watchdog.c"
#include "southbridge/intel/esb6300/reset.c"
#include "superio/winbond/w83627hf/early_serial.c"
+#include <superio/winbond/w83627hf/w83627hf.h>
#include "northbridge/intel/e7520/memory_initialized.c"
#include "cpu/x86/bist.h"
#include <spd.h>
diff --git a/src/mainboard/supermicro/x6dhr_ig/romstage.c b/src/mainboard/supermicro/x6dhr_ig/romstage.c
index 9c61d60..2dc96c1 100644
--- a/src/mainboard/supermicro/x6dhr_ig/romstage.c
+++ b/src/mainboard/supermicro/x6dhr_ig/romstage.c
@@ -7,13 +7,13 @@
#include <console/console.h>
#include "southbridge/intel/i82801ex/early_smbus.c"
#include "northbridge/intel/e7520/raminit.h"
-#include "superio/winbond/w83627hf/w83627hf.h"
#include "cpu/x86/lapic/boot_cpu.c"
#include "cpu/x86/mtrr/earlymtrr.c"
#include "debug.c"
#include "watchdog.c"
#include "southbridge/intel/i82801ex/reset.c"
#include "superio/winbond/w83627hf/early_serial.c"
+#include <superio/winbond/w83627hf/w83627hf.h>
#include "northbridge/intel/e7520/memory_initialized.c"
#include "cpu/x86/bist.h"
#include <spd.h>
diff --git a/src/mainboard/supermicro/x6dhr_ig2/romstage.c b/src/mainboard/supermicro/x6dhr_ig2/romstage.c
index 4690a0c..0f2a644 100644
--- a/src/mainboard/supermicro/x6dhr_ig2/romstage.c
+++ b/src/mainboard/supermicro/x6dhr_ig2/romstage.c
@@ -7,13 +7,13 @@
#include <console/console.h>
#include "southbridge/intel/i82801ex/early_smbus.c"
#include "northbridge/intel/e7520/raminit.h"
-#include "superio/winbond/w83627hf/w83627hf.h"
#include "cpu/x86/lapic/boot_cpu.c"
#include "cpu/x86/mtrr/earlymtrr.c"
#include "debug.c"
#include "watchdog.c"
#include "southbridge/intel/i82801ex/reset.c"
#include "superio/winbond/w83627hf/early_serial.c"
+#include <superio/winbond/w83627hf/w83627hf.h>
#include "northbridge/intel/e7520/memory_initialized.c"
#include "cpu/x86/bist.h"
#include <spd.h>
diff --git a/src/mainboard/winent/pl6064/romstage.c b/src/mainboard/winent/pl6064/romstage.c
index 124183c..6a38355 100644
--- a/src/mainboard/winent/pl6064/romstage.c
+++ b/src/mainboard/winent/pl6064/romstage.c
@@ -34,7 +34,8 @@
#include <spd.h>
#include "southbridge/amd/cs5536/early_smbus.c"
#include "southbridge/amd/cs5536/early_setup.c"
-#include "superio/winbond/w83627hf/early_serial.c"
+#include <superio/winbond/common/winbond.h>
+#include <superio/winbond/w83627hf/w83627hf.h>
#include "northbridge/amd/lx/raminit.h"
#define SERIAL_DEV PNP_DEV(0x2e, W83627HF_SP1)
@@ -66,7 +67,7 @@ void main(unsigned long bist)
* early MSR setup for CS5536.
*/
w83627hf_set_clksel_48(SERIAL_DEV);
- w83627hf_enable_serial(SERIAL_DEV, CONFIG_TTYS0_BASE);
+ winbond_enable_serial(SERIAL_DEV, CONFIG_TTYS0_BASE);
console_init();
/* Halt if there was a built in self test failure */
diff --git a/src/superio/winbond/w83627hf/Makefile.inc b/src/superio/winbond/w83627hf/Makefile.inc
index be5ec48..d6a8869 100644
--- a/src/superio/winbond/w83627hf/Makefile.inc
+++ b/src/superio/winbond/w83627hf/Makefile.inc
@@ -20,5 +20,5 @@
## Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
##
+romstage-$(CONFIG_SUPERIO_WINBOND_W83627HF) += early_serial.c
ramstage-$(CONFIG_SUPERIO_WINBOND_W83627HF) += superio.c
-
diff --git a/src/superio/winbond/w83627hf/early_init.c b/src/superio/winbond/w83627hf/early_init.c
deleted file mode 100644
index 66b4098..0000000
--- a/src/superio/winbond/w83627hf/early_init.c
+++ /dev/null
@@ -1,38 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2000 AG Electronics Ltd.
- * Copyright 2003-2004 Linux Networx
- * Copyright 2004 Tyan
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#include <arch/io.h>
-#include "w83627hf.h"
-
-void w83627hf_disable_dev(device_t dev)
-{
- pnp_set_logical_device(dev);
- pnp_set_enable(dev, 0);
-}
-
-void w83627hf_enable_dev(device_t dev, u16 iobase)
-{
- pnp_set_logical_device(dev);
- pnp_set_enable(dev, 0);
- pnp_set_iobase(dev, PNP_IDX_IO0, iobase);
- pnp_set_enable(dev, 1);
-}
diff --git a/src/superio/winbond/w83627hf/early_serial.c b/src/superio/winbond/w83627hf/early_serial.c
index db2827b..35c0b86 100644
--- a/src/superio/winbond/w83627hf/early_serial.c
+++ b/src/superio/winbond/w83627hf/early_serial.c
@@ -22,6 +22,7 @@
*/
#include <arch/io.h>
+#include <device/pnp.h>
#include "w83627hf.h"
static void pnp_enter_ext_func_mode(device_t dev)
@@ -37,6 +38,15 @@ static void pnp_exit_ext_func_mode(device_t dev)
outb(0xaa, port);
}
+/*
+ * FIXME: The following ROMCC boards are blocking the removal this superio's
+ * model specific w83627hf_enable_serial() symbol.
+ *
+ * mainboard/supermicro/x6dai_g
+ * mainboard/supermicro/x6dhe_g
+ * mainboard/supermicro/x6dhr_ig
+ * mainboard/supermicro/x6dhr_ig2
+ */
void w83627hf_enable_serial(device_t dev, u16 iobase)
{
pnp_enter_ext_func_mode(dev);
diff --git a/src/superio/winbond/w83627hf/w83627hf.h b/src/superio/winbond/w83627hf/w83627hf.h
index 468cb55..bf7186e 100644
--- a/src/superio/winbond/w83627hf/w83627hf.h
+++ b/src/superio/winbond/w83627hf/w83627hf.h
@@ -20,8 +20,8 @@
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*/
-#ifndef SUPERIO_WINBOND_W83627HF_W83627HF_H
-#define SUPERIO_WINBOND_W83627HF_W83627HF_H
+#ifndef SUPERIO_WINBOND_W83627HF_H
+#define SUPERIO_WINBOND_W83627HF_H
#define W83627HF_FDC 0 /* Floppy */
#define W83627HF_PP 1 /* Parallel port */
@@ -113,11 +113,9 @@
#define W83627HF_XSCNF 0x15
#define W83627HF_XWBCNF 0x16
-#if defined(__PRE_RAM__)
-void w83627hf_disable_dev(device_t dev);
-void w83627hf_enable_dev(device_t dev, u16 iobase);
-void w83627hf_enable_serial(device_t dev, u16 iobase);
+#include <arch/io.h>
+
void w83627hf_set_clksel_48(device_t dev);
-#endif
+void w83627hf_enable_serial(device_t dev, u16 iobase);
-#endif
+#endif /* SUPERIO_WINBOND_W83627HF_H */
1
0
New patch to review for coreboot: 371faf8 mainboard/*: Use std memset/memcpy func over AGESA
by Edward O'Callaghan April 30, 2014
by Edward O'Callaghan April 30, 2014
April 30, 2014
Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5625
-gerrit
commit 371faf8201e2c1c9832a91b5502f912708d4ece1
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Wed Apr 30 23:13:08 2014 +1000
mainboard/*: Use std memset/memcpy func over AGESA
In amd/{persimmon,inagua} and derived boards avoid using AGESA
reimplementation of memcpy as following the reasoning in:
e2f3bfc jetway/nf81-t56n-lf: Use std memset/memcpy func over AGESA
Change-Id: I943b46103c3bf1c5fd88b25e9f9595b9adfcafeb
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
src/mainboard/amd/inagua/PlatformGnbPcie.c | 23 +++++-----------------
src/mainboard/amd/persimmon/PlatformGnbPcie.c | 23 +++++-----------------
src/mainboard/amd/south_station/PlatformGnbPcie.c | 23 +++++-----------------
src/mainboard/amd/union_station/PlatformGnbPcie.c | 23 +++++-----------------
src/mainboard/asrock/e350m1/PlatformGnbPcie.c | 23 +++++-----------------
src/mainboard/gizmosphere/gizmo/PlatformGnbPcie.c | 23 +++++-----------------
.../lippert/frontrunner-af/PlatformGnbPcie.c | 23 +++++-----------------
src/mainboard/lippert/toucan-af/PlatformGnbPcie.c | 23 +++++-----------------
8 files changed, 40 insertions(+), 144 deletions(-)
diff --git a/src/mainboard/amd/inagua/PlatformGnbPcie.c b/src/mainboard/amd/inagua/PlatformGnbPcie.c
index c64e523..bf07ff3 100644
--- a/src/mainboard/amd/inagua/PlatformGnbPcie.c
+++ b/src/mainboard/amd/inagua/PlatformGnbPcie.c
@@ -24,6 +24,8 @@
#include "PlatformGnbPcieComplex.h"
#include "Filecode.h"
+#include <string.h>
+
#define FILECODE PROC_RECOVERY_MEM_NB_ON_MRNON_FILECODE
/*---------------------------------------------------------------------------------------*/
@@ -127,24 +129,9 @@ OemCustomizeInitEarly (
AllocHeapParams.BufferPtr += sizeof(PortList);
BrazosPcieDdiPtr = (PCIe_DDI_DESCRIPTOR *) AllocHeapParams.BufferPtr;
- LibAmdMemFill (BrazosPcieComplexListPtr,
- 0,
- sizeof(Brazos),
- &InitEarly->StdHeader);
-
- LibAmdMemFill (BrazosPciePortPtr,
- 0,
- sizeof(PortList),
- &InitEarly->StdHeader);
-
- LibAmdMemFill (BrazosPcieDdiPtr,
- 0,
- sizeof(DdiList),
- &InitEarly->StdHeader);
-
- LibAmdMemCopy (BrazosPcieComplexListPtr, &Brazos, sizeof(Brazos), &InitEarly->StdHeader);
- LibAmdMemCopy (BrazosPciePortPtr, &PortList[0], sizeof(PortList), &InitEarly->StdHeader);
- LibAmdMemCopy (BrazosPcieDdiPtr, &DdiList[0], sizeof(DdiList), &InitEarly->StdHeader);
+ memcpy(BrazosPcieComplexListPtr, &Brazos, sizeof(Brazos));
+ memcpy(BrazosPciePortPtr, &PortList[0], sizeof(PortList));
+ memcpy(BrazosPcieDdiPtr, &DdiList[0], sizeof(DdiList));
((PCIe_COMPLEX_DESCRIPTOR*)BrazosPcieComplexListPtr)->PciePortList = (PCIe_PORT_DESCRIPTOR*)BrazosPciePortPtr;
diff --git a/src/mainboard/amd/persimmon/PlatformGnbPcie.c b/src/mainboard/amd/persimmon/PlatformGnbPcie.c
index 3cd69f1..4ec5f2b 100644
--- a/src/mainboard/amd/persimmon/PlatformGnbPcie.c
+++ b/src/mainboard/amd/persimmon/PlatformGnbPcie.c
@@ -25,6 +25,8 @@
#include "Filecode.h"
#include "BiosCallOuts.h"
+#include <string.h>
+
#define FILECODE PROC_RECOVERY_MEM_NB_ON_MRNON_FILECODE
/*---------------------------------------------------------------------------------------*/
@@ -136,24 +138,9 @@ PCIe_COMPLEX_DESCRIPTOR Brazos = {
AllocHeapParams.BufferPtr += sizeof(PortList);
BrazosPcieDdiPtr = (PCIe_DDI_DESCRIPTOR *) AllocHeapParams.BufferPtr;
- LibAmdMemFill (BrazosPcieComplexListPtr,
- 0,
- sizeof(Brazos),
- &InitEarly->StdHeader);
-
- LibAmdMemFill (BrazosPciePortPtr,
- 0,
- sizeof(PortList),
- &InitEarly->StdHeader);
-
- LibAmdMemFill (BrazosPcieDdiPtr,
- 0,
- sizeof(DdiList),
- &InitEarly->StdHeader);
-
- LibAmdMemCopy (BrazosPcieComplexListPtr, &Brazos, sizeof(Brazos), &InitEarly->StdHeader);
- LibAmdMemCopy (BrazosPciePortPtr, &PortList[0], sizeof(PortList), &InitEarly->StdHeader);
- LibAmdMemCopy (BrazosPcieDdiPtr, &DdiList[0], sizeof(DdiList), &InitEarly->StdHeader);
+ memcpy(BrazosPcieComplexListPtr, &Brazos, sizeof(Brazos));
+ memcpy(BrazosPciePortPtr, &PortList[0], sizeof(PortList));
+ memcpy(BrazosPcieDdiPtr, &DdiList[0], sizeof(DdiList));
((PCIe_COMPLEX_DESCRIPTOR*)BrazosPcieComplexListPtr)->PciePortList = (PCIe_PORT_DESCRIPTOR*)BrazosPciePortPtr;
diff --git a/src/mainboard/amd/south_station/PlatformGnbPcie.c b/src/mainboard/amd/south_station/PlatformGnbPcie.c
index a8511ea..4ad49be 100644
--- a/src/mainboard/amd/south_station/PlatformGnbPcie.c
+++ b/src/mainboard/amd/south_station/PlatformGnbPcie.c
@@ -24,6 +24,8 @@
#include "PlatformGnbPcieComplex.h"
#include "Filecode.h"
+#include <string.h>
+
#define FILECODE PROC_RECOVERY_MEM_NB_ON_MRNON_FILECODE
/*---------------------------------------------------------------------------------------*/
@@ -135,24 +137,9 @@ PCIe_COMPLEX_DESCRIPTOR Brazos = {
AllocHeapParams.BufferPtr += sizeof(PortList);
BrazosPcieDdiPtr = (PCIe_DDI_DESCRIPTOR *) AllocHeapParams.BufferPtr;
- LibAmdMemFill (BrazosPcieComplexListPtr,
- 0,
- sizeof(Brazos),
- &InitEarly->StdHeader);
-
- LibAmdMemFill (BrazosPciePortPtr,
- 0,
- sizeof(PortList),
- &InitEarly->StdHeader);
-
- LibAmdMemFill (BrazosPcieDdiPtr,
- 0,
- sizeof(DdiList),
- &InitEarly->StdHeader);
-
- LibAmdMemCopy (BrazosPcieComplexListPtr, &Brazos, sizeof(Brazos), &InitEarly->StdHeader);
- LibAmdMemCopy (BrazosPciePortPtr, &PortList[0], sizeof(PortList), &InitEarly->StdHeader);
- LibAmdMemCopy (BrazosPcieDdiPtr, &DdiList[0], sizeof(DdiList), &InitEarly->StdHeader);
+ memcpy(BrazosPcieComplexListPtr, &Brazos, sizeof(Brazos));
+ memcpy(BrazosPciePortPtr, &PortList[0], sizeof(PortList));
+ memcpy(BrazosPcieDdiPtr, &DdiList[0], sizeof(DdiList));
((PCIe_COMPLEX_DESCRIPTOR*)BrazosPcieComplexListPtr)->PciePortList = (PCIe_PORT_DESCRIPTOR*)BrazosPciePortPtr;
diff --git a/src/mainboard/amd/union_station/PlatformGnbPcie.c b/src/mainboard/amd/union_station/PlatformGnbPcie.c
index 829b6c9..be1c3a5 100644
--- a/src/mainboard/amd/union_station/PlatformGnbPcie.c
+++ b/src/mainboard/amd/union_station/PlatformGnbPcie.c
@@ -24,6 +24,8 @@
#include "PlatformGnbPcieComplex.h"
#include "Filecode.h"
+#include <string.h>
+
#define FILECODE PROC_RECOVERY_MEM_NB_ON_MRNON_FILECODE
/*---------------------------------------------------------------------------------------*/
@@ -137,24 +139,9 @@ PCIe_COMPLEX_DESCRIPTOR Brazos = {
AllocHeapParams.BufferPtr += sizeof(PortList);
BrazosPcieDdiPtr = (PCIe_DDI_DESCRIPTOR *) AllocHeapParams.BufferPtr;
- LibAmdMemFill (BrazosPcieComplexListPtr,
- 0,
- sizeof(Brazos),
- &InitEarly->StdHeader);
-
- LibAmdMemFill (BrazosPciePortPtr,
- 0,
- sizeof(PortList),
- &InitEarly->StdHeader);
-
- LibAmdMemFill (BrazosPcieDdiPtr,
- 0,
- sizeof(DdiList),
- &InitEarly->StdHeader);
-
- LibAmdMemCopy (BrazosPcieComplexListPtr, &Brazos, sizeof(Brazos), &InitEarly->StdHeader);
- LibAmdMemCopy (BrazosPciePortPtr, &PortList[0], sizeof(PortList), &InitEarly->StdHeader);
- LibAmdMemCopy (BrazosPcieDdiPtr, &DdiList[0], sizeof(DdiList), &InitEarly->StdHeader);
+ memcpy(BrazosPcieComplexListPtr, &Brazos, sizeof(Brazos));
+ memcpy(BrazosPciePortPtr, &PortList[0], sizeof(PortList));
+ memcpy(BrazosPcieDdiPtr, &DdiList[0], sizeof(DdiList));
((PCIe_COMPLEX_DESCRIPTOR*)BrazosPcieComplexListPtr)->PciePortList = (PCIe_PORT_DESCRIPTOR*)BrazosPciePortPtr;
diff --git a/src/mainboard/asrock/e350m1/PlatformGnbPcie.c b/src/mainboard/asrock/e350m1/PlatformGnbPcie.c
index 50a8db9..ed20b59 100644
--- a/src/mainboard/asrock/e350m1/PlatformGnbPcie.c
+++ b/src/mainboard/asrock/e350m1/PlatformGnbPcie.c
@@ -24,6 +24,8 @@
#include "PlatformGnbPcieComplex.h"
#include "Filecode.h"
+#include <string.h>
+
#define FILECODE PROC_RECOVERY_MEM_NB_ON_MRNON_FILECODE
/*---------------------------------------------------------------------------------------*/
@@ -137,24 +139,9 @@ PCIe_COMPLEX_DESCRIPTOR Brazos = {
AllocHeapParams.BufferPtr += sizeof(PortList);
BrazosPcieDdiPtr = (PCIe_DDI_DESCRIPTOR *) AllocHeapParams.BufferPtr;
- LibAmdMemFill (BrazosPcieComplexListPtr,
- 0,
- sizeof(Brazos),
- &InitEarly->StdHeader);
-
- LibAmdMemFill (BrazosPciePortPtr,
- 0,
- sizeof(PortList),
- &InitEarly->StdHeader);
-
- LibAmdMemFill (BrazosPcieDdiPtr,
- 0,
- sizeof(DdiList),
- &InitEarly->StdHeader);
-
- LibAmdMemCopy (BrazosPcieComplexListPtr, &Brazos, sizeof(Brazos), &InitEarly->StdHeader);
- LibAmdMemCopy (BrazosPciePortPtr, &PortList[0], sizeof(PortList), &InitEarly->StdHeader);
- LibAmdMemCopy (BrazosPcieDdiPtr, &DdiList[0], sizeof(DdiList), &InitEarly->StdHeader);
+ memcpy(BrazosPcieComplexListPtr, &Brazos, sizeof(Brazos));
+ memcpy(BrazosPciePortPtr, &PortList[0], sizeof(PortList));
+ memcpy(BrazosPcieDdiPtr, &DdiList[0], sizeof(DdiList));
((PCIe_COMPLEX_DESCRIPTOR*)BrazosPcieComplexListPtr)->PciePortList = (PCIe_PORT_DESCRIPTOR*)BrazosPciePortPtr;
diff --git a/src/mainboard/gizmosphere/gizmo/PlatformGnbPcie.c b/src/mainboard/gizmosphere/gizmo/PlatformGnbPcie.c
index a2560e0..26bf10b 100755
--- a/src/mainboard/gizmosphere/gizmo/PlatformGnbPcie.c
+++ b/src/mainboard/gizmosphere/gizmo/PlatformGnbPcie.c
@@ -26,6 +26,8 @@
#include "Filecode.h"
#include "BiosCallOuts.h"
+#include <string.h>
+
/*---------------------------------------------------------------------------------------*/
/**
* OemCustomizeInitEarly
@@ -135,24 +137,9 @@ PCIe_COMPLEX_DESCRIPTOR Brazos = {
AllocHeapParams.BufferPtr += sizeof(PortList);
BrazosPcieDdiPtr = (PCIe_DDI_DESCRIPTOR *) AllocHeapParams.BufferPtr;
- LibAmdMemFill (BrazosPcieComplexListPtr,
- 0,
- sizeof(Brazos),
- &InitEarly->StdHeader);
-
- LibAmdMemFill (BrazosPciePortPtr,
- 0,
- sizeof(PortList),
- &InitEarly->StdHeader);
-
- LibAmdMemFill (BrazosPcieDdiPtr,
- 0,
- sizeof(DdiList),
- &InitEarly->StdHeader);
-
- LibAmdMemCopy (BrazosPcieComplexListPtr, &Brazos, sizeof(Brazos), &InitEarly->StdHeader);
- LibAmdMemCopy (BrazosPciePortPtr, &PortList[0], sizeof(PortList), &InitEarly->StdHeader);
- LibAmdMemCopy (BrazosPcieDdiPtr, &DdiList[0], sizeof(DdiList), &InitEarly->StdHeader);
+ memcpy(BrazosPcieComplexListPtr, &Brazos, sizeof(Brazos));
+ memcpy(BrazosPciePortPtr, &PortList[0], sizeof(PortList));
+ memcpy(BrazosPcieDdiPtr, &DdiList[0], sizeof(DdiList));
((PCIe_COMPLEX_DESCRIPTOR*)BrazosPcieComplexListPtr)->PciePortList = (PCIe_PORT_DESCRIPTOR*)BrazosPciePortPtr;
diff --git a/src/mainboard/lippert/frontrunner-af/PlatformGnbPcie.c b/src/mainboard/lippert/frontrunner-af/PlatformGnbPcie.c
index 9724be1..ccf262c 100644
--- a/src/mainboard/lippert/frontrunner-af/PlatformGnbPcie.c
+++ b/src/mainboard/lippert/frontrunner-af/PlatformGnbPcie.c
@@ -25,6 +25,8 @@
#include "Filecode.h"
#include "BiosCallOuts.h"
+#include <string.h>
+
#define FILECODE PROC_RECOVERY_MEM_NB_ON_MRNON_FILECODE
/*---------------------------------------------------------------------------------------*/
@@ -136,24 +138,9 @@ PCIe_COMPLEX_DESCRIPTOR Brazos = {
AllocHeapParams.BufferPtr += sizeof(PortList);
BrazosPcieDdiPtr = (PCIe_DDI_DESCRIPTOR *) AllocHeapParams.BufferPtr;
- LibAmdMemFill (BrazosPcieComplexListPtr,
- 0,
- sizeof(Brazos),
- &InitEarly->StdHeader);
-
- LibAmdMemFill (BrazosPciePortPtr,
- 0,
- sizeof(PortList),
- &InitEarly->StdHeader);
-
- LibAmdMemFill (BrazosPcieDdiPtr,
- 0,
- sizeof(DdiList),
- &InitEarly->StdHeader);
-
- LibAmdMemCopy (BrazosPcieComplexListPtr, &Brazos, sizeof(Brazos), &InitEarly->StdHeader);
- LibAmdMemCopy (BrazosPciePortPtr, &PortList[0], sizeof(PortList), &InitEarly->StdHeader);
- LibAmdMemCopy (BrazosPcieDdiPtr, &DdiList[0], sizeof(DdiList), &InitEarly->StdHeader);
+ memcpy(BrazosPcieComplexListPtr, &Brazos, sizeof(Brazos));
+ memcpy(BrazosPciePortPtr, &PortList[0], sizeof(PortList));
+ memcpy(BrazosPcieDdiPtr, &DdiList[0], sizeof(DdiList));
((PCIe_COMPLEX_DESCRIPTOR*)BrazosPcieComplexListPtr)->PciePortList = (PCIe_PORT_DESCRIPTOR*)BrazosPciePortPtr;
diff --git a/src/mainboard/lippert/toucan-af/PlatformGnbPcie.c b/src/mainboard/lippert/toucan-af/PlatformGnbPcie.c
index 61318b1..0cdfcda 100644
--- a/src/mainboard/lippert/toucan-af/PlatformGnbPcie.c
+++ b/src/mainboard/lippert/toucan-af/PlatformGnbPcie.c
@@ -25,6 +25,8 @@
#include "Filecode.h"
#include "BiosCallOuts.h"
+#include <string.h>
+
#define FILECODE PROC_RECOVERY_MEM_NB_ON_MRNON_FILECODE
/*---------------------------------------------------------------------------------------*/
@@ -136,24 +138,9 @@ PCIe_COMPLEX_DESCRIPTOR Brazos = {
AllocHeapParams.BufferPtr += sizeof(PortList);
BrazosPcieDdiPtr = (PCIe_DDI_DESCRIPTOR *) AllocHeapParams.BufferPtr;
- LibAmdMemFill (BrazosPcieComplexListPtr,
- 0,
- sizeof(Brazos),
- &InitEarly->StdHeader);
-
- LibAmdMemFill (BrazosPciePortPtr,
- 0,
- sizeof(PortList),
- &InitEarly->StdHeader);
-
- LibAmdMemFill (BrazosPcieDdiPtr,
- 0,
- sizeof(DdiList),
- &InitEarly->StdHeader);
-
- LibAmdMemCopy (BrazosPcieComplexListPtr, &Brazos, sizeof(Brazos), &InitEarly->StdHeader);
- LibAmdMemCopy (BrazosPciePortPtr, &PortList[0], sizeof(PortList), &InitEarly->StdHeader);
- LibAmdMemCopy (BrazosPcieDdiPtr, &DdiList[0], sizeof(DdiList), &InitEarly->StdHeader);
+ memcpy(BrazosPcieComplexListPtr, &Brazos, sizeof(Brazos));
+ memcpy(BrazosPciePortPtr, &PortList[0], sizeof(PortList));
+ memcpy(BrazosPcieDdiPtr, &DdiList[0], sizeof(DdiList));
((PCIe_COMPLEX_DESCRIPTOR*)BrazosPcieComplexListPtr)->PciePortList = (PCIe_PORT_DESCRIPTOR*)BrazosPciePortPtr;
1
0
Patch set updated for coreboot: 0bbef0f src/*: Remove the last remnants of struct keyboard
by Edward O'Callaghan April 30, 2014
by Edward O'Callaghan April 30, 2014
April 30, 2014
Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5624
-gerrit
commit 0bbef0fbe52e8bbde362e040c74340f70b716e5c
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Wed Apr 30 20:41:41 2014 +1000
src/*: Remove the last remnants of struct keyboard
Change-Id: I7d0e8d2119a470428cfc01c0738b8988ab75ba2d
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
src/ec/compal/ene932/chip.h | 2 --
src/ec/compal/ene932/ec.c | 1 +
src/ec/google/chromeec/chip.h | 4 +---
src/ec/google/chromeec/ec_lpc.c | 1 +
src/ec/quanta/ene_kb3940q/chip.h | 2 --
src/ec/quanta/ene_kb3940q/ec.c | 1 +
src/ec/quanta/it8518/chip.h | 2 --
src/ec/quanta/it8518/ec.c | 1 +
src/include/pc80/keyboard.h | 4 ----
src/southbridge/via/vt82c686/chip.h | 31 -------------------------------
src/superio/smsc/fdc37n972/chip.h | 32 --------------------------------
src/superio/smsc/fdc37n972/fdc37n972.c | 1 -
src/superio/smsc/sio10n268/chip.h | 31 -------------------------------
src/superio/smsc/sio10n268/sio10n268.c | 1 -
14 files changed, 5 insertions(+), 109 deletions(-)
diff --git a/src/ec/compal/ene932/chip.h b/src/ec/compal/ene932/chip.h
index 5d209ff..7dcd3eb 100644
--- a/src/ec/compal/ene932/chip.h
+++ b/src/ec/compal/ene932/chip.h
@@ -23,13 +23,11 @@
#define _EC_COMPAL_ENE932_CHIP_H
#include <device/device.h>
-#include <pc80/keyboard.h>
struct chip_operations;
extern struct chip_operations ec_compal_ene932_ops;
struct ec_compal_ene932_config {
- struct pc_keyboard keyboard;
};
#endif /* _EC_COMPAL_ENE932_CHIP_H */
diff --git a/src/ec/compal/ene932/ec.c b/src/ec/compal/ene932/ec.c
index e60a0ac..2e83b4c 100644
--- a/src/ec/compal/ene932/ec.c
+++ b/src/ec/compal/ene932/ec.c
@@ -27,6 +27,7 @@
#include <stdlib.h>
#include <arch/io.h>
#include <delay.h>
+#include <pc80/keyboard.h>
#include "ec.h"
#include "chip.h"
diff --git a/src/ec/google/chromeec/chip.h b/src/ec/google/chromeec/chip.h
index eb35632..1cd97f4 100644
--- a/src/ec/google/chromeec/chip.h
+++ b/src/ec/google/chromeec/chip.h
@@ -21,12 +21,10 @@
#define EC_GOOGLE_CHROMEEC_CHIP_H
#include <device/device.h>
-#include <pc80/keyboard.h>
extern struct chip_operations ec_google_chromeec_ops;
struct ec_google_chromeec_config {
- struct pc_keyboard keyboard;
};
-#endif
+#endif /* EC_GOOGLE_CHROMEEC_CHIP_H */
diff --git a/src/ec/google/chromeec/ec_lpc.c b/src/ec/google/chromeec/ec_lpc.c
index 5f377c9..8b1cb2f 100644
--- a/src/ec/google/chromeec/ec_lpc.c
+++ b/src/ec/google/chromeec/ec_lpc.c
@@ -24,6 +24,7 @@
#include <arch/io.h>
#include <delay.h>
#include <device/pnp.h>
+#include <pc80/keyboard.h>
#include "chip.h"
#include "ec.h"
#include "ec_commands.h"
diff --git a/src/ec/quanta/ene_kb3940q/chip.h b/src/ec/quanta/ene_kb3940q/chip.h
index ce95c1c..6a28c43 100644
--- a/src/ec/quanta/ene_kb3940q/chip.h
+++ b/src/ec/quanta/ene_kb3940q/chip.h
@@ -23,13 +23,11 @@
#define _EC_QUANTA_ENE_KB3940Q_CHIP_H
#include <device/device.h>
-#include <pc80/keyboard.h>
struct chip_operations;
extern struct chip_operations ec_quanta_ene_kb3940q_ops;
struct ec_quanta_ene_kb3940q_config {
- struct pc_keyboard keyboard;
};
#endif /* _EC_QUANTA_ENE_KB3940Q_CHIP_H */
diff --git a/src/ec/quanta/ene_kb3940q/ec.c b/src/ec/quanta/ene_kb3940q/ec.c
index 57854d0..7d0e6ae 100644
--- a/src/ec/quanta/ene_kb3940q/ec.c
+++ b/src/ec/quanta/ene_kb3940q/ec.c
@@ -28,6 +28,7 @@
#include <stdlib.h>
#include <delay.h>
#include <elog.h>
+#include <pc80/keyboard.h>
#include "ec.h"
#include "chip.h"
diff --git a/src/ec/quanta/it8518/chip.h b/src/ec/quanta/it8518/chip.h
index cc7d374..52e842b 100644
--- a/src/ec/quanta/it8518/chip.h
+++ b/src/ec/quanta/it8518/chip.h
@@ -23,13 +23,11 @@
#define _EC_QUANTA_IT8518_CHIP_H
#include <device/device.h>
-#include <pc80/keyboard.h>
struct chip_operations;
extern struct chip_operations ec_quanta_it8518_ops;
struct ec_quanta_it8518_config {
- struct pc_keyboard keyboard;
};
#endif /* _EC_QUANTA_IT8518_CHIP_H */
diff --git a/src/ec/quanta/it8518/ec.c b/src/ec/quanta/it8518/ec.c
index 7588a1e..9e6d3d0 100644
--- a/src/ec/quanta/it8518/ec.c
+++ b/src/ec/quanta/it8518/ec.c
@@ -22,6 +22,7 @@
#include <console/console.h>
+#include <pc80/keyboard.h>
#ifndef __PRE_RAM__
#include <device/device.h>
diff --git a/src/include/pc80/keyboard.h b/src/include/pc80/keyboard.h
index 5f97723..16cb959 100644
--- a/src/include/pc80/keyboard.h
+++ b/src/include/pc80/keyboard.h
@@ -1,10 +1,6 @@
#ifndef PC80_KEYBOARD_H
#define PC80_KEYBOARD_H
-struct pc_keyboard {
- /* No initialization parameters for now */
-};
-
void pc_keyboard_init(void);
void set_kbc_ps2_mode(void);
diff --git a/src/southbridge/via/vt82c686/chip.h b/src/southbridge/via/vt82c686/chip.h
deleted file mode 100644
index 5c4959f..0000000
--- a/src/southbridge/via/vt82c686/chip.h
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2007 Corey Osgood <corey_osgood(a)verizon.net>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef _SUPERIO_VIA_VT82C686
-#define _SUPERIO_VIA_VT82C686
-
-#include <device/device.h>
-#include <pc80/keyboard.h>
-
-struct superio_via_vt82c686_config {
- struct pc_keyboard keyboard;
-};
-
-#endif /* _SUPERIO_VIA_VT82C686 */
diff --git a/src/superio/smsc/fdc37n972/chip.h b/src/superio/smsc/fdc37n972/chip.h
deleted file mode 100644
index e4126a7..0000000
--- a/src/superio/smsc/fdc37n972/chip.h
+++ /dev/null
@@ -1,32 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2008 coresystems GmbH
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_SMSC_FDC37N972_CHIP_H
-#define SUPERIO_SMSC_FDC37N972_CHIP_H
-
-#include <device/device.h>
-#include <pc80/keyboard.h>
-
-struct superio_smsc_fdc37n972_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
-
diff --git a/src/superio/smsc/fdc37n972/fdc37n972.c b/src/superio/smsc/fdc37n972/fdc37n972.c
index f7ae697..78d8350 100644
--- a/src/superio/smsc/fdc37n972/fdc37n972.c
+++ b/src/superio/smsc/fdc37n972/fdc37n972.c
@@ -21,7 +21,6 @@
#include <device/device.h>
#include <device/pnp.h>
#include <pc80/keyboard.h>
-#include "chip.h"
#include "fdc37n972.h"
static void init(device_t dev)
diff --git a/src/superio/smsc/sio10n268/chip.h b/src/superio/smsc/sio10n268/chip.h
deleted file mode 100644
index cde51d1..0000000
--- a/src/superio/smsc/sio10n268/chip.h
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2008 coresystems GmbH
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_SMSC_SIO10N268_CHIP_H
-#define SUPERIO_SMSC_SIO10N268_CHIP_H
-
-#include <device/device.h>
-#include <pc80/keyboard.h>
-
-struct superio_smsc_sio10n268_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/smsc/sio10n268/sio10n268.c b/src/superio/smsc/sio10n268/sio10n268.c
index 40c01b4..f69e3bf 100644
--- a/src/superio/smsc/sio10n268/sio10n268.c
+++ b/src/superio/smsc/sio10n268/sio10n268.c
@@ -21,7 +21,6 @@
#include <device/device.h>
#include <device/pnp.h>
#include <pc80/keyboard.h>
-#include "chip.h"
#include "sio10n268.h"
static void init(device_t dev)
1
0
New patch to review for coreboot: b500b9a src/*: Remove the last remnants of struct keyboard
by Edward O'Callaghan April 30, 2014
by Edward O'Callaghan April 30, 2014
April 30, 2014
Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5624
-gerrit
commit b500b9ac087f5e74623392fbb0467a572febe832
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Wed Apr 30 20:41:41 2014 +1000
src/*: Remove the last remnants of struct keyboard
Change-Id: I7d0e8d2119a470428cfc01c0738b8988ab75ba2d
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
src/ec/compal/ene932/chip.h | 2 --
src/ec/google/chromeec/chip.h | 4 +---
src/ec/quanta/ene_kb3940q/chip.h | 2 --
src/ec/quanta/it8518/chip.h | 2 --
src/include/pc80/keyboard.h | 4 ----
src/southbridge/via/vt82c686/chip.h | 31 -------------------------------
src/superio/smsc/fdc37n972/chip.h | 32 --------------------------------
src/superio/smsc/sio10n268/chip.h | 31 -------------------------------
src/superio/smsc/sio10n268/sio10n268.c | 1 -
9 files changed, 1 insertion(+), 108 deletions(-)
diff --git a/src/ec/compal/ene932/chip.h b/src/ec/compal/ene932/chip.h
index 5d209ff..7dcd3eb 100644
--- a/src/ec/compal/ene932/chip.h
+++ b/src/ec/compal/ene932/chip.h
@@ -23,13 +23,11 @@
#define _EC_COMPAL_ENE932_CHIP_H
#include <device/device.h>
-#include <pc80/keyboard.h>
struct chip_operations;
extern struct chip_operations ec_compal_ene932_ops;
struct ec_compal_ene932_config {
- struct pc_keyboard keyboard;
};
#endif /* _EC_COMPAL_ENE932_CHIP_H */
diff --git a/src/ec/google/chromeec/chip.h b/src/ec/google/chromeec/chip.h
index eb35632..1cd97f4 100644
--- a/src/ec/google/chromeec/chip.h
+++ b/src/ec/google/chromeec/chip.h
@@ -21,12 +21,10 @@
#define EC_GOOGLE_CHROMEEC_CHIP_H
#include <device/device.h>
-#include <pc80/keyboard.h>
extern struct chip_operations ec_google_chromeec_ops;
struct ec_google_chromeec_config {
- struct pc_keyboard keyboard;
};
-#endif
+#endif /* EC_GOOGLE_CHROMEEC_CHIP_H */
diff --git a/src/ec/quanta/ene_kb3940q/chip.h b/src/ec/quanta/ene_kb3940q/chip.h
index ce95c1c..6a28c43 100644
--- a/src/ec/quanta/ene_kb3940q/chip.h
+++ b/src/ec/quanta/ene_kb3940q/chip.h
@@ -23,13 +23,11 @@
#define _EC_QUANTA_ENE_KB3940Q_CHIP_H
#include <device/device.h>
-#include <pc80/keyboard.h>
struct chip_operations;
extern struct chip_operations ec_quanta_ene_kb3940q_ops;
struct ec_quanta_ene_kb3940q_config {
- struct pc_keyboard keyboard;
};
#endif /* _EC_QUANTA_ENE_KB3940Q_CHIP_H */
diff --git a/src/ec/quanta/it8518/chip.h b/src/ec/quanta/it8518/chip.h
index cc7d374..52e842b 100644
--- a/src/ec/quanta/it8518/chip.h
+++ b/src/ec/quanta/it8518/chip.h
@@ -23,13 +23,11 @@
#define _EC_QUANTA_IT8518_CHIP_H
#include <device/device.h>
-#include <pc80/keyboard.h>
struct chip_operations;
extern struct chip_operations ec_quanta_it8518_ops;
struct ec_quanta_it8518_config {
- struct pc_keyboard keyboard;
};
#endif /* _EC_QUANTA_IT8518_CHIP_H */
diff --git a/src/include/pc80/keyboard.h b/src/include/pc80/keyboard.h
index 5f97723..16cb959 100644
--- a/src/include/pc80/keyboard.h
+++ b/src/include/pc80/keyboard.h
@@ -1,10 +1,6 @@
#ifndef PC80_KEYBOARD_H
#define PC80_KEYBOARD_H
-struct pc_keyboard {
- /* No initialization parameters for now */
-};
-
void pc_keyboard_init(void);
void set_kbc_ps2_mode(void);
diff --git a/src/southbridge/via/vt82c686/chip.h b/src/southbridge/via/vt82c686/chip.h
deleted file mode 100644
index 5c4959f..0000000
--- a/src/southbridge/via/vt82c686/chip.h
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2007 Corey Osgood <corey_osgood(a)verizon.net>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef _SUPERIO_VIA_VT82C686
-#define _SUPERIO_VIA_VT82C686
-
-#include <device/device.h>
-#include <pc80/keyboard.h>
-
-struct superio_via_vt82c686_config {
- struct pc_keyboard keyboard;
-};
-
-#endif /* _SUPERIO_VIA_VT82C686 */
diff --git a/src/superio/smsc/fdc37n972/chip.h b/src/superio/smsc/fdc37n972/chip.h
deleted file mode 100644
index e4126a7..0000000
--- a/src/superio/smsc/fdc37n972/chip.h
+++ /dev/null
@@ -1,32 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2008 coresystems GmbH
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_SMSC_FDC37N972_CHIP_H
-#define SUPERIO_SMSC_FDC37N972_CHIP_H
-
-#include <device/device.h>
-#include <pc80/keyboard.h>
-
-struct superio_smsc_fdc37n972_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
-
diff --git a/src/superio/smsc/sio10n268/chip.h b/src/superio/smsc/sio10n268/chip.h
deleted file mode 100644
index cde51d1..0000000
--- a/src/superio/smsc/sio10n268/chip.h
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2008 coresystems GmbH
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_SMSC_SIO10N268_CHIP_H
-#define SUPERIO_SMSC_SIO10N268_CHIP_H
-
-#include <device/device.h>
-#include <pc80/keyboard.h>
-
-struct superio_smsc_sio10n268_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/smsc/sio10n268/sio10n268.c b/src/superio/smsc/sio10n268/sio10n268.c
index 40c01b4..f69e3bf 100644
--- a/src/superio/smsc/sio10n268/sio10n268.c
+++ b/src/superio/smsc/sio10n268/sio10n268.c
@@ -21,7 +21,6 @@
#include <device/device.h>
#include <device/pnp.h>
#include <pc80/keyboard.h>
-#include "chip.h"
#include "sio10n268.h"
static void init(device_t dev)
1
0
New patch to review for coreboot: 4a1bd8f src/*: Remove the last remittance of struct keyboard
by Edward O'Callaghan April 30, 2014
by Edward O'Callaghan April 30, 2014
April 30, 2014
Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5623
-gerrit
commit 4a1bd8fc6c59c74351f153195941f8ae735d43ee
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Wed Apr 30 20:06:39 2014 +1000
src/*: Remove the last remittance of struct keyboard
Change-Id: I652fbce053a11da1865a03b11cf995129d1ea4b0
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
src/ec/compal/ene932/chip.h | 5 -----
src/ec/google/chromeec/chip.h | 7 +------
src/ec/quanta/ene_kb3940q/chip.h | 5 -----
src/ec/quanta/it8518/chip.h | 5 -----
src/include/pc80/keyboard.h | 4 ----
src/southbridge/via/vt82c686/chip.h | 31 -------------------------------
src/superio/smsc/fdc37n972/chip.h | 32 --------------------------------
src/superio/smsc/sio10n268/chip.h | 31 -------------------------------
8 files changed, 1 insertion(+), 119 deletions(-)
diff --git a/src/ec/compal/ene932/chip.h b/src/ec/compal/ene932/chip.h
index 5d209ff..e244534 100644
--- a/src/ec/compal/ene932/chip.h
+++ b/src/ec/compal/ene932/chip.h
@@ -23,13 +23,8 @@
#define _EC_COMPAL_ENE932_CHIP_H
#include <device/device.h>
-#include <pc80/keyboard.h>
struct chip_operations;
extern struct chip_operations ec_compal_ene932_ops;
-struct ec_compal_ene932_config {
- struct pc_keyboard keyboard;
-};
-
#endif /* _EC_COMPAL_ENE932_CHIP_H */
diff --git a/src/ec/google/chromeec/chip.h b/src/ec/google/chromeec/chip.h
index eb35632..6ae4f14 100644
--- a/src/ec/google/chromeec/chip.h
+++ b/src/ec/google/chromeec/chip.h
@@ -21,12 +21,7 @@
#define EC_GOOGLE_CHROMEEC_CHIP_H
#include <device/device.h>
-#include <pc80/keyboard.h>
extern struct chip_operations ec_google_chromeec_ops;
-struct ec_google_chromeec_config {
- struct pc_keyboard keyboard;
-};
-
-#endif
+#endif /* EC_GOOGLE_CHROMEEC_CHIP_H */
diff --git a/src/ec/quanta/ene_kb3940q/chip.h b/src/ec/quanta/ene_kb3940q/chip.h
index ce95c1c..eaae0e3 100644
--- a/src/ec/quanta/ene_kb3940q/chip.h
+++ b/src/ec/quanta/ene_kb3940q/chip.h
@@ -23,13 +23,8 @@
#define _EC_QUANTA_ENE_KB3940Q_CHIP_H
#include <device/device.h>
-#include <pc80/keyboard.h>
struct chip_operations;
extern struct chip_operations ec_quanta_ene_kb3940q_ops;
-struct ec_quanta_ene_kb3940q_config {
- struct pc_keyboard keyboard;
-};
-
#endif /* _EC_QUANTA_ENE_KB3940Q_CHIP_H */
diff --git a/src/ec/quanta/it8518/chip.h b/src/ec/quanta/it8518/chip.h
index cc7d374..cd46023 100644
--- a/src/ec/quanta/it8518/chip.h
+++ b/src/ec/quanta/it8518/chip.h
@@ -23,13 +23,8 @@
#define _EC_QUANTA_IT8518_CHIP_H
#include <device/device.h>
-#include <pc80/keyboard.h>
struct chip_operations;
extern struct chip_operations ec_quanta_it8518_ops;
-struct ec_quanta_it8518_config {
- struct pc_keyboard keyboard;
-};
-
#endif /* _EC_QUANTA_IT8518_CHIP_H */
diff --git a/src/include/pc80/keyboard.h b/src/include/pc80/keyboard.h
index 5f97723..16cb959 100644
--- a/src/include/pc80/keyboard.h
+++ b/src/include/pc80/keyboard.h
@@ -1,10 +1,6 @@
#ifndef PC80_KEYBOARD_H
#define PC80_KEYBOARD_H
-struct pc_keyboard {
- /* No initialization parameters for now */
-};
-
void pc_keyboard_init(void);
void set_kbc_ps2_mode(void);
diff --git a/src/southbridge/via/vt82c686/chip.h b/src/southbridge/via/vt82c686/chip.h
deleted file mode 100644
index 5c4959f..0000000
--- a/src/southbridge/via/vt82c686/chip.h
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2007 Corey Osgood <corey_osgood(a)verizon.net>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef _SUPERIO_VIA_VT82C686
-#define _SUPERIO_VIA_VT82C686
-
-#include <device/device.h>
-#include <pc80/keyboard.h>
-
-struct superio_via_vt82c686_config {
- struct pc_keyboard keyboard;
-};
-
-#endif /* _SUPERIO_VIA_VT82C686 */
diff --git a/src/superio/smsc/fdc37n972/chip.h b/src/superio/smsc/fdc37n972/chip.h
deleted file mode 100644
index e4126a7..0000000
--- a/src/superio/smsc/fdc37n972/chip.h
+++ /dev/null
@@ -1,32 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2008 coresystems GmbH
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_SMSC_FDC37N972_CHIP_H
-#define SUPERIO_SMSC_FDC37N972_CHIP_H
-
-#include <device/device.h>
-#include <pc80/keyboard.h>
-
-struct superio_smsc_fdc37n972_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
-
diff --git a/src/superio/smsc/sio10n268/chip.h b/src/superio/smsc/sio10n268/chip.h
deleted file mode 100644
index cde51d1..0000000
--- a/src/superio/smsc/sio10n268/chip.h
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2008 coresystems GmbH
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_SMSC_SIO10N268_CHIP_H
-#define SUPERIO_SMSC_SIO10N268_CHIP_H
-
-#include <device/device.h>
-#include <pc80/keyboard.h>
-
-struct superio_smsc_sio10n268_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
1
0
New patch to review for coreboot: 0a59270 superio/*: Remove redundant chip.h header
by Edward O'Callaghan April 30, 2014
by Edward O'Callaghan April 30, 2014
April 30, 2014
Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5622
-gerrit
commit 0a59270cfe7a2166e11aad3501f6083eb46530e5
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Wed Apr 30 19:25:05 2014 +1000
superio/*: Remove redundant chip.h header
Change-Id: If7141112ea67071ee05c52f455c3b2496aa7e17e
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
src/superio/fintek/f71863fg/chip.h | 31 -------------------------------
src/superio/fintek/f71869ad/chip.h | 30 ------------------------------
src/superio/fintek/f71872/chip.h | 31 -------------------------------
src/superio/fintek/f71889/chip.h | 31 -------------------------------
src/superio/fintek/f81865f/chip.h | 33 ---------------------------------
src/superio/ite/it8671f/chip.h | 32 --------------------------------
src/superio/ite/it8712f/chip.h | 32 --------------------------------
src/superio/ite/it8716f/chip.h | 31 -------------------------------
src/superio/ite/it8718f/chip.h | 32 --------------------------------
src/superio/ite/it8721f/chip.h | 30 ------------------------------
src/superio/nsc/pc8374/chip.h | 32 --------------------------------
src/superio/nsc/pc87309/chip.h | 31 -------------------------------
src/superio/nsc/pc87351/chip.h | 32 --------------------------------
src/superio/nsc/pc87360/chip.h | 32 --------------------------------
src/superio/nsc/pc87366/chip.h | 32 --------------------------------
src/superio/nsc/pc87417/chip.h | 33 ---------------------------------
src/superio/nsc/pc87427/chip.h | 32 --------------------------------
src/superio/nuvoton/wpcm450/chip.h | 31 -------------------------------
src/superio/renesas/m3885x/chip.h | 30 ------------------------------
src/superio/smsc/fdc37m60x/chip.h | 32 --------------------------------
src/superio/smsc/kbc1100/chip.h | 30 ------------------------------
src/superio/smsc/lpc47b272/chip.h | 31 -------------------------------
src/superio/smsc/lpc47b397/chip.h | 33 ---------------------------------
src/superio/smsc/lpc47m10x/chip.h | 35 -----------------------------------
src/superio/smsc/lpc47m15x/chip.h | 30 ------------------------------
src/superio/smsc/lpc47n227/chip.h | 31 -------------------------------
src/superio/smsc/mec1308/chip.h | 29 -----------------------------
src/superio/smsc/sch4037/chip.h | 30 ------------------------------
src/superio/smsc/sio1036/chip.h | 29 -----------------------------
src/superio/smsc/smscsuperio/chip.h | 32 --------------------------------
src/superio/winbond/w83627dhg/chip.h | 31 -------------------------------
src/superio/winbond/w83627ehg/chip.h | 32 --------------------------------
src/superio/winbond/w83627hf/chip.h | 33 ---------------------------------
src/superio/winbond/w83627thg/chip.h | 33 ---------------------------------
src/superio/winbond/w83627uhg/chip.h | 31 -------------------------------
src/superio/winbond/w83977f/chip.h | 31 -------------------------------
src/superio/winbond/w83977tf/chip.h | 33 ---------------------------------
37 files changed, 1164 deletions(-)
diff --git a/src/superio/fintek/f71863fg/chip.h b/src/superio/fintek/f71863fg/chip.h
deleted file mode 100644
index a2c04d7..0000000
--- a/src/superio/fintek/f71863fg/chip.h
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2007 Corey Osgood <corey(a)slightlyhackish.com>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_FINTEK_F71863FG_CHIP_H
-#define SUPERIO_FINTEK_F71863FG_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_fintek_f71863fg_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif /* SUPERIO_FINTEK_F71863FG_CHIP_H */
diff --git a/src/superio/fintek/f71869ad/chip.h b/src/superio/fintek/f71869ad/chip.h
deleted file mode 100644
index ea2ee6e..0000000
--- a/src/superio/fintek/f71869ad/chip.h
+++ /dev/null
@@ -1,30 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2014 Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_FINTEK_F71869AD_CHIP_H
-#define SUPERIO_FINTEK_F71869AD_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_fintek_f71869ad_config {
- struct pc_keyboard keyboard;
-};
-
-#endif /* SUPERIO_FINTEK_F71869AD_CHIP_H */
diff --git a/src/superio/fintek/f71872/chip.h b/src/superio/fintek/f71872/chip.h
deleted file mode 100644
index 3368ef2..0000000
--- a/src/superio/fintek/f71872/chip.h
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2007 Corey Osgood <corey(a)slightlyhackish.com>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_FINTEK_F71872_CHIP_H
-#define SUPERIO_FINTEK_F71872_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_fintek_f71872_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif /* SUPERIO_FINTEK_F71872_CHIP_H */
diff --git a/src/superio/fintek/f71889/chip.h b/src/superio/fintek/f71889/chip.h
deleted file mode 100644
index e5a15d3..0000000
--- a/src/superio/fintek/f71889/chip.h
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2010 Alec Ari <neotheuser(a)ymail.com>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_FINTEK_F71889_CHIP_H
-#define SUPERIO_FINTEK_F71889_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_fintek_f71889_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif /* SUPERIO_FINTEK_F71889_CHIP_H */
diff --git a/src/superio/fintek/f81865f/chip.h b/src/superio/fintek/f81865f/chip.h
deleted file mode 100644
index ae4da18..0000000
--- a/src/superio/fintek/f81865f/chip.h
+++ /dev/null
@@ -1,33 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2011 Advanced Micro Devices, Inc.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_FINTEK_F81865_CHIP_H
-#define SUPERIO_FINTEK_F81865_CHIP_H
-
-#include <pc80/keyboard.h>
-
-/* This chip doesn't have keyboard and mouse support. */
-
-struct superio_fintek_f81865f_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif /* SUPERIO_FINTEK_F81865_CHIP_H */
diff --git a/src/superio/ite/it8671f/chip.h b/src/superio/ite/it8671f/chip.h
deleted file mode 100644
index 7d5c1d4..0000000
--- a/src/superio/ite/it8671f/chip.h
+++ /dev/null
@@ -1,32 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2006 Uwe Hermann <uwe(a)hermann-uwe.de>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_ITE_IT8671F_CHIP_H
-#define SUPERIO_ITE_IT8671F_CHIP_H
-
-#include <device/device.h>
-#include <pc80/keyboard.h>
-
-struct superio_ite_it8671f_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/ite/it8712f/chip.h b/src/superio/ite/it8712f/chip.h
deleted file mode 100644
index 1f159cd..0000000
--- a/src/superio/ite/it8712f/chip.h
+++ /dev/null
@@ -1,32 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2006 Uwe Hermann <uwe(a)hermann-uwe.de>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_ITE_IT8712F_CHIP_H
-#define SUPERIO_ITE_IT8712F_CHIP_H
-
-#include <device/device.h>
-#include <pc80/keyboard.h>
-
-struct superio_ite_it8712f_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/ite/it8716f/chip.h b/src/superio/ite/it8716f/chip.h
deleted file mode 100644
index afdcd96..0000000
--- a/src/superio/ite/it8716f/chip.h
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2006 Uwe Hermann <uwe(a)hermann-uwe.de>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_ITE_IT8716F_CHIP_H
-#define SUPERIO_ITE_IT8716F_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_ite_it8716f_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif /* SUPERIO_ITE_IT8716F_CHIP_H */
diff --git a/src/superio/ite/it8718f/chip.h b/src/superio/ite/it8718f/chip.h
deleted file mode 100644
index a23687c..0000000
--- a/src/superio/ite/it8718f/chip.h
+++ /dev/null
@@ -1,32 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2006 Uwe Hermann <uwe(a)hermann-uwe.de>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_ITE_IT8718F_CHIP_H
-#define SUPERIO_ITE_IT8718F_CHIP_H
-
-#include <device/device.h>
-#include <pc80/keyboard.h>
-
-struct superio_ite_it8718f_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/ite/it8721f/chip.h b/src/superio/ite/it8721f/chip.h
deleted file mode 100644
index e903cd6..0000000
--- a/src/superio/ite/it8721f/chip.h
+++ /dev/null
@@ -1,30 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2011 QingPei Wang <wangqingpei(a)gmail.com>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_ITE_IT8721F_CHIP_H
-#define SUPERIO_ITE_IT8721F_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_ite_it8721f_config {
- struct pc_keyboard keyboard;
-};
-
-#endif /* SUPERIO_ITE_IT8721F_CHIP_H */
diff --git a/src/superio/nsc/pc8374/chip.h b/src/superio/nsc/pc8374/chip.h
deleted file mode 100644
index 40a9b60..0000000
--- a/src/superio/nsc/pc8374/chip.h
+++ /dev/null
@@ -1,32 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2000 AG Electronics Ltd.
- * Copyright (C) 2003-2004 Linux Networx
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_NSC_PC8374_CHIP_H
-#define SUPERIO_NSC_PC8374_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_nsc_pc8374_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/nsc/pc87309/chip.h b/src/superio/nsc/pc87309/chip.h
deleted file mode 100644
index e769a85..0000000
--- a/src/superio/nsc/pc87309/chip.h
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2007 Uwe Hermann <uwe(a)hermann-uwe.de>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_NSC_PC87309_CHIP_H
-#define SUPERIO_NSC_PC87309_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_nsc_pc87309_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/nsc/pc87351/chip.h b/src/superio/nsc/pc87351/chip.h
deleted file mode 100644
index 39faee4..0000000
--- a/src/superio/nsc/pc87351/chip.h
+++ /dev/null
@@ -1,32 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2000 AG Electronics Ltd.
- * Copyright (C) 2003-2004 Linux Networx
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_NSC_PC87351_CHIP_H
-#define SUPERIO_NSC_PC87351_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_nsc_pc87351_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/nsc/pc87360/chip.h b/src/superio/nsc/pc87360/chip.h
deleted file mode 100644
index e48c0f2..0000000
--- a/src/superio/nsc/pc87360/chip.h
+++ /dev/null
@@ -1,32 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2000 AG Electronics Ltd.
- * Copyright (C) 2003-2004 Linux Networx
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_NSC_PC87360_CHIP_H
-#define SUPERIO_NSC_PC87360_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_nsc_pc87360_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/nsc/pc87366/chip.h b/src/superio/nsc/pc87366/chip.h
deleted file mode 100644
index 0ce8abd..0000000
--- a/src/superio/nsc/pc87366/chip.h
+++ /dev/null
@@ -1,32 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2000 AG Electronics Ltd.
- * Copyright (C) 2003-2004 Linux Networx
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_NSC_PC87366_CHIP_H
-#define SUPERIO_NSC_PC87366_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_nsc_pc87366_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/nsc/pc87417/chip.h b/src/superio/nsc/pc87417/chip.h
deleted file mode 100644
index 8c1e8e0..0000000
--- a/src/superio/nsc/pc87417/chip.h
+++ /dev/null
@@ -1,33 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2000 AG Electronics Ltd.
- * Copyright (C) 2003-2004 Linux Networx
- * Copyright (C) 2004 Tyan by yhlu
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_NSC_PC87417_CHIP_H
-#define SUPERIO_NSC_PC87417_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_nsc_pc87417_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/nsc/pc87427/chip.h b/src/superio/nsc/pc87427/chip.h
deleted file mode 100644
index 4500aa9..0000000
--- a/src/superio/nsc/pc87427/chip.h
+++ /dev/null
@@ -1,32 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2000 AG Electronics Ltd.
- * Copyright (C) 2003-2004 Linux Networx
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_NSC_PC87427_CHIP_H
-#define SUPERIO_NSC_PC87427_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_nsc_pc87427_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/nuvoton/wpcm450/chip.h b/src/superio/nuvoton/wpcm450/chip.h
deleted file mode 100644
index cc4054f..0000000
--- a/src/superio/nuvoton/wpcm450/chip.h
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2011 Advanced Micro Devices, Inc.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_NUVOTON_WPCM450_CHIP_H
-#define SUPERIO_NUVOTON_WPCM450_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_nuvoton_wpcm450_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/renesas/m3885x/chip.h b/src/superio/renesas/m3885x/chip.h
deleted file mode 100644
index 8ca54c7..0000000
--- a/src/superio/renesas/m3885x/chip.h
+++ /dev/null
@@ -1,30 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2009 coresystems GmbH
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_RENESAS_M3885X_CHIP_H
-#define SUPERIO_RENESAS_M3885X_CHIP_H
-
-#include <device/device.h>
-#include <pc80/keyboard.h>
-
-struct superio_renesas_m3885x_config {
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/smsc/fdc37m60x/chip.h b/src/superio/smsc/fdc37m60x/chip.h
deleted file mode 100644
index a143f65..0000000
--- a/src/superio/smsc/fdc37m60x/chip.h
+++ /dev/null
@@ -1,32 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2006 Uwe Hermann <uwe(a)hermann-uwe.de>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_SMSC_FDC37M60X_CHIP_H
-#define SUPERIO_SMSC_FDC37M60X_CHIP_H
-
-#include <device/device.h>
-#include <pc80/keyboard.h>
-
-struct superio_smsc_fdc37m60x_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/smsc/kbc1100/chip.h b/src/superio/smsc/kbc1100/chip.h
deleted file mode 100644
index 59ffb42..0000000
--- a/src/superio/smsc/kbc1100/chip.h
+++ /dev/null
@@ -1,30 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2011 Advanced Micro Devices, Inc.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_SMSC_KBC1100_CHIP_H
-#define SUPERIO_SMSC_KBC1100_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_smsc_kbc1100_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
\ No newline at end of file
diff --git a/src/superio/smsc/lpc47b272/chip.h b/src/superio/smsc/lpc47b272/chip.h
deleted file mode 100644
index 5a42f4a..0000000
--- a/src/superio/smsc/lpc47b272/chip.h
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2005 Digital Design Corporation
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_SMSC_LPC47B272_CHIP_H
-#define SUPERIO_SMSC_LPC47B272_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_smsc_lpc47b272_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/smsc/lpc47b397/chip.h b/src/superio/smsc/lpc47b397/chip.h
deleted file mode 100644
index e4ad5b1..0000000
--- a/src/superio/smsc/lpc47b397/chip.h
+++ /dev/null
@@ -1,33 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2000 AG Electronics Ltd.
- * Copyright (C) 2003-2004 Linux Networx
- * Copyright (C) 2004 Tyan
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_SMSC_LPC47B397_CHIP_H
-#define SUPERIO_SMSC_LPC47B397_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_smsc_lpc47b397_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/smsc/lpc47m10x/chip.h b/src/superio/smsc/lpc47m10x/chip.h
deleted file mode 100644
index 2d947be..0000000
--- a/src/superio/smsc/lpc47m10x/chip.h
+++ /dev/null
@@ -1,35 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2000 AG Electronics Ltd.
- * Copyright (C) 2003-2004 Linux Networx
- * Copyright (C) 2004 Tyan
- * Copyright (C) 2005 Digital Design Corporation
- * Copyright (C) 2006 Ron Minnich, LANL
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_SMSC_LPC47M10X_CHIP_H
-#define SUPERIO_SMSC_LPC47M10X_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_smsc_lpc47m10x_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/smsc/lpc47m15x/chip.h b/src/superio/smsc/lpc47m15x/chip.h
deleted file mode 100644
index 757fa73..0000000
--- a/src/superio/smsc/lpc47m15x/chip.h
+++ /dev/null
@@ -1,30 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2009 coresystems GmbH
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_SMSC_LPC47M15X_CHIP_H
-#define SUPERIO_SMSC_LPC47M15X_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_smsc_lpc47m15x_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/smsc/lpc47n227/chip.h b/src/superio/smsc/lpc47n227/chip.h
deleted file mode 100644
index 3f26d8b..0000000
--- a/src/superio/smsc/lpc47n227/chip.h
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2008-2009 coresystems GmbH
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_SMSC_LPC47N227_CHIP_H
-#define SUPERIO_SMSC_LPC47N227_CHIP_H
-
-#include <device/device.h>
-#include <pc80/keyboard.h>
-
-struct superio_smsc_lpc47n227_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/smsc/mec1308/chip.h b/src/superio/smsc/mec1308/chip.h
deleted file mode 100644
index 733cd34..0000000
--- a/src/superio/smsc/mec1308/chip.h
+++ /dev/null
@@ -1,29 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2011 Google Inc.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_SMSC_MEC1308_CHIP_H
-#define SUPERIO_SMSC_MEC1308_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_smsc_mec1308_config {
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/smsc/sch4037/chip.h b/src/superio/smsc/sch4037/chip.h
deleted file mode 100644
index b1fee00..0000000
--- a/src/superio/smsc/sch4037/chip.h
+++ /dev/null
@@ -1,30 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2012 Advanced Micro Devices, Inc.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_SCH_4037_CHIP_H
-#define SUPERIO_SCH_4037_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_smsc_sch4037_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif //SUPERIO_SCH_4037_CHIP_H
\ No newline at end of file
diff --git a/src/superio/smsc/sio1036/chip.h b/src/superio/smsc/sio1036/chip.h
deleted file mode 100644
index c19ce43..0000000
--- a/src/superio/smsc/sio1036/chip.h
+++ /dev/null
@@ -1,29 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2012 Advanced Micro Devices, Inc.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_SMSC_SIO1036_CHIP_H
-#define SUPERIO_SMSC_SIO1036_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_smsc_sio1036_config {
-};
-
-#endif //SUPERIO_SMSC_SIO1036_CHIP_H
-
diff --git a/src/superio/smsc/smscsuperio/chip.h b/src/superio/smsc/smscsuperio/chip.h
deleted file mode 100644
index 1e5acfe..0000000
--- a/src/superio/smsc/smscsuperio/chip.h
+++ /dev/null
@@ -1,32 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2007 Uwe Hermann <uwe(a)hermann-uwe.de>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_SMSC_SMSCSUPERIO_CHIP_H
-#define SUPERIO_SMSC_SMSCSUPERIO_CHIP_H
-
-#include <device/device.h>
-#include <pc80/keyboard.h>
-
-struct superio_smsc_smscsuperio_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/winbond/w83627dhg/chip.h b/src/superio/winbond/w83627dhg/chip.h
deleted file mode 100644
index 0bc5374..0000000
--- a/src/superio/winbond/w83627dhg/chip.h
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2008 Uwe Hermann <uwe(a)hermann-uwe.de>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_WINBOND_W83627DHG_CHIP_H
-#define SUPERIO_WINBOND_W83627DHG_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_winbond_w83627dhg_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/winbond/w83627ehg/chip.h b/src/superio/winbond/w83627ehg/chip.h
deleted file mode 100644
index e073d7d..0000000
--- a/src/superio/winbond/w83627ehg/chip.h
+++ /dev/null
@@ -1,32 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2007 AMD
- * Written by Yinghai Lu <yinghai.lu(a)amd.com> for AMD.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_WINBOND_W83627EHG_CHIP_H
-#define SUPERIO_WINBOND_W83627EHG_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_winbond_w83627ehg_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/winbond/w83627hf/chip.h b/src/superio/winbond/w83627hf/chip.h
deleted file mode 100644
index ae527ea..0000000
--- a/src/superio/winbond/w83627hf/chip.h
+++ /dev/null
@@ -1,33 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2000 AG Electronics Ltd.
- * Copyright (C) 2003-2004 Linux Networx
- * Copyright (C) 2004 Tyan
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_WINBOND_W83627HF_CHIP_H
-#define SUPERIO_WINBOND_W83627HF_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_winbond_w83627hf_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/winbond/w83627thg/chip.h b/src/superio/winbond/w83627thg/chip.h
deleted file mode 100644
index 1677b97..0000000
--- a/src/superio/winbond/w83627thg/chip.h
+++ /dev/null
@@ -1,33 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2000 AG Electronics Ltd.
- * Copyright (C) 2003-2004 Linux Networx
- * Copyright (C) 2004 Tyan
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_WINBOND_W83627THG_CHIP_H
-#define SUPERIO_WINBOND_W83627THG_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_winbond_w83627thg_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif /* SUPERIO_WINBOND_W83627THG_CHIP_H */
diff --git a/src/superio/winbond/w83627uhg/chip.h b/src/superio/winbond/w83627uhg/chip.h
deleted file mode 100644
index cf8edfb..0000000
--- a/src/superio/winbond/w83627uhg/chip.h
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2009 Dynon Avionics
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 $
- */
-
-#ifndef SUPERIO_WINBOND_W83627UHG_CHIP_H
-#define SUPERIO_WINBOND_W83627UHG_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_winbond_w83627uhg_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/winbond/w83977f/chip.h b/src/superio/winbond/w83977f/chip.h
deleted file mode 100644
index 7363445..0000000
--- a/src/superio/winbond/w83977f/chip.h
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2007 Nikolay Petukhov <nikolay.petukhov(a)gmail.com>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_WINBOND_W83977F_CHIP_H
-#define SUPERIO_WINBOND_W83977F_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_winbond_w83977f_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/winbond/w83977tf/chip.h b/src/superio/winbond/w83977tf/chip.h
deleted file mode 100644
index ddcf1b5..0000000
--- a/src/superio/winbond/w83977tf/chip.h
+++ /dev/null
@@ -1,33 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2000 AG Electronics Ltd.
- * Copyright (C) 2003-2004 Linux Networx
- * Copyright (C) 2004 Tyan
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_WINBOND_W83977TF_CHIP_H
-#define SUPERIO_WINBOND_W83977TF_CHIP_H
-
-#include <pc80/keyboard.h>
-
-struct superio_winbond_w83977tf_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
1
0
Patch set updated for coreboot: 8224be1 superio/*: Deal with some chip.h special cases
by Edward O'Callaghan April 30, 2014
by Edward O'Callaghan April 30, 2014
April 30, 2014
Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5621
-gerrit
commit 8224be1b9233d26c4ca22dda84d0bc16d890e647
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Wed Apr 30 17:17:03 2014 +1000
superio/*: Deal with some chip.h special cases
While backing out the empty pc80 keyboard struct we encountered some
special cases where chip.h is used for other purposes. Deal with these
cases.
Change-Id: Ib11a46cfd14d050d5daa213623b9d8a401c06410
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
src/superio/ite/it8772f/chip.h | 5 +----
src/superio/nsc/pc97307/chip.h | 37 ---------------------------------
src/superio/nsc/pc97307/superio.c | 1 -
src/superio/nsc/pc97317/chip.h | 38 ----------------------------------
src/superio/nsc/pc97317/superio.c | 1 -
src/superio/winbond/w83697hf/chip.h | 2 +-
src/superio/winbond/w83697hf/superio.c | 2 +-
7 files changed, 3 insertions(+), 83 deletions(-)
diff --git a/src/superio/ite/it8772f/chip.h b/src/superio/ite/it8772f/chip.h
index 9c9aaa6..4a1b696 100644
--- a/src/superio/ite/it8772f/chip.h
+++ b/src/superio/ite/it8772f/chip.h
@@ -22,11 +22,8 @@
#define SUPERIO_ITE_IT8772F_CHIP_H
#include <device/device.h>
-#include <pc80/keyboard.h>
struct superio_ite_it8772f_config {
- struct pc_keyboard keyboard;
-
/* GPIO SimpleIO register values via devicetree.cb */
u8 gpio_set1;
u8 gpio_set2;
@@ -51,4 +48,4 @@ struct superio_ite_it8772f_config {
u8 fan3_enable;
};
-#endif
+#endif /* SUPERIO_ITE_IT8772F_CHIP_H */
diff --git a/src/superio/nsc/pc97307/chip.h b/src/superio/nsc/pc97307/chip.h
deleted file mode 100644
index e0c1330..0000000
--- a/src/superio/nsc/pc97307/chip.h
+++ /dev/null
@@ -1,37 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2000 AG Electronics Ltd.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_NSC_PC97307_CHIP_H
-#define SUPERIO_NSC_PC97307_CHIP_H
-
-#ifndef PNP_INDEX_REG
-#define PNP_INDEX_REG 0x15C
-#endif
-#ifndef PNP_DATA_REG
-#define PNP_DATA_REG 0x15D
-#endif
-
-#include <pc80/keyboard.h>
-
-struct superio_nsc_pc97307_config {
-
- struct pc_keyboard keyboard;
-};
-#endif
diff --git a/src/superio/nsc/pc97307/superio.c b/src/superio/nsc/pc97307/superio.c
index af7ca77..6b15901 100644
--- a/src/superio/nsc/pc97307/superio.c
+++ b/src/superio/nsc/pc97307/superio.c
@@ -24,7 +24,6 @@
#include <device/pnp.h>
#include <stdlib.h>
#include <pc80/keyboard.h>
-#include "chip.h" /* FIXME */
#include "pc97307.h"
static void init(device_t dev)
diff --git a/src/superio/nsc/pc97317/chip.h b/src/superio/nsc/pc97317/chip.h
deleted file mode 100644
index d896916..0000000
--- a/src/superio/nsc/pc97317/chip.h
+++ /dev/null
@@ -1,38 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2000 AG Electronics Ltd.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef SUPERIO_NSC_PC97317_CHIP_H
-#define SUPERIO_NSC_PC97317_CHIP_H
-
-#ifndef PNP_INDEX_REG
-#define PNP_INDEX_REG 0x15C
-#endif
-#ifndef PNP_DATA_REG
-#define PNP_DATA_REG 0x15D
-#endif
-
-#include <pc80/keyboard.h>
-
-struct superio_nsc_pc97317_config {
-
- struct pc_keyboard keyboard;
-};
-
-#endif
diff --git a/src/superio/nsc/pc97317/superio.c b/src/superio/nsc/pc97317/superio.c
index 6f39c5d..09c3b5e 100644
--- a/src/superio/nsc/pc97317/superio.c
+++ b/src/superio/nsc/pc97317/superio.c
@@ -24,7 +24,6 @@
#include <device/pnp.h>
#include <stdlib.h>
#include <pc80/keyboard.h>
-#include "chip.h" /* FIXME */
#include "pc97317.h"
static void init(device_t dev)
diff --git a/src/superio/winbond/w83697hf/chip.h b/src/superio/winbond/w83697hf/chip.h
index df9ce4b..5ca9b20 100644
--- a/src/superio/winbond/w83697hf/chip.h
+++ b/src/superio/winbond/w83697hf/chip.h
@@ -27,4 +27,4 @@ struct superio_winbond_w83697hf_config {
unsigned int hwmon_fan2_divisor;
};
-#endif
+#endif /* SUPERIO_WINBOND_W83697HF_CHIP_H */
diff --git a/src/superio/winbond/w83697hf/superio.c b/src/superio/winbond/w83697hf/superio.c
index 933da95..9d9742c 100644
--- a/src/superio/winbond/w83697hf/superio.c
+++ b/src/superio/winbond/w83697hf/superio.c
@@ -28,7 +28,7 @@
#include <pc80/mc146818rtc.h>
#include <stdlib.h>
#include <pc80/keyboard.h>
-#include "chip.h" /* FIXME */
+#include "chip.h"
#include "w83697hf.h"
static void hwmon_set_fan_divisor(unsigned int base, int num, unsigned int divisor) {
1
0
Patch set updated for coreboot: 86c24fa src/drivers/pc80: Remove empty struct keyboard
by Edward O'Callaghan April 30, 2014
by Edward O'Callaghan April 30, 2014
April 30, 2014
Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5617
-gerrit
commit 86c24fac48112be0105f7fddd1c9c5c3e45df551
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Wed Apr 30 05:01:52 2014 +1000
src/drivers/pc80: Remove empty struct keyboard
This is a empty struct that has propagated through the superio's & ec's
but really does nothing. Time to get rid of it before it adds yet more
cruft. However, since this touches many superio's at once we do this in
stages by first changing the function type to be a pure procedure.
Change-Id: Ibc732e676a9d4f0269114acabc92b15771d27ef2
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
src/drivers/pc80/keyboard.c | 2 +-
src/ec/compal/ene932/ec.c | 5 +----
src/ec/google/chromeec/ec_lpc.c | 5 ++---
src/ec/quanta/ene_kb3940q/ec.c | 4 +---
src/ec/quanta/it8518/ec.c | 5 +----
src/include/pc80/keyboard.h | 2 +-
src/mainboard/emulation/qemu-i440fx/mainboard.c | 2 +-
src/mainboard/emulation/qemu-q35/mainboard.c | 2 +-
src/mainboard/lenovo/t530/mainboard.c | 2 +-
src/mainboard/lenovo/x201/mainboard.c | 2 +-
src/mainboard/lenovo/x230/mainboard.c | 2 +-
src/mainboard/packardbell/ms2290/mainboard.c | 2 +-
src/mainboard/roda/rk9/mainboard.c | 2 +-
src/northbridge/via/cx700/lpc.c | 2 +-
src/northbridge/via/vx800/lpc.c | 2 +-
src/southbridge/dmp/vortex86ex/southbridge.c | 2 +-
src/southbridge/sis/sis966/lpc.c | 16 ++++++++--------
src/southbridge/via/vt8231/vt8231.c | 2 +-
src/southbridge/via/vt8235/vt8235.c | 2 +-
src/southbridge/via/vt8237r/lpc.c | 2 +-
src/superio/fintek/f71863fg/superio.c | 5 ++---
src/superio/fintek/f71869ad/superio.c | 6 ++----
src/superio/fintek/f71872/superio.c | 5 ++---
src/superio/fintek/f71889/superio.c | 5 ++---
src/superio/fintek/f81865f/superio.c | 5 ++---
src/superio/ite/it8671f/superio.c | 4 +---
src/superio/ite/it8712f/superio.c | 4 +---
src/superio/ite/it8716f/superio.c | 4 +---
src/superio/ite/it8718f/superio.c | 4 +---
src/superio/ite/it8721f/superio.c | 4 +---
src/superio/ite/it8772f/superio.c | 4 ++--
src/superio/nsc/pc8374/superio.c | 4 +---
src/superio/nsc/pc87309/superio.c | 4 +---
src/superio/nsc/pc87351/superio.c | 4 +---
src/superio/nsc/pc87360/superio.c | 4 +---
src/superio/nsc/pc87366/superio.c | 4 +---
src/superio/nsc/pc87417/superio.c | 4 +---
src/superio/nsc/pc87427/superio.c | 5 ++---
src/superio/nsc/pc97307/superio.c | 6 +++---
src/superio/nsc/pc97317/superio.c | 6 +++---
src/superio/nuvoton/wpcm450/superio.c | 4 +---
src/superio/renesas/m3885x/superio.c | 5 ++---
src/superio/smsc/fdc37m60x/superio.c | 4 +---
src/superio/smsc/fdc37n972/fdc37n972.c | 5 +----
src/superio/smsc/kbc1100/superio.c | 4 +---
src/superio/smsc/lpc47b272/superio.c | 4 +---
src/superio/smsc/lpc47b397/superio.c | 4 +---
src/superio/smsc/lpc47m10x/superio.c | 4 +---
src/superio/smsc/lpc47m15x/superio.c | 4 +---
src/superio/smsc/lpc47n227/superio.c | 5 ++---
src/superio/smsc/mec1308/superio.c | 4 +---
src/superio/smsc/sch4037/superio.c | 4 +---
src/superio/smsc/sio1007/chip.h | 1 +
src/superio/smsc/sio1036/superio.c | 2 --
src/superio/smsc/sio10n268/sio10n268.c | 4 +---
src/superio/smsc/smscsuperio/superio.c | 4 +---
src/superio/winbond/w83627dhg/superio.c | 4 +---
src/superio/winbond/w83627ehg/superio.c | 4 +---
src/superio/winbond/w83627hf/superio.c | 4 +---
src/superio/winbond/w83627thg/superio.c | 4 +---
src/superio/winbond/w83627uhg/superio.c | 4 +---
src/superio/winbond/w83697hf/superio.c | 3 ++-
src/superio/winbond/w83977f/superio.c | 4 +---
src/superio/winbond/w83977tf/superio.c | 4 +---
64 files changed, 85 insertions(+), 164 deletions(-)
diff --git a/src/drivers/pc80/keyboard.c b/src/drivers/pc80/keyboard.c
index 5b68617..6a573d8 100644
--- a/src/drivers/pc80/keyboard.c
+++ b/src/drivers/pc80/keyboard.c
@@ -193,7 +193,7 @@ static u8 send_keyboard(u8 command)
return regval;
}
-void pc_keyboard_init(struct pc_keyboard *keyboard)
+void pc_keyboard_init(void)
{
u8 retries;
u8 regval;
diff --git a/src/ec/compal/ene932/ec.c b/src/ec/compal/ene932/ec.c
index 3f4d1b9..e60a0ac 100644
--- a/src/ec/compal/ene932/ec.c
+++ b/src/ec/compal/ene932/ec.c
@@ -133,14 +133,11 @@ static u8 ec_io_read(u16 addr)
#ifndef __SMM__
static void ene932_init(device_t dev)
{
- struct ec_compal_ene932_config *conf = dev->chip_info;
-
-
if (!dev->enabled)
return;
printk(BIOS_DEBUG, "Compal ENE932: Initializing keyboard.\n");
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
}
diff --git a/src/ec/google/chromeec/ec_lpc.c b/src/ec/google/chromeec/ec_lpc.c
index a445da8..5f377c9 100644
--- a/src/ec/google/chromeec/ec_lpc.c
+++ b/src/ec/google/chromeec/ec_lpc.c
@@ -138,11 +138,10 @@ int google_chromeec_command(struct chromeec_command *cec_command)
#ifndef __SMM__
static void lpc_ec_init(device_t dev)
{
- struct ec_google_chromeec_config *conf = dev->chip_info;
-
if (!dev->enabled)
return;
- pc_keyboard_init(&conf->keyboard);
+
+ pc_keyboard_init();
google_chromeec_init();
}
diff --git a/src/ec/quanta/ene_kb3940q/ec.c b/src/ec/quanta/ene_kb3940q/ec.c
index 137aa81..57854d0 100644
--- a/src/ec/quanta/ene_kb3940q/ec.c
+++ b/src/ec/quanta/ene_kb3940q/ec.c
@@ -141,13 +141,11 @@ static void ene_kb3940q_log_events(void)
static void ene_kb3940q_init(device_t dev)
{
- struct ec_quanta_ene_kb3940q_config *conf = dev->chip_info;
-
if (!dev->enabled)
return;
printk(BIOS_DEBUG, "Quanta EnE KB3940Q: Initializing keyboard.\n");
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
ene_kb3940q_log_events();
}
diff --git a/src/ec/quanta/it8518/ec.c b/src/ec/quanta/it8518/ec.c
index 6516cee..7588a1e 100644
--- a/src/ec/quanta/it8518/ec.c
+++ b/src/ec/quanta/it8518/ec.c
@@ -162,14 +162,11 @@ void ec_it8518_enable_wake_events(void)
#ifndef __SMM__
static void it8518_init(device_t dev)
{
- struct ec_quanta_it8518_config *conf = dev->chip_info;
-
-
if (!dev->enabled)
return;
printk(BIOS_DEBUG, "Quanta IT8518: Initializing keyboard.\n");
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
}
diff --git a/src/include/pc80/keyboard.h b/src/include/pc80/keyboard.h
index ce281bb..5f97723 100644
--- a/src/include/pc80/keyboard.h
+++ b/src/include/pc80/keyboard.h
@@ -5,7 +5,7 @@ struct pc_keyboard {
/* No initialization parameters for now */
};
-void pc_keyboard_init(struct pc_keyboard *keyboard);
+void pc_keyboard_init(void);
void set_kbc_ps2_mode(void);
#endif /* PC80_KEYBOARD_H */
diff --git a/src/mainboard/emulation/qemu-i440fx/mainboard.c b/src/mainboard/emulation/qemu-i440fx/mainboard.c
index 90ad2db..a8a61c4 100644
--- a/src/mainboard/emulation/qemu-i440fx/mainboard.c
+++ b/src/mainboard/emulation/qemu-i440fx/mainboard.c
@@ -43,7 +43,7 @@ static void qemu_nb_init(device_t dev)
/* This sneaked in here, because Qemu does not
* emulate a SuperIO chip
*/
- pc_keyboard_init(0);
+ pc_keyboard_init();
/* setup IRQ routing */
for (i = 0; i < 32; i++)
diff --git a/src/mainboard/emulation/qemu-q35/mainboard.c b/src/mainboard/emulation/qemu-q35/mainboard.c
index 78c92a9..e991b53 100644
--- a/src/mainboard/emulation/qemu-q35/mainboard.c
+++ b/src/mainboard/emulation/qemu-q35/mainboard.c
@@ -50,7 +50,7 @@ static void qemu_nb_init(device_t dev)
/* This sneaked in here, because Qemu does not
* emulate a SuperIO chip
*/
- pc_keyboard_init(0);
+ pc_keyboard_init();
/* setup IRQ routing for pci slots */
for (i = 0; i < 25; i++)
diff --git a/src/mainboard/lenovo/t530/mainboard.c b/src/mainboard/lenovo/t530/mainboard.c
index 5931d00..6d21b31 100644
--- a/src/mainboard/lenovo/t530/mainboard.c
+++ b/src/mainboard/lenovo/t530/mainboard.c
@@ -161,7 +161,7 @@ static void mainboard_init(device_t dev)
/* This sneaked in here, because X201 SuperIO chip isn't really
connected to anything and hence we don't init it.
*/
- pc_keyboard_init(0);
+ pc_keyboard_init();
/* Enable expresscard hotplug events. */
pci_write_config32(dev_find_slot(0, PCI_DEVFN(0x1c, 2)),
diff --git a/src/mainboard/lenovo/x201/mainboard.c b/src/mainboard/lenovo/x201/mainboard.c
index cf42a9b..d772d19 100644
--- a/src/mainboard/lenovo/x201/mainboard.c
+++ b/src/mainboard/lenovo/x201/mainboard.c
@@ -134,7 +134,7 @@ static void mainboard_init(device_t dev)
/* This sneaked in here, because X201 SuperIO chip isn't really
connected to anything and hence we don't init it.
*/
- pc_keyboard_init(0);
+ pc_keyboard_init();
/* Enable expresscard hotplug events. */
pci_write_config32(dev_find_slot(0, PCI_DEVFN(0x1c, 3)),
diff --git a/src/mainboard/lenovo/x230/mainboard.c b/src/mainboard/lenovo/x230/mainboard.c
index 2e4bc4a..01d1910 100644
--- a/src/mainboard/lenovo/x230/mainboard.c
+++ b/src/mainboard/lenovo/x230/mainboard.c
@@ -161,7 +161,7 @@ static void mainboard_init(device_t dev)
/* This sneaked in here, because X201 SuperIO chip isn't really
connected to anything and hence we don't init it.
*/
- pc_keyboard_init(0);
+ pc_keyboard_init();
/* Enable expresscard hotplug events. */
pci_write_config32(dev_find_slot(0, PCI_DEVFN(0x1c, 2)),
diff --git a/src/mainboard/packardbell/ms2290/mainboard.c b/src/mainboard/packardbell/ms2290/mainboard.c
index e54df19..d7f45c0 100644
--- a/src/mainboard/packardbell/ms2290/mainboard.c
+++ b/src/mainboard/packardbell/ms2290/mainboard.c
@@ -169,7 +169,7 @@ static void mainboard_enable(device_t dev)
/* This sneaked in here, because EasyNote has no SuperIO chip.
*/
- pc_keyboard_init(0);
+ pc_keyboard_init();
verb_setup();
}
diff --git a/src/mainboard/roda/rk9/mainboard.c b/src/mainboard/roda/rk9/mainboard.c
index 33e6bde..0627a5e 100644
--- a/src/mainboard/roda/rk9/mainboard.c
+++ b/src/mainboard/roda/rk9/mainboard.c
@@ -133,7 +133,7 @@ static void mainboard_enable(device_t dev)
/* We have no driver for the embedded controller since the firmware
does most of the job. Hence, initialize keyboards here. */
- pc_keyboard_init(NULL);
+ pc_keyboard_init();
}
struct chip_operations mainboard_ops = {
diff --git a/src/northbridge/via/cx700/lpc.c b/src/northbridge/via/cx700/lpc.c
index 22e8743..c09fbb4 100644
--- a/src/northbridge/via/cx700/lpc.c
+++ b/src/northbridge/via/cx700/lpc.c
@@ -289,7 +289,7 @@ static void cx700_lpc_init(struct device *dev)
isa_dma_init();
/* Initialize keyboard controller */
- pc_keyboard_init(0);
+ pc_keyboard_init();
}
static struct device_operations cx700_lpc_ops = {
diff --git a/src/northbridge/via/vx800/lpc.c b/src/northbridge/via/vx800/lpc.c
index 9e1fa57..1908a8a 100644
--- a/src/northbridge/via/vx800/lpc.c
+++ b/src/northbridge/via/vx800/lpc.c
@@ -330,7 +330,7 @@ static void southbridge_init(struct device *dev)
setup_i8259(); // make sure interupt controller is configured before keyboard init
/* turn on keyboard and RTC, no need to visit this reg twice */
- pc_keyboard_init(0);
+ pc_keyboard_init();
printk(BIOS_DEBUG, "ps2 usb lid, you set who can wakeup system from s3 sleep\n");
S3_ps2_kb_ms_wakeup(dev);
diff --git a/src/southbridge/dmp/vortex86ex/southbridge.c b/src/southbridge/dmp/vortex86ex/southbridge.c
index 2a68e4e..b9432f0 100644
--- a/src/southbridge/dmp/vortex86ex/southbridge.c
+++ b/src/southbridge/dmp/vortex86ex/southbridge.c
@@ -611,7 +611,7 @@ static void southbridge_init(struct device *dev)
retries--;
}
post_code(POST_DMP_KBD_IS_READY);
- pc_keyboard_init(0);
+ pc_keyboard_init();
}
static struct device_operations vortex_sb_ops = {
diff --git a/src/southbridge/sis/sis966/lpc.c b/src/southbridge/sis/sis966/lpc.c
index a4db1e5..824bb6a 100644
--- a/src/southbridge/sis/sis966/lpc.c
+++ b/src/southbridge/sis/sis966/lpc.c
@@ -89,16 +89,16 @@ static void lpc_usb_legacy_init(device_t dev)
static void lpc_init(device_t dev)
{
- uint8_t byte;
- uint8_t byte_old;
- int on;
- int nmi_option;
+ uint8_t byte;
+ uint8_t byte_old;
+ int on;
+ int nmi_option;
- printk(BIOS_DEBUG, "LPC_INIT -------->\n");
- pc_keyboard_init(0);
+ printk(BIOS_DEBUG, "LPC_INIT -------->\n");
+ pc_keyboard_init();
- lpc_usb_legacy_init(dev);
- lpc_common_init(dev);
+ lpc_usb_legacy_init(dev);
+ lpc_common_init(dev);
/* power after power fail */
diff --git a/src/southbridge/via/vt8231/vt8231.c b/src/southbridge/via/vt8231/vt8231.c
index 85f007a..2011d8a 100644
--- a/src/southbridge/via/vt8231/vt8231.c
+++ b/src/southbridge/via/vt8231/vt8231.c
@@ -21,7 +21,7 @@ static void keyboard_on(void)
regval |= 0x0f;
pci_write_config8(lpc_dev, 0x51, regval);
}
- pc_keyboard_init(0);
+ pc_keyboard_init();
}
static void com_port_on(void)
diff --git a/src/southbridge/via/vt8235/vt8235.c b/src/southbridge/via/vt8235/vt8235.c
index 4fa2784..6bc767a 100644
--- a/src/southbridge/via/vt8235/vt8235.c
+++ b/src/southbridge/via/vt8235/vt8235.c
@@ -16,7 +16,7 @@ static void keyboard_on(struct device *dev)
regval &= 0xfd;
pci_write_config8(dev, 0x51, regval);
- pc_keyboard_init(0);
+ pc_keyboard_init();
}
#ifdef UNUSED_CODE
diff --git a/src/southbridge/via/vt8237r/lpc.c b/src/southbridge/via/vt8237r/lpc.c
index 4949de6..5ddd816 100644
--- a/src/southbridge/via/vt8237r/lpc.c
+++ b/src/southbridge/via/vt8237r/lpc.c
@@ -627,7 +627,7 @@ static void init_keyboard(struct device *dev)
{
u8 regval = pci_read_config8(dev, 0x51);
if (regval & 0x1)
- pc_keyboard_init(0);
+ pc_keyboard_init();
}
static void southbridge_init_common(struct device *dev)
diff --git a/src/superio/fintek/f71863fg/superio.c b/src/superio/fintek/f71863fg/superio.c
index 29b5ab3..83ccd38 100644
--- a/src/superio/fintek/f71863fg/superio.c
+++ b/src/superio/fintek/f71863fg/superio.c
@@ -24,12 +24,11 @@
#include <superio/conf_mode.h>
#include <console/console.h>
#include <stdlib.h>
-#include "chip.h"
+#include <pc80/keyboard.h>
#include "f71863fg.h"
static void f71863fg_init(device_t dev)
{
- struct superio_fintek_f71863fg_config *conf = dev->chip_info;
struct resource *res0;
if (!dev->enabled)
@@ -39,7 +38,7 @@ static void f71863fg_init(device_t dev)
/* TODO: Might potentially need code for HWM or FDC etc. */
case F71863FG_KBC:
res0 = find_resource(dev, PNP_IDX_IO0);
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/fintek/f71869ad/superio.c b/src/superio/fintek/f71869ad/superio.c
index 11ad6f8..01544fe 100644
--- a/src/superio/fintek/f71869ad/superio.c
+++ b/src/superio/fintek/f71869ad/superio.c
@@ -24,13 +24,11 @@
#include <superio/conf_mode.h>
#include <console/console.h>
#include <stdlib.h>
-
-#include "chip.h"
+#include <pc80/keyboard.h>
#include "f71869ad.h"
static void f71869ad_init(device_t dev)
{
- struct superio_fintek_f71869ad_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -38,7 +36,7 @@ static void f71869ad_init(device_t dev)
switch(dev->path.pnp.device) {
/* TODO: Might potentially need code for HWM or FDC etc. */
case F71869AD_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/fintek/f71872/superio.c b/src/superio/fintek/f71872/superio.c
index 55a11b1..c6fa7d0 100644
--- a/src/superio/fintek/f71872/superio.c
+++ b/src/superio/fintek/f71872/superio.c
@@ -24,12 +24,11 @@
#include <superio/conf_mode.h>
#include <console/console.h>
#include <stdlib.h>
-#include "chip.h"
+#include <pc80/keyboard.h>
#include "f71872.h"
static void f71872_init(device_t dev)
{
- struct superio_fintek_f71872_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -37,7 +36,7 @@ static void f71872_init(device_t dev)
switch(dev->path.pnp.device) {
/* TODO: Might potentially need code for HWM or FDC etc. */
case F71872_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/fintek/f71889/superio.c b/src/superio/fintek/f71889/superio.c
index 71edafc..14bdd07 100644
--- a/src/superio/fintek/f71889/superio.c
+++ b/src/superio/fintek/f71889/superio.c
@@ -24,12 +24,11 @@
#include <superio/conf_mode.h>
#include <console/console.h>
#include <stdlib.h>
-#include "chip.h"
+#include <pc80/keyboard.h>
#include "f71889.h"
static void f71889_init(device_t dev)
{
- struct superio_fintek_f71889_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -37,7 +36,7 @@ static void f71889_init(device_t dev)
switch (dev->path.pnp.device) {
/* TODO: Might potentially need code for HWM or FDC etc. */
case F71889_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/fintek/f81865f/superio.c b/src/superio/fintek/f81865f/superio.c
index 85ac7ba..bfb53b4 100644
--- a/src/superio/fintek/f81865f/superio.c
+++ b/src/superio/fintek/f81865f/superio.c
@@ -24,12 +24,11 @@
#include <superio/conf_mode.h>
#include <console/console.h>
#include <stdlib.h>
-#include "chip.h"
+#include <pc80/keyboard.h>
#include "f81865f.h"
static void f81865f_init(device_t dev)
{
- struct superio_fintek_f81865f_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -37,7 +36,7 @@ static void f81865f_init(device_t dev)
switch (dev->path.pnp.device) {
/* TODO: Might potentially need code for HWM or FDC etc. */
case F81865F_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/ite/it8671f/superio.c b/src/superio/ite/it8671f/superio.c
index 21c70ec..9da8f1d 100644
--- a/src/superio/ite/it8671f/superio.c
+++ b/src/superio/ite/it8671f/superio.c
@@ -22,12 +22,10 @@
#include <device/pnp.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "it8671f.h"
static void init(device_t dev)
{
- struct superio_ite_it8671f_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -38,7 +36,7 @@ static void init(device_t dev)
case IT8671F_PP: /* TODO. */
break;
case IT8671F_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
case IT8671F_KBCM: /* TODO. */
break;
diff --git a/src/superio/ite/it8712f/superio.c b/src/superio/ite/it8712f/superio.c
index 2fd3440..769afe3 100644
--- a/src/superio/ite/it8712f/superio.c
+++ b/src/superio/ite/it8712f/superio.c
@@ -24,7 +24,6 @@
#include <pc80/keyboard.h>
#include <arch/io.h>
#include <stdlib.h>
-#include "chip.h"
#include "it8712f.h"
static void pnp_enter_ext_func_mode(device_t dev)
@@ -44,7 +43,6 @@ static void pnp_exit_ext_func_mode(device_t dev)
static void it8712f_init(device_t dev)
{
- struct superio_ite_it8712f_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -58,7 +56,7 @@ static void it8712f_init(device_t dev)
break;
case IT8712F_KBCK:
set_kbc_ps2_mode();
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
case IT8712F_KBCM: /* TODO. */
break;
diff --git a/src/superio/ite/it8716f/superio.c b/src/superio/ite/it8716f/superio.c
index 8cf3c5f..bd89a32 100644
--- a/src/superio/ite/it8716f/superio.c
+++ b/src/superio/ite/it8716f/superio.c
@@ -27,7 +27,6 @@
#include <pc80/keyboard.h>
#include <arch/io.h>
#include <stdlib.h>
-#include "chip.h"
#include "it8716f.h"
static void pnp_enter_ext_func_mode(device_t dev)
@@ -76,7 +75,6 @@ static void init_ec(u16 base)
static void it8716f_init(device_t dev)
{
- struct superio_ite_it8716f_config *conf = dev->chip_info;
struct resource *res0;
if (!dev->enabled)
@@ -90,7 +88,7 @@ static void it8716f_init(device_t dev)
init_ec(res0->base + EC_INDEX_PORT);
break;
case IT8716F_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/ite/it8718f/superio.c b/src/superio/ite/it8718f/superio.c
index 998d271..d40407a 100644
--- a/src/superio/ite/it8718f/superio.c
+++ b/src/superio/ite/it8718f/superio.c
@@ -22,12 +22,10 @@
#include <device/pnp.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "it8718f.h"
static void init(device_t dev)
{
- struct superio_ite_it8718f_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -40,7 +38,7 @@ static void init(device_t dev)
case IT8718F_EC: /* TODO. */
break;
case IT8718F_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
case IT8718F_KBCM: /* TODO. */
break;
diff --git a/src/superio/ite/it8721f/superio.c b/src/superio/ite/it8721f/superio.c
index 300a480..66e6ef7 100644
--- a/src/superio/ite/it8721f/superio.c
+++ b/src/superio/ite/it8721f/superio.c
@@ -23,12 +23,10 @@
#include <device/pnp.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "it8721f.h"
static void init(device_t dev)
{
- struct superio_ite_it8721f_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -41,7 +39,7 @@ static void init(device_t dev)
case IT8721F_EC: /* TODO. */
break;
case IT8721F_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
case IT8721F_KBCM: /* TODO. */
break;
diff --git a/src/superio/ite/it8772f/superio.c b/src/superio/ite/it8772f/superio.c
index e7e82a8..721e7f9 100644
--- a/src/superio/ite/it8772f/superio.c
+++ b/src/superio/ite/it8772f/superio.c
@@ -23,7 +23,7 @@
#include <pc80/keyboard.h>
#include <arch/io.h>
#include <stdlib.h>
-#include "chip.h"
+#include "chip.h" /* FIXME */
#include "it8772f.h"
static void pnp_enter_ext_func_mode(device_t dev)
@@ -177,7 +177,7 @@ static void it8772f_init(device_t dev)
case IT8772F_KBCK:
if (!conf->skip_keyboard) {
set_kbc_ps2_mode();
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
}
break;
case IT8772F_KBCM:
diff --git a/src/superio/nsc/pc8374/superio.c b/src/superio/nsc/pc8374/superio.c
index 71cde2c..8b6f1ca 100644
--- a/src/superio/nsc/pc8374/superio.c
+++ b/src/superio/nsc/pc8374/superio.c
@@ -26,19 +26,17 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "pc8374.h"
static void init(device_t dev)
{
- struct superio_nsc_pc8374_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case PC8374_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/nsc/pc87309/superio.c b/src/superio/nsc/pc87309/superio.c
index 52fc14f..f374e6a 100644
--- a/src/superio/nsc/pc87309/superio.c
+++ b/src/superio/nsc/pc87309/superio.c
@@ -23,19 +23,17 @@
#include <device/pnp.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "pc87309.h"
static void init(device_t dev)
{
- struct superio_nsc_pc87309_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch (dev->path.pnp.device) {
case PC87309_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/nsc/pc87351/superio.c b/src/superio/nsc/pc87351/superio.c
index 4d9469e..7f395b4 100644
--- a/src/superio/nsc/pc87351/superio.c
+++ b/src/superio/nsc/pc87351/superio.c
@@ -26,19 +26,17 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "pc87351.h"
static void init(device_t dev)
{
- struct superio_nsc_pc87351_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case PC87351_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/nsc/pc87360/superio.c b/src/superio/nsc/pc87360/superio.c
index 92731e7..a53cc35 100644
--- a/src/superio/nsc/pc87360/superio.c
+++ b/src/superio/nsc/pc87360/superio.c
@@ -26,19 +26,17 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "pc87360.h"
static void init(device_t dev)
{
- struct superio_nsc_pc87360_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case PC87360_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/nsc/pc87366/superio.c b/src/superio/nsc/pc87366/superio.c
index 1874ab3..27bdd6e 100644
--- a/src/superio/nsc/pc87366/superio.c
+++ b/src/superio/nsc/pc87366/superio.c
@@ -26,19 +26,17 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "pc87366.h"
static void init(device_t dev)
{
- struct superio_nsc_pc87366_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case PC87366_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/nsc/pc87417/superio.c b/src/superio/nsc/pc87417/superio.c
index 3b91f36..99882a5 100644
--- a/src/superio/nsc/pc87417/superio.c
+++ b/src/superio/nsc/pc87417/superio.c
@@ -27,19 +27,17 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "pc87417.h"
static void init(device_t dev)
{
- struct superio_nsc_pc87417_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case PC87417_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/nsc/pc87427/superio.c b/src/superio/nsc/pc87427/superio.c
index ae02f01..170fd85 100644
--- a/src/superio/nsc/pc87427/superio.c
+++ b/src/superio/nsc/pc87427/superio.c
@@ -25,19 +25,18 @@
#include <console/console.h>
#include <string.h>
#include <stdlib.h>
-#include "chip.h"
+#include <pc80/keyboard.h>
#include "pc87427.h"
static void init(device_t dev)
{
- struct superio_nsc_pc87427_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case PC87427_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/nsc/pc97307/superio.c b/src/superio/nsc/pc97307/superio.c
index c1cd5c1..af7ca77 100644
--- a/src/superio/nsc/pc97307/superio.c
+++ b/src/superio/nsc/pc97307/superio.c
@@ -23,12 +23,12 @@
#include <device/device.h>
#include <device/pnp.h>
#include <stdlib.h>
-#include "chip.h"
+#include <pc80/keyboard.h>
+#include "chip.h" /* FIXME */
#include "pc97307.h"
static void init(device_t dev)
{
- struct superio_nsc_pc97307_config *conf = dev->chip_info;
u8 reg8;
if (!dev->enabled)
@@ -40,7 +40,7 @@ static void init(device_t dev)
pnp_set_enable(dev, 0); /* Disable keyboard */
pnp_write_config(dev, 0xf0, 0x40); /* Set KBC clock to 8 MHz. */
pnp_set_enable(dev, 1); /* Enable keyboard. */
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
case PC97307_FDC:
/* Set up floppy in PS/2 mode. */
diff --git a/src/superio/nsc/pc97317/superio.c b/src/superio/nsc/pc97317/superio.c
index a69dfe7..6f39c5d 100644
--- a/src/superio/nsc/pc97317/superio.c
+++ b/src/superio/nsc/pc97317/superio.c
@@ -23,12 +23,12 @@
#include <device/device.h>
#include <device/pnp.h>
#include <stdlib.h>
-#include "chip.h"
+#include <pc80/keyboard.h>
+#include "chip.h" /* FIXME */
#include "pc97317.h"
static void init(device_t dev)
{
- struct superio_nsc_pc97317_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -39,7 +39,7 @@ static void init(device_t dev)
pnp_set_enable(dev, 0); /* Disable keyboard */
pnp_write_config(dev, 0xf0, 0x40); /* Set KBC clock to 8 MHz. */
pnp_set_enable(dev, 1); /* Enable keyboard */
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
default:
break;
diff --git a/src/superio/nuvoton/wpcm450/superio.c b/src/superio/nuvoton/wpcm450/superio.c
index fabb546..ff815cf 100644
--- a/src/superio/nuvoton/wpcm450/superio.c
+++ b/src/superio/nuvoton/wpcm450/superio.c
@@ -25,19 +25,17 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "wpcm450.h"
static void init(device_t dev)
{
- struct superio_nuvoton_wpcm450_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case WPCM450_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/renesas/m3885x/superio.c b/src/superio/renesas/m3885x/superio.c
index 34d7d11..17bdadf 100644
--- a/src/superio/renesas/m3885x/superio.c
+++ b/src/superio/renesas/m3885x/superio.c
@@ -25,20 +25,19 @@
#include <string.h>
#include <assert.h>
#include <stdlib.h>
-#include "chip.h"
+#include <pc80/keyboard.h>
void m3885_configure_multikey(void);
static void m3885x_init(device_t dev)
{
- struct superio_renesas_m3885x_config *conf = dev->chip_info;
if (!dev->enabled)
return;
printk(BIOS_DEBUG, "Renesas M3885x: Initializing keyboard.\n");
set_kbc_ps2_mode();
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
m3885_configure_multikey();
}
diff --git a/src/superio/smsc/fdc37m60x/superio.c b/src/superio/smsc/fdc37m60x/superio.c
index 92a1783..f5626ee 100644
--- a/src/superio/smsc/fdc37m60x/superio.c
+++ b/src/superio/smsc/fdc37m60x/superio.c
@@ -22,12 +22,10 @@
#include <device/pnp.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "fdc37m60x.h"
static void init(device_t dev)
{
- struct superio_smsc_fdc37m60x_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -38,7 +36,7 @@ static void init(device_t dev)
case FDC37M60X_PP: /* TODO. */
break;
case FDC37M60X_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
case FDC37M60X_AUX: /* TODO. */
break;
diff --git a/src/superio/smsc/fdc37n972/fdc37n972.c b/src/superio/smsc/fdc37n972/fdc37n972.c
index cab4540..f7ae697 100644
--- a/src/superio/smsc/fdc37n972/fdc37n972.c
+++ b/src/superio/smsc/fdc37n972/fdc37n972.c
@@ -26,8 +26,6 @@
static void init(device_t dev)
{
- struct superio_smsc_fdc37n972_config *conf = dev->chip_info;
-
if (!dev->enabled)
return;
@@ -37,7 +35,7 @@ static void init(device_t dev)
case FDC37N972_PP: /* TODO. */
break;
case FDC37N972_KBDC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
// [..] The rest: TODO
}
@@ -67,4 +65,3 @@ struct chip_operations superio_smsc_fdc37n972_ops = {
CHIP_NAME("SMSC FDC37N972 Super I/O")
.enable_dev = enable_dev,
};
-
diff --git a/src/superio/smsc/kbc1100/superio.c b/src/superio/smsc/kbc1100/superio.c
index 3a834a0..77777d0 100644
--- a/src/superio/smsc/kbc1100/superio.c
+++ b/src/superio/smsc/kbc1100/superio.c
@@ -28,7 +28,6 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "kbc1100.h"
/* Forward declarations */
@@ -60,7 +59,6 @@ static void enable_dev(device_t dev)
static void kbc1100_init(device_t dev)
{
- struct superio_smsc_kbc1100_config *conf = dev->chip_info;
struct resource *res0, *res1;
@@ -74,7 +72,7 @@ static void kbc1100_init(device_t dev)
case KBC1100_KBC:
res0 = find_resource(dev, PNP_IDX_IO0);
res1 = find_resource(dev, PNP_IDX_IO1);
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/smsc/lpc47b272/superio.c b/src/superio/smsc/lpc47b272/superio.c
index fb39c28..30e89a8 100644
--- a/src/superio/smsc/lpc47b272/superio.c
+++ b/src/superio/smsc/lpc47b272/superio.c
@@ -32,7 +32,6 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "lpc47b272.h"
/* Forward declarations */
@@ -85,14 +84,13 @@ static void enable_dev(device_t dev)
*/
static void lpc47b272_init(device_t dev)
{
- struct superio_smsc_lpc47b272_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case LPC47B272_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/smsc/lpc47b397/superio.c b/src/superio/smsc/lpc47b397/superio.c
index 3d81350..8a6c03e 100644
--- a/src/superio/smsc/lpc47b397/superio.c
+++ b/src/superio/smsc/lpc47b397/superio.c
@@ -29,7 +29,6 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "lpc47b397.h"
static void pnp_write_index(u16 port, u8 reg, u8 value)
@@ -56,14 +55,13 @@ static void enable_hwm_smbus(device_t dev)
static void lpc47b397_init(device_t dev)
{
- struct superio_smsc_lpc47b397_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case LPC47B397_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/smsc/lpc47m10x/superio.c b/src/superio/smsc/lpc47m10x/superio.c
index 1a697f2..a9583e4 100644
--- a/src/superio/smsc/lpc47m10x/superio.c
+++ b/src/superio/smsc/lpc47m10x/superio.c
@@ -31,7 +31,6 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "lpc47m10x.h"
/* Forward declarations */
@@ -84,14 +83,13 @@ static void enable_dev(device_t dev)
*/
static void lpc47m10x_init(device_t dev)
{
- struct superio_smsc_lpc47m10x_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case LPC47M10X2_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/smsc/lpc47m15x/superio.c b/src/superio/smsc/lpc47m15x/superio.c
index b142dde..91e77ba 100644
--- a/src/superio/smsc/lpc47m15x/superio.c
+++ b/src/superio/smsc/lpc47m15x/superio.c
@@ -28,7 +28,6 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "lpc47m15x.h"
/* Forward declarations */
@@ -65,14 +64,13 @@ static void enable_dev(device_t dev)
static void lpc47m15x_init(device_t dev)
{
- struct superio_smsc_lpc47m15x_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case LPC47M15X_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/smsc/lpc47n227/superio.c b/src/superio/smsc/lpc47n227/superio.c
index eda2622..b1ef18a 100644
--- a/src/superio/smsc/lpc47n227/superio.c
+++ b/src/superio/smsc/lpc47n227/superio.c
@@ -28,7 +28,7 @@
#include <string.h>
#include <assert.h>
#include <stdlib.h>
-#include "chip.h"
+#include <pc80/keyboard.h>
#include "lpc47n227.h"
/* Forward declarations. */
@@ -128,7 +128,6 @@ void lpc47n227_pnp_enable(device_t dev)
*/
static void lpc47n227_init(device_t dev)
{
- struct superio_smsc_lpc47n227_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -136,7 +135,7 @@ static void lpc47n227_init(device_t dev)
switch (dev->path.pnp.device) {
case LPC47N227_KBDC:
printk(BIOS_DEBUG, "LPC47N227: Initializing keyboard.\n");
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/smsc/mec1308/superio.c b/src/superio/smsc/mec1308/superio.c
index 448800e..54d99fb 100644
--- a/src/superio/smsc/mec1308/superio.c
+++ b/src/superio/smsc/mec1308/superio.c
@@ -28,7 +28,6 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "mec1308.h"
#if CONFIG_HAVE_ACPI_RESUME
#include <arch/acpi.h>
@@ -36,7 +35,6 @@
static void mec1308_init(device_t dev)
{
- struct superio_smsc_mec1308_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -47,7 +45,7 @@ static void mec1308_init(device_t dev)
if (acpi_slp_type == 3)
return;
#endif
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/smsc/sch4037/superio.c b/src/superio/smsc/sch4037/superio.c
index b7f100a..35df431 100644
--- a/src/superio/smsc/sch4037/superio.c
+++ b/src/superio/smsc/sch4037/superio.c
@@ -28,7 +28,6 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "sch4037.h"
/* Forward declarations */
@@ -61,7 +60,6 @@ static void enable_dev(device_t dev)
static void sch4037_init(device_t dev)
{
- struct superio_smsc_sch4037_config *conf = dev->chip_info;
struct resource *res0, *res1;
if (!dev->enabled) {
@@ -73,7 +71,7 @@ static void sch4037_init(device_t dev)
case SCH4037_KBC:
res0 = find_resource(dev, PNP_IDX_IO0);
res1 = find_resource(dev, PNP_IDX_IO1);
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/smsc/sio1007/chip.h b/src/superio/smsc/sio1007/chip.h
index eee2811..345b403 100644
--- a/src/superio/smsc/sio1007/chip.h
+++ b/src/superio/smsc/sio1007/chip.h
@@ -20,6 +20,7 @@
#ifndef SUPERIO_SMSC_1007_CHIP_H
#define SUPERIO_SMSC_1007_CHIP_H
+/* FIXME: wrong place for this! */
void sio1007_setreg(u16 lpc_port, u8 reg, u8 value, u8 mask);
int sio1007_enable_uart_at(u16 port);
diff --git a/src/superio/smsc/sio1036/superio.c b/src/superio/smsc/sio1036/superio.c
index 4980803..7fac79c 100644
--- a/src/superio/smsc/sio1036/superio.c
+++ b/src/superio/smsc/sio1036/superio.c
@@ -28,7 +28,6 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "sio1036.h"
/* Forward declarations */
@@ -60,7 +59,6 @@ static void enable_dev(device_t dev)
static void sio1036_init(device_t dev)
{
- struct superio_smsc_sio1036_config *conf = dev->chip_info;
struct resource *res0, *res1;
diff --git a/src/superio/smsc/sio10n268/sio10n268.c b/src/superio/smsc/sio10n268/sio10n268.c
index 690c160..40c01b4 100644
--- a/src/superio/smsc/sio10n268/sio10n268.c
+++ b/src/superio/smsc/sio10n268/sio10n268.c
@@ -26,8 +26,6 @@
static void init(device_t dev)
{
- struct superio_smsc_sio10n268_config *conf = dev->chip_info;
-
if (!dev->enabled)
return;
@@ -38,7 +36,7 @@ static void init(device_t dev)
break;
case SIO10N268_KBDC:
/* TODO: This is still hardcoded. */
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
// [..] The rest: TODO
}
diff --git a/src/superio/smsc/smscsuperio/superio.c b/src/superio/smsc/smscsuperio/superio.c
index 9990aa2..0d62a9a 100644
--- a/src/superio/smsc/smscsuperio/superio.c
+++ b/src/superio/smsc/smscsuperio/superio.c
@@ -41,7 +41,6 @@
#include <console/console.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
/* The following Super I/O chips are currently supported by this driver: */
#define LPC47M172 0x14
@@ -150,7 +149,6 @@ static const struct logical_devices {
*/
static void smsc_init(device_t dev)
{
- struct superio_smsc_smscsuperio_config *conf = dev->chip_info;
int i, ld;
/* Do not initialize disabled devices. */
@@ -169,7 +167,7 @@ static void smsc_init(device_t dev)
/* A Super I/O was found, so initialize the respective device. */
ld = dev->path.pnp.device;
if (ld == logical_device_table[i].devs[LD_KBC]) {
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
}
}
diff --git a/src/superio/winbond/w83627dhg/superio.c b/src/superio/winbond/w83627dhg/superio.c
index 269e57c..fdab974 100644
--- a/src/superio/winbond/w83627dhg/superio.c
+++ b/src/superio/winbond/w83627dhg/superio.c
@@ -23,7 +23,6 @@
#include <superio/conf_mode.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "w83627dhg.h"
static void w83627dhg_enable_UR2(device_t dev)
@@ -39,7 +38,6 @@ static void w83627dhg_enable_UR2(device_t dev)
static void w83627dhg_init(device_t dev)
{
- struct superio_winbond_w83627dhg_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -49,7 +47,7 @@ static void w83627dhg_init(device_t dev)
w83627dhg_enable_UR2(dev);
break;
case W83627DHG_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/winbond/w83627ehg/superio.c b/src/superio/winbond/w83627ehg/superio.c
index 62d3311..aae0ac4 100644
--- a/src/superio/winbond/w83627ehg/superio.c
+++ b/src/superio/winbond/w83627ehg/superio.c
@@ -30,7 +30,6 @@
#include <pc80/keyboard.h>
#include <pc80/mc146818rtc.h>
#include <stdlib.h>
-#include "chip.h"
#include "w83627ehg.h"
static void pnp_write_index(u16 port, u8 reg, u8 value)
@@ -95,7 +94,6 @@ static void init_hwm(u16 base)
static void w83627ehg_init(device_t dev)
{
- struct superio_winbond_w83627ehg_config *conf = dev->chip_info;
struct resource *res0;
if (!dev->enabled)
@@ -103,7 +101,7 @@ static void w83627ehg_init(device_t dev)
switch(dev->path.pnp.device) {
case W83627EHG_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
case W83627EHG_HWM:
res0 = find_resource(dev, PNP_IDX_IO0);
diff --git a/src/superio/winbond/w83627hf/superio.c b/src/superio/winbond/w83627hf/superio.c
index c881743..5445b47 100644
--- a/src/superio/winbond/w83627hf/superio.c
+++ b/src/superio/winbond/w83627hf/superio.c
@@ -30,7 +30,6 @@
#include <pc80/keyboard.h>
#include <pc80/mc146818rtc.h>
#include <stdlib.h>
-#include "chip.h"
#include "w83627hf.h"
static void pnp_write_index(u16 port, u8 reg, u8 value)
@@ -102,7 +101,6 @@ static void init_hwm(u16 base)
static void w83627hf_init(device_t dev)
{
- struct superio_winbond_w83627hf_config *conf = dev->chip_info;
struct resource *res0;
if (!dev->enabled)
@@ -110,7 +108,7 @@ static void w83627hf_init(device_t dev)
switch(dev->path.pnp.device) {
case W83627HF_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
case W83627HF_HWM:
res0 = find_resource(dev, PNP_IDX_IO0);
diff --git a/src/superio/winbond/w83627thg/superio.c b/src/superio/winbond/w83627thg/superio.c
index f6aba10..75186fa 100644
--- a/src/superio/winbond/w83627thg/superio.c
+++ b/src/superio/winbond/w83627thg/superio.c
@@ -27,19 +27,17 @@
#include <console/console.h>
#include <string.h>
#include <pc80/keyboard.h>
-#include "chip.h"
#include "w83627thg.h"
static void w83627thg_init(device_t dev)
{
- struct superio_winbond_w83627thg_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case W83627THG_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/winbond/w83627uhg/superio.c b/src/superio/winbond/w83627uhg/superio.c
index 5775ff5..f0d9aef 100644
--- a/src/superio/winbond/w83627uhg/superio.c
+++ b/src/superio/winbond/w83627uhg/superio.c
@@ -27,7 +27,6 @@
#include <stdint.h>
#include <stdlib.h>
#include <pc80/keyboard.h>
-#include "chip.h"
#include "w83627uhg.h"
/*
@@ -60,7 +59,6 @@ static void set_uart_clock_source(device_t dev, u8 uart_clock)
static void w83627uhg_init(device_t dev)
{
- struct superio_winbond_w83627uhg_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -85,7 +83,7 @@ static void w83627uhg_init(device_t dev)
set_uart_clock_source(dev, 0);
break;
case W83627UHG_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/winbond/w83697hf/superio.c b/src/superio/winbond/w83697hf/superio.c
index f896078..933da95 100644
--- a/src/superio/winbond/w83697hf/superio.c
+++ b/src/superio/winbond/w83697hf/superio.c
@@ -27,7 +27,8 @@
#include <lib.h>
#include <pc80/mc146818rtc.h>
#include <stdlib.h>
-#include "chip.h"
+#include <pc80/keyboard.h>
+#include "chip.h" /* FIXME */
#include "w83697hf.h"
static void hwmon_set_fan_divisor(unsigned int base, int num, unsigned int divisor) {
diff --git a/src/superio/winbond/w83977f/superio.c b/src/superio/winbond/w83977f/superio.c
index 32ecf18..232f46a 100644
--- a/src/superio/winbond/w83977f/superio.c
+++ b/src/superio/winbond/w83977f/superio.c
@@ -26,19 +26,17 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "w83977f.h"
static void w83977f_init(device_t dev)
{
- struct superio_winbond_w83977f_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case W83977F_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/winbond/w83977tf/superio.c b/src/superio/winbond/w83977tf/superio.c
index 7f2dffb..a61770a 100644
--- a/src/superio/winbond/w83977tf/superio.c
+++ b/src/superio/winbond/w83977tf/superio.c
@@ -28,19 +28,17 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "w83977tf.h"
static void w83977tf_init(device_t dev)
{
- struct superio_winbond_w83977tf_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case W83977TF_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
1
0
Patch set updated for coreboot: 1b493af src/drivers/pc80: Remove empty struct keyboard
by Edward O'Callaghan April 30, 2014
by Edward O'Callaghan April 30, 2014
April 30, 2014
Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5617
-gerrit
commit 1b493af8381e4365665e5a6e74a7844e95439601
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Wed Apr 30 05:01:52 2014 +1000
src/drivers/pc80: Remove empty struct keyboard
This is a empty struct that has propagated through the superio's & ec's
but really does nothing. Time to get rid of it before it adds yet more
cruft. However, since this touches many superio's at once we do this in
stages by first changing the function type to be a pure procedure.
Change-Id: Ibc732e676a9d4f0269114acabc92b15771d27ef2
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
src/drivers/pc80/keyboard.c | 2 +-
src/ec/compal/ene932/ec.c | 5 +----
src/ec/google/chromeec/ec_lpc.c | 5 ++---
src/ec/quanta/ene_kb3940q/ec.c | 4 +---
src/ec/quanta/it8518/ec.c | 5 +----
src/include/pc80/keyboard.h | 2 +-
src/mainboard/emulation/qemu-i440fx/mainboard.c | 2 +-
src/mainboard/emulation/qemu-q35/mainboard.c | 2 +-
src/mainboard/lenovo/t530/mainboard.c | 2 +-
src/mainboard/lenovo/x201/mainboard.c | 2 +-
src/mainboard/lenovo/x230/mainboard.c | 2 +-
src/mainboard/packardbell/ms2290/mainboard.c | 2 +-
src/mainboard/roda/rk9/mainboard.c | 2 +-
src/northbridge/via/cx700/lpc.c | 2 +-
src/northbridge/via/vx800/lpc.c | 2 +-
src/southbridge/dmp/vortex86ex/southbridge.c | 2 +-
src/southbridge/sis/sis966/lpc.c | 16 ++++++++--------
src/southbridge/via/vt8231/vt8231.c | 2 +-
src/southbridge/via/vt8235/vt8235.c | 2 +-
src/southbridge/via/vt8237r/lpc.c | 2 +-
src/superio/fintek/f71863fg/superio.c | 5 ++---
src/superio/fintek/f71869ad/superio.c | 6 ++----
src/superio/fintek/f71872/superio.c | 5 ++---
src/superio/fintek/f71889/superio.c | 5 ++---
src/superio/fintek/f81865f/superio.c | 5 ++---
src/superio/ite/it8671f/superio.c | 4 +---
src/superio/ite/it8712f/superio.c | 4 +---
src/superio/ite/it8716f/superio.c | 4 +---
src/superio/ite/it8718f/superio.c | 4 +---
src/superio/ite/it8721f/superio.c | 4 +---
src/superio/ite/it8772f/superio.c | 4 ++--
src/superio/nsc/pc8374/superio.c | 4 +---
src/superio/nsc/pc87309/superio.c | 4 +---
src/superio/nsc/pc87351/superio.c | 4 +---
src/superio/nsc/pc87360/superio.c | 4 +---
src/superio/nsc/pc87366/superio.c | 4 +---
src/superio/nsc/pc87417/superio.c | 4 +---
src/superio/nsc/pc87427/superio.c | 5 ++---
src/superio/nsc/pc97307/superio.c | 6 +++---
src/superio/nsc/pc97317/superio.c | 6 +++---
src/superio/nuvoton/wpcm450/superio.c | 4 +---
src/superio/renesas/m3885x/superio.c | 5 ++---
src/superio/smsc/fdc37m60x/superio.c | 4 +---
src/superio/smsc/fdc37n972/fdc37n972.c | 2 +-
src/superio/smsc/kbc1100/superio.c | 4 +---
src/superio/smsc/lpc47b272/superio.c | 4 +---
src/superio/smsc/lpc47b397/superio.c | 4 +---
src/superio/smsc/lpc47m10x/superio.c | 4 +---
src/superio/smsc/lpc47m15x/superio.c | 4 +---
src/superio/smsc/lpc47n227/superio.c | 5 ++---
src/superio/smsc/mec1308/superio.c | 4 +---
src/superio/smsc/sch4037/superio.c | 4 +---
src/superio/smsc/sio1007/chip.h | 1 +
src/superio/smsc/sio1036/superio.c | 2 --
src/superio/smsc/sio10n268/sio10n268.c | 4 +---
src/superio/smsc/smscsuperio/superio.c | 4 +---
src/superio/winbond/w83627dhg/superio.c | 4 +---
src/superio/winbond/w83627ehg/superio.c | 4 +---
src/superio/winbond/w83627hf/superio.c | 4 +---
src/superio/winbond/w83627thg/superio.c | 4 +---
src/superio/winbond/w83627uhg/superio.c | 4 +---
src/superio/winbond/w83697hf/superio.c | 3 ++-
src/superio/winbond/w83977f/superio.c | 4 +---
src/superio/winbond/w83977tf/superio.c | 4 +---
64 files changed, 85 insertions(+), 161 deletions(-)
diff --git a/src/drivers/pc80/keyboard.c b/src/drivers/pc80/keyboard.c
index 5b68617..6a573d8 100644
--- a/src/drivers/pc80/keyboard.c
+++ b/src/drivers/pc80/keyboard.c
@@ -193,7 +193,7 @@ static u8 send_keyboard(u8 command)
return regval;
}
-void pc_keyboard_init(struct pc_keyboard *keyboard)
+void pc_keyboard_init(void)
{
u8 retries;
u8 regval;
diff --git a/src/ec/compal/ene932/ec.c b/src/ec/compal/ene932/ec.c
index 3f4d1b9..e60a0ac 100644
--- a/src/ec/compal/ene932/ec.c
+++ b/src/ec/compal/ene932/ec.c
@@ -133,14 +133,11 @@ static u8 ec_io_read(u16 addr)
#ifndef __SMM__
static void ene932_init(device_t dev)
{
- struct ec_compal_ene932_config *conf = dev->chip_info;
-
-
if (!dev->enabled)
return;
printk(BIOS_DEBUG, "Compal ENE932: Initializing keyboard.\n");
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
}
diff --git a/src/ec/google/chromeec/ec_lpc.c b/src/ec/google/chromeec/ec_lpc.c
index a445da8..5f377c9 100644
--- a/src/ec/google/chromeec/ec_lpc.c
+++ b/src/ec/google/chromeec/ec_lpc.c
@@ -138,11 +138,10 @@ int google_chromeec_command(struct chromeec_command *cec_command)
#ifndef __SMM__
static void lpc_ec_init(device_t dev)
{
- struct ec_google_chromeec_config *conf = dev->chip_info;
-
if (!dev->enabled)
return;
- pc_keyboard_init(&conf->keyboard);
+
+ pc_keyboard_init();
google_chromeec_init();
}
diff --git a/src/ec/quanta/ene_kb3940q/ec.c b/src/ec/quanta/ene_kb3940q/ec.c
index 137aa81..57854d0 100644
--- a/src/ec/quanta/ene_kb3940q/ec.c
+++ b/src/ec/quanta/ene_kb3940q/ec.c
@@ -141,13 +141,11 @@ static void ene_kb3940q_log_events(void)
static void ene_kb3940q_init(device_t dev)
{
- struct ec_quanta_ene_kb3940q_config *conf = dev->chip_info;
-
if (!dev->enabled)
return;
printk(BIOS_DEBUG, "Quanta EnE KB3940Q: Initializing keyboard.\n");
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
ene_kb3940q_log_events();
}
diff --git a/src/ec/quanta/it8518/ec.c b/src/ec/quanta/it8518/ec.c
index 6516cee..7588a1e 100644
--- a/src/ec/quanta/it8518/ec.c
+++ b/src/ec/quanta/it8518/ec.c
@@ -162,14 +162,11 @@ void ec_it8518_enable_wake_events(void)
#ifndef __SMM__
static void it8518_init(device_t dev)
{
- struct ec_quanta_it8518_config *conf = dev->chip_info;
-
-
if (!dev->enabled)
return;
printk(BIOS_DEBUG, "Quanta IT8518: Initializing keyboard.\n");
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
}
diff --git a/src/include/pc80/keyboard.h b/src/include/pc80/keyboard.h
index ce281bb..5f97723 100644
--- a/src/include/pc80/keyboard.h
+++ b/src/include/pc80/keyboard.h
@@ -5,7 +5,7 @@ struct pc_keyboard {
/* No initialization parameters for now */
};
-void pc_keyboard_init(struct pc_keyboard *keyboard);
+void pc_keyboard_init(void);
void set_kbc_ps2_mode(void);
#endif /* PC80_KEYBOARD_H */
diff --git a/src/mainboard/emulation/qemu-i440fx/mainboard.c b/src/mainboard/emulation/qemu-i440fx/mainboard.c
index 90ad2db..a8a61c4 100644
--- a/src/mainboard/emulation/qemu-i440fx/mainboard.c
+++ b/src/mainboard/emulation/qemu-i440fx/mainboard.c
@@ -43,7 +43,7 @@ static void qemu_nb_init(device_t dev)
/* This sneaked in here, because Qemu does not
* emulate a SuperIO chip
*/
- pc_keyboard_init(0);
+ pc_keyboard_init();
/* setup IRQ routing */
for (i = 0; i < 32; i++)
diff --git a/src/mainboard/emulation/qemu-q35/mainboard.c b/src/mainboard/emulation/qemu-q35/mainboard.c
index 78c92a9..e991b53 100644
--- a/src/mainboard/emulation/qemu-q35/mainboard.c
+++ b/src/mainboard/emulation/qemu-q35/mainboard.c
@@ -50,7 +50,7 @@ static void qemu_nb_init(device_t dev)
/* This sneaked in here, because Qemu does not
* emulate a SuperIO chip
*/
- pc_keyboard_init(0);
+ pc_keyboard_init();
/* setup IRQ routing for pci slots */
for (i = 0; i < 25; i++)
diff --git a/src/mainboard/lenovo/t530/mainboard.c b/src/mainboard/lenovo/t530/mainboard.c
index 5931d00..6d21b31 100644
--- a/src/mainboard/lenovo/t530/mainboard.c
+++ b/src/mainboard/lenovo/t530/mainboard.c
@@ -161,7 +161,7 @@ static void mainboard_init(device_t dev)
/* This sneaked in here, because X201 SuperIO chip isn't really
connected to anything and hence we don't init it.
*/
- pc_keyboard_init(0);
+ pc_keyboard_init();
/* Enable expresscard hotplug events. */
pci_write_config32(dev_find_slot(0, PCI_DEVFN(0x1c, 2)),
diff --git a/src/mainboard/lenovo/x201/mainboard.c b/src/mainboard/lenovo/x201/mainboard.c
index cf42a9b..d772d19 100644
--- a/src/mainboard/lenovo/x201/mainboard.c
+++ b/src/mainboard/lenovo/x201/mainboard.c
@@ -134,7 +134,7 @@ static void mainboard_init(device_t dev)
/* This sneaked in here, because X201 SuperIO chip isn't really
connected to anything and hence we don't init it.
*/
- pc_keyboard_init(0);
+ pc_keyboard_init();
/* Enable expresscard hotplug events. */
pci_write_config32(dev_find_slot(0, PCI_DEVFN(0x1c, 3)),
diff --git a/src/mainboard/lenovo/x230/mainboard.c b/src/mainboard/lenovo/x230/mainboard.c
index 2e4bc4a..01d1910 100644
--- a/src/mainboard/lenovo/x230/mainboard.c
+++ b/src/mainboard/lenovo/x230/mainboard.c
@@ -161,7 +161,7 @@ static void mainboard_init(device_t dev)
/* This sneaked in here, because X201 SuperIO chip isn't really
connected to anything and hence we don't init it.
*/
- pc_keyboard_init(0);
+ pc_keyboard_init();
/* Enable expresscard hotplug events. */
pci_write_config32(dev_find_slot(0, PCI_DEVFN(0x1c, 2)),
diff --git a/src/mainboard/packardbell/ms2290/mainboard.c b/src/mainboard/packardbell/ms2290/mainboard.c
index e54df19..d7f45c0 100644
--- a/src/mainboard/packardbell/ms2290/mainboard.c
+++ b/src/mainboard/packardbell/ms2290/mainboard.c
@@ -169,7 +169,7 @@ static void mainboard_enable(device_t dev)
/* This sneaked in here, because EasyNote has no SuperIO chip.
*/
- pc_keyboard_init(0);
+ pc_keyboard_init();
verb_setup();
}
diff --git a/src/mainboard/roda/rk9/mainboard.c b/src/mainboard/roda/rk9/mainboard.c
index 33e6bde..0627a5e 100644
--- a/src/mainboard/roda/rk9/mainboard.c
+++ b/src/mainboard/roda/rk9/mainboard.c
@@ -133,7 +133,7 @@ static void mainboard_enable(device_t dev)
/* We have no driver for the embedded controller since the firmware
does most of the job. Hence, initialize keyboards here. */
- pc_keyboard_init(NULL);
+ pc_keyboard_init();
}
struct chip_operations mainboard_ops = {
diff --git a/src/northbridge/via/cx700/lpc.c b/src/northbridge/via/cx700/lpc.c
index 22e8743..c09fbb4 100644
--- a/src/northbridge/via/cx700/lpc.c
+++ b/src/northbridge/via/cx700/lpc.c
@@ -289,7 +289,7 @@ static void cx700_lpc_init(struct device *dev)
isa_dma_init();
/* Initialize keyboard controller */
- pc_keyboard_init(0);
+ pc_keyboard_init();
}
static struct device_operations cx700_lpc_ops = {
diff --git a/src/northbridge/via/vx800/lpc.c b/src/northbridge/via/vx800/lpc.c
index 9e1fa57..1908a8a 100644
--- a/src/northbridge/via/vx800/lpc.c
+++ b/src/northbridge/via/vx800/lpc.c
@@ -330,7 +330,7 @@ static void southbridge_init(struct device *dev)
setup_i8259(); // make sure interupt controller is configured before keyboard init
/* turn on keyboard and RTC, no need to visit this reg twice */
- pc_keyboard_init(0);
+ pc_keyboard_init();
printk(BIOS_DEBUG, "ps2 usb lid, you set who can wakeup system from s3 sleep\n");
S3_ps2_kb_ms_wakeup(dev);
diff --git a/src/southbridge/dmp/vortex86ex/southbridge.c b/src/southbridge/dmp/vortex86ex/southbridge.c
index 2a68e4e..b9432f0 100644
--- a/src/southbridge/dmp/vortex86ex/southbridge.c
+++ b/src/southbridge/dmp/vortex86ex/southbridge.c
@@ -611,7 +611,7 @@ static void southbridge_init(struct device *dev)
retries--;
}
post_code(POST_DMP_KBD_IS_READY);
- pc_keyboard_init(0);
+ pc_keyboard_init();
}
static struct device_operations vortex_sb_ops = {
diff --git a/src/southbridge/sis/sis966/lpc.c b/src/southbridge/sis/sis966/lpc.c
index a4db1e5..824bb6a 100644
--- a/src/southbridge/sis/sis966/lpc.c
+++ b/src/southbridge/sis/sis966/lpc.c
@@ -89,16 +89,16 @@ static void lpc_usb_legacy_init(device_t dev)
static void lpc_init(device_t dev)
{
- uint8_t byte;
- uint8_t byte_old;
- int on;
- int nmi_option;
+ uint8_t byte;
+ uint8_t byte_old;
+ int on;
+ int nmi_option;
- printk(BIOS_DEBUG, "LPC_INIT -------->\n");
- pc_keyboard_init(0);
+ printk(BIOS_DEBUG, "LPC_INIT -------->\n");
+ pc_keyboard_init();
- lpc_usb_legacy_init(dev);
- lpc_common_init(dev);
+ lpc_usb_legacy_init(dev);
+ lpc_common_init(dev);
/* power after power fail */
diff --git a/src/southbridge/via/vt8231/vt8231.c b/src/southbridge/via/vt8231/vt8231.c
index 85f007a..2011d8a 100644
--- a/src/southbridge/via/vt8231/vt8231.c
+++ b/src/southbridge/via/vt8231/vt8231.c
@@ -21,7 +21,7 @@ static void keyboard_on(void)
regval |= 0x0f;
pci_write_config8(lpc_dev, 0x51, regval);
}
- pc_keyboard_init(0);
+ pc_keyboard_init();
}
static void com_port_on(void)
diff --git a/src/southbridge/via/vt8235/vt8235.c b/src/southbridge/via/vt8235/vt8235.c
index 4fa2784..6bc767a 100644
--- a/src/southbridge/via/vt8235/vt8235.c
+++ b/src/southbridge/via/vt8235/vt8235.c
@@ -16,7 +16,7 @@ static void keyboard_on(struct device *dev)
regval &= 0xfd;
pci_write_config8(dev, 0x51, regval);
- pc_keyboard_init(0);
+ pc_keyboard_init();
}
#ifdef UNUSED_CODE
diff --git a/src/southbridge/via/vt8237r/lpc.c b/src/southbridge/via/vt8237r/lpc.c
index 4949de6..5ddd816 100644
--- a/src/southbridge/via/vt8237r/lpc.c
+++ b/src/southbridge/via/vt8237r/lpc.c
@@ -627,7 +627,7 @@ static void init_keyboard(struct device *dev)
{
u8 regval = pci_read_config8(dev, 0x51);
if (regval & 0x1)
- pc_keyboard_init(0);
+ pc_keyboard_init();
}
static void southbridge_init_common(struct device *dev)
diff --git a/src/superio/fintek/f71863fg/superio.c b/src/superio/fintek/f71863fg/superio.c
index 29b5ab3..83ccd38 100644
--- a/src/superio/fintek/f71863fg/superio.c
+++ b/src/superio/fintek/f71863fg/superio.c
@@ -24,12 +24,11 @@
#include <superio/conf_mode.h>
#include <console/console.h>
#include <stdlib.h>
-#include "chip.h"
+#include <pc80/keyboard.h>
#include "f71863fg.h"
static void f71863fg_init(device_t dev)
{
- struct superio_fintek_f71863fg_config *conf = dev->chip_info;
struct resource *res0;
if (!dev->enabled)
@@ -39,7 +38,7 @@ static void f71863fg_init(device_t dev)
/* TODO: Might potentially need code for HWM or FDC etc. */
case F71863FG_KBC:
res0 = find_resource(dev, PNP_IDX_IO0);
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/fintek/f71869ad/superio.c b/src/superio/fintek/f71869ad/superio.c
index 11ad6f8..01544fe 100644
--- a/src/superio/fintek/f71869ad/superio.c
+++ b/src/superio/fintek/f71869ad/superio.c
@@ -24,13 +24,11 @@
#include <superio/conf_mode.h>
#include <console/console.h>
#include <stdlib.h>
-
-#include "chip.h"
+#include <pc80/keyboard.h>
#include "f71869ad.h"
static void f71869ad_init(device_t dev)
{
- struct superio_fintek_f71869ad_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -38,7 +36,7 @@ static void f71869ad_init(device_t dev)
switch(dev->path.pnp.device) {
/* TODO: Might potentially need code for HWM or FDC etc. */
case F71869AD_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/fintek/f71872/superio.c b/src/superio/fintek/f71872/superio.c
index 55a11b1..c6fa7d0 100644
--- a/src/superio/fintek/f71872/superio.c
+++ b/src/superio/fintek/f71872/superio.c
@@ -24,12 +24,11 @@
#include <superio/conf_mode.h>
#include <console/console.h>
#include <stdlib.h>
-#include "chip.h"
+#include <pc80/keyboard.h>
#include "f71872.h"
static void f71872_init(device_t dev)
{
- struct superio_fintek_f71872_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -37,7 +36,7 @@ static void f71872_init(device_t dev)
switch(dev->path.pnp.device) {
/* TODO: Might potentially need code for HWM or FDC etc. */
case F71872_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/fintek/f71889/superio.c b/src/superio/fintek/f71889/superio.c
index 71edafc..14bdd07 100644
--- a/src/superio/fintek/f71889/superio.c
+++ b/src/superio/fintek/f71889/superio.c
@@ -24,12 +24,11 @@
#include <superio/conf_mode.h>
#include <console/console.h>
#include <stdlib.h>
-#include "chip.h"
+#include <pc80/keyboard.h>
#include "f71889.h"
static void f71889_init(device_t dev)
{
- struct superio_fintek_f71889_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -37,7 +36,7 @@ static void f71889_init(device_t dev)
switch (dev->path.pnp.device) {
/* TODO: Might potentially need code for HWM or FDC etc. */
case F71889_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/fintek/f81865f/superio.c b/src/superio/fintek/f81865f/superio.c
index 85ac7ba..bfb53b4 100644
--- a/src/superio/fintek/f81865f/superio.c
+++ b/src/superio/fintek/f81865f/superio.c
@@ -24,12 +24,11 @@
#include <superio/conf_mode.h>
#include <console/console.h>
#include <stdlib.h>
-#include "chip.h"
+#include <pc80/keyboard.h>
#include "f81865f.h"
static void f81865f_init(device_t dev)
{
- struct superio_fintek_f81865f_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -37,7 +36,7 @@ static void f81865f_init(device_t dev)
switch (dev->path.pnp.device) {
/* TODO: Might potentially need code for HWM or FDC etc. */
case F81865F_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/ite/it8671f/superio.c b/src/superio/ite/it8671f/superio.c
index 21c70ec..9da8f1d 100644
--- a/src/superio/ite/it8671f/superio.c
+++ b/src/superio/ite/it8671f/superio.c
@@ -22,12 +22,10 @@
#include <device/pnp.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "it8671f.h"
static void init(device_t dev)
{
- struct superio_ite_it8671f_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -38,7 +36,7 @@ static void init(device_t dev)
case IT8671F_PP: /* TODO. */
break;
case IT8671F_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
case IT8671F_KBCM: /* TODO. */
break;
diff --git a/src/superio/ite/it8712f/superio.c b/src/superio/ite/it8712f/superio.c
index 2fd3440..769afe3 100644
--- a/src/superio/ite/it8712f/superio.c
+++ b/src/superio/ite/it8712f/superio.c
@@ -24,7 +24,6 @@
#include <pc80/keyboard.h>
#include <arch/io.h>
#include <stdlib.h>
-#include "chip.h"
#include "it8712f.h"
static void pnp_enter_ext_func_mode(device_t dev)
@@ -44,7 +43,6 @@ static void pnp_exit_ext_func_mode(device_t dev)
static void it8712f_init(device_t dev)
{
- struct superio_ite_it8712f_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -58,7 +56,7 @@ static void it8712f_init(device_t dev)
break;
case IT8712F_KBCK:
set_kbc_ps2_mode();
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
case IT8712F_KBCM: /* TODO. */
break;
diff --git a/src/superio/ite/it8716f/superio.c b/src/superio/ite/it8716f/superio.c
index 8cf3c5f..bd89a32 100644
--- a/src/superio/ite/it8716f/superio.c
+++ b/src/superio/ite/it8716f/superio.c
@@ -27,7 +27,6 @@
#include <pc80/keyboard.h>
#include <arch/io.h>
#include <stdlib.h>
-#include "chip.h"
#include "it8716f.h"
static void pnp_enter_ext_func_mode(device_t dev)
@@ -76,7 +75,6 @@ static void init_ec(u16 base)
static void it8716f_init(device_t dev)
{
- struct superio_ite_it8716f_config *conf = dev->chip_info;
struct resource *res0;
if (!dev->enabled)
@@ -90,7 +88,7 @@ static void it8716f_init(device_t dev)
init_ec(res0->base + EC_INDEX_PORT);
break;
case IT8716F_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/ite/it8718f/superio.c b/src/superio/ite/it8718f/superio.c
index 998d271..d40407a 100644
--- a/src/superio/ite/it8718f/superio.c
+++ b/src/superio/ite/it8718f/superio.c
@@ -22,12 +22,10 @@
#include <device/pnp.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "it8718f.h"
static void init(device_t dev)
{
- struct superio_ite_it8718f_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -40,7 +38,7 @@ static void init(device_t dev)
case IT8718F_EC: /* TODO. */
break;
case IT8718F_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
case IT8718F_KBCM: /* TODO. */
break;
diff --git a/src/superio/ite/it8721f/superio.c b/src/superio/ite/it8721f/superio.c
index 300a480..66e6ef7 100644
--- a/src/superio/ite/it8721f/superio.c
+++ b/src/superio/ite/it8721f/superio.c
@@ -23,12 +23,10 @@
#include <device/pnp.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "it8721f.h"
static void init(device_t dev)
{
- struct superio_ite_it8721f_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -41,7 +39,7 @@ static void init(device_t dev)
case IT8721F_EC: /* TODO. */
break;
case IT8721F_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
case IT8721F_KBCM: /* TODO. */
break;
diff --git a/src/superio/ite/it8772f/superio.c b/src/superio/ite/it8772f/superio.c
index e7e82a8..721e7f9 100644
--- a/src/superio/ite/it8772f/superio.c
+++ b/src/superio/ite/it8772f/superio.c
@@ -23,7 +23,7 @@
#include <pc80/keyboard.h>
#include <arch/io.h>
#include <stdlib.h>
-#include "chip.h"
+#include "chip.h" /* FIXME */
#include "it8772f.h"
static void pnp_enter_ext_func_mode(device_t dev)
@@ -177,7 +177,7 @@ static void it8772f_init(device_t dev)
case IT8772F_KBCK:
if (!conf->skip_keyboard) {
set_kbc_ps2_mode();
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
}
break;
case IT8772F_KBCM:
diff --git a/src/superio/nsc/pc8374/superio.c b/src/superio/nsc/pc8374/superio.c
index 71cde2c..8b6f1ca 100644
--- a/src/superio/nsc/pc8374/superio.c
+++ b/src/superio/nsc/pc8374/superio.c
@@ -26,19 +26,17 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "pc8374.h"
static void init(device_t dev)
{
- struct superio_nsc_pc8374_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case PC8374_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/nsc/pc87309/superio.c b/src/superio/nsc/pc87309/superio.c
index 52fc14f..f374e6a 100644
--- a/src/superio/nsc/pc87309/superio.c
+++ b/src/superio/nsc/pc87309/superio.c
@@ -23,19 +23,17 @@
#include <device/pnp.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "pc87309.h"
static void init(device_t dev)
{
- struct superio_nsc_pc87309_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch (dev->path.pnp.device) {
case PC87309_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/nsc/pc87351/superio.c b/src/superio/nsc/pc87351/superio.c
index 4d9469e..7f395b4 100644
--- a/src/superio/nsc/pc87351/superio.c
+++ b/src/superio/nsc/pc87351/superio.c
@@ -26,19 +26,17 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "pc87351.h"
static void init(device_t dev)
{
- struct superio_nsc_pc87351_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case PC87351_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/nsc/pc87360/superio.c b/src/superio/nsc/pc87360/superio.c
index 92731e7..a53cc35 100644
--- a/src/superio/nsc/pc87360/superio.c
+++ b/src/superio/nsc/pc87360/superio.c
@@ -26,19 +26,17 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "pc87360.h"
static void init(device_t dev)
{
- struct superio_nsc_pc87360_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case PC87360_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/nsc/pc87366/superio.c b/src/superio/nsc/pc87366/superio.c
index 1874ab3..27bdd6e 100644
--- a/src/superio/nsc/pc87366/superio.c
+++ b/src/superio/nsc/pc87366/superio.c
@@ -26,19 +26,17 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "pc87366.h"
static void init(device_t dev)
{
- struct superio_nsc_pc87366_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case PC87366_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/nsc/pc87417/superio.c b/src/superio/nsc/pc87417/superio.c
index 3b91f36..99882a5 100644
--- a/src/superio/nsc/pc87417/superio.c
+++ b/src/superio/nsc/pc87417/superio.c
@@ -27,19 +27,17 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "pc87417.h"
static void init(device_t dev)
{
- struct superio_nsc_pc87417_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case PC87417_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/nsc/pc87427/superio.c b/src/superio/nsc/pc87427/superio.c
index ae02f01..170fd85 100644
--- a/src/superio/nsc/pc87427/superio.c
+++ b/src/superio/nsc/pc87427/superio.c
@@ -25,19 +25,18 @@
#include <console/console.h>
#include <string.h>
#include <stdlib.h>
-#include "chip.h"
+#include <pc80/keyboard.h>
#include "pc87427.h"
static void init(device_t dev)
{
- struct superio_nsc_pc87427_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case PC87427_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/nsc/pc97307/superio.c b/src/superio/nsc/pc97307/superio.c
index c1cd5c1..af7ca77 100644
--- a/src/superio/nsc/pc97307/superio.c
+++ b/src/superio/nsc/pc97307/superio.c
@@ -23,12 +23,12 @@
#include <device/device.h>
#include <device/pnp.h>
#include <stdlib.h>
-#include "chip.h"
+#include <pc80/keyboard.h>
+#include "chip.h" /* FIXME */
#include "pc97307.h"
static void init(device_t dev)
{
- struct superio_nsc_pc97307_config *conf = dev->chip_info;
u8 reg8;
if (!dev->enabled)
@@ -40,7 +40,7 @@ static void init(device_t dev)
pnp_set_enable(dev, 0); /* Disable keyboard */
pnp_write_config(dev, 0xf0, 0x40); /* Set KBC clock to 8 MHz. */
pnp_set_enable(dev, 1); /* Enable keyboard. */
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
case PC97307_FDC:
/* Set up floppy in PS/2 mode. */
diff --git a/src/superio/nsc/pc97317/superio.c b/src/superio/nsc/pc97317/superio.c
index a69dfe7..6f39c5d 100644
--- a/src/superio/nsc/pc97317/superio.c
+++ b/src/superio/nsc/pc97317/superio.c
@@ -23,12 +23,12 @@
#include <device/device.h>
#include <device/pnp.h>
#include <stdlib.h>
-#include "chip.h"
+#include <pc80/keyboard.h>
+#include "chip.h" /* FIXME */
#include "pc97317.h"
static void init(device_t dev)
{
- struct superio_nsc_pc97317_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -39,7 +39,7 @@ static void init(device_t dev)
pnp_set_enable(dev, 0); /* Disable keyboard */
pnp_write_config(dev, 0xf0, 0x40); /* Set KBC clock to 8 MHz. */
pnp_set_enable(dev, 1); /* Enable keyboard */
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
default:
break;
diff --git a/src/superio/nuvoton/wpcm450/superio.c b/src/superio/nuvoton/wpcm450/superio.c
index fabb546..ff815cf 100644
--- a/src/superio/nuvoton/wpcm450/superio.c
+++ b/src/superio/nuvoton/wpcm450/superio.c
@@ -25,19 +25,17 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "wpcm450.h"
static void init(device_t dev)
{
- struct superio_nuvoton_wpcm450_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case WPCM450_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/renesas/m3885x/superio.c b/src/superio/renesas/m3885x/superio.c
index 34d7d11..17bdadf 100644
--- a/src/superio/renesas/m3885x/superio.c
+++ b/src/superio/renesas/m3885x/superio.c
@@ -25,20 +25,19 @@
#include <string.h>
#include <assert.h>
#include <stdlib.h>
-#include "chip.h"
+#include <pc80/keyboard.h>
void m3885_configure_multikey(void);
static void m3885x_init(device_t dev)
{
- struct superio_renesas_m3885x_config *conf = dev->chip_info;
if (!dev->enabled)
return;
printk(BIOS_DEBUG, "Renesas M3885x: Initializing keyboard.\n");
set_kbc_ps2_mode();
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
m3885_configure_multikey();
}
diff --git a/src/superio/smsc/fdc37m60x/superio.c b/src/superio/smsc/fdc37m60x/superio.c
index 92a1783..f5626ee 100644
--- a/src/superio/smsc/fdc37m60x/superio.c
+++ b/src/superio/smsc/fdc37m60x/superio.c
@@ -22,12 +22,10 @@
#include <device/pnp.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "fdc37m60x.h"
static void init(device_t dev)
{
- struct superio_smsc_fdc37m60x_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -38,7 +36,7 @@ static void init(device_t dev)
case FDC37M60X_PP: /* TODO. */
break;
case FDC37M60X_KBCK:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
case FDC37M60X_AUX: /* TODO. */
break;
diff --git a/src/superio/smsc/fdc37n972/fdc37n972.c b/src/superio/smsc/fdc37n972/fdc37n972.c
index cab4540..a5d1163 100644
--- a/src/superio/smsc/fdc37n972/fdc37n972.c
+++ b/src/superio/smsc/fdc37n972/fdc37n972.c
@@ -37,7 +37,7 @@ static void init(device_t dev)
case FDC37N972_PP: /* TODO. */
break;
case FDC37N972_KBDC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
// [..] The rest: TODO
}
diff --git a/src/superio/smsc/kbc1100/superio.c b/src/superio/smsc/kbc1100/superio.c
index 3a834a0..77777d0 100644
--- a/src/superio/smsc/kbc1100/superio.c
+++ b/src/superio/smsc/kbc1100/superio.c
@@ -28,7 +28,6 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "kbc1100.h"
/* Forward declarations */
@@ -60,7 +59,6 @@ static void enable_dev(device_t dev)
static void kbc1100_init(device_t dev)
{
- struct superio_smsc_kbc1100_config *conf = dev->chip_info;
struct resource *res0, *res1;
@@ -74,7 +72,7 @@ static void kbc1100_init(device_t dev)
case KBC1100_KBC:
res0 = find_resource(dev, PNP_IDX_IO0);
res1 = find_resource(dev, PNP_IDX_IO1);
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/smsc/lpc47b272/superio.c b/src/superio/smsc/lpc47b272/superio.c
index fb39c28..30e89a8 100644
--- a/src/superio/smsc/lpc47b272/superio.c
+++ b/src/superio/smsc/lpc47b272/superio.c
@@ -32,7 +32,6 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "lpc47b272.h"
/* Forward declarations */
@@ -85,14 +84,13 @@ static void enable_dev(device_t dev)
*/
static void lpc47b272_init(device_t dev)
{
- struct superio_smsc_lpc47b272_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case LPC47B272_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/smsc/lpc47b397/superio.c b/src/superio/smsc/lpc47b397/superio.c
index 3d81350..8a6c03e 100644
--- a/src/superio/smsc/lpc47b397/superio.c
+++ b/src/superio/smsc/lpc47b397/superio.c
@@ -29,7 +29,6 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "lpc47b397.h"
static void pnp_write_index(u16 port, u8 reg, u8 value)
@@ -56,14 +55,13 @@ static void enable_hwm_smbus(device_t dev)
static void lpc47b397_init(device_t dev)
{
- struct superio_smsc_lpc47b397_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case LPC47B397_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/smsc/lpc47m10x/superio.c b/src/superio/smsc/lpc47m10x/superio.c
index 1a697f2..a9583e4 100644
--- a/src/superio/smsc/lpc47m10x/superio.c
+++ b/src/superio/smsc/lpc47m10x/superio.c
@@ -31,7 +31,6 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "lpc47m10x.h"
/* Forward declarations */
@@ -84,14 +83,13 @@ static void enable_dev(device_t dev)
*/
static void lpc47m10x_init(device_t dev)
{
- struct superio_smsc_lpc47m10x_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case LPC47M10X2_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/smsc/lpc47m15x/superio.c b/src/superio/smsc/lpc47m15x/superio.c
index b142dde..91e77ba 100644
--- a/src/superio/smsc/lpc47m15x/superio.c
+++ b/src/superio/smsc/lpc47m15x/superio.c
@@ -28,7 +28,6 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "lpc47m15x.h"
/* Forward declarations */
@@ -65,14 +64,13 @@ static void enable_dev(device_t dev)
static void lpc47m15x_init(device_t dev)
{
- struct superio_smsc_lpc47m15x_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case LPC47M15X_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/smsc/lpc47n227/superio.c b/src/superio/smsc/lpc47n227/superio.c
index eda2622..b1ef18a 100644
--- a/src/superio/smsc/lpc47n227/superio.c
+++ b/src/superio/smsc/lpc47n227/superio.c
@@ -28,7 +28,7 @@
#include <string.h>
#include <assert.h>
#include <stdlib.h>
-#include "chip.h"
+#include <pc80/keyboard.h>
#include "lpc47n227.h"
/* Forward declarations. */
@@ -128,7 +128,6 @@ void lpc47n227_pnp_enable(device_t dev)
*/
static void lpc47n227_init(device_t dev)
{
- struct superio_smsc_lpc47n227_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -136,7 +135,7 @@ static void lpc47n227_init(device_t dev)
switch (dev->path.pnp.device) {
case LPC47N227_KBDC:
printk(BIOS_DEBUG, "LPC47N227: Initializing keyboard.\n");
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/smsc/mec1308/superio.c b/src/superio/smsc/mec1308/superio.c
index 448800e..54d99fb 100644
--- a/src/superio/smsc/mec1308/superio.c
+++ b/src/superio/smsc/mec1308/superio.c
@@ -28,7 +28,6 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "mec1308.h"
#if CONFIG_HAVE_ACPI_RESUME
#include <arch/acpi.h>
@@ -36,7 +35,6 @@
static void mec1308_init(device_t dev)
{
- struct superio_smsc_mec1308_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -47,7 +45,7 @@ static void mec1308_init(device_t dev)
if (acpi_slp_type == 3)
return;
#endif
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/smsc/sch4037/superio.c b/src/superio/smsc/sch4037/superio.c
index b7f100a..35df431 100644
--- a/src/superio/smsc/sch4037/superio.c
+++ b/src/superio/smsc/sch4037/superio.c
@@ -28,7 +28,6 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "sch4037.h"
/* Forward declarations */
@@ -61,7 +60,6 @@ static void enable_dev(device_t dev)
static void sch4037_init(device_t dev)
{
- struct superio_smsc_sch4037_config *conf = dev->chip_info;
struct resource *res0, *res1;
if (!dev->enabled) {
@@ -73,7 +71,7 @@ static void sch4037_init(device_t dev)
case SCH4037_KBC:
res0 = find_resource(dev, PNP_IDX_IO0);
res1 = find_resource(dev, PNP_IDX_IO1);
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/smsc/sio1007/chip.h b/src/superio/smsc/sio1007/chip.h
index eee2811..345b403 100644
--- a/src/superio/smsc/sio1007/chip.h
+++ b/src/superio/smsc/sio1007/chip.h
@@ -20,6 +20,7 @@
#ifndef SUPERIO_SMSC_1007_CHIP_H
#define SUPERIO_SMSC_1007_CHIP_H
+/* FIXME: wrong place for this! */
void sio1007_setreg(u16 lpc_port, u8 reg, u8 value, u8 mask);
int sio1007_enable_uart_at(u16 port);
diff --git a/src/superio/smsc/sio1036/superio.c b/src/superio/smsc/sio1036/superio.c
index 4980803..7fac79c 100644
--- a/src/superio/smsc/sio1036/superio.c
+++ b/src/superio/smsc/sio1036/superio.c
@@ -28,7 +28,6 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "sio1036.h"
/* Forward declarations */
@@ -60,7 +59,6 @@ static void enable_dev(device_t dev)
static void sio1036_init(device_t dev)
{
- struct superio_smsc_sio1036_config *conf = dev->chip_info;
struct resource *res0, *res1;
diff --git a/src/superio/smsc/sio10n268/sio10n268.c b/src/superio/smsc/sio10n268/sio10n268.c
index 690c160..40c01b4 100644
--- a/src/superio/smsc/sio10n268/sio10n268.c
+++ b/src/superio/smsc/sio10n268/sio10n268.c
@@ -26,8 +26,6 @@
static void init(device_t dev)
{
- struct superio_smsc_sio10n268_config *conf = dev->chip_info;
-
if (!dev->enabled)
return;
@@ -38,7 +36,7 @@ static void init(device_t dev)
break;
case SIO10N268_KBDC:
/* TODO: This is still hardcoded. */
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
// [..] The rest: TODO
}
diff --git a/src/superio/smsc/smscsuperio/superio.c b/src/superio/smsc/smscsuperio/superio.c
index 9990aa2..0d62a9a 100644
--- a/src/superio/smsc/smscsuperio/superio.c
+++ b/src/superio/smsc/smscsuperio/superio.c
@@ -41,7 +41,6 @@
#include <console/console.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
/* The following Super I/O chips are currently supported by this driver: */
#define LPC47M172 0x14
@@ -150,7 +149,6 @@ static const struct logical_devices {
*/
static void smsc_init(device_t dev)
{
- struct superio_smsc_smscsuperio_config *conf = dev->chip_info;
int i, ld;
/* Do not initialize disabled devices. */
@@ -169,7 +167,7 @@ static void smsc_init(device_t dev)
/* A Super I/O was found, so initialize the respective device. */
ld = dev->path.pnp.device;
if (ld == logical_device_table[i].devs[LD_KBC]) {
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
}
}
diff --git a/src/superio/winbond/w83627dhg/superio.c b/src/superio/winbond/w83627dhg/superio.c
index 269e57c..fdab974 100644
--- a/src/superio/winbond/w83627dhg/superio.c
+++ b/src/superio/winbond/w83627dhg/superio.c
@@ -23,7 +23,6 @@
#include <superio/conf_mode.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "w83627dhg.h"
static void w83627dhg_enable_UR2(device_t dev)
@@ -39,7 +38,6 @@ static void w83627dhg_enable_UR2(device_t dev)
static void w83627dhg_init(device_t dev)
{
- struct superio_winbond_w83627dhg_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -49,7 +47,7 @@ static void w83627dhg_init(device_t dev)
w83627dhg_enable_UR2(dev);
break;
case W83627DHG_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/winbond/w83627ehg/superio.c b/src/superio/winbond/w83627ehg/superio.c
index 62d3311..aae0ac4 100644
--- a/src/superio/winbond/w83627ehg/superio.c
+++ b/src/superio/winbond/w83627ehg/superio.c
@@ -30,7 +30,6 @@
#include <pc80/keyboard.h>
#include <pc80/mc146818rtc.h>
#include <stdlib.h>
-#include "chip.h"
#include "w83627ehg.h"
static void pnp_write_index(u16 port, u8 reg, u8 value)
@@ -95,7 +94,6 @@ static void init_hwm(u16 base)
static void w83627ehg_init(device_t dev)
{
- struct superio_winbond_w83627ehg_config *conf = dev->chip_info;
struct resource *res0;
if (!dev->enabled)
@@ -103,7 +101,7 @@ static void w83627ehg_init(device_t dev)
switch(dev->path.pnp.device) {
case W83627EHG_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
case W83627EHG_HWM:
res0 = find_resource(dev, PNP_IDX_IO0);
diff --git a/src/superio/winbond/w83627hf/superio.c b/src/superio/winbond/w83627hf/superio.c
index c881743..5445b47 100644
--- a/src/superio/winbond/w83627hf/superio.c
+++ b/src/superio/winbond/w83627hf/superio.c
@@ -30,7 +30,6 @@
#include <pc80/keyboard.h>
#include <pc80/mc146818rtc.h>
#include <stdlib.h>
-#include "chip.h"
#include "w83627hf.h"
static void pnp_write_index(u16 port, u8 reg, u8 value)
@@ -102,7 +101,6 @@ static void init_hwm(u16 base)
static void w83627hf_init(device_t dev)
{
- struct superio_winbond_w83627hf_config *conf = dev->chip_info;
struct resource *res0;
if (!dev->enabled)
@@ -110,7 +108,7 @@ static void w83627hf_init(device_t dev)
switch(dev->path.pnp.device) {
case W83627HF_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
case W83627HF_HWM:
res0 = find_resource(dev, PNP_IDX_IO0);
diff --git a/src/superio/winbond/w83627thg/superio.c b/src/superio/winbond/w83627thg/superio.c
index f6aba10..75186fa 100644
--- a/src/superio/winbond/w83627thg/superio.c
+++ b/src/superio/winbond/w83627thg/superio.c
@@ -27,19 +27,17 @@
#include <console/console.h>
#include <string.h>
#include <pc80/keyboard.h>
-#include "chip.h"
#include "w83627thg.h"
static void w83627thg_init(device_t dev)
{
- struct superio_winbond_w83627thg_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case W83627THG_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/winbond/w83627uhg/superio.c b/src/superio/winbond/w83627uhg/superio.c
index 5775ff5..f0d9aef 100644
--- a/src/superio/winbond/w83627uhg/superio.c
+++ b/src/superio/winbond/w83627uhg/superio.c
@@ -27,7 +27,6 @@
#include <stdint.h>
#include <stdlib.h>
#include <pc80/keyboard.h>
-#include "chip.h"
#include "w83627uhg.h"
/*
@@ -60,7 +59,6 @@ static void set_uart_clock_source(device_t dev, u8 uart_clock)
static void w83627uhg_init(device_t dev)
{
- struct superio_winbond_w83627uhg_config *conf = dev->chip_info;
if (!dev->enabled)
return;
@@ -85,7 +83,7 @@ static void w83627uhg_init(device_t dev)
set_uart_clock_source(dev, 0);
break;
case W83627UHG_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/winbond/w83697hf/superio.c b/src/superio/winbond/w83697hf/superio.c
index f896078..933da95 100644
--- a/src/superio/winbond/w83697hf/superio.c
+++ b/src/superio/winbond/w83697hf/superio.c
@@ -27,7 +27,8 @@
#include <lib.h>
#include <pc80/mc146818rtc.h>
#include <stdlib.h>
-#include "chip.h"
+#include <pc80/keyboard.h>
+#include "chip.h" /* FIXME */
#include "w83697hf.h"
static void hwmon_set_fan_divisor(unsigned int base, int num, unsigned int divisor) {
diff --git a/src/superio/winbond/w83977f/superio.c b/src/superio/winbond/w83977f/superio.c
index 32ecf18..232f46a 100644
--- a/src/superio/winbond/w83977f/superio.c
+++ b/src/superio/winbond/w83977f/superio.c
@@ -26,19 +26,17 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "w83977f.h"
static void w83977f_init(device_t dev)
{
- struct superio_winbond_w83977f_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case W83977F_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
diff --git a/src/superio/winbond/w83977tf/superio.c b/src/superio/winbond/w83977tf/superio.c
index 7f2dffb..a61770a 100644
--- a/src/superio/winbond/w83977tf/superio.c
+++ b/src/superio/winbond/w83977tf/superio.c
@@ -28,19 +28,17 @@
#include <string.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
-#include "chip.h"
#include "w83977tf.h"
static void w83977tf_init(device_t dev)
{
- struct superio_winbond_w83977tf_config *conf = dev->chip_info;
if (!dev->enabled)
return;
switch(dev->path.pnp.device) {
case W83977TF_KBC:
- pc_keyboard_init(&conf->keyboard);
+ pc_keyboard_init();
break;
}
}
1
0