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

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

 



Pawel Osciak 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 is fine for me to send such patch in a series of changes. A pure CodingStyle patch
is preferred if you're doing lots of changes, since it is very easy to review those
changes. Yet, I generally hold pure CodingStyle changes to happen at the end of an
rc cycle, to avoid conflicts with real patches, especially when the change is on a
code that use to have lots of changes during a kernel cycle.

In the specific case of videobuf, I prefer to merge any changes functional changes at the
beginning of a -rc cycle, and after having several tested-by replies with different
architectures and boards, as a trouble there will affect almost all drivers.

Cheers,
Mauro
--
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