diff options
| author |   Sai Krishna Potthuri <lakshmi.sai.krishna.potthuri@xilinx.com> | 2017-08-10 08:30:20 +0200 |
|---|---|---|
| committer |   Jonathan Cameron <Jonathan.Cameron@huawei.com> | 2017-08-12 13:10:30 +0100 |
| commit | ce0d376140838b080b8625892203704d6f10784b (patch) | |
| tree | 16d0b5b96721392767999de4e4188e71a316c5e2 /drivers/iio/adc/xilinx-xadc.h | |
| parent | staging: iio: adc: fix error return code in ad7606_par_probe() (diff) | |
| download | wireguard-linux-ce0d376140838b080b8625892203704d6f10784b.tar.xz wireguard-linux-ce0d376140838b080b8625892203704d6f10784b.zip | |
iio: adc: xadc: Fix coding style violations
This patch fix following checkpatch warnings in xadc driver
1. prefer 'unsigned int' to bare use of 'unsigned'.
2. else is not generally useful after a break or return.
3. fill all function definition arguments with identifier name
Signed-off-by: Sai Krishna Potthuri <lakshmis@xilinx.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Diffstat (limited to 'drivers/iio/adc/xilinx-xadc.h')
| -rw-r--r-- | drivers/iio/adc/xilinx-xadc.h | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/iio/adc/xilinx-xadc.h b/drivers/iio/adc/xilinx-xadc.h index f6f081965647..62edbdae1244 100644 --- a/drivers/iio/adc/xilinx-xadc.h +++ b/drivers/iio/adc/xilinx-xadc.h @@ -71,13 +71,13 @@ struct xadc { }; struct xadc_ops { - int (*read)(struct xadc *, unsigned int, uint16_t *); - int (*write)(struct xadc *, unsigned int, uint16_t); + int (*read)(struct xadc *xadc, unsigned int reg, uint16_t *val); + int (*write)(struct xadc *xadc, unsigned int reg, uint16_t val); int (*setup)(struct platform_device *pdev, struct iio_dev *indio_dev, int irq); - void (*update_alarm)(struct xadc *, unsigned int); - unsigned long (*get_dclk_rate)(struct xadc *); - irqreturn_t (*interrupt_handler)(int, void *); + void (*update_alarm)(struct xadc *xadc, unsigned int alarm); + unsigned long (*get_dclk_rate)(struct xadc *xadc); + irqreturn_t (*interrupt_handler)(int irq, void *devid); unsigned int flags; }; |