> Aguirre, Sergio wrote: >> >> 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 :) Come on guys, I really do not give up that easily, I just went on with more important patches. I am just a very agreeable person, that's all :) Best regards -- Pawel Osciak Linux Platform Group Samsung Poland R&D Center -- 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