<feed xmlns='http://www.w3.org/2005/Atom'>
<title>user/sven/linux.git/include/drm/drm_encoder.h, branch v4.11</title>
<subtitle>Linux Kernel
</subtitle>
<id>https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=v4.11</id>
<link rel='self' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=v4.11'/>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/'/>
<updated>2017-01-26T09:49:09Z</updated>
<entry>
<title>drm/doc: Fix typos for early_unregister doc</title>
<updated>2017-01-26T09:49:09Z</updated>
<author>
<name>Daniel Vetter</name>
<email>daniel.vetter@ffwll.ch</email>
</author>
<published>2017-01-25T06:26:55Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=559bdaf729bfd4e112c3efce95eb3c894288d281'/>
<id>urn:sha1:559bdaf729bfd4e112c3efce95eb3c894288d281</id>
<content type='text'>
There's no late_unregister. While at it switch to the new canonical
reference style.

Cc: Chris Wilson &lt;chris@chris-wilson.co.uk&gt;
Acked-by: Chris Wilson &lt;chris@chris-wilson.co.uk&gt;
Signed-off-by: Daniel Vetter &lt;daniel.vetter@intel.com&gt;
Link: http://patchwork.freedesktop.org/patch/msgid/20170125062657.19270-14-daniel.vetter@ffwll.ch
</content>
</entry>
<entry>
<title>drm: Fix compilation warning caused by static inline forward declaration</title>
<updated>2016-12-18T10:59:30Z</updated>
<author>
<name>Laurent Pinchart</name>
<email>laurent.pinchart+renesas@ideasonboard.com</email>
</author>
<published>2016-11-28T19:03:49Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=2407d1dc04789bfb2f39d248644981d2233feb0e'/>
<id>urn:sha1:2407d1dc04789bfb2f39d248644981d2233feb0e</id>
<content type='text'>
The drm_crtc_mask() function used in &lt;drm/drm_encoder.h&gt; is a static
inline defined in &lt;drm/drm_crtc.h&gt;. If the first header is included in a
compilation unit without the second one, the following compilation
warning will be issued.

In file included from &lt;linux&gt;/drivers/gpu/drm/drm_bridge.c:29:0:
&lt;linux&gt;/include/drm/drm_encoder.h:192:95: warning: ‘drm_crtc_mask’ used but never defined
 static inline uint32_t drm_crtc_mask(const struct drm_crtc *crtc);

Fix this by including the header defining the function instead of using
a forward declaration.

Signed-off-by: Laurent Pinchart &lt;laurent.pinchart+renesas@ideasonboard.com&gt;
Reviewed-by: Daniel Vetter &lt;daniel.vetter@ffwll.ch&gt;
Signed-off-by: Archit Taneja &lt;architt@codeaurora.org&gt;
Link: http://patchwork.freedesktop.org/patch/msgid/1481709550-29226-3-git-send-email-laurent.pinchart+renesas@ideasonboard.com
</content>
</entry>
<entry>
<title>drm: Don't include &lt;drm/drm_encoder.h&gt; in &lt;drm/drm_crtc.h&gt;</title>
<updated>2016-12-18T10:59:29Z</updated>
<author>
<name>Laurent Pinchart</name>
<email>laurent.pinchart+renesas@ideasonboard.com</email>
</author>
<published>2016-11-28T18:51:09Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=9338203c4f03ffe323b67f0b2fa17b9811fa9bb6'/>
<id>urn:sha1:9338203c4f03ffe323b67f0b2fa17b9811fa9bb6</id>
<content type='text'>
&lt;drm/drm_crtc.h&gt; used to define most of the in-kernel KMS API. It has
now been split into separate files for each object type, but still
includes most other KMS headers to avoid breaking driver compilation.

As a step towards fixing that problem, remove the inclusion of
&lt;drm/drm_encoder.h&gt; from &lt;drm/drm_crtc.h&gt; and include it instead where
appropriate. Also remove the forward declarations of the drm_encoder and
drm_encoder_helper_funcs structures from &lt;drm/drm_crtc.h&gt; as they're not
needed in the header.

&lt;drm/drm_encoder.h&gt; now has to include &lt;drm/drm_mode.h&gt; and contain a
forward declaration of struct drm_encoder in order to allow including it
as the first header in a compilation unit.

Signed-off-by: Laurent Pinchart &lt;laurent.pinchart+renesas@ideasonboard.com&gt;
Reviewed-by: Daniel Vetter &lt;daniel.vetter@ffwll.ch&gt;
Reviewed-by: Sinclair Yeh &lt;syeh@vmware.com&gt; # For vmwgfx
Reviewed-by: Alex Deucher &lt;alexander.deucher@amd.com&gt;
Signed-off-by: Archit Taneja &lt;architt@codeaurora.org&gt;
Link: http://patchwork.freedesktop.org/patch/msgid/1481709550-29226-2-git-send-email-laurent.pinchart+renesas@ideasonboard.com
</content>
</entry>
<entry>
<title>drm/crtc: constify drm_crtc_mask parameter</title>
<updated>2016-10-17T06:12:05Z</updated>
<author>
<name>Maarten Lankhorst</name>
<email>maarten.lankhorst@linux.intel.com</email>
</author>
<published>2016-10-13T08:38:11Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=1dfdb0ed34b28b086ffc1f9fb9f538ab60298029'/>
<id>urn:sha1:1dfdb0ed34b28b086ffc1f9fb9f538ab60298029</id>
<content type='text'>
Now that drm_crtc_index takes a const, the same can be done for drm_crtc_mask.

