Commit 546615f9 authored by H Hartley Sweeten's avatar H Hartley Sweeten Committed by Greg Kroah-Hartman

staging: comedi: ni_mio_common: use 'unsigned int' instead of kernel types

Generally comedi drivers use 'unsigned int' types instead of the kernel type
'u32' for unsigned 32-bit values.

For aesthetics, change all the 'u32' and 'uin32_t' types and fix the
checkpatch.pl issues about:
CHECK: Prefer kernel type 'u32' over 'uint32_t'

The various i/o helpers use for reading and writing the 32/16/8-bit registers
all start with an 'unsigned int' value for writing and finally return an
'unsigned int' value. For aesthetics, change all the 'uint16_t' and 'uint8_t'
types for insigned int and fix the checkpatch.pl issues about:
CHECK: Prefer kernel type 'u16' over 'uint16_t'
Signed-off-by: default avatarH Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: default avatarIan Abbott <abbotti@mev.co.uk>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent fe20a34a
...@@ -243,7 +243,7 @@ enum timebase_nanoseconds { ...@@ -243,7 +243,7 @@ enum timebase_nanoseconds {
static const int num_adc_stages_611x = 3; static const int num_adc_stages_611x = 3;
static void ni_writel(struct comedi_device *dev, uint32_t data, int reg) static void ni_writel(struct comedi_device *dev, unsigned int data, int reg)
{ {
if (dev->mmio) if (dev->mmio)
writel(data, dev->mmio + reg); writel(data, dev->mmio + reg);
...@@ -251,7 +251,7 @@ static void ni_writel(struct comedi_device *dev, uint32_t data, int reg) ...@@ -251,7 +251,7 @@ static void ni_writel(struct comedi_device *dev, uint32_t data, int reg)
outl(data, dev->iobase + reg); outl(data, dev->iobase + reg);
} }
static void ni_writew(struct comedi_device *dev, uint16_t data, int reg) static void ni_writew(struct comedi_device *dev, unsigned int data, int reg)
{ {
if (dev->mmio) if (dev->mmio)
writew(data, dev->mmio + reg); writew(data, dev->mmio + reg);
...@@ -259,7 +259,7 @@ static void ni_writew(struct comedi_device *dev, uint16_t data, int reg) ...@@ -259,7 +259,7 @@ static void ni_writew(struct comedi_device *dev, uint16_t data, int reg)
outw(data, dev->iobase + reg); outw(data, dev->iobase + reg);
} }
static void ni_writeb(struct comedi_device *dev, uint8_t data, int reg) static void ni_writeb(struct comedi_device *dev, unsigned int data, int reg)
{ {
if (dev->mmio) if (dev->mmio)
writeb(data, dev->mmio + reg); writeb(data, dev->mmio + reg);
...@@ -267,7 +267,7 @@ static void ni_writeb(struct comedi_device *dev, uint8_t data, int reg) ...@@ -267,7 +267,7 @@ static void ni_writeb(struct comedi_device *dev, uint8_t data, int reg)
outb(data, dev->iobase + reg); outb(data, dev->iobase + reg);
} }
static uint32_t ni_readl(struct comedi_device *dev, int reg) static unsigned int ni_readl(struct comedi_device *dev, int reg)
{ {
if (dev->mmio) if (dev->mmio)
return readl(dev->mmio + reg); return readl(dev->mmio + reg);
...@@ -275,7 +275,7 @@ static uint32_t ni_readl(struct comedi_device *dev, int reg) ...@@ -275,7 +275,7 @@ static uint32_t ni_readl(struct comedi_device *dev, int reg)
return inl(dev->iobase + reg); return inl(dev->iobase + reg);
} }
static uint16_t ni_readw(struct comedi_device *dev, int reg) static unsigned int ni_readw(struct comedi_device *dev, int reg)
{ {
if (dev->mmio) if (dev->mmio)
return readw(dev->mmio + reg); return readw(dev->mmio + reg);
...@@ -283,7 +283,7 @@ static uint16_t ni_readw(struct comedi_device *dev, int reg) ...@@ -283,7 +283,7 @@ static uint16_t ni_readw(struct comedi_device *dev, int reg)
return inw(dev->iobase + reg); return inw(dev->iobase + reg);
} }
static uint8_t ni_readb(struct comedi_device *dev, int reg) static unsigned int ni_readb(struct comedi_device *dev, int reg)
{ {
if (dev->mmio) if (dev->mmio)
return readb(dev->mmio + reg); return readb(dev->mmio + reg);
...@@ -458,7 +458,8 @@ static unsigned int m_series_stc_read(struct comedi_device *dev, ...@@ -458,7 +458,8 @@ static unsigned int m_series_stc_read(struct comedi_device *dev,
} }
} }
static void ni_stc_writew(struct comedi_device *dev, uint16_t data, int reg) static void ni_stc_writew(struct comedi_device *dev,
unsigned int data, int reg)
{ {
struct ni_private *devpriv = dev->private; struct ni_private *devpriv = dev->private;
unsigned long flags; unsigned long flags;
...@@ -477,7 +478,8 @@ static void ni_stc_writew(struct comedi_device *dev, uint16_t data, int reg) ...@@ -477,7 +478,8 @@ static void ni_stc_writew(struct comedi_device *dev, uint16_t data, int reg)
} }
} }
static void ni_stc_writel(struct comedi_device *dev, uint32_t data, int reg) static void ni_stc_writel(struct comedi_device *dev,
unsigned int data, int reg)
{ {
struct ni_private *devpriv = dev->private; struct ni_private *devpriv = dev->private;
...@@ -489,11 +491,11 @@ static void ni_stc_writel(struct comedi_device *dev, uint32_t data, int reg) ...@@ -489,11 +491,11 @@ static void ni_stc_writel(struct comedi_device *dev, uint32_t data, int reg)
} }
} }
static uint16_t ni_stc_readw(struct comedi_device *dev, int reg) static unsigned int ni_stc_readw(struct comedi_device *dev, int reg)
{ {
struct ni_private *devpriv = dev->private; struct ni_private *devpriv = dev->private;
unsigned long flags; unsigned long flags;
uint16_t val; unsigned int val;
if (devpriv->is_m_series) { if (devpriv->is_m_series) {
val = m_series_stc_read(dev, reg); val = m_series_stc_read(dev, reg);
...@@ -510,10 +512,10 @@ static uint16_t ni_stc_readw(struct comedi_device *dev, int reg) ...@@ -510,10 +512,10 @@ static uint16_t ni_stc_readw(struct comedi_device *dev, int reg)
return val; return val;
} }
static uint32_t ni_stc_readl(struct comedi_device *dev, int reg) static unsigned int ni_stc_readl(struct comedi_device *dev, int reg)
{ {
struct ni_private *devpriv = dev->private; struct ni_private *devpriv = dev->private;
uint32_t val; unsigned int val;
if (devpriv->is_m_series) { if (devpriv->is_m_series) {
val = m_series_stc_read(dev, reg); val = m_series_stc_read(dev, reg);
...@@ -779,7 +781,7 @@ static void ni_e_series_enable_second_irq(struct comedi_device *dev, ...@@ -779,7 +781,7 @@ static void ni_e_series_enable_second_irq(struct comedi_device *dev,
unsigned int gpct_index, short enable) unsigned int gpct_index, short enable)
{ {
struct ni_private *devpriv = dev->private; struct ni_private *devpriv = dev->private;
uint16_t val = 0; unsigned int val = 0;
int reg; int reg;
if (devpriv->is_m_series || gpct_index > 1) if (devpriv->is_m_series || gpct_index > 1)
...@@ -839,8 +841,8 @@ static void ni_clear_ai_fifo(struct comedi_device *dev) ...@@ -839,8 +841,8 @@ static void ni_clear_ai_fifo(struct comedi_device *dev)
} }
} }
static inline void ni_ao_win_outw(struct comedi_device *dev, uint16_t data, static inline void ni_ao_win_outw(struct comedi_device *dev,
int addr) unsigned int data, int addr)
{ {
struct ni_private *devpriv = dev->private; struct ni_private *devpriv = dev->private;
unsigned long flags; unsigned long flags;
...@@ -851,8 +853,8 @@ static inline void ni_ao_win_outw(struct comedi_device *dev, uint16_t data, ...@@ -851,8 +853,8 @@ static inline void ni_ao_win_outw(struct comedi_device *dev, uint16_t data,
spin_unlock_irqrestore(&devpriv->window_lock, flags); spin_unlock_irqrestore(&devpriv->window_lock, flags);
} }
static inline void ni_ao_win_outl(struct comedi_device *dev, uint32_t data, static inline void ni_ao_win_outl(struct comedi_device *dev,
int addr) unsigned int data, int addr)
{ {
struct ni_private *devpriv = dev->private; struct ni_private *devpriv = dev->private;
unsigned long flags; unsigned long flags;
...@@ -989,7 +991,7 @@ static void ni_ao_fifo_load(struct comedi_device *dev, ...@@ -989,7 +991,7 @@ static void ni_ao_fifo_load(struct comedi_device *dev,
struct ni_private *devpriv = dev->private; struct ni_private *devpriv = dev->private;
int i; int i;
unsigned short d; unsigned short d;
u32 packed_data; unsigned int packed_data;
for (i = 0; i < n; i++) { for (i = 0; i < n; i++) {
comedi_buf_read_samples(s, &d, 1); comedi_buf_read_samples(s, &d, 1);
...@@ -1079,7 +1081,7 @@ static void ni_ai_fifo_read(struct comedi_device *dev, ...@@ -1079,7 +1081,7 @@ static void ni_ai_fifo_read(struct comedi_device *dev,
{ {
struct ni_private *devpriv = dev->private; struct ni_private *devpriv = dev->private;
struct comedi_async *async = s->async; struct comedi_async *async = s->async;
u32 dl; unsigned int dl;
unsigned short data; unsigned short data;
int i; int i;
...@@ -1150,7 +1152,7 @@ static void ni_handle_fifo_dregs(struct comedi_device *dev) ...@@ -1150,7 +1152,7 @@ static void ni_handle_fifo_dregs(struct comedi_device *dev)
{ {
struct ni_private *devpriv = dev->private; struct ni_private *devpriv = dev->private;
struct comedi_subdevice *s = dev->read_subdev; struct comedi_subdevice *s = dev->read_subdev;
u32 dl; unsigned int dl;
unsigned short data; unsigned short data;
unsigned short fifo_empty; unsigned short fifo_empty;
int i; int i;
...@@ -1211,7 +1213,7 @@ static void get_last_sample_611x(struct comedi_device *dev) ...@@ -1211,7 +1213,7 @@ static void get_last_sample_611x(struct comedi_device *dev)
struct ni_private *devpriv = dev->private; struct ni_private *devpriv = dev->private;
struct comedi_subdevice *s = dev->read_subdev; struct comedi_subdevice *s = dev->read_subdev;
unsigned short data; unsigned short data;
u32 dl; unsigned int dl;
if (!devpriv->is_611x) if (!devpriv->is_611x)
return; return;
...@@ -1229,7 +1231,7 @@ static void get_last_sample_6143(struct comedi_device *dev) ...@@ -1229,7 +1231,7 @@ static void get_last_sample_6143(struct comedi_device *dev)
struct ni_private *devpriv = dev->private; struct ni_private *devpriv = dev->private;
struct comedi_subdevice *s = dev->read_subdev; struct comedi_subdevice *s = dev->read_subdev;
unsigned short data; unsigned short data;
u32 dl; unsigned int dl;
if (!devpriv->is_6143) if (!devpriv->is_6143)
return; return;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment