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
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date:	Tue, 23 Mar 2010 14:06:55 +0300
From:	Dan Carpenter <error27@...il.com>
To:	Olimpiu Pascariu <olimpiu.pascariu@...il.com>, gregkh@...e.de,
	mchehab@...hat.com, palash.bandyopadhyay@...exant.com,
	devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org,
	linux-media@...r.kernel.org
Subject: Re: [PATCH 5/5] Staging: cx25821: fix coding style issues in
	cx25821-medusa-video.c

On Tue, Mar 23, 2010 at 02:03:00PM +0300, Dan Carpenter wrote:
> On Mon, Mar 22, 2010 at 10:10:30PM +0200, Olimpiu Pascariu wrote:
> > >From 32591165a537a03f472c68289798044d6eeea2e0 Mon Sep 17 00:00:00 2001
> > From: Olimpiu Pascariu <olimpiu.pascariu@...il.com>
> > Date: Mon, 22 Mar 2010 22:07:20 +0200
> > Subject: [PATCH 5/5] Staging: cx25821: fix coding style issues in cx25821-medusa-video.c
> >  This is a patch to cx25821-medusa-video.c file that fixes up warnings and errors found by the checkpatch.pl tool
> >  Signed-off-by: Olimpiu Pascariu <olimpiu.pascariu@...il.com>
> 
> 	[snip]
> 
> > +		/*
> > +		 * clear VPRES_VERT_EN bit, fixes the chroma run away problem
> > +		 * when the input switching rate < 16 fields
> > +		*/
>                ^^^
> 	Missing a space here.
> 
> Otherwise looks good.
> 
> Acked-by: Dan Carpenter <error27@...il.com>
> 
> regard,
> dan carpenter

Oh sorry.  I thought this was a new patch.  This is fine to go in as is
by me...  Maybe hold off for a couple days and see if anyone else has
any comments.

regards,
dan carpenter  
--
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

Powered by Openwall GNU/*/Linux Powered by OpenVZ