sravan/system76-coreboot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

drivers/i2c: Use I2C HID driver for wacom devices

Browse Source 
Wacom I2C driver does the same thing as I2C HID driver, other than
defining macros for Wacom HID. Instead of maintaining two separate
drivers providing the same functionality, update all wacom devices to
use generic I2C HID driver.

BUG=None
BRANCH=None
TEST=Verified that ACPI nodes for wacom devices are unchanged.

Change-Id: Ibb3226d1f3934f5c3c5d98b939756775d11b792c
Signed-off-by: Furquan Shaikh <furquan@chromium.org>
Reviewed-on: https://review.coreboot.org/18401
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Duncan Laurie <dlaurie@chromium.org>
This commit is contained in:
Furquan Shaikh
2017-02-19 01:18:09 -08:00
committed by Furquan Shaikh
parent 658a6dc78d
commit 5360c7ef94
14 changed files with 32 additions and 145 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
src/drivers/i2c/wacom/Kconfig
View File

@ -1,3 +0,0 @@
config DRIVERS_I2C_WACOM
bool
select DRIVERS_I2C_GENERIC

1
src/drivers/i2c/wacom/Makefile.inc
View File

@ -1 +0,0 @@
ramstage-$(CONFIG_DRIVERS_I2C_WACOM) += wacom.c

32
src/drivers/i2c/wacom/chip.h
View File

@ -1,32 +0,0 @@
/*
* 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 __I2C_WACOM_CHIP_H__
#define __I2C_WACOM_CHIP_H__
#include <drivers/i2c/generic/chip.h>
#define WCOM50C1_HID "WCOM50C1"
#define WCOMNTN2_HID "WCOMNTN2"
#define PNP0C50_CID "PNP0C50"
#define WCOM_TS_DESC "WCOM Touchscreen"
#define WCOM_DT_DESC "WCOM Digitizer"
struct drivers_i2c_wacom_config {
struct drivers_i2c_generic_config generic;
uint8_t hid_desc_reg_offset;
};
#endif /* __I2C_WACOM_CHIP_H__ */

60
src/drivers/i2c/wacom/wacom.c
View File

@ -1,60 +0,0 @@
/*
* 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 "chip.h"
static void i2c_wacom_fill_dsm(struct device *dev)
{
struct drivers_i2c_wacom_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_wacom_fill_ssdt_generator(struct device *dev)
{
struct drivers_i2c_wacom_config *config = dev->chip_info;
i2c_generic_fill_ssdt(dev, &i2c_wacom_fill_dsm, &config->generic);
}
static const char *i2c_wacom_acpi_name(struct device *dev)
{
return "WCOM";
}
static struct device_operations i2c_wacom_ops = {
.read_resources = DEVICE_NOOP,
.set_resources = DEVICE_NOOP,
.enable_resources = DEVICE_NOOP,
#if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES)
.acpi_name = &i2c_wacom_acpi_name,
.acpi_fill_ssdt_generator = &i2c_wacom_fill_ssdt_generator,
#endif
};
static void i2c_wacom_enable(struct device *dev)
{
dev->ops = &i2c_wacom_ops;
}
struct chip_operations drivers_i2c_wacom_ops = {
CHIP_NAME("Wacom I2C Device")
.enable_dev = &i2c_wacom_enable
};