[coreboot-gerrit] Patch set updated for coreboot: drivers/i2c/alps: Add support for ALPS Touchpad driver

Barnali Sarkar (barnali.sarkar@intel.com) gerrit at coreboot.org
Tue Nov 15 11:41:04 CET 2016


Barnali Sarkar (barnali.sarkar at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17390

-gerrit

commit f2c698e99ca83a5eec4a9c09168495242daf0423
Author: Barnali Sarkar <barnali.sarkar at intel.com>
Date:   Sat Nov 12 00:55:53 2016 +0530

    drivers/i2c/alps: Add support for ALPS Touchpad driver
    
    Add support for I2C ALPS Touchpad Device Driver.
    
    BUG=none
    BRANCH=none
    TEST=Build and booted successfully on KBL RVP
    
    Change-Id: I78b77bd7c4694ccf61260724f593bd59545c70e6
    Signed-off-by: Barnali Sarkar <barnali.sarkar at intel.com>
---
 src/drivers/i2c/alps/Kconfig      |  3 ++
 src/drivers/i2c/alps/Makefile.inc |  1 +
 src/drivers/i2c/alps/alps.c       | 59 +++++++++++++++++++++++++++++++++++++++
 src/drivers/i2c/alps/chip.h       | 30 ++++++++++++++++++++
 4 files changed, 93 insertions(+)

diff --git a/src/drivers/i2c/alps/Kconfig b/src/drivers/i2c/alps/Kconfig
new file mode 100644
index 0000000..b49581d
--- /dev/null
+++ b/src/drivers/i2c/alps/Kconfig
@@ -0,0 +1,3 @@
+config DRIVERS_I2C_ALPS
+	bool
+	select DRIVERS_I2C_GENERIC
diff --git a/src/drivers/i2c/alps/Makefile.inc b/src/drivers/i2c/alps/Makefile.inc
new file mode 100644
index 0000000..aa18034
--- /dev/null
+++ b/src/drivers/i2c/alps/Makefile.inc
@@ -0,0 +1 @@
+ramstage-$(CONFIG_DRIVERS_I2C_ALPS) += alps.c
diff --git a/src/drivers/i2c/alps/alps.c b/src/drivers/i2c/alps/alps.c
new file mode 100644
index 0000000..2686f6c
--- /dev/null
+++ b/src/drivers/i2c/alps/alps.c
@@ -0,0 +1,59 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright (C) 2016 Intel 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; 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.
+ */
+
+#include <arch/acpigen_dsm.h>
+#include <device/device.h>
+#include <stdint.h>
+#include "chip.h"
+
+static void i2c_alps_fill_dsm(struct device *dev)
+{
+	struct drivers_i2c_alps_config *config = dev->chip_info;
+	struct dsm_i2c_hid_config dsm_config = {
+		.hid_desc_reg_offset = config->hid_desc_reg_offset,
+	};
+
+	acpigen_write_dsm_i2c_hid(&dsm_config);
+}
+
+static void i2c_alps_fill_ssdt_generator(struct device *dev)
+{
+	i2c_generic_fill_ssdt(dev, &i2c_alps_fill_dsm);
+}
+
+static const char *i2c_alps_acpi_name(struct device *dev)
+{
+	return "ALPS";
+}
+
+static struct device_operations i2c_alps_ops = {
+	.read_resources		  = DEVICE_NOOP,
+	.set_resources		  = DEVICE_NOOP,
+	.enable_resources	  = DEVICE_NOOP,
+#if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES)
+	.acpi_name		  = &i2c_alps_acpi_name,
+	.acpi_fill_ssdt_generator = &i2c_alps_fill_ssdt_generator,
+#endif
+};
+
+static void i2c_alps_enable(struct device *dev)
+{
+	dev->ops = &i2c_alps_ops;
+}
+
+struct chip_operations drivers_i2c_alps_ops = {
+	CHIP_NAME("ALPS I2C Device")
+	.enable_dev = &i2c_alps_enable
+};
diff --git a/src/drivers/i2c/alps/chip.h b/src/drivers/i2c/alps/chip.h
new file mode 100644
index 0000000..fb0b0fa
--- /dev/null
+++ b/src/drivers/i2c/alps/chip.h
@@ -0,0 +1,30 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright (C) 2016 Intel 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; 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.
+ */
+
+#ifndef __I2C_ALPS_CHIP_H__
+#define __I2C_ALPS_CHIP_H__
+
+#include <drivers/i2c/generic/chip.h>
+
+#define ALPS0001_HID		"ALPS0001"
+#define PNP0C50_CID		"PNP0C50"
+#define ALPS_TP_DESC		"ALPS Touchpad"
+
+struct drivers_i2c_alps_config {
+	struct drivers_i2c_generic_config generic;
+	uint8_t hid_desc_reg_offset;
+};
+
+#endif /* __I2C_ALPS_CHIP_H__ */



More information about the coreboot-gerrit mailing list