staging: comedi: ni_stc.h: tidy up Serial_Command register and bits

Rename the CamelCase. Use the BIT() macro to define the bits.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
H Hartley Sweeten 2015-05-01 14:59:53 -07:00 committed by Greg Kroah-Hartman
parent 906170b882
commit 2ed183fff4
2 changed files with 26 additions and 16 deletions

View file

@ -4129,6 +4129,7 @@ static void ni_write_caldac(struct comedi_device *dev, int addr, int val)
const struct ni_board_struct *board = dev->board_ptr; const struct ni_board_struct *board = dev->board_ptr;
struct ni_private *devpriv = dev->private; struct ni_private *devpriv = dev->private;
unsigned int loadbit = 0, bits = 0, bit, bitstring = 0; unsigned int loadbit = 0, bits = 0, bit, bitstring = 0;
unsigned int cmd;
int i; int i;
int type; int type;
@ -4142,7 +4143,7 @@ static void ni_write_caldac(struct comedi_device *dev, int addr, int val)
break; break;
if (addr < caldacs[type].n_chans) { if (addr < caldacs[type].n_chans) {
bits = caldacs[type].packbits(addr, val, &bitstring); bits = caldacs[type].packbits(addr, val, &bitstring);
loadbit = SerDacLd(i); loadbit = NI_E_SERIAL_CMD_DAC_LD(i);
break; break;
} }
addr -= caldacs[type].n_chans; addr -= caldacs[type].n_chans;
@ -4153,15 +4154,15 @@ static void ni_write_caldac(struct comedi_device *dev, int addr, int val)
return; return;
for (bit = 1 << (bits - 1); bit; bit >>= 1) { for (bit = 1 << (bits - 1); bit; bit >>= 1) {
ni_writeb(dev, ((bit & bitstring) ? 0x02 : 0), Serial_Command); cmd = (bit & bitstring) ? NI_E_SERIAL_CMD_SDATA : 0;
ni_writeb(dev, cmd, NI_E_SERIAL_CMD_REG);
udelay(1); udelay(1);
ni_writeb(dev, 1 | ((bit & bitstring) ? 0x02 : 0), ni_writeb(dev, NI_E_SERIAL_CMD_SCLK | cmd, NI_E_SERIAL_CMD_REG);
Serial_Command);
udelay(1); udelay(1);
} }
ni_writeb(dev, loadbit, Serial_Command); ni_writeb(dev, loadbit, NI_E_SERIAL_CMD_REG);
udelay(1); udelay(1);
ni_writeb(dev, 0, Serial_Command); ni_writeb(dev, 0, NI_E_SERIAL_CMD_REG);
} }
static int ni_calib_insn_write(struct comedi_device *dev, static int ni_calib_insn_write(struct comedi_device *dev,
@ -4243,25 +4244,30 @@ static void caldac_setup(struct comedi_device *dev, struct comedi_subdevice *s)
static int ni_read_eeprom(struct comedi_device *dev, int addr) static int ni_read_eeprom(struct comedi_device *dev, int addr)
{ {
unsigned int cmd = NI_E_SERIAL_CMD_EEPROM_CS;
int bit; int bit;
int bitstring; int bitstring;
bitstring = 0x0300 | ((addr & 0x100) << 3) | (addr & 0xff); bitstring = 0x0300 | ((addr & 0x100) << 3) | (addr & 0xff);
ni_writeb(dev, 0x04, Serial_Command); ni_writeb(dev, cmd, NI_E_SERIAL_CMD_REG);
for (bit = 0x8000; bit; bit >>= 1) { for (bit = 0x8000; bit; bit >>= 1) {
ni_writeb(dev, 0x04 | ((bit & bitstring) ? 0x02 : 0), if (bit & bitstring)
Serial_Command); cmd |= NI_E_SERIAL_CMD_SDATA;
ni_writeb(dev, 0x05 | ((bit & bitstring) ? 0x02 : 0), else
Serial_Command); cmd &= ~NI_E_SERIAL_CMD_SDATA;
ni_writeb(dev, cmd, NI_E_SERIAL_CMD_REG);
ni_writeb(dev, NI_E_SERIAL_CMD_SCLK | cmd, NI_E_SERIAL_CMD_REG);
} }
cmd = NI_E_SERIAL_CMD_EEPROM_CS;
bitstring = 0; bitstring = 0;
for (bit = 0x80; bit; bit >>= 1) { for (bit = 0x80; bit; bit >>= 1) {
ni_writeb(dev, 0x04, Serial_Command); ni_writeb(dev, cmd, NI_E_SERIAL_CMD_REG);
ni_writeb(dev, 0x05, Serial_Command); ni_writeb(dev, NI_E_SERIAL_CMD_SCLK | cmd, NI_E_SERIAL_CMD_REG);
if (ni_readb(dev, NI_E_STATUS_REG) & NI_E_STATUS_PROMOUT) if (ni_readb(dev, NI_E_STATUS_REG) & NI_E_STATUS_PROMOUT)
bitstring |= bit; bitstring |= bit;
} }
ni_writeb(dev, 0x00, Serial_Command); ni_writeb(dev, 0, NI_E_SERIAL_CMD_REG);
return bitstring; return bitstring;
} }

View file

@ -571,7 +571,12 @@
#define NI_E_STATUS_AI_FIFO_LOWER_NE BIT(3) #define NI_E_STATUS_AI_FIFO_LOWER_NE BIT(3)
#define NI_E_STATUS_PROMOUT BIT(0) #define NI_E_STATUS_PROMOUT BIT(0)
#define Serial_Command 0x0d #define NI_E_SERIAL_CMD_REG 0x0d /* w8 */
#define NI_E_SERIAL_CMD_DAC_LD(x) BIT(3 + (x))
#define NI_E_SERIAL_CMD_EEPROM_CS BIT(2)
#define NI_E_SERIAL_CMD_SDATA BIT(1)
#define NI_E_SERIAL_CMD_SCLK BIT(0)
#define Misc_Command 0x0f #define Misc_Command 0x0f
#define Port_A 0x19 #define Port_A 0x19
#define Port_B 0x1b #define Port_B 0x1b
@ -765,7 +770,6 @@ enum cs5529_status_bits {
CSS_OSC_DETECT = 0x2, /* indicates adc error */ CSS_OSC_DETECT = 0x2, /* indicates adc error */
CSS_OVERRANGE = 0x4, CSS_OVERRANGE = 0x4,
}; };
#define SerDacLd(x) (0x08<<(x))
/* /*
This is stuff unique to the NI E series drivers, This is stuff unique to the NI E series drivers,