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>

diff --git a/drivers/iio/adc/xilinx-xadc-events.c b/drivers/iio/adc/xilinx-xadc-events.c
index 6d5c2a6f4e6ea7382a461bef2d33476468e183fb..dc067030825303bbfe899e765035af4b2f29cfa5 100644 (file)
--- a/drivers/iio/adc/xilinx-xadc-events.c
+++ b/drivers/iio/adc/xilinx-xadc-events.c
@@ -68,7 +68,7 @@ void xadc_handle_events(struct iio_dev *indio_dev, unsigned long events)
                xadc_handle_event(indio_dev, i);
 }
 
-static unsigned xadc_get_threshold_offset(const struct iio_chan_spec *chan,
+static unsigned int xadc_get_threshold_offset(const struct iio_chan_spec *chan,
        enum iio_event_direction dir)
 {
        unsigned int offset;
@@ -90,26 +90,24 @@ static unsigned xadc_get_threshold_offset(const struct iio_chan_spec *chan,
 
 static unsigned int xadc_get_alarm_mask(const struct iio_chan_spec *chan)
 {
-       if (chan->type == IIO_TEMP) {
+       if (chan->type == IIO_TEMP)
                return XADC_ALARM_OT_MASK;
-       } else {
-               switch (chan->channel) {
-               case 0:
-                       return XADC_ALARM_VCCINT_MASK;
-               case 1:
-                       return XADC_ALARM_VCCAUX_MASK;
-               case 2:
-                       return XADC_ALARM_VCCBRAM_MASK;
-               case 3:
-                       return XADC_ALARM_VCCPINT_MASK;
-               case 4:
-                       return XADC_ALARM_VCCPAUX_MASK;
-               case 5:
-                       return XADC_ALARM_VCCODDR_MASK;
-               default:
-                       /* We will never get here */
-                       return 0;
-               }
+       switch (chan->channel) {
+       case 0:
+               return XADC_ALARM_VCCINT_MASK;
+       case 1:
+               return XADC_ALARM_VCCAUX_MASK;
+       case 2:
+               return XADC_ALARM_VCCBRAM_MASK;
+       case 3:
+               return XADC_ALARM_VCCPINT_MASK;
+       case 4:
+               return XADC_ALARM_VCCPAUX_MASK;
+       case 5:
+               return XADC_ALARM_VCCODDR_MASK;
+       default:
+               /* We will never get here */
+               return 0;
        }
 }
 

diff --git a/drivers/iio/adc/xilinx-xadc.h b/drivers/iio/adc/xilinx-xadc.h
index f6f08196564709f1adc18cf0c683811e85fc2a30..62edbdae1244cd4704e3695aa5d34f4b0967f473 100644 (file)
--- 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;
 };