Messages in this thread |  | | Subject | Re: [PATCH v9 52/55] input: touchscreen: atmel_mxt_ts: Added sysfs entry for touchscreen status | From | Dmitry Osipenko <> | Date | Fri, 27 Mar 2020 16:43:53 +0300 |
| |
27.03.2020 16:37, Dmitry Osipenko пишет: > 27.03.2020 15:49, Wang, Jiada пишет: >>>> +static void mxt_watchdog_work(struct work_struct *work) >>>> +{ >>>> + struct mxt_data *data = >>>> + container_of(work, struct mxt_data, watchdog_work); >>>> + u16 info_buf; >>>> + int ret = 0; >>>> + u8 size = 2;
There is no need to initialize the ret variable.
>>>> + if (data->suspended || data->in_bootloader) >>>> + return; >>>> + >>>> + ret = __mxt_read_reg(data->client, 0, size, &info_buf);
The "size" argument could be replaced with "sizeof(info_buf)".
|  |