> -----Original Message----- > From: Hans Verkuil [mailto:hverkuil@xxxxxxxxx] > Sent: Wednesday, March 17, 2010 9:29 AM > To: Pawel Osciak > Cc: Aguirre, Sergio; linux-media@xxxxxxxxxxxxxxx; Marek Szyprowski; > kyungmin.park@xxxxxxxxxxx > Subject: RE: [PATCH v2] v4l: videobuf: code cleanup. > > > >> Aguirre, Sergio wrote: > >>> 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 :) > > > > Hm, right... > > I'm not an expert either, but it does seem reasonable. It was just a > part > > of the > > roadmap we agreed on in Norway, so I simply went ahead with it. Merging > > with other > > patches would pollute them so I just posted it separately. I will leave > > the > > decision up to Mauro then. I have some more "normal" patches lined up, > > so please let me know. I'm guessing we are cancelling the clean-up then > > though. It wasn't my intention to cancel your effort :) Please don't give up because of my comment. > > As I said, you give up way too easily. There are good reasons for doing a > simple straightforward cleanup patch first before tackling all the more > complex issues. Let's get this in first, then the future patches will only > do the actual functional changes instead of them having to do codingstyle > cleanups at the same time. I want to avoid that. Sounds reasonable. I wont say naything more about the topic. I think you guys have cleared it enough for me :) 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