Minki/linux
1
0
Fork 1
mirror of https://github.com/torvalds/linux.git synced 2024-11-10 22:21:40 +00:00
Code Issues Packages Projects Releases Wiki Activity

Input: iqs7211 - point to match data directly

Browse Source 
Point the OF match table directly to the struct that describes the
device as opposed to an intermediate enum; doing so simplifies the
code and avoids a clang warning.

As part of this change, the I2C device ID table is removed, as the
device cannot probe without an OF node due to the unique nature of
the hardware's interrupt pin.

Fixes: f2ba47e65f ("Input: add support for Azoteq IQS7210A/7211A/E")
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202307131717.LtwApG0z-lkp@intel.com/
Signed-off-by: Jeff LaBundy <jeff@labundy.com>
Link: https://lore.kernel.org/r/ZLA+cuciIeVcCvm6@nixie71
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
This commit is contained in:
Jeff LaBundy 2023-07-13 13:12:02 -05:00 committed by Dmitry Torokhov
parent f2ba47e65f
commit 677232f879
1 changed files with 7 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
drivers/input/touchscreen/iqs7211.c
View File

@ -2420,33 +2420,24 @@ ATTRIBUTE_GROUPS(iqs7211);
static const struct of_device_id iqs7211_of_match[] = {
{
.compatible = "azoteq,iqs7210a",
.data = (void *)IQS7210A,
.data = &iqs7211_devs[IQS7210A],
},
{
.compatible = "azoteq,iqs7211a",
.data = (void *)IQS7211A,
.data = &iqs7211_devs[IQS7211A],
},
{
.compatible = "azoteq,iqs7211e",
.data = (void *)IQS7211E,
.data = &iqs7211_devs[IQS7211E],
},
{ }
};
MODULE_DEVICE_TABLE(of, iqs7211_of_match);
static const struct i2c_device_id iqs7211_id[] = {
{ "iqs7210a", IQS7210A },
{ "iqs7211a", IQS7211A },
{ "iqs7211e", IQS7211E },
{ }
};
MODULE_DEVICE_TABLE(i2c, iqs7211_id);
static int iqs7211_probe(struct i2c_client *client)
{
struct iqs7211_private *iqs7211;
enum iqs7211_reg_grp_id reg_grp;
enum iqs7211_dev_id dev_id;
unsigned long irq_flags;
bool shared_irq;
int error, irq;
@ -2460,13 +2451,11 @@ static int iqs7211_probe(struct i2c_client *client)
INIT_LIST_HEAD(&iqs7211->reg_field_head);
if (client->dev.of_node)
dev_id = (enum iqs7211_dev_id)of_device_get_match_data(&client->dev);
else
dev_id = i2c_match_id(iqs7211_id, client)->driver_data;
iqs7211->dev_desc = device_get_match_data(&client->dev);
if (!iqs7211->dev_desc)
return -ENODEV;
shared_irq = iqs7211_devs[dev_id].num_ctx == IQS7211_MAX_CTX;
iqs7211->dev_desc = &iqs7211_devs[dev_id];
shared_irq = iqs7211->dev_desc->num_ctx == IQS7211_MAX_CTX;
/*
* The RDY pin behaves as an interrupt, but must also be polled ahead
@ -2554,7 +2543,6 @@ static int iqs7211_probe(struct i2c_client *client)
static struct i2c_driver iqs7211_i2c_driver = {
.probe = iqs7211_probe,
.id_table = iqs7211_id,
.driver = {
.name = "iqs7211",
.of_match_table = iqs7211_of_match,