Staging: comedi: icp_multi: fix style issue printk
add KERN_ facility to printk (mostly KERN_DEBUG, some KERN_WARNING) Signed-off-by: Dirk Hohndel <hohndel@infradead.org> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This commit is contained in:
parent
a622afcb7e
commit
ca5edf2f4a
1 changed files with 55 additions and 38 deletions
|
@ -256,7 +256,7 @@ static int icp_multi_insn_read_ai(struct comedi_device *dev,
|
||||||
int n, timeout;
|
int n, timeout;
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk("icp multi EDBG: BGN: icp_multi_insn_read_ai(...)\n");
|
printk(KERN_DEBUG "icp multi EDBG: BGN: icp_multi_insn_read_ai(...)\n");
|
||||||
#endif
|
#endif
|
||||||
/* Disable A/D conversion ready interrupt */
|
/* Disable A/D conversion ready interrupt */
|
||||||
devpriv->IntEnable &= ~ADC_READY;
|
devpriv->IntEnable &= ~ADC_READY;
|
||||||
|
@ -270,7 +270,7 @@ static int icp_multi_insn_read_ai(struct comedi_device *dev,
|
||||||
setup_channel_list(dev, s, &insn->chanspec, 1);
|
setup_channel_list(dev, s, &insn->chanspec, 1);
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk("icp_multi A ST=%4x IO=%p\n",
|
printk(KERN_DEBUG "icp_multi A ST=%4x IO=%p\n",
|
||||||
readw(devpriv->io_addr + ICP_MULTI_ADC_CSR),
|
readw(devpriv->io_addr + ICP_MULTI_ADC_CSR),
|
||||||
devpriv->io_addr + ICP_MULTI_ADC_CSR);
|
devpriv->io_addr + ICP_MULTI_ADC_CSR);
|
||||||
#endif
|
#endif
|
||||||
|
@ -283,14 +283,14 @@ static int icp_multi_insn_read_ai(struct comedi_device *dev,
|
||||||
devpriv->AdcCmdStatus &= ~ADC_ST;
|
devpriv->AdcCmdStatus &= ~ADC_ST;
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk("icp multi B n=%d ST=%4x\n", n,
|
printk(KERN_DEBUG "icp multi B n=%d ST=%4x\n", n,
|
||||||
readw(devpriv->io_addr + ICP_MULTI_ADC_CSR));
|
readw(devpriv->io_addr + ICP_MULTI_ADC_CSR));
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
udelay(1);
|
udelay(1);
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk("icp multi C n=%d ST=%4x\n", n,
|
printk(KERN_DEBUG "icp multi C n=%d ST=%4x\n", n,
|
||||||
readw(devpriv->io_addr + ICP_MULTI_ADC_CSR));
|
readw(devpriv->io_addr + ICP_MULTI_ADC_CSR));
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
|
@ -303,7 +303,8 @@ static int icp_multi_insn_read_ai(struct comedi_device *dev,
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
if (!(timeout % 10))
|
if (!(timeout % 10))
|
||||||
printk("icp multi D n=%d tm=%d ST=%4x\n", n,
|
printk(KERN_DEBUG
|
||||||
|
"icp multi D n=%d tm=%d ST=%4x\n", n,
|
||||||
timeout,
|
timeout,
|
||||||
readw(devpriv->io_addr +
|
readw(devpriv->io_addr +
|
||||||
ICP_MULTI_ADC_CSR));
|
ICP_MULTI_ADC_CSR));
|
||||||
|
@ -328,9 +329,9 @@ static int icp_multi_insn_read_ai(struct comedi_device *dev,
|
||||||
data[n] = 0;
|
data[n] = 0;
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk
|
printk(KERN_DEBUG
|
||||||
("icp multi EDBG: END: icp_multi_insn_read_ai(...) n=%d\n",
|
"icp multi EDBG: END: icp_multi_insn_read_ai(...) n=%d\n",
|
||||||
n);
|
n);
|
||||||
#endif
|
#endif
|
||||||
return -ETIME;
|
return -ETIME;
|
||||||
|
|
||||||
|
@ -348,7 +349,8 @@ static int icp_multi_insn_read_ai(struct comedi_device *dev,
|
||||||
writew(devpriv->IntStatus, devpriv->io_addr + ICP_MULTI_INT_STAT);
|
writew(devpriv->IntStatus, devpriv->io_addr + ICP_MULTI_INT_STAT);
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk("icp multi EDBG: END: icp_multi_insn_read_ai(...) n=%d\n", n);
|
printk(KERN_DEBUG
|
||||||
|
"icp multi EDBG: END: icp_multi_insn_read_ai(...) n=%d\n", n);
|
||||||
#endif
|
#endif
|
||||||
return n;
|
return n;
|
||||||
}
|
}
|
||||||
|
@ -378,7 +380,8 @@ static int icp_multi_insn_write_ao(struct comedi_device *dev,
|
||||||
int n, chan, range, timeout;
|
int n, chan, range, timeout;
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk("icp multi EDBG: BGN: icp_multi_insn_write_ao(...)\n");
|
printk(KERN_DEBUG
|
||||||
|
"icp multi EDBG: BGN: icp_multi_insn_write_ao(...)\n");
|
||||||
#endif
|
#endif
|
||||||
/* Disable D/A conversion ready interrupt */
|
/* Disable D/A conversion ready interrupt */
|
||||||
devpriv->IntEnable &= ~DAC_READY;
|
devpriv->IntEnable &= ~DAC_READY;
|
||||||
|
@ -414,7 +417,8 @@ static int icp_multi_insn_write_ao(struct comedi_device *dev,
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
if (!(timeout % 10))
|
if (!(timeout % 10))
|
||||||
printk("icp multi A n=%d tm=%d ST=%4x\n", n,
|
printk(KERN_DEBUG
|
||||||
|
"icp multi A n=%d tm=%d ST=%4x\n", n,
|
||||||
timeout,
|
timeout,
|
||||||
readw(devpriv->io_addr +
|
readw(devpriv->io_addr +
|
||||||
ICP_MULTI_DAC_CSR));
|
ICP_MULTI_DAC_CSR));
|
||||||
|
@ -439,8 +443,8 @@ static int icp_multi_insn_write_ao(struct comedi_device *dev,
|
||||||
devpriv->ao_data[chan] = 0;
|
devpriv->ao_data[chan] = 0;
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk
|
printk(KERN_DEBUG
|
||||||
("icp multi EDBG: END: icp_multi_insn_write_ao(...) n=%d\n",
|
"icp multi EDBG: END: icp_multi_insn_write_ao(...) n=%d\n",
|
||||||
n);
|
n);
|
||||||
#endif
|
#endif
|
||||||
return -ETIME;
|
return -ETIME;
|
||||||
|
@ -460,7 +464,8 @@ static int icp_multi_insn_write_ao(struct comedi_device *dev,
|
||||||
}
|
}
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk("icp multi EDBG: END: icp_multi_insn_write_ao(...) n=%d\n", n);
|
printk(KERN_DEBUG
|
||||||
|
"icp multi EDBG: END: icp_multi_insn_write_ao(...) n=%d\n", n);
|
||||||
#endif
|
#endif
|
||||||
return n;
|
return n;
|
||||||
}
|
}
|
||||||
|
@ -549,14 +554,14 @@ static int icp_multi_insn_bits_do(struct comedi_device *dev,
|
||||||
struct comedi_insn *insn, unsigned int *data)
|
struct comedi_insn *insn, unsigned int *data)
|
||||||
{
|
{
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk("icp multi EDBG: BGN: icp_multi_insn_bits_do(...)\n");
|
printk(KERN_DEBUG "icp multi EDBG: BGN: icp_multi_insn_bits_do(...)\n");
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
if (data[0]) {
|
if (data[0]) {
|
||||||
s->state &= ~data[0];
|
s->state &= ~data[0];
|
||||||
s->state |= (data[0] & data[1]);
|
s->state |= (data[0] & data[1]);
|
||||||
|
|
||||||
printk("Digital outputs = %4x \n", s->state);
|
printk(KERN_DEBUG "Digital outputs = %4x \n", s->state);
|
||||||
|
|
||||||
writew(s->state, devpriv->io_addr + ICP_MULTI_DO);
|
writew(s->state, devpriv->io_addr + ICP_MULTI_DO);
|
||||||
}
|
}
|
||||||
|
@ -564,7 +569,7 @@ static int icp_multi_insn_bits_do(struct comedi_device *dev,
|
||||||
data[1] = readw(devpriv->io_addr + ICP_MULTI_DI);
|
data[1] = readw(devpriv->io_addr + ICP_MULTI_DI);
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk("icp multi EDBG: END: icp_multi_insn_bits_do(...)\n");
|
printk(KERN_DEBUG "icp multi EDBG: END: icp_multi_insn_bits_do(...)\n");
|
||||||
#endif
|
#endif
|
||||||
return 2;
|
return 2;
|
||||||
}
|
}
|
||||||
|
@ -641,7 +646,8 @@ static irqreturn_t interrupt_service_icp_multi(int irq, void *d)
|
||||||
int int_no;
|
int int_no;
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk("icp multi EDBG: BGN: interrupt_service_icp_multi(%d,...)\n",
|
printk(KERN_DEBUG
|
||||||
|
"icp multi EDBG: BGN: interrupt_service_icp_multi(%d,...)\n",
|
||||||
irq);
|
irq);
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
|
@ -652,7 +658,8 @@ static irqreturn_t interrupt_service_icp_multi(int irq, void *d)
|
||||||
return IRQ_NONE;
|
return IRQ_NONE;
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk("icp multi EDBG: interrupt_service_icp_multi() ST: %4x\n",
|
printk(KERN_DEBUG
|
||||||
|
"icp multi EDBG: interrupt_service_icp_multi() ST: %4x\n",
|
||||||
readw(devpriv->io_addr + ICP_MULTI_INT_STAT));
|
readw(devpriv->io_addr + ICP_MULTI_INT_STAT));
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
|
@ -680,7 +687,8 @@ static irqreturn_t interrupt_service_icp_multi(int irq, void *d)
|
||||||
}
|
}
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk("icp multi EDBG: END: interrupt_service_icp_multi(...)\n");
|
printk(KERN_DEBUG
|
||||||
|
"icp multi EDBG: END: interrupt_service_icp_multi(...)\n");
|
||||||
#endif
|
#endif
|
||||||
return IRQ_HANDLED;
|
return IRQ_HANDLED;
|
||||||
}
|
}
|
||||||
|
@ -713,7 +721,8 @@ static int check_channel_list(struct comedi_device *dev,
|
||||||
unsigned int i;
|
unsigned int i;
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk("icp multi EDBG: check_channel_list(...,%d)\n", n_chan);
|
printk(KERN_DEBUG
|
||||||
|
"icp multi EDBG: check_channel_list(...,%d)\n", n_chan);
|
||||||
#endif
|
#endif
|
||||||
/* Check that we at least have one channel to check */
|
/* Check that we at least have one channel to check */
|
||||||
if (n_chan < 1) {
|
if (n_chan < 1) {
|
||||||
|
@ -726,7 +735,7 @@ static int check_channel_list(struct comedi_device *dev,
|
||||||
if (CR_AREF(chanlist[i]) == AREF_DIFF) {
|
if (CR_AREF(chanlist[i]) == AREF_DIFF) {
|
||||||
if (CR_CHAN(chanlist[i]) > this_board->n_aichand) {
|
if (CR_CHAN(chanlist[i]) > this_board->n_aichand) {
|
||||||
comedi_error(dev,
|
comedi_error(dev,
|
||||||
"Incorrect differential ai channel number");
|
"Incorrect differential ai ch-nr");
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
|
@ -769,7 +778,8 @@ static void setup_channel_list(struct comedi_device *dev,
|
||||||
unsigned int diff;
|
unsigned int diff;
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk("icp multi EDBG: setup_channel_list(...,%d)\n", n_chan);
|
printk(KERN_DEBUG
|
||||||
|
"icp multi EDBG: setup_channel_list(...,%d)\n", n_chan);
|
||||||
#endif
|
#endif
|
||||||
devpriv->act_chanlist_len = n_chan;
|
devpriv->act_chanlist_len = n_chan;
|
||||||
devpriv->act_chanlist_pos = 0;
|
devpriv->act_chanlist_pos = 0;
|
||||||
|
@ -810,7 +820,8 @@ static void setup_channel_list(struct comedi_device *dev,
|
||||||
devpriv->io_addr + ICP_MULTI_ADC_CSR);
|
devpriv->io_addr + ICP_MULTI_ADC_CSR);
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk("GS: %2d. [%4x]=%4x %4x\n", i, chanprog, range,
|
printk(KERN_DEBUG
|
||||||
|
"GS: %2d. [%4x]=%4x %4x\n", i, chanprog, range,
|
||||||
devpriv->act_chanlist[i]);
|
devpriv->act_chanlist[i]);
|
||||||
#endif
|
#endif
|
||||||
}
|
}
|
||||||
|
@ -837,7 +848,8 @@ static int icp_multi_reset(struct comedi_device *dev)
|
||||||
unsigned int i;
|
unsigned int i;
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk("icp_multi EDBG: BGN: icp_multi_reset(...)\n");
|
printk(KERN_DEBUG
|
||||||
|
"icp_multi EDBG: BGN: icp_multi_reset(...)\n");
|
||||||
#endif
|
#endif
|
||||||
/* Clear INT enables and requests */
|
/* Clear INT enables and requests */
|
||||||
writew(0, devpriv->io_addr + ICP_MULTI_INT_EN);
|
writew(0, devpriv->io_addr + ICP_MULTI_INT_EN);
|
||||||
|
@ -868,7 +880,8 @@ static int icp_multi_reset(struct comedi_device *dev)
|
||||||
writew(0, devpriv->io_addr + ICP_MULTI_DO);
|
writew(0, devpriv->io_addr + ICP_MULTI_DO);
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk("icp multi EDBG: END: icp_multi_reset(...)\n");
|
printk(KERN_DEBUG
|
||||||
|
"icp multi EDBG: END: icp_multi_reset(...)\n");
|
||||||
#endif
|
#endif
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
@ -900,7 +913,8 @@ static int icp_multi_attach(struct comedi_device *dev,
|
||||||
resource_size_t io_addr[5], iobase;
|
resource_size_t io_addr[5], iobase;
|
||||||
unsigned char pci_bus, pci_slot, pci_func;
|
unsigned char pci_bus, pci_slot, pci_func;
|
||||||
|
|
||||||
printk("icp_multi EDBG: BGN: icp_multi_attach(...)\n");
|
printk(KERN_WARNING
|
||||||
|
"icp_multi EDBG: BGN: icp_multi_attach(...)\n");
|
||||||
|
|
||||||
/* Alocate private data storage space */
|
/* Alocate private data storage space */
|
||||||
ret = alloc_private(dev, sizeof(struct icp_multi_private));
|
ret = alloc_private(dev, sizeof(struct icp_multi_private));
|
||||||
|
@ -918,7 +932,8 @@ static int icp_multi_attach(struct comedi_device *dev,
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
printk("Anne's comedi%d: icp_multi: board=%s", dev->minor,
|
printk(KERN_WARNING
|
||||||
|
"Anne's comedi%d: icp_multi: board=%s", dev->minor,
|
||||||
this_board->name);
|
this_board->name);
|
||||||
|
|
||||||
card = select_and_alloc_pci_card(PCI_VENDOR_ID_ICP,
|
card = select_and_alloc_pci_card(PCI_VENDOR_ID_ICP,
|
||||||
|
@ -932,24 +947,26 @@ static int icp_multi_attach(struct comedi_device *dev,
|
||||||
|
|
||||||
if ((pci_card_data(card, &pci_bus, &pci_slot, &pci_func, &io_addr[0],
|
if ((pci_card_data(card, &pci_bus, &pci_slot, &pci_func, &io_addr[0],
|
||||||
&irq)) < 0) {
|
&irq)) < 0) {
|
||||||
printk(" - Can't get configuration data!\n");
|
printk(KERN_WARNING " - Can't get configuration data!\n");
|
||||||
return -EIO;
|
return -EIO;
|
||||||
}
|
}
|
||||||
|
|
||||||
iobase = io_addr[2];
|
iobase = io_addr[2];
|
||||||
devpriv->phys_iobase = iobase;
|
devpriv->phys_iobase = iobase;
|
||||||
|
|
||||||
printk(", b:s:f=%d:%d:%d, io=0x%8llx \n", pci_bus, pci_slot, pci_func,
|
printk(KERN_WARNING
|
||||||
|
", b:s:f=%d:%d:%d, io=0x%8llx \n", pci_bus, pci_slot, pci_func,
|
||||||
(unsigned long long)iobase);
|
(unsigned long long)iobase);
|
||||||
|
|
||||||
devpriv->io_addr = ioremap(iobase, ICP_MULTI_SIZE);
|
devpriv->io_addr = ioremap(iobase, ICP_MULTI_SIZE);
|
||||||
|
|
||||||
if (devpriv->io_addr == NULL) {
|
if (devpriv->io_addr == NULL) {
|
||||||
printk("ioremap failed.\n");
|
printk(KERN_WARNING "ioremap failed.\n");
|
||||||
return -ENOMEM;
|
return -ENOMEM;
|
||||||
}
|
}
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk("0x%08llx mapped to %p, ", (unsigned long long)iobase,
|
printk(KERN_DEBUG
|
||||||
|
"0x%08llx mapped to %p, ", (unsigned long long)iobase,
|
||||||
devpriv->io_addr);
|
devpriv->io_addr);
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
|
@ -977,20 +994,20 @@ static int icp_multi_attach(struct comedi_device *dev,
|
||||||
if (irq) {
|
if (irq) {
|
||||||
if (request_irq(irq, interrupt_service_icp_multi,
|
if (request_irq(irq, interrupt_service_icp_multi,
|
||||||
IRQF_SHARED, "Inova Icp Multi", dev)) {
|
IRQF_SHARED, "Inova Icp Multi", dev)) {
|
||||||
printk
|
printk(KERN_WARNING
|
||||||
(", unable to allocate IRQ %u, DISABLING IT",
|
"unable to allocate IRQ %u, DISABLING IT",
|
||||||
irq);
|
irq);
|
||||||
irq = 0; /* Can't use IRQ */
|
irq = 0; /* Can't use IRQ */
|
||||||
} else
|
} else
|
||||||
printk(", irq=%u", irq);
|
printk(KERN_WARNING ", irq=%u", irq);
|
||||||
} else
|
} else
|
||||||
printk(", IRQ disabled");
|
printk(KERN_WARNING ", IRQ disabled");
|
||||||
} else
|
} else
|
||||||
irq = 0;
|
irq = 0;
|
||||||
|
|
||||||
dev->irq = irq;
|
dev->irq = irq;
|
||||||
|
|
||||||
printk(".\n");
|
printk(KERN_WARNING ".\n");
|
||||||
|
|
||||||
subdev = 0;
|
subdev = 0;
|
||||||
|
|
||||||
|
@ -1065,7 +1082,7 @@ static int icp_multi_attach(struct comedi_device *dev,
|
||||||
devpriv->valid = 1;
|
devpriv->valid = 1;
|
||||||
|
|
||||||
#ifdef ICP_MULTI_EXTDEBUG
|
#ifdef ICP_MULTI_EXTDEBUG
|
||||||
printk("icp multi EDBG: END: icp_multi_attach(...)\n");
|
printk(KERN_DEBUG "icp multi EDBG: END: icp_multi_attach(...)\n");
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
|
|
Loading…
Reference in a new issue