diff options
| author | Fugang Duan <b38611@freescale.com> | 2016-01-11 14:40:32 +0800 |
|---|---|---|
| committer | Leonard Crestez <leonard.crestez@nxp.com> | 2018-08-24 12:41:33 +0300 |
| commit | 98ff21f1fecabfa63c4ba18a5d54f110c9d62fba (patch) | |
| tree | b3422149a4abdfbc4ce76c04e30bfea6b0ecef77 /drivers/input/touchscreen | |
| parent | 2ce81cb1ddc8d09b67b05d8dfe83e902578d4241 (diff) | |
MLK-12176 Input: ads7846: add return value check after calling .of_property_read_u32()
Add return value check after calling .of_property_read_u32().
The issue was reported by coverity.
Signed-off-by: Fugang Duan <B38611@freescale.com>
Diffstat (limited to 'drivers/input/touchscreen')
| -rw-r--r-- | drivers/input/touchscreen/ads7846.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/drivers/input/touchscreen/ads7846.c b/drivers/input/touchscreen/ads7846.c index 1ce3ecbe37f8..fc21a7459e3e 100644 --- a/drivers/input/touchscreen/ads7846.c +++ b/drivers/input/touchscreen/ads7846.c @@ -1188,6 +1188,7 @@ static const struct ads7846_platform_data *ads7846_probe_dt(struct device *dev) struct ads7846_platform_data *pdata; struct device_node *node = dev->of_node; const struct of_device_id *match; + int ret; if (!node) { dev_err(dev, "Device does not have associated DT data\n"); @@ -1233,8 +1234,11 @@ static const struct ads7846_platform_data *ads7846_probe_dt(struct device *dev) of_property_read_u16(node, "ti,debounce-tol", &pdata->debounce_tol); of_property_read_u16(node, "ti,debounce-rep", &pdata->debounce_rep); - of_property_read_u32(node, "ti,pendown-gpio-debounce", + ret = of_property_read_u32(node, "ti,pendown-gpio-debounce", &pdata->gpio_pendown_debounce); + if (!ret) + dev_info(dev, "Set pendown gpio debounce time to %d microseconds\n", + pdata->gpio_pendown_debounce); pdata->wakeup = of_property_read_bool(node, "wakeup-source") || of_property_read_bool(node, "linux,wakeup"); |