Development: How to submit patches: Difference between revisions

From LinuxTVWiki
Jump to navigation Jump to search
Line 8: Line 8:


Follow the guidelines in [http://lxr.linux.no/source/Documentation/SubmittingPatches Documentation/SubmittingPatches] (cf. [http://linux.yyz.us/patch-format.html jgarzik's version]), including:
Follow the guidelines in [http://lxr.linux.no/source/Documentation/SubmittingPatches Documentation/SubmittingPatches] (cf. [http://linux.yyz.us/patch-format.html jgarzik's version]), including:
:* Verify best-practice [[Documentation/CodingStyle | coding style]]
:* Verify best-practice kernel [[Documentation/CodingStyle | coding style]]
:* Use [PATCH] in the subject line to get attention
:* Use [PATCH] in the subject line to get attention
:* Explain what the patch does and what hardware it applies to
:* Explain what the patch does and what hardware it applies to

Revision as of 05:24, 21 March 2006

Patch Preparation

Patches should be created against the v4l-dvb mercurial tree; see How to build from mercurial.

For references on how to develop a driver, take a look at Index of Documentation for People Interested in Writing and/or Understanding the Linux Kernel.

Post your patches to the video4linux mailing list for review and testing.

Follow the guidelines in Documentation/SubmittingPatches (cf. jgarzik's version), including:

  • Verify best-practice kernel coding style
  • Use [PATCH] in the subject line to get attention
  • Explain what the patch does and what hardware it applies to
  • Document your work where appropriate, in the form of patches to Documentation/video4linux files
  • Add a Signed-off-by: Your name <name@yoursite.com> as a Developer's Certificate of Origin 1.1
  • Fix any problems and repeat until everyone is happy ;)
  • Send the patch inline, not as an attachment

Subscription to the V4l mailing list is recommended but not required.

The patch will be applied to the main mercurial tree. Once tested and integrated, patches are periodically sent upstream to the linux kernel mailing list.