diff mbox

[2/2] Input: goodix - add more entries in i2c_device_id

Message ID 20171012150443.27542-2-m.niestroj@grinn-global.com (mailing list archive)
State New, archived
Headers show

Commit Message

Marcin Niestroj Oct. 12, 2017, 3:04 p.m. UTC
Add corresponding entry in i2c_device_id table for each of_device_id
entry. This makes it possible to autoload module for goodix
touchscreen device-tree entry, based on modalias (which is generated
by stripping manufacturer prefix from compatible string).

Signed-off-by: Marcin Niestroj <m.niestroj@grinn-global.com>
---
 drivers/input/touchscreen/goodix.c | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

Comments

Rob Herring Oct. 12, 2017, 6:08 p.m. UTC | #1
On Thu, Oct 12, 2017 at 10:04 AM, Marcin Niestroj
<m.niestroj@grinn-global.com> wrote:
> Add corresponding entry in i2c_device_id table for each of_device_id
> entry. This makes it possible to autoload module for goodix
> touchscreen device-tree entry, based on modalias (which is generated
> by stripping manufacturer prefix from compatible string).

Stripping the vendor prefix is behavior we don't want to expand. Add
or extend the OF match table instead.

Rob
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/input/touchscreen/goodix.c b/drivers/input/touchscreen/goodix.c
index 9d50d9688975..4b702b363677 100644
--- a/drivers/input/touchscreen/goodix.c
+++ b/drivers/input/touchscreen/goodix.c
@@ -902,7 +902,15 @@  static int __maybe_unused goodix_resume(struct device *dev)
 static SIMPLE_DEV_PM_OPS(goodix_pm_ops, goodix_suspend, goodix_resume);
 
 static const struct i2c_device_id goodix_ts_id[] = {
-	{ "GDIX1001:00", 0 },
+	{ "GDIX1001:00" },
+	{ "gt1151" },
+	{ "gt911" },
+	{ "gt9110" },
+	{ "gt912" },
+	{ "gt927" },
+	{ "gt9271" },
+	{ "gt928" },
+	{ "gt967" },
 	{ }
 };
 MODULE_DEVICE_TABLE(i2c, goodix_ts_id);