linux-kernel - Re: [PATCH 10/10] Staging: comedi: fix brace printk() issue in pcl818.c

Open Source and information security mailing list archives

Date:	Mon, 11 Jul 2011 10:20:29 +0200
From:	Tobias Klauser <tklauser@...tanz.ch>
To:	Ravishankar <ravishankarkm32@...il.com>
Cc:	gregkh@...e.de, wfp5p@...ginia.edu, devel@...verdev.osuosl.org,
	linux-kernel@...r.kernel.org,
	Ravishankar <ravi.shankar@...enturtles.in>
Subject: Re: [PATCH 10/10] Staging: comedi: fix brace printk() issue in
	pcl818.c

On 2011-07-11 at 09:51:52 +0200, Ravishankar <ravishankarkm32@...il.com> wrote:
> From: Ravishankar <ravi.shankar@...enturtles.in>
> 
> This is a patch to the pcl818.c file that fixes up a printk() warning found by the checkpatch.pl tool
> 
> Signed-off-by: Ravishankar <ravishankarkm32@...il.com>
> ---
>  drivers/staging/comedi/drivers/pcl818.c |    8 ++++----
>  1 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/staging/comedi/drivers/pcl818.c b/drivers/staging/comedi/drivers/pcl818.c
> index 2fdb8c9..4069fee 100644
> --- a/drivers/staging/comedi/drivers/pcl818.c
> +++ b/drivers/staging/comedi/drivers/pcl818.c
> @@ -1777,7 +1777,7 @@ static int pcl818_attach(struct comedi_device *dev, struct comedi_devconfig *it)
>  					     irq);
>  					irq = 0;	/* Can't use IRQ */
>  				} else {
> -					printk(", irq=%u", irq);
> +					printk(KERN_INFO ", irq=%u", irq);

This should be KERN_CONT.

>  				}
>  			}
>  		}
> @@ -1883,7 +1883,7 @@ no_dma:
>  		if (check_single_ended(dev->iobase)) {
>  			s->n_chan = this_board->n_aichan_se;
>  			s->subdev_flags |= SDF_COMMON | SDF_GROUND;
> -			printk(", %dchans S.E. DAC", s->n_chan);
> +			printk(KERN_INFO ", %dchans S.E. DAC", s->n_chan);

Ditto.

>  		} else {
>  			s->n_chan = this_board->n_aichan_diff;
>  			s->subdev_flags |= SDF_DIFF;
> -- 
> 1.6.5.2
> 
> _______________________________________________
> devel mailing list
> devel@...uxdriverproject.org
> http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
> 
--
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/

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives