[coreboot-gerrit] New patch to review for coreboot: superio/fintek: Add support for Fintek F71889A.

Marty Plummer (ntzrmtthihu777@gmail.com) gerrit at coreboot.org
Thu Dec 1 09:16:26 CET 2016


Marty Plummer (ntzrmtthihu777 at gmail.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17669

-gerrit

commit d0d0730d1c454cb7b98206370786fcc20f9338ef
Author: Marty Plummer <ntzrmtthihu777 at gmail.com>
Date:   Thu Dec 1 02:14:39 2016 -0600

    superio/fintek: Add support for Fintek F71889A.
    
    Datasheet: F71889A rev V0.21P
    
    Change-Id: I91c60a3b48cd4872ae7a27de8f49faa40e877a27
    Signed-off-by: Marty Plummer <ntzrmtthihu777 at gmail.com>
---
 src/superio/fintek/Makefile.inc         |  1 +
 src/superio/fintek/f71889a/Kconfig      | 20 ++++++++++
 src/superio/fintek/f71889a/Makefile.inc | 18 +++++++++
 src/superio/fintek/f71889a/f71889a.h    | 31 ++++++++++++++
 src/superio/fintek/f71889a/superio.c    | 71 +++++++++++++++++++++++++++++++++
 5 files changed, 141 insertions(+)

diff --git a/src/superio/fintek/Makefile.inc b/src/superio/fintek/Makefile.inc
index 0d0ae66..5d117e2 100644
--- a/src/superio/fintek/Makefile.inc
+++ b/src/superio/fintek/Makefile.inc
@@ -24,3 +24,4 @@ subdirs-y += f71872
 subdirs-y += f81216h
 subdirs-y += f81865f
 subdirs-y += f81866d
+supdirs-y += f71889a
diff --git a/src/superio/fintek/f71889a/Kconfig b/src/superio/fintek/f71889a/Kconfig
new file mode 100644
index 0000000..641621e
--- /dev/null
+++ b/src/superio/fintek/f71889a/Kconfig
@@ -0,0 +1,20 @@
+#
+# This file is part of the coreboot project.
+#
+# Copyright (C) 2016  Marty Plummer <netz.kernel at 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 3 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.
+#
+
+config SUPERIO_FINTEK_F71889A
+	bool
+	select SUPERIO_FINTEK_COMMON_ROMSTAGE
+
diff --git a/src/superio/fintek/f71889a/Makefile.inc b/src/superio/fintek/f71889a/Makefile.inc
new file mode 100644
index 0000000..4642c46
--- /dev/null
+++ b/src/superio/fintek/f71889a/Makefile.inc
@@ -0,0 +1,18 @@
+##
+## This file is part of the coreboot project.
+##
+## Copyright (C) 2016  Marty Plummer <netz.kernel at 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 3 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.
+##
+
+ramstage-$(CONFIG_SUPERIO_FINTEK_F71889A) += superio.c
+
diff --git a/src/superio/fintek/f71889a/f71889a.h b/src/superio/fintek/f71889a/f71889a.h
new file mode 100644
index 0000000..a129e41
--- /dev/null
+++ b/src/superio/fintek/f71889a/f71889a.h
@@ -0,0 +1,31 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright (C) 2016  Marty Plummer <netz.kernel at 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 3 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.
+ */
+
+#ifndef SUPERIO_FINTEK_F71889A_H
+#define SUPERIO_FINTEK_F71889A_H
+
+/* Logical Device Numbers (LDN). */
+#define F71889A_SP1	0x01	/* UART1 */
+#define F71889A_SP2	0x02	/* UART2 */
+#define F71889A_PP	0x03	/* Parallel Port */
+#define F71889A_HWM	0x04	/* Hardware Monitor */
+#define F71889A_KBC	0x05	/* Keyboard/Mouse */
+#define F71889A_GPIO	0x06	/* GPIO */
+#define F71889A_VID	0x07	/* VID */
+#define F71889A_PM	0x0a	/* ACPI/PME */
+
+#endif /* SUPERIO_FINTEK_F71889A_H */
+
diff --git a/src/superio/fintek/f71889a/superio.c b/src/superio/fintek/f71889a/superio.c
new file mode 100644
index 0000000..34d4806
--- /dev/null
+++ b/src/superio/fintek/f71889a/superio.c
@@ -0,0 +1,71 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright (C) 2016  Marty Plummer <netz.kernel at 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 3 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.
+ */
+
+#include <arch/io.h>
+#include <device/device.h>
+#include <device/pnp.h>
+#include <superio/conf_mode.h>
+#include <console/console.h>
+#include <stdlib.h>
+#include <pc80/keyboard.h>
+#include "f71889a.h"
+
+static void f71889a_init(struct device *dev)
+{
+	if (!dev->enabled)
+		return;
+
+	switch (dev->path.pnp.device) {
+
+	case F71889A_KBC:
+		pc_keyboard_init(NO_AUX_DEVICE);
+		break;
+	}
+}
+
+static struct device_operations ops = {
+	.read_resources   = pnp_read_resources,
+	.set_resources    = pnp_set_resources,
+	.enable_resources = pnp_enable_resources,
+	.enable           = pnp_alt_enable,
+	.init             = f71889a_init,
+	.ops_pnp_mode     = &pnp_conf_mode_8787_aa,
+};
+
+static struct pnp_info pnp_dev_info[] = {
+	/* TODO: Some of the 0x07f8 etc. values may not be correct.
+	 * double check bitmasks
+	 */
+	{ &ops, F71889A_SP1,  PNP_IO0 | PNP_IRQ0, {0x07f8, 0}, },
+	{ &ops, F71889A_SP2,  PNP_IO0 | PNP_IRQ0, {0x07f8, 0}, },
+	{ &ops, F71889A_PP,   PNP_IO0 | PNP_IRQ0 | PNP_DRQ0, {0x07f8, 0}, },
+	{ &ops, F71889A_HWM,  PNP_IO0 | PNP_IRQ0, {0x07f8, 0}, },
+	{ &ops, F71889A_KBC,  PNP_IO0 | PNP_IRQ0 | PNP_IRQ1, {0x07ff, 0}, },
+	{ &ops, F71889A_GPIO, PNP_IRQ0, },
+	{ &ops, F71889A_VID,  PNP_IO0, {0x0ff8, 0}, },
+	{ &ops, F71889A_PM, },
+};
+
+static void enable_dev(struct device *dev)
+{
+	pnp_enable_devices(dev, &ops, ARRAY_SIZE(pnp_dev_info), pnp_dev_info);
+}
+
+struct chip_operations superio_fintek_f71889a_ops = {
+	CHIP_NAME("Fintek F71889A Super I/O")
+	.enable_Dev = enable_dev
+};
+



More information about the coreboot-gerrit mailing list