From: Ravishankarkm <ravi.shankar@xxxxxxxxxxxxxxx> This is a patch to the das1800.c file that fixes up a printk warning found by the checkpatch.pl tool Signed-off-by: Ravishankarkm <ravi.shankar@xxxxxxxxxxxxxxx> --- drivers/staging/comedi/drivers/das1800.c | 57 +++++++++++++----------------- 1 files changed, 25 insertions(+), 32 deletions(-) diff --git a/drivers/staging/comedi/drivers/das1800.c b/drivers/staging/comedi/drivers/das1800.c index 222232c..e548a03 100644 --- a/drivers/staging/comedi/drivers/das1800.c +++ b/drivers/staging/comedi/drivers/das1800.c @@ -573,22 +573,21 @@ static int das1800_init_dma(struct comedi_device *dev, unsigned int dma0, devpriv->dma_bits |= DMA_CH7_CH5; break; default: - printk(" only supports dma channels 5 through 7\n" - " Dual dma only allows the following combinations:\n" - " dma 5,6 / 6,7 / or 7,5\n"); + pr_warn(" only supports dma channels 5 through 7\n" + " Dual dma only allows the following combinations:\n" + " dma 5,6 / 6,7 / or 7,5\n"); return -EINVAL; break; } if (request_dma(dma0, driver_das1800.driver_name)) { - printk(" failed to allocate dma channel %i\n", dma0); + pr_warn(" failed to allocate dma channel %i\n", dma0); return -EINVAL; } devpriv->dma0 = dma0; devpriv->dma_current = dma0; if (dma1) { if (request_dma(dma1, driver_das1800.driver_name)) { - printk(" failed to allocate dma channel %i\n", - dma1); + pr_warn(" failed to allocate dma channel %i\n", + dma1); return -EINVAL; } devpriv->dma1 = dma1; @@ -834,69 +833,63 @@ static int das1800_probe(struct comedi_device *dev) case 0x3: if (board == das1801st_da || board == das1802st_da || board == das1701st_da || board == das1702st_da) { - printk(" Board model: %s\n", - das1800_boards[board].name); + pr_debug(" Board model: %s\n", + das1800_boards[board].name); return board; } - printk - (" Board model (probed, not recommended): das-1800st-da series\n"); + pr_debug(" Board model (probed, not recommended): das-1800st-da series\n"); return das1801st; break; case 0x4: if (board == das1802hr_da || board == das1702hr_da) { - printk(" Board model: %s\n", - das1800_boards[board].name); + pr_debug(" Board model: %s\n", + das1800_boards[board].name); return board; } - printk - (" Board model (probed, not recommended): das-1802hr-da\n"); + pr_debug(" Board model (probed, not recommended): das-1802hr-da\n"); return das1802hr; break; case 0x5: if (board == das1801ao || board == das1802ao || board == das1701ao || board == das1702ao) { - printk(" Board model: %s\n", - das1800_boards[board].name); + pr_debug(" Board model: %s\n", + das1800_boards[board].name); return board; } - printk - (" Board model (probed, not recommended): das-1800ao series\n"); + pr_debug(" Board model (probed, not recommended): das-1800ao series\n"); return das1801ao; break; case 0x6: if (board == das1802hr || board == das1702hr) { - printk(" Board model: %s\n", - das1800_boards[board].name); + pr_debug(" Board model: %s\n", + das1800_boards[board].name); return board; } - printk(" Board model (probed, not recommended): das-1802hr\n"); + pr_debug(" Board model (probed, not recommended): das-1802hr\n"); return das1802hr; break; case 0x7: if (board == das1801st || board == das1802st || board == das1701st || board == das1702st) { - printk(" Board model: %s\n", - das1800_boards[board].name); + pr_debug(" Board model: %s\n", + das1800_boards[board].name); return board; } - printk - (" Board model (probed, not recommended): das-1800st series\n"); + pr_debug(" Board model (probed, not recommended): das-1800st series\n"); return das1801st; break; case 0x8: if (board == das1801hc || board == das1802hc) { - printk(" Board model: %s\n", - das1800_boards[board].name); + pr_debug(" Board model: %s\n", + das1800_boards[board].name); return board; } - printk - (" Board model (probed, not recommended): das-1800hc series\n"); + pr_debug(" Board model (probed, not recommended): das-1800hc series\n"); return das1801hc; break; default: - printk - (" Board model: probe returned 0x%x (unknown, please report)\n", - id); + pr_debug(" Board model: probe returned 0x%x (unknown, please report)\n", + id); return board; break; } -- 1.7.6 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel