* re: Input: atmel_mxt_ts - initialise IRQ before probing
@ 2014-07-25 12:20 Dan Carpenter
0 siblings, 0 replies; only message in thread
From: Dan Carpenter @ 2014-07-25 12:20 UTC (permalink / raw)
Cc: Nick Dyer, linux-input
Hello Nick Dyer,
The patch dd24dcf566d0: "Input: atmel_mxt_ts - initialise IRQ before
probing" from Jul 23, 2014, leads to the following static checker
warning:
drivers/input/touchscreen/atmel_mxt_ts.c:1701 mxt_initialize()
warn: we tested 'error' before and it was 'false'
drivers/input/touchscreen/atmel_mxt_ts.c
1693 /* Get object table information */
1694 error = mxt_get_object_table(data);
1695 if (error) {
1696 dev_err(&client->dev, "Error %d reading object table\n", error);
1697 return error;
1698 }
1699
1700 mxt_acquire_irq(data);
Probably error = mxt_acquire_irq(data) was intended here?
1701 if (error)
1702 goto err_free_object_table;
1703
1704 request_firmware_nowait(THIS_MODULE, true, MXT_CFG_NAME,
1705 &data->client->dev, GFP_KERNEL, data,
1706 mxt_config_cb);
1707
1708 return 0;
1709
1710 err_free_object_table:
1711 mxt_free_object_table(data);
1712 return error;
1713 }
regards,
dan carpenter
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2014-07-25 12:20 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-07-25 12:20 Input: atmel_mxt_ts - initialise IRQ before probing Dan Carpenter
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).