Duncan Laurie (dlaurie@chromium.org) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17856
-gerrit
commit 369c16ed82a966d9b8dbab1a79c1b1bdc30902cd Author: Duncan Laurie dlaurie@chromium.org Date: Tue Dec 13 16:43:40 2016 -0800
drivers/i2c/hid: Add generic I2C HID driver
Add a generic I2C-HID driver for these types of devices that do not need extra functionality. This allows a new device to be added without having to write a new driver.
The i2c-hid PNP0C50 is automatically added as the _CID for the device in the ACPI Device.
BUG=chrome-os-partner:58666 TEST=used on eve to describe a new i2c-hid touch controller
Change-Id: I94e9531a72f9bf1d6b3ade362b88883b21b83d0a Signed-off-by: Duncan Laurie dlaurie@chromium.org --- src/drivers/i2c/hid/Kconfig | 3 ++ src/drivers/i2c/hid/Makefile.inc | 1 + src/drivers/i2c/hid/chip.h | 28 +++++++++++++++++ src/drivers/i2c/hid/hid.c | 67 ++++++++++++++++++++++++++++++++++++++++ 4 files changed, 99 insertions(+)
diff --git a/src/drivers/i2c/hid/Kconfig b/src/drivers/i2c/hid/Kconfig new file mode 100644 index 0000000..22531bb --- /dev/null +++ b/src/drivers/i2c/hid/Kconfig @@ -0,0 +1,3 @@ +config DRIVERS_I2C_HID + bool + select DRIVERS_I2C_GENERIC diff --git a/src/drivers/i2c/hid/Makefile.inc b/src/drivers/i2c/hid/Makefile.inc new file mode 100644 index 0000000..12162ac --- /dev/null +++ b/src/drivers/i2c/hid/Makefile.inc @@ -0,0 +1 @@ +ramstage-$(CONFIG_DRIVERS_I2C_HID) += hid.c diff --git a/src/drivers/i2c/hid/chip.h b/src/drivers/i2c/hid/chip.h new file mode 100644 index 0000000..7bce167 --- /dev/null +++ b/src/drivers/i2c/hid/chip.h @@ -0,0 +1,28 @@ +/* + * This file is part of the coreboot project. + * + * Copyright 2016 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. + */ + +#ifndef __DRIVERS_I2C_HID_CHIP_H__ +#define __DRIVERS_I2C_HID_CHIP_H__ + +#include <drivers/i2c/generic/chip.h> + +#define I2C_HID_CID "PNP0C50" + +struct drivers_i2c_hid_config { + struct drivers_i2c_generic_config generic; + uint8_t hid_desc_reg_offset; +}; + +#endif /* __I2C_HID_CHIP_H__ */ diff --git a/src/drivers/i2c/hid/hid.c b/src/drivers/i2c/hid/hid.c new file mode 100644 index 0000000..9eb56c2 --- /dev/null +++ b/src/drivers/i2c/hid/hid.c @@ -0,0 +1,67 @@ +/* + * This file is part of the coreboot project. + * + * Copyright 2016 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. + */ + +#include <arch/acpigen_dsm.h> +#include <device/device.h> +#include <stdint.h> +#include <string.h> +#include "chip.h" + +#if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES) +static void i2c_hid_fill_dsm(struct device *dev) +{ + struct drivers_i2c_hid_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_hid_fill_ssdt_generator(struct device *dev) +{ + struct drivers_i2c_hid_config *config = dev->chip_info; + config->generic.cid = I2C_HID_CID; + i2c_generic_fill_ssdt(dev, &i2c_hid_fill_dsm, &config->generic); +} + +static const char *i2c_hid_acpi_name(struct device *dev) +{ + static char name[5]; + snprintf(name, sizeof(name), "H%03.3X", dev->path.i2c.device); + name[4] = '\0'; + return name; +} +#endif + +static struct device_operations i2c_hid_ops = { + .read_resources = DEVICE_NOOP, + .set_resources = DEVICE_NOOP, + .enable_resources = DEVICE_NOOP, +#if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES) + .acpi_name = &i2c_hid_acpi_name, + .acpi_fill_ssdt_generator = &i2c_hid_fill_ssdt_generator, +#endif +}; + +static void i2c_hid_enable(struct device *dev) +{ + dev->ops = &i2c_hid_ops; +} + +struct chip_operations drivers_i2c_hid_ops = { + CHIP_NAME("I2C HID Device") + .enable_dev = &i2c_hid_enable +};