Signed-off-by: Maarten Lankhorst &lt;maarten.lankhorst@linux.intel.com&gt;
Reviewed-by: Jani Nikula &lt;jani.nikula@intel.com&gt;
Signed-off-by: Daniel Vetter &lt;daniel.vetter@ffwll.ch&gt;
Link: http://patchwork.freedesktop.org/patch/msgid/6e32d013-6fee-34ad-f8d2-59139f3dc4c1@linux.intel.com
</content>
</entry>
<entry>
<title>drm: Move a few macros away from drm_crtc.h</title>
<updated>2016-09-19T13:03:46Z</updated>
<author>
<name>Daniel Vetter</name>
<email>daniel.vetter@ffwll.ch</email>
</author>
<published>2016-08-31T16:09:04Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=afb21ea63d815d05f6081ee3efef6772a16317eb'/>
<id>urn:sha1:afb21ea63d815d05f6081ee3efef6772a16317eb</id>
<content type='text'>
Now that there's less stuff in there I noticed that I overlooked them.
Sprinkle some docs over them while at it.

Reviewed-by: Sean Paul &lt;seanpaul@chromium.org&gt;
Signed-off-by: Daniel Vetter &lt;daniel.vetter@intel.com&gt;
Link: http://patchwork.freedesktop.org/patch/msgid/20160831160913.12991-2-daniel.vetter@ffwll.ch
</content>
</entry>
<entry>
<title>drm: Extract drm_mode_object.[hc]</title>
<updated>2016-08-29T13:37:13Z</updated>
<author>
<name>Daniel Vetter</name>
<email>daniel.vetter@ffwll.ch</email>
</author>
<published>2016-08-29T08:27:51Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=949619f32eee37a6385de1e976523501c8256768'/>
<id>urn:sha1:949619f32eee37a6385de1e976523501c8256768</id>
<content type='text'>
Just for the struct drm_mode_object base class. The header file was
already partially extracted to help untangle the include loops.

v2:
- Also move the generic get/set property ioctls. At first this seemed
  like a bad idea since it requires making drm_mode_crtc_set_obj_prop
  non-static. But eventually that will get split away too (like
  the connector version already is) for both crtc and planes. Hence I
  reconsidered.

- drm_mode_object.[hc] instead of drm_modeset.[hc], which requires
  renaming the drm_modeset.h header I already started building up.
  This is more consistent (matches the name of the main structure),
  and I want to be able to use drm_modeset.[hc] for the basic modeset
  init/cleanup functionality like drm_mode_config_init.

Reviewed-by: Archit Taneja &lt;architt@codeaurora.org&gt;
Signed-off-by: Daniel Vetter &lt;daniel.vetter@intel.com&gt;
Link: http://patchwork.freedesktop.org/patch/msgid/20160829082757.17913-3-daniel.vetter@ffwll.ch
</content>
</entry>
<entry>
<title>drm/doc: Polish kerneldoc for encoders</title>
<updated>2016-08-29T13:37:08Z</updated>
<author>
<name>Daniel Vetter</name>
<email>daniel.vetter@ffwll.ch</email>
</author>
<published>2016-08-29T08:27:50Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=e03e6de03e931bd3d93b1e2a9dc7d0bf0b505287'/>
<id>urn:sha1:e03e6de03e931bd3d93b1e2a9dc7d0bf0b505287</id>
<content type='text'>
- Move missing bits into struct drm_encoder docs.
- Explain that encoders are 95% internal and only 5% uapi, and that in
  general the uapi part is broken.
- Remove verbose comments for functions not exposed to drivers.

v2: Review from Archit:
- Appease checkpatch in the moved code.
- Make it clearer that bridges are not exposed to userspace.

Reviewed-by: Archit Taneja &lt;architt@codeaurora.org&gt;
Signed-off-by: Daniel Vetter &lt;daniel.vetter@intel.com&gt;
Link: http://patchwork.freedesktop.org/patch/msgid/20160829082757.17913-2-daniel.vetter@ffwll.ch
</content>
</entry>
<entry>
<title>drm: Extract drm_encoder.[hc]</title>
<updated>2016-08-29T13:37:03Z</updated>
<author>
<name>Daniel Vetter</name>
<email>daniel.vetter@ffwll.ch</email>
</author>
<published>2016-08-29T08:27:49Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=321a95ae35f2ec4f58c4cda28d3606cc6f2b97d1'/>
<id>urn:sha1:321a95ae35f2ec4f58c4cda28d3606cc6f2b97d1</id>
<content type='text'>
Same treatment as before. Only hiccup is drm_crtc_mask, which
unfortunately can't be resolved until drm_crtc.h is less of a monster.
Untangle the header loop with a forward declaration for that static
inline.

Reviewed-by: Archit Taneja &lt;architt@codeaurora.org&gt;
Signed-off-by: Daniel Vetter &lt;daniel.vetter@intel.com&gt;
Link: http://patchwork.freedesktop.org/patch/msgid/20160829082757.17913-1-daniel.vetter@ffwll.ch
</content>
</entry>
</feed>
