[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Tue, 07 Aug 2007 21:34:00 -0400
From: James Smart <James.Smart@...lex.Com>
To: Joe Perches <joe@...ches.com>
CC: linux-kernel@...r.kernel.org, Greg KH <gregkh@...e.de>,
Jeff Garzik <jeff@...zik.org>, Tony Luck <tony.luck@...el.com>,
Jiri Slaby <jirislaby@...il.com>,
Roland Dreier <rolandd@...co.com>,
David Brownell <dbrownell@...rs.sourceforge.net>,
Hansjoerg Lipp <hjlipp@....de>,
Jean Delvare <khali@...ux-fr.org>,
"Mark M. Hoffman" <mhoffman@...htlink.com>,
Jeremy Fitzhardinge <jeremy@...source.com>
Subject: Re: [PATCH] Add missing newlines to some uses of dev_<level> messages
ACK
-- james s
Joe Perches wrote:
> Found these while looking at printk uses.
>
> Add missing newlines to dev_<level> uses
> Add missing KERN_<level> prefixes to multiline dev_<level>s
> Fixed a wierd->weird spelling typo
> Added a newline to a printk
>
> Signed-off-by: Joe Perches <joe@...ches.com>
>
> arch/ia64/sn/kernel/xpnet.c | 13 ++++++-----
> drivers/block/xsysace.c | 4 +-
> drivers/char/isicom.c | 10 ++++++--
> drivers/hwmon/adm1026.c | 2 +-
> drivers/hwmon/lm63.c | 2 +-
> drivers/hwmon/vt1211.c | 2 +-
> drivers/hwmon/w83791d.c | 2 +-
> drivers/hwmon/w83792d.c | 4 +-
> drivers/i2c/busses/i2c-pmcmsp.c | 8 +++---
> drivers/i2c/busses/i2c-pnx.c | 2 +-
> drivers/i2c/chips/menelaus.c | 2 +-
> drivers/infiniband/hw/ehca/ehca_main.c | 4 +-
> drivers/isdn/gigaset/common.c | 3 +-
> drivers/mtd/nand/s3c2410.c | 2 +-
> drivers/net/ax88796.c | 2 +-
> drivers/net/bnx2.c | 2 +-
> drivers/net/ehea/ehea_main.c | 9 ++++---
> drivers/net/myri10ge/myri10ge.c | 3 +-
> drivers/net/sky2.c | 2 +-
> drivers/net/usb/mcs7830.c | 16 +++++++-------
> drivers/net/xen-netfront.c | 2 +-
> drivers/power/ds2760_battery.c | 2 +-
> drivers/ps3/ps3stor_lib.c | 2 +-
> drivers/rtc/rtc-sysfs.c | 5 +--
> drivers/scsi/lpfc/lpfc_sli.c | 8 ++++--
> drivers/scsi/qla4xxx/ql4_os.c | 2 +-
> drivers/spi/atmel_spi.c | 2 +-
> drivers/spi/spi_bfin5xx.c | 2 +-
> drivers/spi/spi_imx.c | 4 +-
> drivers/usb/class/cdc-acm.c | 34 ++++++++++++++++----------------
> drivers/usb/core/endpoint.c | 2 +-
> drivers/usb/core/message.c | 4 +-
> drivers/usb/host/ohci-s3c2410.c | 2 +-
> drivers/usb/misc/adutux.c | 4 +-
> drivers/usb/misc/iowarrior.c | 8 +++---
> drivers/usb/misc/phidgetmotorcontrol.c | 2 +-
> drivers/usb/serial/ftdi_sio.c | 4 +-
> drivers/usb/serial/io_edgeport.c | 2 +-
> drivers/usb/serial/ipw.c | 18 ++++++++--------
> drivers/usb/serial/mos7720.c | 4 +-
> drivers/usb/serial/oti6858.c | 2 +-
> drivers/usb/serial/sierra.c | 10 ++++----
> drivers/video/gxt4500.c | 2 +-
> drivers/video/omap/blizzard.c | 2 +-
> drivers/video/omap/dispc.c | 6 ++--
> drivers/video/omap/hwa742.c | 2 +-
> drivers/video/omap/rfbi.c | 4 +-
> drivers/video/s3c2410fb.c | 2 +-
> drivers/w1/masters/ds1wm.c | 2 +-
> drivers/xen/xenbus/xenbus_probe.c | 5 ++-
> sound/arm/aaci.c | 4 +-
> 51 files changed, 129 insertions(+), 119 deletions(-)
>
> diff --git a/arch/ia64/sn/kernel/xpnet.c b/arch/ia64/sn/kernel/xpnet.c
> index e58fcad..a5df672 100644
> --- a/arch/ia64/sn/kernel/xpnet.c
> +++ b/arch/ia64/sn/kernel/xpnet.c
> @@ -269,8 +269,9 @@ xpnet_receive(partid_t partid, int channel, struct xpnet_message *msg)
> skb->protocol = eth_type_trans(skb, xpnet_device);
> skb->ip_summed = CHECKSUM_UNNECESSARY;
>
> - dev_dbg(xpnet, "passing skb to network layer; \n\tskb->head=0x%p "
> - "skb->data=0x%p skb->tail=0x%p skb->end=0x%p skb->len=%d\n",
> + dev_dbg(xpnet, "passing skb to network layer\n"
> + KERN_DEBUG "\tskb->head=0x%p skb->data=0x%p skb->tail=0x%p "
> + "skb->end=0x%p skb->len=%d\n",
> (void *)skb->head, (void *)skb->data, skb_tail_pointer(skb),
> skb_end_pointer(skb), skb->len);
>
> @@ -576,10 +577,10 @@ xpnet_dev_hard_start_xmit(struct sk_buff *skb, struct net_device *dev)
> msg->tailout_ignore = end_addr - (u64)skb_tail_pointer(skb);
> msg->buf_pa = __pa(start_addr);
>
> - dev_dbg(xpnet, "sending XPC message to %d:%d\nmsg->buf_pa="
> - "0x%lx, msg->size=%u, msg->leadin_ignore=%u, "
> - "msg->tailout_ignore=%u\n", dest_partid,
> - XPC_NET_CHANNEL, msg->buf_pa, msg->size,
> + dev_dbg(xpnet, "sending XPC message to %d:%d\n"
> + KERN_DEBUG "msg->buf_pa=0x%lx, msg->size=%u, "
> + "msg->leadin_ignore=%u, msg->tailout_ignore=%u\n",
> + dest_partid, XPC_NET_CHANNEL, msg->buf_pa, msg->size,
> msg->leadin_ignore, msg->tailout_ignore);
>
>
> diff --git a/drivers/block/xsysace.c b/drivers/block/xsysace.c
> index cb27e88..e988710 100644
> --- a/drivers/block/xsysace.c
> +++ b/drivers/block/xsysace.c
> @@ -383,8 +383,8 @@ static inline void ace_dump_mem(void *base, int len)
> static void ace_dump_regs(struct ace_device *ace)
> {
> dev_info(ace->dev, " ctrl: %.8x seccnt/cmd: %.4x ver:%.4x\n"
> - " status:%.8x mpu_lba:%.8x busmode:%4x\n"
> - " error: %.8x cfg_lba:%.8x fatstat:%.4x\n",
> + KERN_INFO " status:%.8x mpu_lba:%.8x busmode:%4x\n"
> + KERN_INFO " error: %.8x cfg_lba:%.8x fatstat:%.4x\n",
> ace_in32(ace, ACE_CTRL),
> ace_in(ace, ACE_SECCNTCMD),
> ace_in(ace, ACE_VERSION),
> diff --git a/drivers/char/isicom.c b/drivers/char/isicom.c
> index 77a7a4a..85d596a 100644
> --- a/drivers/char/isicom.c
> +++ b/drivers/char/isicom.c
> @@ -1529,7 +1529,7 @@ static int __devinit reset_card(struct pci_dev *pdev,
> portcount = inw(base + 0x2);
> if (!inw(base + 0xe) & 0x1 || (portcount != 0 && portcount != 4 &&
> portcount != 8 && portcount != 16)) {
> - dev_err(&pdev->dev, "ISILoad:PCI Card%d reset failure.",
> + dev_err(&pdev->dev, "ISILoad:PCI Card%d reset failure.\n",
> card + 1);
> retval = -EIO;
> goto end;
> @@ -1622,7 +1622,9 @@ static int __devinit load_firmware(struct pci_dev *pdev,
>
> if ((status = inw(base + 0x4)) != 0) {
> dev_warn(&pdev->dev, "Card%d rejected load header:\n"
> - "Address:0x%x\nCount:0x%x\nStatus:0x%x\n",
> + KERN_WARNING "Address:0x%x\n"
> + KERN_WARNING "Count:0x%x\n"
> + KERN_WARNING "Status:0x%x\n",
> index + 1, frame->addr, frame->count, status);
> goto errrelfw;
> }
> @@ -1666,7 +1668,9 @@ static int __devinit load_firmware(struct pci_dev *pdev,
>
> if ((status = inw(base + 0x4)) != 0) {
> dev_warn(&pdev->dev, "Card%d rejected verify header:\n"
> - "Address:0x%x\nCount:0x%x\nStatus: 0x%x\n",
> + KERN_WARNING "Address:0x%x\n"
> + KERN_WARNING "Count:0x%x\n"
> + KERN_WARNING "Status: 0x%x\n",
> index + 1, frame->addr, frame->count, status);
> goto errrelfw;
> }
> diff --git a/drivers/hwmon/adm1026.c b/drivers/hwmon/adm1026.c
> index ba80cd3..8a54f85 100644
> --- a/drivers/hwmon/adm1026.c
> +++ b/drivers/hwmon/adm1026.c
> @@ -1651,7 +1651,7 @@ static int adm1026_detect(struct i2c_adapter *adapter, int address,
> break;
> default :
> dev_err(&adapter->dev, ": Internal error, invalid "
> - "kind (%d)!", kind);
> + "kind (%d)!\n", kind);
> err = -EFAULT;
> goto exitfree;
> }
> diff --git a/drivers/hwmon/lm63.c b/drivers/hwmon/lm63.c
> index 2162d69..1bd0894 100644
> --- a/drivers/hwmon/lm63.c
> +++ b/drivers/hwmon/lm63.c
> @@ -533,7 +533,7 @@ static void lm63_init_client(struct i2c_client *client)
>
> /* Start converting if needed */
> if (data->config & 0x40) { /* standby */
> - dev_dbg(&client->dev, "Switching to operational mode");
> + dev_dbg(&client->dev, "Switching to operational mode\n");
> data->config &= 0xA7;
> i2c_smbus_write_byte_data(client, LM63_REG_CONFIG1,
> data->config);
> diff --git a/drivers/hwmon/vt1211.c b/drivers/hwmon/vt1211.c
> index 9f3e332..a5c0036 100644
> --- a/drivers/hwmon/vt1211.c
> +++ b/drivers/hwmon/vt1211.c
> @@ -795,7 +795,7 @@ static ssize_t set_pwm_auto_point_pwm(struct device *dev,
>
> if ((val < 0) || (val > 255)) {
> dev_err(dev, "pwm value %ld is out of range. "
> - "Choose a value between 0 and 255." , val);
> + "Choose a value between 0 and 255.\n" , val);
> return -EINVAL;
> }
>
> diff --git a/drivers/hwmon/w83791d.c b/drivers/hwmon/w83791d.c
> index 9e5f885..e94ffb9 100644
> --- a/drivers/hwmon/w83791d.c
> +++ b/drivers/hwmon/w83791d.c
> @@ -982,7 +982,7 @@ static int w83791d_detect(struct i2c_adapter *adapter, int address, int kind)
> if (kind == w83791d) {
> client_name = "w83791d";
> } else {
> - dev_err(dev, "w83791d: Internal error: unknown kind (%d)?!?",
> + dev_err(dev, "w83791d: Internal error: unknown kind (%d)?!?\n",
> kind);
> goto error1;
> }
> diff --git a/drivers/hwmon/w83792d.c b/drivers/hwmon/w83792d.c
> index b0fa296..4897adf 100644
> --- a/drivers/hwmon/w83792d.c
> +++ b/drivers/hwmon/w83792d.c
> @@ -1338,8 +1338,8 @@ w83792d_detect(struct i2c_adapter *adapter, int address, int kind)
> if (kind == w83792d) {
> client_name = "w83792d";
> } else {
> - dev_err(dev, "w83792d: Internal error: unknown"
> - " kind (%d)?!?", kind);
> + dev_err(dev, "w83792d: Internal error: unknown kind (%d)?!?\n",
> + kind);
> goto ERROR1;
> }
>
> diff --git a/drivers/i2c/busses/i2c-pmcmsp.c b/drivers/i2c/busses/i2c-pmcmsp.c
> index 17cecf1..be99c02 100644
> --- a/drivers/i2c/busses/i2c-pmcmsp.c
> +++ b/drivers/i2c/busses/i2c-pmcmsp.c
> @@ -591,18 +591,18 @@ static int pmcmsptwi_master_xfer(struct i2c_adapter *adap,
> if (msg->flags & I2C_M_TEN)
> pmcmsptwi_set_twi_config(&oldcfg, data);
>
> - dev_dbg(&adap->dev, "I2C %s of %d bytes ",
> - (msg->flags & I2C_M_RD) ? "read" : "write", msg->len);
> + dev_dbg(&adap->dev, "I2C %s of %d bytes %s\n",
> + (msg->flags & I2C_M_RD) ? "read" : "write", msg->len,
> + (ret == MSP_TWI_XFER_OK) ? "succeeded" : "failed");
> +
> if (ret != MSP_TWI_XFER_OK) {
> /*
> * TODO: We could potentially loop and retry in the case
> * of MSP_TWI_XFER_TIMEOUT.
> */
> - dev_dbg(&adap->dev, "failed\n");
> return -1;
> }
>
> - dev_dbg(&adap->dev, "succeeded\n");
> return 0;
> }
>
> diff --git a/drivers/i2c/busses/i2c-pnx.c b/drivers/i2c/busses/i2c-pnx.c
> index 17376fe..f8d0dff 100644
> --- a/drivers/i2c/busses/i2c-pnx.c
> +++ b/drivers/i2c/busses/i2c-pnx.c
> @@ -575,7 +575,7 @@ static int __devinit i2c_pnx_probe(struct platform_device *pdev)
> else {
> freq_mhz = PNX_DEFAULT_FREQ;
> dev_info(&pdev->dev, "Setting bus frequency to default value: "
> - "%d MHz", freq_mhz);
> + "%d MHz\n", freq_mhz);
> }
>
> i2c_pnx->adapter->algo = &pnx_algorithm;
> diff --git a/drivers/i2c/chips/menelaus.c b/drivers/i2c/chips/menelaus.c
> index 48a7e2f..76d4eda 100644
> --- a/drivers/i2c/chips/menelaus.c
> +++ b/drivers/i2c/chips/menelaus.c
> @@ -1198,7 +1198,7 @@ static int menelaus_probe(struct i2c_client *client)
> err = request_irq(client->irq, menelaus_irq, IRQF_DISABLED,
> DRIVER_NAME, menelaus);
> if (err) {
> - dev_dbg(&client->dev, "can't get IRQ %d, err %d",
> + dev_dbg(&client->dev, "can't get IRQ %d, err %d\n",
> client->irq, err);
> goto fail1;
> }
> diff --git a/drivers/infiniband/hw/ehca/ehca_main.c b/drivers/infiniband/hw/ehca/ehca_main.c
> index 99036b6..3d849c3 100644
> --- a/drivers/infiniband/hw/ehca/ehca_main.c
> +++ b/drivers/infiniband/hw/ehca/ehca_main.c
> @@ -577,12 +577,12 @@ static ssize_t ehca_show_##name(struct device *dev, \
> \
> rblock = ehca_alloc_fw_ctrlblock(GFP_KERNEL); \
> if (!rblock) { \
> - dev_err(dev, "Can't allocate rblock memory."); \
> + dev_err(dev, "Can't allocate rblock memory.\n"); \
> return 0; \
> } \
> \
> if (hipz_h_query_hca(shca->ipz_hca_handle, rblock) != H_SUCCESS) { \
> - dev_err(dev, "Can't query device properties"); \
> + dev_err(dev, "Can't query device properties\n"); \
> ehca_free_fw_ctrlblock(rblock); \
> return 0; \
> } \
> diff --git a/drivers/isdn/gigaset/common.c b/drivers/isdn/gigaset/common.c
> index 6df336b..acd4171 100644
> --- a/drivers/isdn/gigaset/common.c
> +++ b/drivers/isdn/gigaset/common.c
> @@ -534,7 +534,8 @@ int gigaset_fill_inbuf(struct inbuf_t *inbuf, const unsigned char *src,
> n = RBUFSIZE - tail;
> if (!n) {
> dev_err(inbuf->cs->dev,
> - "buffer overflow (%u bytes lost)", bytesleft);
> + "buffer overflow (%u bytes lost)\n",
> + bytesleft);
> break;
> }
> if (n > bytesleft)
> diff --git a/drivers/mtd/nand/s3c2410.c b/drivers/mtd/nand/s3c2410.c
> index 5fac4c4..618ba94 100644
> --- a/drivers/mtd/nand/s3c2410.c
> +++ b/drivers/mtd/nand/s3c2410.c
> @@ -696,7 +696,7 @@ static int s3c24xx_nand_probe(struct platform_device *pdev,
>
> info->clk = clk_get(&pdev->dev, "nand");
> if (IS_ERR(info->clk)) {
> - dev_err(&pdev->dev, "failed to get clock");
> + dev_err(&pdev->dev, "failed to get clock\n");
> err = -ENOENT;
> goto exit_error;
> }
> diff --git a/drivers/net/ax88796.c b/drivers/net/ax88796.c
> index 83da177..f5283ef 100644
> --- a/drivers/net/ax88796.c
> +++ b/drivers/net/ax88796.c
> @@ -850,7 +850,7 @@ static int ax_probe(struct platform_device *pdev)
>
> ax->map2 = ioremap(res->start, size);
> if (ax->map2 == NULL) {
> - dev_err(&pdev->dev, "cannot map reset register");
> + dev_err(&pdev->dev, "cannot map reset register\n");
> ret = -ENXIO;
> goto exit_mem2;
> }
> diff --git a/drivers/net/bnx2.c b/drivers/net/bnx2.c
> index 24e7f9a..ec16b29 100644
> --- a/drivers/net/bnx2.c
> +++ b/drivers/net/bnx2.c
> @@ -6484,7 +6484,7 @@ bnx2_init_board(struct pci_dev *pdev, struct net_device *dev)
> /* enable device (incl. PCI PM wakeup), and bus-mastering */
> rc = pci_enable_device(pdev);
> if (rc) {
> - dev_err(&pdev->dev, "Cannot enable PCI device, aborting.");
> + dev_err(&pdev->dev, "Cannot enable PCI device, aborting.\n");
> goto err_out;
> }
>
> diff --git a/drivers/net/ehea/ehea_main.c b/drivers/net/ehea/ehea_main.c
> index 58702f5..5dc15f7 100644
> --- a/drivers/net/ehea/ehea_main.c
> +++ b/drivers/net/ehea/ehea_main.c
> @@ -2935,7 +2935,8 @@ static int __devinit ehea_probe_adapter(struct ibmebus_dev *dev,
> /* get adapter properties */
> ret = ehea_sense_adapter_attr(adapter);
> if (ret) {
> - dev_err(&dev->ofdev.dev, "sense_adapter_attr failed: %d", ret);
> + dev_err(&dev->ofdev.dev, "sense_adapter_attr failed: %d\n",
> + ret);
> goto out_free_ad;
> }
>
> @@ -2943,7 +2944,7 @@ static int __devinit ehea_probe_adapter(struct ibmebus_dev *dev,
> EHEA_NEQ, EHEA_MAX_ENTRIES_EQ, 1);
> if (!adapter->neq) {
> ret = -EIO;
> - dev_err(&dev->ofdev.dev, "NEQ creation failed");
> + dev_err(&dev->ofdev.dev, "NEQ creation failed\n");
> goto out_free_ad;
> }
>
> @@ -2954,7 +2955,7 @@ static int __devinit ehea_probe_adapter(struct ibmebus_dev *dev,
> ehea_interrupt_neq, IRQF_DISABLED,
> "ehea_neq", adapter);
> if (ret) {
> - dev_err(&dev->ofdev.dev, "requesting NEQ IRQ failed");
> + dev_err(&dev->ofdev.dev, "requesting NEQ IRQ failed\n");
> goto out_kill_eq;
> }
>
> @@ -2970,7 +2971,7 @@ static int __devinit ehea_probe_adapter(struct ibmebus_dev *dev,
>
> ret = ehea_setup_ports(adapter);
> if (ret) {
> - dev_err(&dev->ofdev.dev, "setup_ports failed");
> + dev_err(&dev->ofdev.dev, "setup_ports failed\n");
> goto out_rem_dev_sysfs;
> }
>
> diff --git a/drivers/net/myri10ge/myri10ge.c b/drivers/net/myri10ge/myri10ge.c
> index deca653..5a17db1 100644
> --- a/drivers/net/myri10ge/myri10ge.c
> +++ b/drivers/net/myri10ge/myri10ge.c
> @@ -2898,7 +2898,8 @@ static int myri10ge_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
> if (status != 0) {
> dac_enabled = 0;
> dev_err(&pdev->dev,
> - "64-bit pci address mask was refused, trying 32-bit");
> + "64-bit pci address mask was refused, "
> + "trying 32-bit\n");
> status = pci_set_dma_mask(pdev, DMA_32BIT_MASK);
> }
> if (status != 0) {
> diff --git a/drivers/net/sky2.c b/drivers/net/sky2.c
> index e7a2ead..73a4608 100644
> --- a/drivers/net/sky2.c
> +++ b/drivers/net/sky2.c
> @@ -3748,7 +3748,7 @@ static __devinit struct net_device *sky2_init_netdev(struct sky2_hw *hw,
> struct net_device *dev = alloc_etherdev(sizeof(*sky2));
>
> if (!dev) {
> - dev_err(&hw->pdev->dev, "etherdev alloc failed");
> + dev_err(&hw->pdev->dev, "etherdev alloc failed\n");
> return NULL;
> }
>
> diff --git a/drivers/net/usb/mcs7830.c b/drivers/net/usb/mcs7830.c
> index 6240b97..f55a595 100644
> --- a/drivers/net/usb/mcs7830.c
> +++ b/drivers/net/usb/mcs7830.c
> @@ -114,8 +114,8 @@ static void mcs7830_async_cmd_callback(struct urb *urb)
> struct usb_ctrlrequest *req = (struct usb_ctrlrequest *)urb->context;
>
> if (urb->status < 0)
> - printk(KERN_DEBUG "mcs7830_async_cmd_callback() failed with %d",
> - urb->status);
> + printk(KERN_DEBUG "%s() failed with %d\n",
> + __FUNCTION__, urb->status);
>
> kfree(req);
> usb_free_urb(urb);
> @@ -129,15 +129,15 @@ static void mcs7830_set_reg_async(struct usbnet *dev, u16 index, u16 size, void
>
> urb = usb_alloc_urb(0, GFP_ATOMIC);
> if (!urb) {
> - dev_dbg(&dev->udev->dev, "Error allocating URB "
> - "in write_cmd_async!");
> + dev_dbg(&dev->udev->dev,
> + "Error allocating URB in write_cmd_async!\n");
> return;
> }
>
> req = kmalloc(sizeof *req, GFP_ATOMIC);
> if (!req) {
> - dev_err(&dev->udev->dev, "Failed to allocate memory for "
> - "control request");
> + dev_err(&dev->udev->dev,
> + "Failed to allocate memory for control request\n");
> goto out;
> }
> req->bRequestType = MCS7830_WR_BMREQ;
> @@ -153,8 +153,8 @@ static void mcs7830_set_reg_async(struct usbnet *dev, u16 index, u16 size, void
>
> ret = usb_submit_urb(urb, GFP_ATOMIC);
> if (ret < 0) {
> - dev_err(&dev->udev->dev, "Error submitting the control "
> - "message: ret=%d", ret);
> + dev_err(&dev->udev->dev,
> + "Error submitting the control message: ret=%d\n", ret);
> goto out;
> }
> return;
> diff --git a/drivers/net/xen-netfront.c b/drivers/net/xen-netfront.c
> index 489f69c..6013a2b 100644
> --- a/drivers/net/xen-netfront.c
> +++ b/drivers/net/xen-netfront.c
> @@ -1570,7 +1570,7 @@ static int xennet_connect(struct net_device *dev)
>
> if (!feature_rx_copy) {
> dev_info(&dev->dev,
> - "backend does not support copying recieve path");
> + "backend does not support copying receive path\n");
> return -ENODEV;
> }
>
> diff --git a/drivers/power/ds2760_battery.c b/drivers/power/ds2760_battery.c
> index be7021e..bdb9b72 100644
> --- a/drivers/power/ds2760_battery.c
> +++ b/drivers/power/ds2760_battery.c
> @@ -366,7 +366,7 @@ static int ds2760_battery_probe(struct platform_device *pdev)
>
> retval = power_supply_register(&pdev->dev, &di->bat);
> if (retval) {
> - dev_err(di->dev, "failed to register battery");
> + dev_err(di->dev, "failed to register battery\n");
> goto batt_failed;
> }
>
> diff --git a/drivers/ps3/ps3stor_lib.c b/drivers/ps3/ps3stor_lib.c
> index 3a9824e..55955f1 100644
> --- a/drivers/ps3/ps3stor_lib.c
> +++ b/drivers/ps3/ps3stor_lib.c
> @@ -66,7 +66,7 @@ static int ps3stor_probe_access(struct ps3_storage_device *dev)
> if (n > 1)
> dev_info(&dev->sbd.core,
> "%s:%u: %lu accessible regions found. Only the first "
> - "one will be used",
> + "one will be used\n",
> __func__, __LINE__, n);
> dev->region_idx = __ffs(dev->accessible_regions);
> dev_info(&dev->sbd.core,
> diff --git a/drivers/rtc/rtc-sysfs.c b/drivers/rtc/rtc-sysfs.c
> index 69df94b..2466b9c 100644
> --- a/drivers/rtc/rtc-sysfs.c
> +++ b/drivers/rtc/rtc-sysfs.c
> @@ -176,9 +176,8 @@ void rtc_sysfs_add_device(struct rtc_device *rtc)
>
> err = device_create_file(&rtc->dev, &dev_attr_wakealarm);
> if (err)
> - dev_err(rtc->dev.parent, "failed to create "
> - "alarm attribute, %d",
> - err);
> + dev_err(rtc->dev.parent,
> + "failed to create alarm attribute, %d\n", err);
> }
>
> void rtc_sysfs_del_device(struct rtc_device *rtc)
> diff --git a/drivers/scsi/lpfc/lpfc_sli.c b/drivers/scsi/lpfc/lpfc_sli.c
> index ce5ff2b..dabd170 100644
> --- a/drivers/scsi/lpfc/lpfc_sli.c
> +++ b/drivers/scsi/lpfc/lpfc_sli.c
> @@ -1243,7 +1243,8 @@ void lpfc_sli_poll_fcp_ring(struct lpfc_hba *phba)
> memset(adaptermsg, 0, LPFC_MAX_ADPTMSG);
> memcpy(&adaptermsg[0], (uint8_t *) irsp,
> MAX_MSG_DATA);
> - dev_warn(&((phba->pcidev)->dev), "lpfc%d: %s",
> + dev_warn(&((phba->pcidev)->dev),
> + "lpfc%d: %s\n",
> phba->brd_no, adaptermsg);
> } else {
> /* Unknown IOCB command */
> @@ -1430,7 +1431,8 @@ lpfc_sli_handle_fast_ring_event(struct lpfc_hba *phba,
> memset(adaptermsg, 0, LPFC_MAX_ADPTMSG);
> memcpy(&adaptermsg[0], (uint8_t *) irsp,
> MAX_MSG_DATA);
> - dev_warn(&((phba->pcidev)->dev), "lpfc%d: %s",
> + dev_warn(&((phba->pcidev)->dev),
> + "lpfc%d: %s\n",
> phba->brd_no, adaptermsg);
> } else {
> /* Unknown IOCB command */
> @@ -1681,7 +1683,7 @@ lpfc_sli_handle_slow_ring_event(struct lpfc_hba *phba,
> memcpy(&adaptermsg[0], (uint8_t *) irsp,
> MAX_MSG_DATA);
> dev_warn(&((phba->pcidev)->dev),
> - "lpfc%d: %s",
> + "lpfc%d: %s\n",
> phba->brd_no, adaptermsg);
> } else {
> /* Unknown IOCB command */
> diff --git a/drivers/scsi/qla4xxx/ql4_os.c b/drivers/scsi/qla4xxx/ql4_os.c
> index b1d565c..8fa5aea 100644
> --- a/drivers/scsi/qla4xxx/ql4_os.c
> +++ b/drivers/scsi/qla4xxx/ql4_os.c
> @@ -1285,7 +1285,7 @@ static int __devinit qla4xxx_probe_adapter(struct pci_dev *pdev,
>
> ret = scsi_init_shared_tag_map(host, MAX_SRBS);
> if (ret) {
> - dev_warn(&ha->pdev->dev, "scsi_init_shared_tag_map failed");
> + dev_warn(&ha->pdev->dev, "scsi_init_shared_tag_map failed\n");
> goto probe_failed;
> }
>
> diff --git a/drivers/spi/atmel_spi.c b/drivers/spi/atmel_spi.c
> index ad14405..f24f442 100644
> --- a/drivers/spi/atmel_spi.c
> +++ b/drivers/spi/atmel_spi.c
> @@ -364,7 +364,7 @@ atmel_spi_interrupt(int irq, void *dev_id)
> break;
> if (!timeout)
> dev_warn(master->cdev.dev,
> - "timeout waiting for TXEMPTY");
> + "timeout waiting for TXEMPTY\n");
> while (spi_readl(as, SR) & SPI_BIT(RDRF))
> spi_readl(as, RDR);
>
> diff --git a/drivers/spi/spi_bfin5xx.c b/drivers/spi/spi_bfin5xx.c
> index 48587c2..de2403c 100644
> --- a/drivers/spi/spi_bfin5xx.c
> +++ b/drivers/spi/spi_bfin5xx.c
> @@ -1071,7 +1071,7 @@ static int setup(struct spi_device *spi)
> return -ENODEV;
> }
>
> - dev_dbg(&spi->dev, "setup spi chip %s, width is %d, dma is %d,",
> + dev_dbg(&spi->dev, "setup spi chip %s, width is %d, dma is %d\n",
> spi->modalias, chip->width, chip->enable_dma);
> dev_dbg(&spi->dev, "ctl_reg is 0x%x, flag_reg is 0x%x\n",
> chip->ctl_reg, chip->flag);
> diff --git a/drivers/spi/spi_imx.c b/drivers/spi/spi_imx.c
> index aee9ad6..c7a98ad 100644
> --- a/drivers/spi/spi_imx.c
> +++ b/drivers/spi/spi_imx.c
> @@ -1194,7 +1194,7 @@ static int setup(struct spi_device *spi)
> chip = kzalloc(sizeof(struct chip_data), GFP_KERNEL);
> if (!chip) {
> dev_err(&spi->dev,
> - "setup - cannot allocate controller state");
> + "setup - cannot allocate controller state\n");
> return -ENOMEM;
> }
> chip->control = SPI_DEFAULT_CONTROL;
> @@ -1206,7 +1206,7 @@ static int setup(struct spi_device *spi)
> if (!chip_info) {
> dev_err(&spi->dev,
> "setup - "
> - "cannot allocate controller data");
> + "cannot allocate controller data\n");
> status = -ENOMEM;
> goto err_first_setup;
> }
> diff --git a/drivers/usb/class/cdc-acm.c b/drivers/usb/class/cdc-acm.c
> index fe940e0..40131f3 100644
> --- a/drivers/usb/class/cdc-acm.c
> +++ b/drivers/usb/class/cdc-acm.c
> @@ -332,7 +332,7 @@ static void acm_read_bulk(struct urb *urb)
> return;
>
> if (status)
> - dev_dbg(&acm->data->dev, "bulk rx status %d", status);
> + dev_dbg(&acm->data->dev, "bulk rx status %d\n", status);
>
> buf = rcv->buffer;
> buf->size = urb->actual_length;
> @@ -831,13 +831,13 @@ static int acm_probe (struct usb_interface *intf,
>
> /* normal probing*/
> if (!buffer) {
> - err("Wierd descriptor references\n");
> + err("Weird descriptor references\n");
> return -EINVAL;
> }
>
> if (!buflen) {
> if (intf->cur_altsetting->endpoint->extralen && intf->cur_altsetting->endpoint->extra) {
> - dev_dbg(&intf->dev,"Seeking extra descriptors on endpoint");
> + dev_dbg(&intf->dev,"Seeking extra descriptors on endpoint\n");
> buflen = intf->cur_altsetting->endpoint->extralen;
> buffer = intf->cur_altsetting->endpoint->extra;
> } else {
> @@ -887,24 +887,24 @@ next_desc:
>
> if (!union_header) {
> if (call_interface_num > 0) {
> - dev_dbg(&intf->dev,"No union descriptor, using call management descriptor");
> + dev_dbg(&intf->dev,"No union descriptor, using call management descriptor\n");
> data_interface = usb_ifnum_to_if(usb_dev, (data_interface_num = call_interface_num));
> control_interface = intf;
> } else {
> - dev_dbg(&intf->dev,"No union descriptor, giving up");
> + dev_dbg(&intf->dev,"No union descriptor, giving up\n");
> return -ENODEV;
> }
> } else {
> control_interface = usb_ifnum_to_if(usb_dev, union_header->bMasterInterface0);
> data_interface = usb_ifnum_to_if(usb_dev, (data_interface_num = union_header->bSlaveInterface0));
> if (!control_interface || !data_interface) {
> - dev_dbg(&intf->dev,"no interfaces");
> + dev_dbg(&intf->dev,"no interfaces\n");
> return -ENODEV;
> }
> }
>
> if (data_interface_num != call_interface_num)
> - dev_dbg(&intf->dev,"Seperate call control interface. That is not fully supported.");
> + dev_dbg(&intf->dev,"Seperate call control interface. That is not fully supported.\n");
>
> skip_normal_probe:
>
> @@ -912,7 +912,7 @@ skip_normal_probe:
> if (data_interface->cur_altsetting->desc.bInterfaceClass != CDC_DATA_INTERFACE_TYPE) {
> if (control_interface->cur_altsetting->desc.bInterfaceClass == CDC_DATA_INTERFACE_TYPE) {
> struct usb_interface *t;
> - dev_dbg(&intf->dev,"Your device has switched interfaces.");
> + dev_dbg(&intf->dev,"Your device has switched interfaces.\n");
>
> t = control_interface;
> control_interface = data_interface;
> @@ -923,7 +923,7 @@ skip_normal_probe:
> }
>
> if (usb_interface_claimed(data_interface)) { /* valid in this context */
> - dev_dbg(&intf->dev,"The data interface isn't available");
> + dev_dbg(&intf->dev,"The data interface isn't available\n");
> return -EBUSY;
> }
>
> @@ -940,7 +940,7 @@ skip_normal_probe:
> if (!usb_endpoint_dir_in(epread)) {
> /* descriptors are swapped */
> struct usb_endpoint_descriptor *t;
> - dev_dbg(&intf->dev,"The data interface has switched endpoints");
> + dev_dbg(&intf->dev,"The data interface has switched endpoints\n");
>
> t = epread;
> epread = epwrite;
> @@ -955,7 +955,7 @@ skip_normal_probe:
> }
>
> if (!(acm = kzalloc(sizeof(struct acm), GFP_KERNEL))) {
> - dev_dbg(&intf->dev, "out of memory (acm kzalloc)");
> + dev_dbg(&intf->dev, "out of memory (acm kzalloc)\n");
> goto alloc_fail;
> }
>
> @@ -981,26 +981,26 @@ skip_normal_probe:
>
> buf = usb_buffer_alloc(usb_dev, ctrlsize, GFP_KERNEL, &acm->ctrl_dma);
> if (!buf) {
> - dev_dbg(&intf->dev, "out of memory (ctrl buffer alloc)");
> + dev_dbg(&intf->dev, "out of memory (ctrl buffer alloc)\n");
> goto alloc_fail2;
> }
> acm->ctrl_buffer = buf;
>
> if (acm_write_buffers_alloc(acm) < 0) {
> - dev_dbg(&intf->dev, "out of memory (write buffer alloc)");
> + dev_dbg(&intf->dev, "out of memory (write buffer alloc)\n");
> goto alloc_fail4;
> }
>
> acm->ctrlurb = usb_alloc_urb(0, GFP_KERNEL);
> if (!acm->ctrlurb) {
> - dev_dbg(&intf->dev, "out of memory (ctrlurb kmalloc)");
> + dev_dbg(&intf->dev, "out of memory (ctrlurb kmalloc)\n");
> goto alloc_fail5;
> }
> for (i = 0; i < num_rx_buf; i++) {
> struct acm_ru *rcv = &(acm->ru[i]);
>
> if (!(rcv->urb = usb_alloc_urb(0, GFP_KERNEL))) {
> - dev_dbg(&intf->dev, "out of memory (read urbs usb_alloc_urb)");
> + dev_dbg(&intf->dev, "out of memory (read urbs usb_alloc_urb)\n");
> goto alloc_fail7;
> }
>
> @@ -1011,13 +1011,13 @@ skip_normal_probe:
> struct acm_rb *buf = &(acm->rb[i]);
>
> if (!(buf->base = usb_buffer_alloc(acm->dev, readsize, GFP_KERNEL, &buf->dma))) {
> - dev_dbg(&intf->dev, "out of memory (read bufs usb_buffer_alloc)");
> + dev_dbg(&intf->dev, "out of memory (read bufs usb_buffer_alloc)\n");
> goto alloc_fail7;
> }
> }
> acm->writeurb = usb_alloc_urb(0, GFP_KERNEL);
> if (!acm->writeurb) {
> - dev_dbg(&intf->dev, "out of memory (writeurb kmalloc)");
> + dev_dbg(&intf->dev, "out of memory (writeurb kmalloc)\n");
> goto alloc_fail7;
> }
>
> diff --git a/drivers/usb/core/endpoint.c b/drivers/usb/core/endpoint.c
> index e0ec704..ac89e2f 100644
> --- a/drivers/usb/core/endpoint.c
> +++ b/drivers/usb/core/endpoint.c
> @@ -292,7 +292,7 @@ int usb_create_ep_files(struct device *parent,
>
> retval = endpoint_get_minor(ep_dev);
> if (retval) {
> - dev_err(parent, "can not allocate minor number for %s",
> + dev_err(parent, "can not allocate minor number for %s\n",
> ep_dev->dev.bus_id);
> goto error_register;
> }
> diff --git a/drivers/usb/core/message.c b/drivers/usb/core/message.c
> index b6bd05e..2e2bc4d 100644
> --- a/drivers/usb/core/message.c
> +++ b/drivers/usb/core/message.c
> @@ -1508,7 +1508,7 @@ int usb_set_configuration(struct usb_device *dev, int configuration)
> new_interfaces = kmalloc(nintf * sizeof(*new_interfaces),
> GFP_KERNEL);
> if (!new_interfaces) {
> - dev_err(&dev->dev, "Out of memory");
> + dev_err(&dev->dev, "Out of memory\n");
> return -ENOMEM;
> }
>
> @@ -1517,7 +1517,7 @@ int usb_set_configuration(struct usb_device *dev, int configuration)
> sizeof(struct usb_interface),
> GFP_KERNEL);
> if (!new_interfaces[n]) {
> - dev_err(&dev->dev, "Out of memory");
> + dev_err(&dev->dev, "Out of memory\n");
> ret = -ENOMEM;
> free_interfaces:
> while (--n >= 0)
> diff --git a/drivers/usb/host/ohci-s3c2410.c b/drivers/usb/host/ohci-s3c2410.c
> index 6829814..44b79e8 100644
> --- a/drivers/usb/host/ohci-s3c2410.c
> +++ b/drivers/usb/host/ohci-s3c2410.c
> @@ -358,7 +358,7 @@ static int usb_hcd_s3c2410_probe (const struct hc_driver *driver,
> hcd->rsrc_len = dev->resource[0].end - dev->resource[0].start + 1;
>
> if (!request_mem_region(hcd->rsrc_start, hcd->rsrc_len, hcd_name)) {
> - dev_err(&dev->dev, "request_mem_region failed");
> + dev_err(&dev->dev, "request_mem_region failed\n");
> retval = -EBUSY;
> goto err_put;
> }
> diff --git a/drivers/usb/misc/adutux.c b/drivers/usb/misc/adutux.c
> index e9fdbc8..1f67812 100644
> --- a/drivers/usb/misc/adutux.c
> +++ b/drivers/usb/misc/adutux.c
> @@ -804,7 +804,7 @@ static int adu_probe(struct usb_interface *interface,
> dev->minor = interface->minor;
>
> /* let the user know what node this device is now attached to */
> - dev_info(&interface->dev, "ADU%d %s now attached to /dev/usb/adutux%d",
> + dev_info(&interface->dev, "ADU%d %s now attached to /dev/usb/adutux%d\n",
> udev->descriptor.idProduct, dev->serial_number,
> (dev->minor - ADU_MINOR_BASE));
> exit:
> @@ -850,7 +850,7 @@ static void adu_disconnect(struct usb_interface *interface)
> mutex_unlock(&dev->mtx);
> }
>
> - dev_info(&interface->dev, "ADU device adutux%d now disconnected",
> + dev_info(&interface->dev, "ADU device adutux%d now disconnected\n",
> (minor - ADU_MINOR_BASE));
>
> dbg(2," %s : leave", __FUNCTION__);
> diff --git a/drivers/usb/misc/iowarrior.c b/drivers/usb/misc/iowarrior.c
> index 46d9f27..d372fbc 100644
> --- a/drivers/usb/misc/iowarrior.c
> +++ b/drivers/usb/misc/iowarrior.c
> @@ -216,7 +216,7 @@ static void iowarrior_callback(struct urb *urb)
> exit:
> retval = usb_submit_urb(urb, GFP_ATOMIC);
> if (retval)
> - dev_err(&dev->interface->dev, "%s - usb_submit_urb failed with result %d",
> + dev_err(&dev->interface->dev, "%s - usb_submit_urb failed with result %d\n",
> __FUNCTION__, retval);
>
> }
> @@ -451,7 +451,7 @@ static ssize_t iowarrior_write(struct file *file,
> break;
> default:
> /* what do we have here ? An unsupported Product-ID ? */
> - dev_err(&dev->interface->dev, "%s - not supported for product=0x%x",
> + dev_err(&dev->interface->dev, "%s - not supported for product=0x%x\n",
> __FUNCTION__, dev->product_id);
> retval = -EFAULT;
> goto exit;
> @@ -526,7 +526,7 @@ static int iowarrior_ioctl(struct inode *inode, struct file *file,
> } else {
> retval = -EINVAL;
> dev_err(&dev->interface->dev,
> - "ioctl 'IOW_WRITE' is not supported for product=0x%x.",
> + "ioctl 'IOW_WRITE' is not supported for product=0x%x.\n",
> dev->product_id);
> }
> break;
> @@ -752,7 +752,7 @@ static int iowarrior_probe(struct usb_interface *interface,
> /* allocate memory for our device state and intialize it */
> dev = kzalloc(sizeof(struct iowarrior), GFP_KERNEL);
> if (dev == NULL) {
> - dev_err(&interface->dev, "Out of memory");
> + dev_err(&interface->dev, "Out of memory\n");
> return retval;
> }
>
> diff --git a/drivers/usb/misc/phidgetmotorcontrol.c b/drivers/usb/misc/phidgetmotorcontrol.c
> index df0ebcd..2ad09b1 100644
> --- a/drivers/usb/misc/phidgetmotorcontrol.c
> +++ b/drivers/usb/misc/phidgetmotorcontrol.c
> @@ -155,7 +155,7 @@ resubmit:
> retval = usb_submit_urb(urb, GFP_ATOMIC);
> if (retval)
> dev_err(&mc->intf->dev,
> - "can't resubmit intr, %s-%s/motorcontrol0, retval %d",
> + "can't resubmit intr, %s-%s/motorcontrol0, retval %d\n",
> mc->udev->bus->bus_name,
> mc->udev->devpath, retval);
> }
> diff --git a/drivers/usb/serial/ftdi_sio.c b/drivers/usb/serial/ftdi_sio.c
> index 7b1673a..4341cad 100644
> --- a/drivers/usb/serial/ftdi_sio.c
> +++ b/drivers/usb/serial/ftdi_sio.c
> @@ -1068,7 +1068,7 @@ static ssize_t show_latency_timer(struct device *dev, struct device_attribute *a
> (char*) &latency, 1, WDR_TIMEOUT);
>
> if (rv < 0) {
> - dev_err(dev, "Unable to read latency timer: %i", rv);
> + dev_err(dev, "Unable to read latency timer: %i\n", rv);
> return -EIO;
> }
> return sprintf(buf, "%i\n", latency);
> @@ -1095,7 +1095,7 @@ static ssize_t store_latency_timer(struct device *dev, struct device_attribute *
> buf, 0, WDR_TIMEOUT);
>
> if (rv < 0) {
> - dev_err(dev, "Unable to write latency timer: %i", rv);
> + dev_err(dev, "Unable to write latency timer: %i\n", rv);
> return -EIO;
> }
>
> diff --git a/drivers/usb/serial/io_edgeport.c b/drivers/usb/serial/io_edgeport.c
> index 2ecb1d2..8dd3abc 100644
> --- a/drivers/usb/serial/io_edgeport.c
> +++ b/drivers/usb/serial/io_edgeport.c
> @@ -2882,7 +2882,7 @@ static int edge_startup (struct usb_serial *serial)
> (edge_serial->product_info.NumPorts != serial->num_ports)) {
> dev_warn(&serial->dev->dev, "Device Reported %d serial ports "
> "vs. core thinking we have %d ports, email "
> - "greg@...ah.com this information.",
> + "greg@...ah.com this information.\n",
> edge_serial->product_info.NumPorts,
> serial->num_ports);
> }
> diff --git a/drivers/usb/serial/ipw.c b/drivers/usb/serial/ipw.c
> index 1b94daa..cbe5530 100644
> --- a/drivers/usb/serial/ipw.c
> +++ b/drivers/usb/serial/ipw.c
> @@ -227,7 +227,7 @@ static int ipw_open(struct usb_serial_port *port, struct file *filp)
> 0,
> 100000);
> if (result < 0)
> - dev_err(&port->dev, "Init of modem failed (error = %d)", result);
> + dev_err(&port->dev, "Init of modem failed (error = %d)\n", result);
>
> /* reset the bulk pipes */
> usb_clear_halt(dev, usb_rcvbulkpipe(dev, port->bulk_in_endpointAddress));
> @@ -255,7 +255,7 @@ static int ipw_open(struct usb_serial_port *port, struct file *filp)
> 0,
> 100000);
> if (result < 0)
> - dev_err(&port->dev, "Enabling bulk RxRead failed (error = %d)", result);
> + dev_err(&port->dev, "Enabling bulk RxRead failed (error = %d)\n", result);
>
> /*--4: setup the initial flowcontrol */
> dbg("%s:setting init flowcontrol (%s)",__FUNCTION__,buf_flow_init);
> @@ -268,7 +268,7 @@ static int ipw_open(struct usb_serial_port *port, struct file *filp)
> 0x10,
> 200000);
> if (result < 0)
> - dev_err(&port->dev, "initial flowcontrol failed (error = %d)", result);
> + dev_err(&port->dev, "initial flowcontrol failed (error = %d)\n", result);
>
>
> /*--5: raise the dtr */
> @@ -282,7 +282,7 @@ static int ipw_open(struct usb_serial_port *port, struct file *filp)
> 0,
> 200000);
> if (result < 0)
> - dev_err(&port->dev, "setting dtr failed (error = %d)", result);
> + dev_err(&port->dev, "setting dtr failed (error = %d)\n", result);
>
> /*--6: raise the rts */
> dbg("%s:raising rts",__FUNCTION__);
> @@ -295,7 +295,7 @@ static int ipw_open(struct usb_serial_port *port, struct file *filp)
> 0,
> 200000);
> if (result < 0)
> - dev_err(&port->dev, "setting dtr failed (error = %d)", result);
> + dev_err(&port->dev, "setting dtr failed (error = %d)\n", result);
>
> kfree(buf_flow_init);
> return 0;
> @@ -322,7 +322,7 @@ static void ipw_close(struct usb_serial_port *port, struct file * filp)
> 0,
> 200000);
> if (result < 0)
> - dev_err(&port->dev, "dropping dtr failed (error = %d)", result);
> + dev_err(&port->dev, "dropping dtr failed (error = %d)\n", result);
>
> /*--2: drop the rts */
> dbg("%s:dropping rts",__FUNCTION__);
> @@ -334,7 +334,7 @@ static void ipw_close(struct usb_serial_port *port, struct file * filp)
> 0,
> 200000);
> if (result < 0)
> - dev_err(&port->dev, "dropping rts failed (error = %d)", result);
> + dev_err(&port->dev, "dropping rts failed (error = %d)\n", result);
>
>
> /*--3: purge */
> @@ -347,7 +347,7 @@ static void ipw_close(struct usb_serial_port *port, struct file * filp)
> 0,
> 200000);
> if (result < 0)
> - dev_err(&port->dev, "purge failed (error = %d)", result);
> + dev_err(&port->dev, "purge failed (error = %d)\n", result);
>
>
> /* send RXBULK_off (tell modem to stop transmitting bulk data on rx chan) */
> @@ -361,7 +361,7 @@ static void ipw_close(struct usb_serial_port *port, struct file * filp)
> 100000);
>
> if (result < 0)
> - dev_err(&port->dev, "Disabling bulk RxRead failed (error = %d)", result);
> + dev_err(&port->dev, "Disabling bulk RxRead failed (error = %d)\n", result);
>
> /* shutdown any in-flight urbs that we know about */
> usb_kill_urb(port->read_urb);
> diff --git a/drivers/usb/serial/mos7720.c b/drivers/usb/serial/mos7720.c
> index 01e811b..e02c198 100644
> --- a/drivers/usb/serial/mos7720.c
> +++ b/drivers/usb/serial/mos7720.c
> @@ -478,7 +478,7 @@ static int mos7720_open(struct usb_serial_port *port, struct file * filp)
> response = usb_submit_urb(port0->interrupt_in_urb, GFP_KERNEL);
> if (response)
> dev_err(&port->dev,
> - "%s - Error %d submitting control urb",
> + "%s - Error %d submitting control urb\n",
> __FUNCTION__, response);
> }
>
> @@ -492,7 +492,7 @@ static int mos7720_open(struct usb_serial_port *port, struct file * filp)
> response = usb_submit_urb(port->read_urb, GFP_KERNEL);
> if (response)
> dev_err(&port->dev,
> - "%s - Error %d submitting read urb", __FUNCTION__, response);
> + "%s - Error %d submitting read urb\n", __FUNCTION__, response);
>
> /* initialize our icount structure */
> memset(&(mos7720_port->icount), 0x00, sizeof(mos7720_port->icount));
> diff --git a/drivers/usb/serial/oti6858.c b/drivers/usb/serial/oti6858.c
> index d7db71e..9c42785 100644
> --- a/drivers/usb/serial/oti6858.c
> +++ b/drivers/usb/serial/oti6858.c
> @@ -256,7 +256,7 @@ static void setup_line(struct work_struct *work)
> 100);
>
> if (result != OTI6858_CTRL_PKT_SIZE) {
> - dev_err(&port->dev, "%s(): error reading status", __FUNCTION__);
> + dev_err(&port->dev, "%s(): error reading status\n", __FUNCTION__);
> kfree(new_setup);
> /* we will try again */
> schedule_delayed_work(&priv->delayed_setup_work, msecs_to_jiffies(2));
> diff --git a/drivers/usb/serial/sierra.c b/drivers/usb/serial/sierra.c
> index 0bb8de4..959b3e4 100644
> --- a/drivers/usb/serial/sierra.c
> +++ b/drivers/usb/serial/sierra.c
> @@ -48,7 +48,7 @@ enum devicetype {
> static int sierra_set_power_state(struct usb_device *udev, __u16 swiState)
> {
> int result;
> - dev_dbg(&udev->dev, "%s", "SET POWER STATE");
> + dev_dbg(&udev->dev, "%s", "SET POWER STATE\n");
> result = usb_control_msg(udev, usb_sndctrlpipe(udev, 0),
> 0x00, /* __u8 request */
> 0x40, /* __u8 request type */
> @@ -63,7 +63,7 @@ static int sierra_set_power_state(struct usb_device *udev, __u16 swiState)
> static int sierra_set_ms_mode(struct usb_device *udev, __u16 eSocMode)
> {
> int result;
> - dev_dbg(&udev->dev, "%s", "DEVICE MODE SWITCH");
> + dev_dbg(&udev->dev, "%s", "DEVICE MODE SWITCH\n");
> result = usb_control_msg(udev, usb_sndctrlpipe(udev, 0),
> SWIMS_USB_REQUEST_SetMode, /* __u8 request */
> SWIMS_USB_REQUEST_TYPE_SetMode, /* __u8 request type */
> @@ -397,7 +397,7 @@ static void sierra_indat_callback(struct urb *urb)
> err = usb_submit_urb(urb, GFP_ATOMIC);
> if (err)
> dev_err(&port->dev, "resubmit read urb failed."
> - "(%d)", err);
> + "(%d)\n", err);
> }
> }
> return;
> @@ -525,7 +525,7 @@ static int sierra_open(struct usb_serial_port *port, struct file *filp)
>
> result = usb_submit_urb(urb, GFP_KERNEL);
> if (result) {
> - dev_err(&port->dev, "submit urb %d failed (%d) %d",
> + dev_err(&port->dev, "submit urb %d failed (%d) %d\n",
> i, result, urb->transfer_buffer_length);
> }
> }
> @@ -538,7 +538,7 @@ static int sierra_open(struct usb_serial_port *port, struct file *filp)
> if (port->interrupt_in_urb) {
> result = usb_submit_urb(port->interrupt_in_urb, GFP_KERNEL);
> if (result)
> - dev_err(&port->dev, "submit irq_in urb failed %d",
> + dev_err(&port->dev, "submit irq_in urb failed %d\n",
> result);
> }
> return 0;
> diff --git a/drivers/video/gxt4500.c b/drivers/video/gxt4500.c
> index 23a6bcc..e92337b 100644
> --- a/drivers/video/gxt4500.c
> +++ b/drivers/video/gxt4500.c
> @@ -636,7 +636,7 @@ static int __devinit gxt4500_probe(struct pci_dev *pdev,
>
> info = framebuffer_alloc(sizeof(struct gxt4500_par), &pdev->dev);
> if (!info) {
> - dev_err(&pdev->dev, "gxt4500: cannot alloc FB info record");
> + dev_err(&pdev->dev, "gxt4500: cannot alloc FB info record\n");
> goto err_free_fb;
> }
> par = info->par;
> diff --git a/drivers/video/omap/blizzard.c b/drivers/video/omap/blizzard.c
> index e682940..4d8ad9c 100644
> --- a/drivers/video/omap/blizzard.c
> +++ b/drivers/video/omap/blizzard.c
> @@ -225,7 +225,7 @@ static void blizzard_restart_sdram(void)
> while (!(blizzard_read_reg(BLIZZARD_MEM_BANK0_STATUS) & 0x01)) {
> if (time_after(jiffies, tmo)) {
> dev_err(blizzard.fbdev->dev,
> - "s1d1374x: SDRAM not ready");
> + "s1d1374x: SDRAM not ready\n");
> break;
> }
> msleep(1);
> diff --git a/drivers/video/omap/dispc.c b/drivers/video/omap/dispc.c
> index f4c2343..ab32ceb 100644
> --- a/drivers/video/omap/dispc.c
> +++ b/drivers/video/omap/dispc.c
> @@ -880,19 +880,19 @@ static irqreturn_t omap_dispc_irq_handler(int irq, void *dev)
> static int get_dss_clocks(void)
> {
> if (IS_ERR((dispc.dss_ick = clk_get(dispc.fbdev->dev, "dss_ick")))) {
> - dev_err(dispc.fbdev->dev, "can't get dss_ick");
> + dev_err(dispc.fbdev->dev, "can't get dss_ick\n");
> return PTR_ERR(dispc.dss_ick);
> }
>
> if (IS_ERR((dispc.dss1_fck = clk_get(dispc.fbdev->dev, "dss1_fck")))) {
> - dev_err(dispc.fbdev->dev, "can't get dss1_fck");
> + dev_err(dispc.fbdev->dev, "can't get dss1_fck\n");
> clk_put(dispc.dss_ick);
> return PTR_ERR(dispc.dss1_fck);
> }
>
> if (IS_ERR((dispc.dss_54m_fck =
> clk_get(dispc.fbdev->dev, "dss_54m_fck")))) {
> - dev_err(dispc.fbdev->dev, "can't get dss_54m_fck");
> + dev_err(dispc.fbdev->dev, "can't get dss_54m_fck\n");
> clk_put(dispc.dss_ick);
> clk_put(dispc.dss1_fck);
> return PTR_ERR(dispc.dss_54m_fck);
> diff --git a/drivers/video/omap/hwa742.c b/drivers/video/omap/hwa742.c
> index dc48e02..1e642b7 100644
> --- a/drivers/video/omap/hwa742.c
> +++ b/drivers/video/omap/hwa742.c
> @@ -508,7 +508,7 @@ int hwa742_update_window_async(struct fb_info *fbi,
> if (unlikely(win->format &
> ~(0x03 | OMAPFB_FORMAT_FLAG_DOUBLE |
> OMAPFB_FORMAT_FLAG_TEARSYNC | OMAPFB_FORMAT_FLAG_FORCE_VSYNC))) {
> - dev_dbg(hwa742.fbdev->dev, "invalid window flag");
> + dev_dbg(hwa742.fbdev->dev, "invalid window flag\n");
> r = -EINVAL;
> goto out;
> }
> diff --git a/drivers/video/omap/rfbi.c b/drivers/video/omap/rfbi.c
> index 2b42698..789cfd2 100644
> --- a/drivers/video/omap/rfbi.c
> +++ b/drivers/video/omap/rfbi.c
> @@ -84,12 +84,12 @@ static inline u32 rfbi_read_reg(int idx)
> static int rfbi_get_clocks(void)
> {
> if (IS_ERR((rfbi.dss_ick = clk_get(rfbi.fbdev->dev, "dss_ick")))) {
> - dev_err(rfbi.fbdev->dev, "can't get dss_ick");
> + dev_err(rfbi.fbdev->dev, "can't get dss_ick\n");
> return PTR_ERR(rfbi.dss_ick);
> }
>
> if (IS_ERR((rfbi.dss1_fck = clk_get(rfbi.fbdev->dev, "dss1_fck")))) {
> - dev_err(rfbi.fbdev->dev, "can't get dss1_fck");
> + dev_err(rfbi.fbdev->dev, "can't get dss1_fck\n");
> clk_put(rfbi.dss_ick);
> return PTR_ERR(rfbi.dss1_fck);
> }
> diff --git a/drivers/video/s3c2410fb.c b/drivers/video/s3c2410fb.c
> index 8a4c647..6185e4c 100644
> --- a/drivers/video/s3c2410fb.c
> +++ b/drivers/video/s3c2410fb.c
> @@ -800,7 +800,7 @@ static int __init s3c2410fb_probe(struct platform_device *pdev)
>
> res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> if (res == NULL) {
> - dev_err(&pdev->dev, "failed to get memory registersn");
> + dev_err(&pdev->dev, "failed to get memory registers\n");
> ret = -ENXIO;
> goto dealloc_fb;
> }
> diff --git a/drivers/w1/masters/ds1wm.c b/drivers/w1/masters/ds1wm.c
> index 4b69664..5747997 100644
> --- a/drivers/w1/masters/ds1wm.c
> +++ b/drivers/w1/masters/ds1wm.c
> @@ -307,7 +307,7 @@ static void ds1wm_search(void *data, u8 search_type,
> rom_id |= (unsigned long long) r << (i * 4);
>
> }
> - dev_dbg(&ds1wm_data->pdev->dev, "found 0x%08llX", rom_id);
> + dev_dbg(&ds1wm_data->pdev->dev, "found 0x%08llX\n", rom_id);
>
> ds1wm_write_register(ds1wm_data, DS1WM_CMD, ~DS1WM_CMD_SRA);
> ds1wm_reset(ds1wm_data);
> diff --git a/drivers/xen/xenbus/xenbus_probe.c b/drivers/xen/xenbus/xenbus_probe.c
> index 0b769f7..4750de3 100644
> --- a/drivers/xen/xenbus/xenbus_probe.c
> +++ b/drivers/xen/xenbus/xenbus_probe.c
> @@ -185,13 +185,14 @@ static void otherend_changed(struct xenbus_watch *watch,
> if (!dev->otherend ||
> strncmp(dev->otherend, vec[XS_WATCH_PATH],
> strlen(dev->otherend))) {
> - dev_dbg(&dev->dev, "Ignoring watch at %s", vec[XS_WATCH_PATH]);
> + dev_dbg(&dev->dev, "Ignoring watch at %s\n",
> + vec[XS_WATCH_PATH]);
> return;
> }
>
> state = xenbus_read_driver_state(dev->otherend);
>
> - dev_dbg(&dev->dev, "state is %d, (%s), %s, %s",
> + dev_dbg(&dev->dev, "state is %d, (%s), %s, %s\n",
> state, xenbus_strstate(state), dev->otherend_watch.node,
> vec[XS_WATCH_PATH]);
>
> diff --git a/sound/arm/aaci.c b/sound/arm/aaci.c
> index b9eca9f..3b73ba7 100644
> --- a/sound/arm/aaci.c
> +++ b/sound/arm/aaci.c
> @@ -209,7 +209,7 @@ static void aaci_fifo_irq(struct aaci *aaci, int channel, u32 mask)
> void *ptr;
>
> if (!aacirun->substream || !aacirun->start) {
> - dev_warn(&aaci->dev->dev, "RX interrupt???");
> + dev_warn(&aaci->dev->dev, "RX interrupt???\n");
> writel(0, aacirun->base + AACI_IE);
> return;
> }
> @@ -263,7 +263,7 @@ static void aaci_fifo_irq(struct aaci *aaci, int channel, u32 mask)
> void *ptr;
>
> if (!aacirun->substream || !aacirun->start) {
> - dev_warn(&aaci->dev->dev, "TX interrupt???");
> + dev_warn(&aaci->dev->dev, "TX interrupt???\n");
> writel(0, aacirun->base + AACI_IE);
> return;
> }
>
>
>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists