@@ -38,21 +38,6 @@ config I2C_HID_OF
will be called i2c-hid-of. It will also build/depend on the
module i2c-hid.
-config I2C_HID_OF_GOODIX
- tristate "Driver for Goodix hid-i2c based devices on OF systems"
- depends on OF
- select I2C_HID_CORE
- help
- Say Y here if you want support for Goodix i2c devices that use
- the i2c-hid protocol on Open Firmware (Device Tree)-based
- systems.
-
- If unsure, say N.
-
- This support is also available as a module. If so, the module
- will be called i2c-hid-of-goodix. It will also build/depend on
- the module i2c-hid.
-
config I2C_HID_CORE
tristate
endif
@@ -10,4 +10,3 @@ i2c-hid-$(CONFIG_DMI) += i2c-hid-dmi-quirks.o
obj-$(CONFIG_I2C_HID_ACPI) += i2c-hid-acpi.o
obj-$(CONFIG_I2C_HID_OF) += i2c-hid-of.o
-obj-$(CONFIG_I2C_HID_OF_GOODIX) += i2c-hid-of-goodix.o
deleted file mode 100644
@@ -1,125 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0
-/*
- * Driver for Goodix touchscreens that use the i2c-hid protocol.
- *
- * Copyright 2020 Google LLC
- */
-
-#include <linux/delay.h>
-#include <linux/device.h>
-#include <linux/gpio/consumer.h>
-#include <linux/i2c.h>
-#include <linux/kernel.h>
-#include <linux/module.h>
-#include <linux/of.h>
-#include <linux/pm.h>
-#include <linux/regulator/consumer.h>
-
-#include "i2c-hid.h"
-
-struct goodix_i2c_hid_timing_data {
- unsigned int post_gpio_reset_delay_ms;
- unsigned int post_power_delay_ms;
-};
-
-struct i2c_hid_of_goodix {
- struct i2chid_ops ops;
-
- struct regulator *vdd;
- struct regulator *vddio;
- struct gpio_desc *reset_gpio;
- const struct goodix_i2c_hid_timing_data *timings;
-};
-
-static int goodix_i2c_hid_power_up(struct i2chid_ops *ops)
-{
- struct i2c_hid_of_goodix *ihid_goodix =
- container_of(ops, struct i2c_hid_of_goodix, ops);
- int ret;
-
- ret = regulator_enable(ihid_goodix->vdd);
- if (ret)
- return ret;
-
- ret = regulator_enable(ihid_goodix->vddio);
- if (ret)
- return ret;
-
- if (ihid_goodix->timings->post_power_delay_ms)
- msleep(ihid_goodix->timings->post_power_delay_ms);
-
- gpiod_set_value_cansleep(ihid_goodix->reset_gpio, 0);
- if (ihid_goodix->timings->post_gpio_reset_delay_ms)
- msleep(ihid_goodix->timings->post_gpio_reset_delay_ms);
-
- return 0;
-}
-
-static void goodix_i2c_hid_power_down(struct i2chid_ops *ops)
-{
- struct i2c_hid_of_goodix *ihid_goodix =
- container_of(ops, struct i2c_hid_of_goodix, ops);
-
- gpiod_set_value_cansleep(ihid_goodix->reset_gpio, 1);
- regulator_disable(ihid_goodix->vddio);
- regulator_disable(ihid_goodix->vdd);
-}
-
-static int i2c_hid_of_goodix_probe(struct i2c_client *client)
-{
- struct i2c_hid_of_goodix *ihid_goodix;
-
- ihid_goodix = devm_kzalloc(&client->dev, sizeof(*ihid_goodix),
- GFP_KERNEL);
- if (!ihid_goodix)
- return -ENOMEM;
-
- ihid_goodix->ops.power_up = goodix_i2c_hid_power_up;
- ihid_goodix->ops.power_down = goodix_i2c_hid_power_down;
-
- /* Start out with reset asserted */
- ihid_goodix->reset_gpio =
- devm_gpiod_get_optional(&client->dev, "reset", GPIOD_OUT_HIGH);
- if (IS_ERR(ihid_goodix->reset_gpio))
- return PTR_ERR(ihid_goodix->reset_gpio);
-
- ihid_goodix->vdd = devm_regulator_get(&client->dev, "vdd");
- if (IS_ERR(ihid_goodix->vdd))
- return PTR_ERR(ihid_goodix->vdd);
-
- ihid_goodix->vddio = devm_regulator_get(&client->dev, "mainboard-vddio");
- if (IS_ERR(ihid_goodix->vddio))
- return PTR_ERR(ihid_goodix->vddio);
-
- ihid_goodix->timings = device_get_match_data(&client->dev);
-
- return i2c_hid_core_probe(client, &ihid_goodix->ops, 0x0001, 0);
-}
-
-static const struct goodix_i2c_hid_timing_data goodix_gt7375p_timing_data = {
- .post_power_delay_ms = 10,
- .post_gpio_reset_delay_ms = 180,
-};
-
-static const struct of_device_id goodix_i2c_hid_of_match[] = {
- { .compatible = "goodix,gt7375p", .data = &goodix_gt7375p_timing_data },
- { }
-};
-MODULE_DEVICE_TABLE(of, goodix_i2c_hid_of_match);
-
-static struct i2c_driver goodix_i2c_hid_ts_driver = {
- .driver = {
- .name = "i2c_hid_of_goodix",
- .pm = &i2c_hid_core_pm,
- .probe_type = PROBE_PREFER_ASYNCHRONOUS,
- .of_match_table = of_match_ptr(goodix_i2c_hid_of_match),
- },
- .probe_new = i2c_hid_of_goodix_probe,
- .remove = i2c_hid_core_remove,
- .shutdown = i2c_hid_core_shutdown,
-};
-module_i2c_driver(goodix_i2c_hid_ts_driver);
-
-MODULE_AUTHOR("Douglas Anderson <dianders@chromium.org>");
-MODULE_DESCRIPTION("Goodix i2c-hid touchscreen driver");
-MODULE_LICENSE("GPL v2");
@@ -180,9 +180,23 @@ static const struct i2c_hid_of_chip_data elan_ekth6915_chip_data = {
.hid_descriptor_address = 0x0001,
};
+static const char * const goodix_gt7375p_supply_names[] = {
+ "vdd",
+ "mainboard-vddio",
+};
+
+static const struct i2c_hid_of_chip_data goodix_gt7375p_chip_data = {
+ .supply_names = goodix_gt7375p_supply_names,
+ .num_supplies = ARRAY_SIZE(goodix_gt7375p_supply_names),
+ .post_power_delay_ms = 10,
+ .post_reset_delay_ms = 180,
+ .hid_descriptor_address = 0x0001,
+};
+
static const struct of_device_id i2c_hid_of_match[] = {
{ .compatible = "hid-over-i2c" },
{ .compatible = "elan,ekth6915", .data = &elan_ekth6915_chip_data },
+ { .compatible = "goodix,gt7375p", .data = &goodix_gt7375p_chip_data },
{},
};
MODULE_DEVICE_TABLE(of, i2c_hid_of_match);
The generic i2c-hid-of driver nowsupports a reset GPIO and allows setting the supply-names, hid_descriptor_address and delays through match_data / chip_data. This means that i2c-hid-of can be easily made to support the goodix,gt7375p compatible directly and the i2c-hid-of-goodix driver can be dropped. Signed-off-by: Hans de Goede <hdegoede@redhat.com> --- drivers/hid/i2c-hid/Kconfig | 15 --- drivers/hid/i2c-hid/Makefile | 1 - drivers/hid/i2c-hid/i2c-hid-of-goodix.c | 125 ------------------------ drivers/hid/i2c-hid/i2c-hid-of.c | 14 +++ 4 files changed, 14 insertions(+), 141 deletions(-) delete mode 100644 drivers/hid/i2c-hid/i2c-hid-of-goodix.c