RE: [PATCH v2] v4l: videobuf: code cleanup.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> -----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

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux