diff options
author | Lee Jones <lee.jones@linaro.org> | 2014-07-25 10:31:02 -0400 |
---|---|---|
committer | Lee Jones <lee.jones@linaro.org> | 2014-07-25 10:31:02 -0400 |
commit | 0309528aae8ac0a4d7b32428f6a7debc84becc9c (patch) | |
tree | cbd248342ed5a8f348ee097af931403fdfd1faa8 | |
parent | c06f308aa3318c60d5d6dd41a6244ce2177a7f45 (diff) |
mfd: ezx-pcap: Repair coding style errors picked up with checkpatch
This is part of an effort to clean-up the MFD subsystem.
WARNING: sizeof t should be sizeof(t)
+ memset(&t, 0, sizeof t);
WARNING: void function return statements are not generally useful
+ return;
+}
total: 0 errors, 2 warnings, 542 lines checked
Signed-off-by: Lee Jones <lee.jones@linaro.org>
-rw-r--r-- | drivers/mfd/ezx-pcap.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/mfd/ezx-pcap.c b/drivers/mfd/ezx-pcap.c index 2ed774e7d342..5991faddd3c6 100644 --- a/drivers/mfd/ezx-pcap.c +++ b/drivers/mfd/ezx-pcap.c | |||
@@ -62,7 +62,7 @@ static int ezx_pcap_putget(struct pcap_chip *pcap, u32 *data) | |||
62 | struct spi_message m; | 62 | struct spi_message m; |
63 | int status; | 63 | int status; |
64 | 64 | ||
65 | memset(&t, 0, sizeof t); | 65 | memset(&t, 0, sizeof(t)); |
66 | spi_message_init(&m); | 66 | spi_message_init(&m); |
67 | t.len = sizeof(u32); | 67 | t.len = sizeof(u32); |
68 | spi_message_add_tail(&t, &m); | 68 | spi_message_add_tail(&t, &m); |
@@ -211,7 +211,6 @@ static void pcap_irq_handler(unsigned int irq, struct irq_desc *desc) | |||
211 | 211 | ||
212 | desc->irq_data.chip->irq_ack(&desc->irq_data); | 212 | desc->irq_data.chip->irq_ack(&desc->irq_data); |
213 | queue_work(pcap->workqueue, &pcap->isr_work); | 213 | queue_work(pcap->workqueue, &pcap->isr_work); |
214 | return; | ||
215 | } | 214 | } |
216 | 215 | ||
217 | /* ADC */ | 216 | /* ADC */ |