8676df5030
freedesktop.org has adopted a formal&enforced code of conduct: https://www.fooishbar.org/blog/fdo-contributor-covenant/ https://www.freedesktop.org/wiki/CodeOfConduct/ Besides formalizing things a bit more I don't think this changes anything for us, we've already peer-enforced respectful and constructive interactions since a long time. But it's good to document things properly. v2: Drop confusing note from commit message and clarify the grammer (Chris, Alex and others). Cc: Daniel Stone <daniels@collabora.com> Cc: Keith Packard <keithp@keithp.com> Cc: tfheen@err.no Signed-off-by: Daniel Vetter <daniel.vetter@intel.com> Reviewed-by: Daniel Stone <daniels@collabora.com> Reviewed-by: Sumit Semwal <sumit.semwal@linaro.org> Acked-by: Archit Taneja <architt@codeaurora.org> Reviewed-by: Martin Peres <martin.peres@free.fr> Acked-by: Thierry Reding <treding@nvidia.com> Acked-by: Jani Nikula <jani.nikula@intel.com> Acked-by: Vincent Abriou <vincent.abriou@st.com> Acked-by: Neil Armstrong <narmstrong@baylibre.com> Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Acked-by: Brian Starkey <brian.starkey@arm.com> Acked-by: Rob Clark <robdclark@gmail.com> Reviewed-by: David Herrmann <dh.herrmann@gmail.com> Acked-by: Sean Paul <seanpaul@chromium.org> Reviewed-by: Harry Wentland <harry.wentland@amd.com> Reviewed-by: Eric Anholt <eric@anholt.net> Acked-by: Alex Deucher <alexander.deucher@amd.com> Acked-by: Gustavo Padovan <gustavo.padovan@collabora.com> Acked-by: Michel Dänzer <michel.daenzer@amd.com> Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Acked-by: Sumit Semwal <sumit.semwal@linaro.org> Acked-by: Keith Packard <keithp@keithp.com> Acked-by: Gabriel Krisman Bertazi <krisman@collabora.co.uk> Acked-by: Adam Jackson <ajax@redhat.com> Signed-off-by: Dave Airlie <airlied@redhat.com>
99 lines
4.5 KiB
ReStructuredText
99 lines
4.5 KiB
ReStructuredText
============
|
|
Introduction
|
|
============
|
|
|
|
The Linux DRM layer contains code intended to support the needs of
|
|
complex graphics devices, usually containing programmable pipelines well
|
|
suited to 3D graphics acceleration. Graphics drivers in the kernel may
|
|
make use of DRM functions to make tasks like memory management,
|
|
interrupt handling and DMA easier, and provide a uniform interface to
|
|
applications.
|
|
|
|
A note on versions: this guide covers features found in the DRM tree,
|
|
including the TTM memory manager, output configuration and mode setting,
|
|
and the new vblank internals, in addition to all the regular features
|
|
found in current kernels.
|
|
|
|
[Insert diagram of typical DRM stack here]
|
|
|
|
Style Guidelines
|
|
================
|
|
|
|
For consistency this documentation uses American English. Abbreviations
|
|
are written as all-uppercase, for example: DRM, KMS, IOCTL, CRTC, and so
|
|
on. To aid in reading, documentations make full use of the markup
|
|
characters kerneldoc provides: @parameter for function parameters,
|
|
@member for structure members (within the same structure), &struct structure to
|
|
reference structures and function() for functions. These all get automatically
|
|
hyperlinked if kerneldoc for the referenced objects exists. When referencing
|
|
entries in function vtables (and structure members in general) please use
|
|
&vtable_name.vfunc. Unfortunately this does not yet yield a direct link to the
|
|
member, only the structure.
|
|
|
|
Except in special situations (to separate locked from unlocked variants)
|
|
locking requirements for functions aren't documented in the kerneldoc.
|
|
Instead locking should be check at runtime using e.g.
|
|
``WARN_ON(!mutex_is_locked(...));``. Since it's much easier to ignore
|
|
documentation than runtime noise this provides more value. And on top of
|
|
that runtime checks do need to be updated when the locking rules change,
|
|
increasing the chances that they're correct. Within the documentation
|
|
the locking rules should be explained in the relevant structures: Either
|
|
in the comment for the lock explaining what it protects, or data fields
|
|
need a note about which lock protects them, or both.
|
|
|
|
Functions which have a non-\ ``void`` return value should have a section
|
|
called "Returns" explaining the expected return values in different
|
|
cases and their meanings. Currently there's no consensus whether that
|
|
section name should be all upper-case or not, and whether it should end
|
|
in a colon or not. Go with the file-local style. Other common section
|
|
names are "Notes" with information for dangerous or tricky corner cases,
|
|
and "FIXME" where the interface could be cleaned up.
|
|
|
|
Also read the :ref:`guidelines for the kernel documentation at large <doc_guide>`.
|
|
|
|
Getting Started
|
|
===============
|
|
|
|
Developers interested in helping out with the DRM subsystem are very welcome.
|
|
Often people will resort to sending in patches for various issues reported by
|
|
checkpatch or sparse. We welcome such contributions.
|
|
|
|
Anyone looking to kick it up a notch can find a list of janitorial tasks on
|
|
the :ref:`TODO list <todo>`.
|
|
|
|
Contribution Process
|
|
====================
|
|
|
|
Mostly the DRM subsystem works like any other kernel subsystem, see :ref:`the
|
|
main process guidelines and documentation <process_index>` for how things work.
|
|
Here we just document some of the specialities of the GPU subsystem.
|
|
|
|
Feature Merge Deadlines
|
|
-----------------------
|
|
|
|
All feature work must be in the linux-next tree by the -rc6 release of the
|
|
current release cycle, otherwise they must be postponed and can't reach the next
|
|
merge window. All patches must have landed in the drm-next tree by latest -rc7,
|
|
but if your branch is not in linux-next then this must have happened by -rc6
|
|
already.
|
|
|
|
After that point only bugfixes (like after the upstream merge window has closed
|
|
with the -rc1 release) are allowed. No new platform enabling or new drivers are
|
|
allowed.
|
|
|
|
This means that there's a blackout-period of about one month where feature work
|
|
can't be merged. The recommended way to deal with that is having a -next tree
|
|
that's always open, but making sure to not feed it into linux-next during the
|
|
blackout period. As an example, drm-misc works like that.
|
|
|
|
Code of Conduct
|
|
---------------
|
|
|
|
As a freedesktop.org project, dri-devel, and the DRM community, follows the
|
|
Contributor Covenant, found at: https://www.freedesktop.org/wiki/CodeOfConduct
|
|
|
|
Please conduct yourself in a respectful and civilised manner when
|
|
interacting with community members on mailing lists, IRC, or bug
|
|
trackers. The community represents the project as a whole, and abusive
|
|
or bullying behaviour is not tolerated by the project.
|