> -----Original Message----- > From: Hans Verkuil [mailto:hverkuil@xxxxxxxxx] > Sent: Wednesday, March 17, 2010 9:26 AM > To: Aguirre, Sergio > Cc: Pawel Osciak; linux-media@xxxxxxxxxxxxxxx; m.szyprowski@xxxxxxxxxxx; > kyungmin.park@xxxxxxxxxxx > Subject: RE: [PATCH v2] v4l: videobuf: code cleanup. > > > > Hi, > > > >> -----Original Message----- > >> From: linux-media-owner@xxxxxxxxxxxxxxx [mailto:linux-media- > >> owner@xxxxxxxxxxxxxxx] On Behalf Of Pawel Osciak > >> Sent: Wednesday, March 17, 2010 8:04 AM > >> To: linux-media@xxxxxxxxxxxxxxx > >> Cc: p.osciak@xxxxxxxxxxx; m.szyprowski@xxxxxxxxxxx; > >> kyungmin.park@xxxxxxxxxxx > >> Subject: [PATCH v2] v4l: videobuf: code cleanup. > >> > >> Make videobuf pass checkpatch; minor code cleanups. > > > > I thought this kind patches were frowned upon.. > > > > http://www.mjmwired.net/kernel/Documentation/development- > process/4.Coding#41 > > > > But maybe it's acceptable in this case... I'm not an expert on community > > policies :) > > It is true that you shouldn't do this 'just to clean up code'. But in this > case we want to do a lot of work on the videobuf framework, and it helps a > lot if it is first brought up to date with the coding standards. > > It's just step one in a much longer process :-) Ok, understood. No problem with that at all... I just wanted clarification on this coding policy. :) Regards, Sergio > > Regards, > > Hans > > -- > Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html