2008-11-08 05:05:41 +07:00
|
|
|
/*
|
|
|
|
* Copyright © 2006 Keith Packard
|
|
|
|
* Copyright © 2007-2008 Dave Airlie
|
|
|
|
* Copyright © 2007-2008 Intel Corporation
|
|
|
|
* Jesse Barnes <jesse.barnes@intel.com>
|
|
|
|
*
|
|
|
|
* Permission is hereby granted, free of charge, to any person obtaining a
|
|
|
|
* copy of this software and associated documentation files (the "Software"),
|
|
|
|
* to deal in the Software without restriction, including without limitation
|
|
|
|
* the rights to use, copy, modify, merge, publish, distribute, sublicense,
|
|
|
|
* and/or sell copies of the Software, and to permit persons to whom the
|
|
|
|
* Software is furnished to do so, subject to the following conditions:
|
|
|
|
*
|
|
|
|
* The above copyright notice and this permission notice shall be included in
|
|
|
|
* all copies or substantial portions of the Software.
|
|
|
|
*
|
|
|
|
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
|
|
|
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
|
|
|
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
|
|
|
|
* THE COPYRIGHT HOLDER(S) OR AUTHOR(S) BE LIABLE FOR ANY CLAIM, DAMAGES OR
|
|
|
|
* OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
|
|
|
|
* ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
|
|
|
|
* OTHER DEALINGS IN THE SOFTWARE.
|
|
|
|
*/
|
|
|
|
#ifndef __DRM_CRTC_H__
|
|
|
|
#define __DRM_CRTC_H__
|
|
|
|
|
|
|
|
#include <linux/i2c.h>
|
|
|
|
#include <linux/spinlock.h>
|
|
|
|
#include <linux/types.h>
|
|
|
|
#include <linux/idr.h>
|
|
|
|
#include <linux/fb.h>
|
2013-12-19 17:04:07 +07:00
|
|
|
#include <linux/hdmi.h>
|
2014-07-22 17:09:10 +07:00
|
|
|
#include <linux/media-bus-format.h>
|
2014-08-29 17:12:40 +07:00
|
|
|
#include <uapi/drm/drm_mode.h>
|
|
|
|
#include <uapi/drm/drm_fourcc.h>
|
2013-11-20 00:10:12 +07:00
|
|
|
#include <drm/drm_modeset_lock.h>
|
2011-11-15 05:51:28 +07:00
|
|
|
|
2008-11-08 05:05:41 +07:00
|
|
|
struct drm_device;
|
|
|
|
struct drm_mode_set;
|
|
|
|
struct drm_framebuffer;
|
2012-05-16 04:09:01 +07:00
|
|
|
struct drm_object_properties;
|
2012-11-21 21:00:47 +07:00
|
|
|
struct drm_file;
|
|
|
|
struct drm_clip_rect;
|
2014-06-15 17:07:12 +07:00
|
|
|
struct device_node;
|
2014-10-29 17:34:56 +07:00
|
|
|
struct fence;
|
2008-11-08 05:05:41 +07:00
|
|
|
|
|
|
|
#define DRM_MODE_OBJECT_CRTC 0xcccccccc
|
|
|
|
#define DRM_MODE_OBJECT_CONNECTOR 0xc0c0c0c0
|
|
|
|
#define DRM_MODE_OBJECT_ENCODER 0xe0e0e0e0
|
|
|
|
#define DRM_MODE_OBJECT_MODE 0xdededede
|
|
|
|
#define DRM_MODE_OBJECT_PROPERTY 0xb0b0b0b0
|
|
|
|
#define DRM_MODE_OBJECT_FB 0xfbfbfbfb
|
|
|
|
#define DRM_MODE_OBJECT_BLOB 0xbbbbbbbb
|
2011-11-15 05:51:27 +07:00
|
|
|
#define DRM_MODE_OBJECT_PLANE 0xeeeeeeee
|
2014-05-30 22:37:03 +07:00
|
|
|
#define DRM_MODE_OBJECT_ANY 0
|
2008-11-08 05:05:41 +07:00
|
|
|
|
|
|
|
struct drm_mode_object {
|
|
|
|
uint32_t id;
|
|
|
|
uint32_t type;
|
2012-05-16 04:09:01 +07:00
|
|
|
struct drm_object_properties *properties;
|
|
|
|
};
|
|
|
|
|
2012-06-12 21:27:01 +07:00
|
|
|
#define DRM_OBJECT_MAX_PROPERTY 24
|
2012-05-16 04:09:01 +07:00
|
|
|
struct drm_object_properties {
|
2014-12-19 04:01:50 +07:00
|
|
|
int count, atomic_count;
|
2014-12-17 06:05:30 +07:00
|
|
|
/* NOTE: if we ever start dynamically destroying properties (ie.
|
|
|
|
* not at drm_mode_config_cleanup() time), then we'd have to do
|
|
|
|
* a better job of detaching property from mode objects to avoid
|
|
|
|
* dangling property pointers:
|
|
|
|
*/
|
|
|
|
struct drm_property *properties[DRM_OBJECT_MAX_PROPERTY];
|
2014-12-17 06:05:31 +07:00
|
|
|
/* do not read/write values directly, but use drm_object_property_get_value()
|
|
|
|
* and drm_object_property_set_value():
|
|
|
|
*/
|
2012-05-16 04:09:01 +07:00
|
|
|
uint64_t values[DRM_OBJECT_MAX_PROPERTY];
|
2008-11-08 05:05:41 +07:00
|
|
|
};
|
|
|
|
|
2012-09-13 10:22:31 +07:00
|
|
|
static inline int64_t U642I64(uint64_t val)
|
|
|
|
{
|
|
|
|
return (int64_t)*((int64_t *)&val);
|
|
|
|
}
|
|
|
|
static inline uint64_t I642U64(int64_t val)
|
|
|
|
{
|
|
|
|
return (uint64_t)*((uint64_t *)&val);
|
|
|
|
}
|
|
|
|
|
2015-11-02 22:14:08 +07:00
|
|
|
/*
|
|
|
|
* Rotation property bits. DRM_ROTATE_<degrees> rotates the image by the
|
|
|
|
* specified amount in degrees in counter clockwise direction. DRM_REFLECT_X and
|
|
|
|
* DRM_REFLECT_Y reflects the image along the specified axis prior to rotation
|
|
|
|
*/
|
2015-10-01 14:00:57 +07:00
|
|
|
#define DRM_ROTATE_MASK 0x0f
|
2014-07-08 12:01:51 +07:00
|
|
|
#define DRM_ROTATE_0 0
|
|
|
|
#define DRM_ROTATE_90 1
|
|
|
|
#define DRM_ROTATE_180 2
|
|
|
|
#define DRM_ROTATE_270 3
|
2015-10-01 14:00:57 +07:00
|
|
|
#define DRM_REFLECT_MASK (~DRM_ROTATE_MASK)
|
2014-07-08 12:01:51 +07:00
|
|
|
#define DRM_REFLECT_X 4
|
|
|
|
#define DRM_REFLECT_Y 5
|
|
|
|
|
2014-01-23 21:52:20 +07:00
|
|
|
enum drm_connector_force {
|
|
|
|
DRM_FORCE_UNSPECIFIED,
|
|
|
|
DRM_FORCE_OFF,
|
|
|
|
DRM_FORCE_ON, /* force on analog part normally */
|
|
|
|
DRM_FORCE_ON_DIGITAL, /* for DVI-I use digital connector */
|
2008-11-08 05:05:41 +07:00
|
|
|
};
|
|
|
|
|
2014-01-23 21:52:20 +07:00
|
|
|
#include <drm/drm_modes.h>
|
2013-09-25 22:45:21 +07:00
|
|
|
|
2008-11-08 05:05:41 +07:00
|
|
|
enum drm_connector_status {
|
|
|
|
connector_status_connected = 1,
|
|
|
|
connector_status_disconnected = 2,
|
|
|
|
connector_status_unknown = 3,
|
|
|
|
};
|
|
|
|
|
|
|
|
enum subpixel_order {
|
|
|
|
SubPixelUnknown = 0,
|
|
|
|
SubPixelHorizontalRGB,
|
|
|
|
SubPixelHorizontalBGR,
|
|
|
|
SubPixelVerticalRGB,
|
|
|
|
SubPixelVerticalBGR,
|
|
|
|
SubPixelNone,
|
|
|
|
};
|
|
|
|
|
2011-04-16 03:48:57 +07:00
|
|
|
#define DRM_COLOR_FORMAT_RGB444 (1<<0)
|
|
|
|
#define DRM_COLOR_FORMAT_YCRCB444 (1<<1)
|
|
|
|
#define DRM_COLOR_FORMAT_YCRCB422 (1<<2)
|
2008-11-08 05:05:41 +07:00
|
|
|
/*
|
|
|
|
* Describes a given display (e.g. CRT or flat panel) and its limitations.
|
|
|
|
*/
|
|
|
|
struct drm_display_info {
|
|
|
|
char name[DRM_DISPLAY_INFO_LEN];
|
2010-08-04 01:38:16 +07:00
|
|
|
|
2008-11-08 05:05:41 +07:00
|
|
|
/* Physical size */
|
|
|
|
unsigned int width_mm;
|
|
|
|
unsigned int height_mm;
|
|
|
|
|
|
|
|
/* Clock limits FIXME: storage format */
|
|
|
|
unsigned int min_vfreq, max_vfreq;
|
|
|
|
unsigned int min_hfreq, max_hfreq;
|
|
|
|
unsigned int pixel_clock;
|
2011-04-16 02:49:23 +07:00
|
|
|
unsigned int bpc;
|
2008-11-08 05:05:41 +07:00
|
|
|
|
|
|
|
enum subpixel_order subpixel_order;
|
2011-04-16 03:48:57 +07:00
|
|
|
u32 color_formats;
|
2008-11-08 05:05:41 +07:00
|
|
|
|
2014-07-22 17:09:10 +07:00
|
|
|
const u32 *bus_formats;
|
|
|
|
unsigned int num_bus_formats;
|
|
|
|
|
2014-06-05 20:52:10 +07:00
|
|
|
/* Mask of supported hdmi deep color modes */
|
|
|
|
u8 edid_hdmi_dc_modes;
|
|
|
|
|
2011-08-03 23:22:54 +07:00
|
|
|
u8 cea_rev;
|
2008-11-08 05:05:41 +07:00
|
|
|
};
|
|
|
|
|
2014-10-20 13:17:17 +07:00
|
|
|
/* data corresponds to displayid vend/prod/serial */
|
|
|
|
struct drm_tile_group {
|
|
|
|
struct kref refcount;
|
|
|
|
struct drm_device *dev;
|
|
|
|
int id;
|
|
|
|
u8 group_data[8];
|
|
|
|
};
|
|
|
|
|
2015-12-04 15:46:01 +07:00
|
|
|
/**
|
|
|
|
* struct drm_framebuffer_funcs - framebuffer hooks
|
|
|
|
*/
|
2008-11-08 05:05:41 +07:00
|
|
|
struct drm_framebuffer_funcs {
|
2015-12-04 15:46:01 +07:00
|
|
|
/**
|
|
|
|
* @destroy:
|
|
|
|
*
|
|
|
|
* Clean up framebuffer resources, specifically also unreference the
|
|
|
|
* backing storage. The core guarantees to call this function for every
|
|
|
|
* framebuffer successfully created by ->fb_create() in
|
2015-12-08 15:49:19 +07:00
|
|
|
* &drm_mode_config_funcs. Drivers must also call
|
|
|
|
* drm_framebuffer_cleanup() to release DRM core resources for this
|
|
|
|
* framebuffer.
|
2015-12-04 15:46:01 +07:00
|
|
|
*/
|
2008-11-08 05:05:41 +07:00
|
|
|
void (*destroy)(struct drm_framebuffer *framebuffer);
|
2015-12-04 15:46:01 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @create_handle:
|
|
|
|
*
|
|
|
|
* Create a buffer handle in the driver-specific buffer manager (either
|
|
|
|
* GEM or TTM) valid for the passed-in struct &drm_file. This is used by
|
|
|
|
* the core to implement the GETFB IOCTL, which returns (for
|
|
|
|
* sufficiently priviledged user) also a native buffer handle. This can
|
|
|
|
* be used for seamless transitions between modesetting clients by
|
|
|
|
* copying the current screen contents to a private buffer and blending
|
|
|
|
* between that and the new contents.
|
|
|
|
*
|
2015-12-08 15:49:19 +07:00
|
|
|
* GEM based drivers should call drm_gem_handle_create() to create the
|
|
|
|
* handle.
|
|
|
|
*
|
2015-12-04 15:46:01 +07:00
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 on success or a negative error code on failure.
|
|
|
|
*/
|
2008-11-08 05:05:41 +07:00
|
|
|
int (*create_handle)(struct drm_framebuffer *fb,
|
|
|
|
struct drm_file *file_priv,
|
|
|
|
unsigned int *handle);
|
2015-12-04 15:46:01 +07:00
|
|
|
/**
|
|
|
|
* @dirty:
|
2009-12-04 06:25:47 +07:00
|
|
|
*
|
2015-12-04 15:46:01 +07:00
|
|
|
* Optional callback for the dirty fb IOCTL.
|
2009-12-04 06:25:47 +07:00
|
|
|
*
|
2015-12-04 15:46:01 +07:00
|
|
|
* Userspace can notify the driver via this callback that an area of the
|
|
|
|
* framebuffer has changed and should be flushed to the display
|
|
|
|
* hardware. This can also be used internally, e.g. by the fbdev
|
|
|
|
* emulation, though that's not the case currently.
|
|
|
|
*
|
|
|
|
* See documentation in drm_mode.h for the struct drm_mode_fb_dirty_cmd
|
|
|
|
* for more information as all the semantics and arguments have a one to
|
|
|
|
* one mapping on this function.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 on success or a negative error code on failure.
|
2009-12-04 06:25:47 +07:00
|
|
|
*/
|
2010-10-05 17:43:02 +07:00
|
|
|
int (*dirty)(struct drm_framebuffer *framebuffer,
|
|
|
|
struct drm_file *file_priv, unsigned flags,
|
2009-12-04 06:25:47 +07:00
|
|
|
unsigned color, struct drm_clip_rect *clips,
|
|
|
|
unsigned num_clips);
|
2008-11-08 05:05:41 +07:00
|
|
|
};
|
|
|
|
|
|
|
|
struct drm_framebuffer {
|
|
|
|
struct drm_device *dev;
|
2012-09-06 04:48:38 +07:00
|
|
|
/*
|
|
|
|
* Note that the fb is refcounted for the benefit of driver internals,
|
|
|
|
* for example some hw, disabling a CRTC/plane is asynchronous, and
|
|
|
|
* scanout does not actually complete until the next vblank. So some
|
|
|
|
* cleanup (like releasing the reference(s) on the backing GEM bo(s))
|
|
|
|
* should be deferred. In cases like this, the driver would like to
|
|
|
|
* hold a ref to the fb even though it has already been removed from
|
|
|
|
* userspace perspective.
|
|
|
|
*/
|
|
|
|
struct kref refcount;
|
drm: revamp locking around fb creation/destruction
Well, at least step 1. The goal here is that framebuffer objects can
survive outside of the mode_config lock, with just a reference held
as protection. The first step to get there is to introduce a special
fb_lock which protects fb lookup, creation and destruction, to make
them appear atomic.
This new fb_lock can nest within the mode_config lock. But the idea is
(once the reference counting part is completed) that we only quickly
take that fb_lock to lookup a framebuffer and grab a reference,
without any other locks involved.
vmwgfx is the only driver which does framebuffer lookups itself, also
wrap those calls to drm_mode_object_find with the new lock.
Also protect the fb_list walking in i915 and omapdrm with the new lock.
As a slight complication there's also the list of user-created fbs
attached to the file private. The problem now is that at fclose() time
we need to walk that list, eventually do a modeset call to remove the
fb from active usage (and are required to be able to take the
mode_config lock), but in the end we need to grab the new fb_lock to
remove the fb from the list. The easiest solution is to add another
mutex to protect this per-file list.
Currently that new fbs_lock nests within the modeset locks and so
appears redudant. But later patches will switch around this sequence
so that taking the modeset locks in the fb destruction path is
optional in the fastpath. Ultimately the goal is that addfb and rmfb
do not require the mode_config lock, since otherwise they have the
potential to introduce stalls in the pageflip sequence of a compositor
(if the compositor e.g. switches to a fullscreen client or if it
enables a plane). But that requires a few more steps and hoops to jump
through.
Note that framebuffer creation/destruction is now double-protected -
once by the fb_lock and in parts by the idr_lock. The later would be
unnecessariy if framebuffers would have their own idr allocator. But
that's material for another patch (series).
v2: Properly initialize the fb->filp_head list in _init, otherwise the
newly added WARN to check whether the fb isn't on a fpriv list any
more will fail for driver-private objects.
v3: Fixup two error-case unlock bugs spotted by Richard Wilbur.
Reviewed-by: Rob Clark <rob@ti.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2012-12-11 03:19:18 +07:00
|
|
|
/*
|
|
|
|
* Place on the dev->mode_config.fb_list, access protected by
|
|
|
|
* dev->mode_config.fb_lock.
|
|
|
|
*/
|
2008-11-08 05:05:41 +07:00
|
|
|
struct list_head head;
|
|
|
|
struct drm_mode_object base;
|
|
|
|
const struct drm_framebuffer_funcs *funcs;
|
2011-12-20 05:06:49 +07:00
|
|
|
unsigned int pitches[4];
|
|
|
|
unsigned int offsets[4];
|
2015-02-05 21:41:52 +07:00
|
|
|
uint64_t modifier[4];
|
2008-11-08 05:05:41 +07:00
|
|
|
unsigned int width;
|
|
|
|
unsigned int height;
|
|
|
|
/* depth can be 15 or 16 */
|
|
|
|
unsigned int depth;
|
|
|
|
int bits_per_pixel;
|
|
|
|
int flags;
|
2011-11-15 05:51:28 +07:00
|
|
|
uint32_t pixel_format; /* fourcc format */
|
2008-11-08 05:05:41 +07:00
|
|
|
struct list_head filp_head;
|
|
|
|
};
|
|
|
|
|
|
|
|
struct drm_property_blob {
|
|
|
|
struct drm_mode_object base;
|
2015-04-21 01:22:55 +07:00
|
|
|
struct drm_device *dev;
|
|
|
|
struct kref refcount;
|
2015-05-22 19:34:51 +07:00
|
|
|
struct list_head head_global;
|
|
|
|
struct list_head head_file;
|
2014-05-13 16:36:13 +07:00
|
|
|
size_t length;
|
2012-03-13 17:35:49 +07:00
|
|
|
unsigned char data[];
|
2008-11-08 05:05:41 +07:00
|
|
|
};
|
|
|
|
|
|
|
|
struct drm_property_enum {
|
|
|
|
uint64_t value;
|
|
|
|
struct list_head head;
|
|
|
|
char name[DRM_PROP_NAME_LEN];
|
|
|
|
};
|
|
|
|
|
|
|
|
struct drm_property {
|
|
|
|
struct list_head head;
|
|
|
|
struct drm_mode_object base;
|
|
|
|
uint32_t flags;
|
|
|
|
char name[DRM_PROP_NAME_LEN];
|
|
|
|
uint32_t num_values;
|
|
|
|
uint64_t *values;
|
2014-05-30 22:37:03 +07:00
|
|
|
struct drm_device *dev;
|
2008-11-08 05:05:41 +07:00
|
|
|
|
2014-11-20 00:38:10 +07:00
|
|
|
struct list_head enum_list;
|
2008-11-08 05:05:41 +07:00
|
|
|
};
|
|
|
|
|
|
|
|
struct drm_crtc;
|
|
|
|
struct drm_connector;
|
|
|
|
struct drm_encoder;
|
2009-11-18 00:43:55 +07:00
|
|
|
struct drm_pending_vblank_event;
|
2011-11-15 05:51:27 +07:00
|
|
|
struct drm_plane;
|
2013-08-15 03:47:37 +07:00
|
|
|
struct drm_bridge;
|
2014-10-28 02:28:44 +07:00
|
|
|
struct drm_atomic_state;
|
|
|
|
|
2015-12-04 15:45:45 +07:00
|
|
|
struct drm_crtc_helper_funcs;
|
|
|
|
struct drm_encoder_helper_funcs;
|
|
|
|
struct drm_connector_helper_funcs;
|
|
|
|
struct drm_plane_helper_funcs;
|
|
|
|
|
2014-10-28 02:28:44 +07:00
|
|
|
/**
|
2014-07-26 02:30:38 +07:00
|
|
|
* struct drm_crtc_state - mutable CRTC state
|
2014-11-27 21:49:39 +07:00
|
|
|
* @crtc: backpointer to the CRTC
|
2014-10-28 02:28:44 +07:00
|
|
|
* @enable: whether the CRTC should be enabled, gates all other state
|
2014-11-26 22:57:41 +07:00
|
|
|
* @active: whether the CRTC is actively displaying (used for DPMS)
|
2015-07-21 18:28:57 +07:00
|
|
|
* @planes_changed: planes on this crtc are updated
|
|
|
|
* @mode_changed: crtc_state->mode or crtc_state->enable has been changed
|
|
|
|
* @active_changed: crtc_state->active has been toggled.
|
|
|
|
* @connectors_changed: connectors to this crtc have been updated
|
2014-11-22 03:28:31 +07:00
|
|
|
* @plane_mask: bitmask of (1 << drm_plane_index(plane)) of attached planes
|
2016-01-04 18:53:18 +07:00
|
|
|
* @connector_mask: bitmask of (1 << drm_connector_index(connector)) of attached connectors
|
drm: Atomic crtc/connector updates using crtc/plane helper interfaces
So this is finally the integration of the crtc and plane helper
interfaces into the atomic helper functions.
In the check function we now have a few steps:
- First we update the output routing and figure out which crtcs need a
full mode set. Suitable encoders are selected using ->best_encoder,
with the same semantics as the crtc helpers of implicitly disabling
all connectors currently using the encoder.
- Then we pull all other connectors into the state update which feed
from a crtc which changes. This must be done do catch mode changes
and similar updates - atomic updates are differences on top of the
current state.
- Then we call all the various ->mode_fixup to compute the adjusted
mode. Note that here we have a slight semantic difference compared
to the crtc helpers: We have not yet updated the encoder->crtc link
when calling the encoder's ->mode_fixup function. But that's a
requirement when converting to atomic since we want to prepare the
entire state completely contained with the over drm_atomic_state
structure. So this must be carefully checked when converting drivers
over to atomic helpers.
- Finally we do call the atomic_check functions on planes and crtcs.
The commit function is also quite a beast:
- The only step that can fail is done first, namely pinning the
framebuffers. After that we cross the point of no return, an async
commit would push all that into the worker thread.
- The disabling of encoders and connectors is a bit tricky, since
depending upon the final state we need to select different crtc
helper functions.
- Software tracking is a bit clarified compared to the crtc helpers:
We commit the software state before starting to touch the hardware,
like crtc helpers. But since we just swap them we still have the old
state (i.e. the current hw state) around, which is really handy to
write simple disable functions. So no more
drm_crtc_helper_disable_all_unused_functions kind of fun because
we're leaving unused crtcs/encoders behind. Everything gets shut
down in-order now, which is one of the key differences of the i915
helpers compared to crtc helpers and a really nice additional
guarantee.
- Like with the plane helpers the atomic commit function waits for one
vblank to pass before calling the framebuffer cleanup function.
Compared to Rob's helper approach there's a bunch of upsides:
- All the interfaces which can fail are called in the ->check hook
(i.e. ->best_match and the various ->mode_fixup hooks). This means
that drivers can just reuse those functions and don't need to move
everything into ->atomic_check callbacks. If drivers have no need
for additional constraint checking beyong their existing crtc
helper callbacks they don't need to do anything.
- The actual commit operation is properly stage: First we prepare
framebuffers, which can potentially still fail (due to memory
exhausting). This is important for the async case, where this must
be done synchronously to correctly return errors.
- The output configuration changes (done with crtc helper functions)
and the plane update (using atomic plane helpers) are correctly
interleaved: First we shut down any crtcs that need changing, then
we update planes and finally we enable everything again. Hardware
without GO bits must be more careful with ordering, which this
sequence enables.
- Also for hardware with shared output resources (like display PLLs)
we first must shut down the old configuration before we can enable
the new one. Otherwise we can hit an impossible intermediate state
where there's not enough PLLs (which is the point behind atomic
updates).
v2:
- Ensure that users of ->check update crtc_state->enable correctly.
- Update the legacy state in crtc/plane structures. Eventually we want
to remove that, but for now the drm core still expects this (especially
the plane->fb pointer).
v3: A few changes for better async handling:
- Reorder the software side state commit so that it happens all before
we touch the hardware. This way async support becomes very easy
since we can punt all the actual hw touching to a worker thread. And
as long as we synchronize with that thread (flushing or cancelling,
depending upon what the driver can handle) before we commit the next
software state there's no need for any locking in the worker thread
at all. Which greatly simplifies things.
And as long as we synchronize with all relevant threads we can have
a lot of them (e.g. per-crtc for per-crtc updates) running in
parallel.
- Expose pre/post plane commit steps separately. We need to expose the
actual hw commit step anyway for drivers to be able to implement
asynchronous commit workers. But if we expose pre/post and plane
commit steps individually we allow drivers to selectively use atomic
helpers.
- I've forgotten to call encoder/bridge ->mode_set functions, fix
this.
v4: Add debug output and fix a mixup between current and new state
that resulted in crtcs not getting updated correctly. And in an
Oops ...
v5:
- Be kind to driver writers in the vblank wait functions.. if thing
aren't working yet, and vblank irq will never come, then let's not
block forever.. especially under console-lock.
- Correctly clear connector_state->best_encoder when disabling.
Spotted while trying to understand a report from Rob Clark.
- Only steal encoder if it actually changed, otherwise hilarity ensues
if we steal from the current connector and so set the ->crtc pointer
unexpectedly to NULL. Reported by Rob Clark.
- Bail out in disable_outputs if an output currently doesn't have a
best_encoder - this means it's already disabled.
v6: Fixupe kerneldoc as reported by Paulo. And also fix up kerneldoc
in drm_crtc.h.
v7: Take ownership of the atomic state and clean it up with
drm_atomic_state_free().
v8 Various improvements all over:
- Polish code comments and kerneldoc.
- Improve debug output to make sure all failure cases are logged.
- Treat enabled crtc with no connectors as invalid input from userspace.
- Don't ignore the return value from mode_fixup().
v9:
- Improve debug output for crtc_state->mode_changed.
v10:
- Fixup the vblank waiting code to properly balance the vblank_get/put
calls.
- Better comments when checking/computing crtc->mode_changed
v11: Fixup the encoder stealing logic: We can't look at encoder->crtc
since that's not in the atomic state structures and might be updated
asynchronously in and async commit. Instead we need to inspect all the
connector states and check whether the encoder is currently in used
and if so, on which crtc.
v12: Review from Sean:
- A few spelling fixes.
- Flatten control flow indent by converting if blocks to early
continue/return in 2 places.
- Capture connectors_for_crtc return value in int num_connectors
instead of bool has_connectors and do an explicit int->bool
conversion with !!. I think the helper is more useful for drivers if
it returns the number of connectors (e.g. to detect cloning
configurations), so decided to keep that return value.
Cc: Sean Paul <seanpaul@chromium.org>
Cc: Paulo Zanoni <przanoni@gmail.com>
Cc: Rob Clark <robdclark@gmail.com>
Reviewed-by: Sean Paul <seanpaul@chromium.org>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-09-16 22:50:47 +07:00
|
|
|
* @last_vblank_count: for helpers and drivers to capture the vblank of the
|
|
|
|
* update to ensure framebuffer cleanup isn't done too early
|
2014-10-29 17:13:47 +07:00
|
|
|
* @adjusted_mode: for use by helpers and drivers to compute adjusted mode timings
|
2014-10-28 02:28:44 +07:00
|
|
|
* @mode: current mode timings
|
|
|
|
* @event: optional pointer to a DRM event to signal upon completion of the
|
|
|
|
* state update
|
|
|
|
* @state: backpointer to global drm_atomic_state
|
2014-11-26 22:57:41 +07:00
|
|
|
*
|
|
|
|
* Note that the distinction between @enable and @active is rather subtile:
|
|
|
|
* Flipping @active while @enable is set without changing anything else may
|
|
|
|
* never return in a failure from the ->atomic_check callback. Userspace assumes
|
|
|
|
* that a DPMS On will always succeed. In other words: @enable controls resource
|
|
|
|
* assignment, @active controls the actual hardware state.
|
2014-10-28 02:28:44 +07:00
|
|
|
*/
|
|
|
|
struct drm_crtc_state {
|
2014-11-27 21:49:39 +07:00
|
|
|
struct drm_crtc *crtc;
|
|
|
|
|
2014-07-26 02:30:38 +07:00
|
|
|
bool enable;
|
2014-11-26 22:57:41 +07:00
|
|
|
bool active;
|
2014-10-28 02:28:44 +07:00
|
|
|
|
2014-11-05 06:14:14 +07:00
|
|
|
/* computed state bits used by helpers and drivers */
|
|
|
|
bool planes_changed : 1;
|
drm: Atomic crtc/connector updates using crtc/plane helper interfaces
So this is finally the integration of the crtc and plane helper
interfaces into the atomic helper functions.
In the check function we now have a few steps:
- First we update the output routing and figure out which crtcs need a
full mode set. Suitable encoders are selected using ->best_encoder,
with the same semantics as the crtc helpers of implicitly disabling
all connectors currently using the encoder.
- Then we pull all other connectors into the state update which feed
from a crtc which changes. This must be done do catch mode changes
and similar updates - atomic updates are differences on top of the
current state.
- Then we call all the various ->mode_fixup to compute the adjusted
mode. Note that here we have a slight semantic difference compared
to the crtc helpers: We have not yet updated the encoder->crtc link
when calling the encoder's ->mode_fixup function. But that's a
requirement when converting to atomic since we want to prepare the
entire state completely contained with the over drm_atomic_state
structure. So this must be carefully checked when converting drivers
over to atomic helpers.
- Finally we do call the atomic_check functions on planes and crtcs.
The commit function is also quite a beast:
- The only step that can fail is done first, namely pinning the
framebuffers. After that we cross the point of no return, an async
commit would push all that into the worker thread.
- The disabling of encoders and connectors is a bit tricky, since
depending upon the final state we need to select different crtc
helper functions.
- Software tracking is a bit clarified compared to the crtc helpers:
We commit the software state before starting to touch the hardware,
like crtc helpers. But since we just swap them we still have the old
state (i.e. the current hw state) around, which is really handy to
write simple disable functions. So no more
drm_crtc_helper_disable_all_unused_functions kind of fun because
we're leaving unused crtcs/encoders behind. Everything gets shut
down in-order now, which is one of the key differences of the i915
helpers compared to crtc helpers and a really nice additional
guarantee.
- Like with the plane helpers the atomic commit function waits for one
vblank to pass before calling the framebuffer cleanup function.
Compared to Rob's helper approach there's a bunch of upsides:
- All the interfaces which can fail are called in the ->check hook
(i.e. ->best_match and the various ->mode_fixup hooks). This means
that drivers can just reuse those functions and don't need to move
everything into ->atomic_check callbacks. If drivers have no need
for additional constraint checking beyong their existing crtc
helper callbacks they don't need to do anything.
- The actual commit operation is properly stage: First we prepare
framebuffers, which can potentially still fail (due to memory
exhausting). This is important for the async case, where this must
be done synchronously to correctly return errors.
- The output configuration changes (done with crtc helper functions)
and the plane update (using atomic plane helpers) are correctly
interleaved: First we shut down any crtcs that need changing, then
we update planes and finally we enable everything again. Hardware
without GO bits must be more careful with ordering, which this
sequence enables.
- Also for hardware with shared output resources (like display PLLs)
we first must shut down the old configuration before we can enable
the new one. Otherwise we can hit an impossible intermediate state
where there's not enough PLLs (which is the point behind atomic
updates).
v2:
- Ensure that users of ->check update crtc_state->enable correctly.
- Update the legacy state in crtc/plane structures. Eventually we want
to remove that, but for now the drm core still expects this (especially
the plane->fb pointer).
v3: A few changes for better async handling:
- Reorder the software side state commit so that it happens all before
we touch the hardware. This way async support becomes very easy
since we can punt all the actual hw touching to a worker thread. And
as long as we synchronize with that thread (flushing or cancelling,
depending upon what the driver can handle) before we commit the next
software state there's no need for any locking in the worker thread
at all. Which greatly simplifies things.
And as long as we synchronize with all relevant threads we can have
a lot of them (e.g. per-crtc for per-crtc updates) running in
parallel.
- Expose pre/post plane commit steps separately. We need to expose the
actual hw commit step anyway for drivers to be able to implement
asynchronous commit workers. But if we expose pre/post and plane
commit steps individually we allow drivers to selectively use atomic
helpers.
- I've forgotten to call encoder/bridge ->mode_set functions, fix
this.
v4: Add debug output and fix a mixup between current and new state
that resulted in crtcs not getting updated correctly. And in an
Oops ...
v5:
- Be kind to driver writers in the vblank wait functions.. if thing
aren't working yet, and vblank irq will never come, then let's not
block forever.. especially under console-lock.
- Correctly clear connector_state->best_encoder when disabling.
Spotted while trying to understand a report from Rob Clark.
- Only steal encoder if it actually changed, otherwise hilarity ensues
if we steal from the current connector and so set the ->crtc pointer
unexpectedly to NULL. Reported by Rob Clark.
- Bail out in disable_outputs if an output currently doesn't have a
best_encoder - this means it's already disabled.
v6: Fixupe kerneldoc as reported by Paulo. And also fix up kerneldoc
in drm_crtc.h.
v7: Take ownership of the atomic state and clean it up with
drm_atomic_state_free().
v8 Various improvements all over:
- Polish code comments and kerneldoc.
- Improve debug output to make sure all failure cases are logged.
- Treat enabled crtc with no connectors as invalid input from userspace.
- Don't ignore the return value from mode_fixup().
v9:
- Improve debug output for crtc_state->mode_changed.
v10:
- Fixup the vblank waiting code to properly balance the vblank_get/put
calls.
- Better comments when checking/computing crtc->mode_changed
v11: Fixup the encoder stealing logic: We can't look at encoder->crtc
since that's not in the atomic state structures and might be updated
asynchronously in and async commit. Instead we need to inspect all the
connector states and check whether the encoder is currently in used
and if so, on which crtc.
v12: Review from Sean:
- A few spelling fixes.
- Flatten control flow indent by converting if blocks to early
continue/return in 2 places.
- Capture connectors_for_crtc return value in int num_connectors
instead of bool has_connectors and do an explicit int->bool
conversion with !!. I think the helper is more useful for drivers if
it returns the number of connectors (e.g. to detect cloning
configurations), so decided to keep that return value.
Cc: Sean Paul <seanpaul@chromium.org>
Cc: Paulo Zanoni <przanoni@gmail.com>
Cc: Rob Clark <robdclark@gmail.com>
Reviewed-by: Sean Paul <seanpaul@chromium.org>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-09-16 22:50:47 +07:00
|
|
|
bool mode_changed : 1;
|
2015-01-22 22:36:21 +07:00
|
|
|
bool active_changed : 1;
|
2015-07-21 18:28:57 +07:00
|
|
|
bool connectors_changed : 1;
|
drm: Atomic crtc/connector updates using crtc/plane helper interfaces
So this is finally the integration of the crtc and plane helper
interfaces into the atomic helper functions.
In the check function we now have a few steps:
- First we update the output routing and figure out which crtcs need a
full mode set. Suitable encoders are selected using ->best_encoder,
with the same semantics as the crtc helpers of implicitly disabling
all connectors currently using the encoder.
- Then we pull all other connectors into the state update which feed
from a crtc which changes. This must be done do catch mode changes
and similar updates - atomic updates are differences on top of the
current state.
- Then we call all the various ->mode_fixup to compute the adjusted
mode. Note that here we have a slight semantic difference compared
to the crtc helpers: We have not yet updated the encoder->crtc link
when calling the encoder's ->mode_fixup function. But that's a
requirement when converting to atomic since we want to prepare the
entire state completely contained with the over drm_atomic_state
structure. So this must be carefully checked when converting drivers
over to atomic helpers.
- Finally we do call the atomic_check functions on planes and crtcs.
The commit function is also quite a beast:
- The only step that can fail is done first, namely pinning the
framebuffers. After that we cross the point of no return, an async
commit would push all that into the worker thread.
- The disabling of encoders and connectors is a bit tricky, since
depending upon the final state we need to select different crtc
helper functions.
- Software tracking is a bit clarified compared to the crtc helpers:
We commit the software state before starting to touch the hardware,
like crtc helpers. But since we just swap them we still have the old
state (i.e. the current hw state) around, which is really handy to
write simple disable functions. So no more
drm_crtc_helper_disable_all_unused_functions kind of fun because
we're leaving unused crtcs/encoders behind. Everything gets shut
down in-order now, which is one of the key differences of the i915
helpers compared to crtc helpers and a really nice additional
guarantee.
- Like with the plane helpers the atomic commit function waits for one
vblank to pass before calling the framebuffer cleanup function.
Compared to Rob's helper approach there's a bunch of upsides:
- All the interfaces which can fail are called in the ->check hook
(i.e. ->best_match and the various ->mode_fixup hooks). This means
that drivers can just reuse those functions and don't need to move
everything into ->atomic_check callbacks. If drivers have no need
for additional constraint checking beyong their existing crtc
helper callbacks they don't need to do anything.
- The actual commit operation is properly stage: First we prepare
framebuffers, which can potentially still fail (due to memory
exhausting). This is important for the async case, where this must
be done synchronously to correctly return errors.
- The output configuration changes (done with crtc helper functions)
and the plane update (using atomic plane helpers) are correctly
interleaved: First we shut down any crtcs that need changing, then
we update planes and finally we enable everything again. Hardware
without GO bits must be more careful with ordering, which this
sequence enables.
- Also for hardware with shared output resources (like display PLLs)
we first must shut down the old configuration before we can enable
the new one. Otherwise we can hit an impossible intermediate state
where there's not enough PLLs (which is the point behind atomic
updates).
v2:
- Ensure that users of ->check update crtc_state->enable correctly.
- Update the legacy state in crtc/plane structures. Eventually we want
to remove that, but for now the drm core still expects this (especially
the plane->fb pointer).
v3: A few changes for better async handling:
- Reorder the software side state commit so that it happens all before
we touch the hardware. This way async support becomes very easy
since we can punt all the actual hw touching to a worker thread. And
as long as we synchronize with that thread (flushing or cancelling,
depending upon what the driver can handle) before we commit the next
software state there's no need for any locking in the worker thread
at all. Which greatly simplifies things.
And as long as we synchronize with all relevant threads we can have
a lot of them (e.g. per-crtc for per-crtc updates) running in
parallel.
- Expose pre/post plane commit steps separately. We need to expose the
actual hw commit step anyway for drivers to be able to implement
asynchronous commit workers. But if we expose pre/post and plane
commit steps individually we allow drivers to selectively use atomic
helpers.
- I've forgotten to call encoder/bridge ->mode_set functions, fix
this.
v4: Add debug output and fix a mixup between current and new state
that resulted in crtcs not getting updated correctly. And in an
Oops ...
v5:
- Be kind to driver writers in the vblank wait functions.. if thing
aren't working yet, and vblank irq will never come, then let's not
block forever.. especially under console-lock.
- Correctly clear connector_state->best_encoder when disabling.
Spotted while trying to understand a report from Rob Clark.
- Only steal encoder if it actually changed, otherwise hilarity ensues
if we steal from the current connector and so set the ->crtc pointer
unexpectedly to NULL. Reported by Rob Clark.
- Bail out in disable_outputs if an output currently doesn't have a
best_encoder - this means it's already disabled.
v6: Fixupe kerneldoc as reported by Paulo. And also fix up kerneldoc
in drm_crtc.h.
v7: Take ownership of the atomic state and clean it up with
drm_atomic_state_free().
v8 Various improvements all over:
- Polish code comments and kerneldoc.
- Improve debug output to make sure all failure cases are logged.
- Treat enabled crtc with no connectors as invalid input from userspace.
- Don't ignore the return value from mode_fixup().
v9:
- Improve debug output for crtc_state->mode_changed.
v10:
- Fixup the vblank waiting code to properly balance the vblank_get/put
calls.
- Better comments when checking/computing crtc->mode_changed
v11: Fixup the encoder stealing logic: We can't look at encoder->crtc
since that's not in the atomic state structures and might be updated
asynchronously in and async commit. Instead we need to inspect all the
connector states and check whether the encoder is currently in used
and if so, on which crtc.
v12: Review from Sean:
- A few spelling fixes.
- Flatten control flow indent by converting if blocks to early
continue/return in 2 places.
- Capture connectors_for_crtc return value in int num_connectors
instead of bool has_connectors and do an explicit int->bool
conversion with !!. I think the helper is more useful for drivers if
it returns the number of connectors (e.g. to detect cloning
configurations), so decided to keep that return value.
Cc: Sean Paul <seanpaul@chromium.org>
Cc: Paulo Zanoni <przanoni@gmail.com>
Cc: Rob Clark <robdclark@gmail.com>
Reviewed-by: Sean Paul <seanpaul@chromium.org>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-09-16 22:50:47 +07:00
|
|
|
|
2014-11-22 03:28:31 +07:00
|
|
|
/* attached planes bitmask:
|
|
|
|
* WARNING: transitional helpers do not maintain plane_mask so
|
|
|
|
* drivers not converted over to atomic helpers should not rely
|
|
|
|
* on plane_mask being accurate!
|
|
|
|
*/
|
|
|
|
u32 plane_mask;
|
|
|
|
|
2016-01-04 18:53:18 +07:00
|
|
|
u32 connector_mask;
|
|
|
|
|
drm: Atomic crtc/connector updates using crtc/plane helper interfaces
So this is finally the integration of the crtc and plane helper
interfaces into the atomic helper functions.
In the check function we now have a few steps:
- First we update the output routing and figure out which crtcs need a
full mode set. Suitable encoders are selected using ->best_encoder,
with the same semantics as the crtc helpers of implicitly disabling
all connectors currently using the encoder.
- Then we pull all other connectors into the state update which feed
from a crtc which changes. This must be done do catch mode changes
and similar updates - atomic updates are differences on top of the
current state.
- Then we call all the various ->mode_fixup to compute the adjusted
mode. Note that here we have a slight semantic difference compared
to the crtc helpers: We have not yet updated the encoder->crtc link
when calling the encoder's ->mode_fixup function. But that's a
requirement when converting to atomic since we want to prepare the
entire state completely contained with the over drm_atomic_state
structure. So this must be carefully checked when converting drivers
over to atomic helpers.
- Finally we do call the atomic_check functions on planes and crtcs.
The commit function is also quite a beast:
- The only step that can fail is done first, namely pinning the
framebuffers. After that we cross the point of no return, an async
commit would push all that into the worker thread.
- The disabling of encoders and connectors is a bit tricky, since
depending upon the final state we need to select different crtc
helper functions.
- Software tracking is a bit clarified compared to the crtc helpers:
We commit the software state before starting to touch the hardware,
like crtc helpers. But since we just swap them we still have the old
state (i.e. the current hw state) around, which is really handy to
write simple disable functions. So no more
drm_crtc_helper_disable_all_unused_functions kind of fun because
we're leaving unused crtcs/encoders behind. Everything gets shut
down in-order now, which is one of the key differences of the i915
helpers compared to crtc helpers and a really nice additional
guarantee.
- Like with the plane helpers the atomic commit function waits for one
vblank to pass before calling the framebuffer cleanup function.
Compared to Rob's helper approach there's a bunch of upsides:
- All the interfaces which can fail are called in the ->check hook
(i.e. ->best_match and the various ->mode_fixup hooks). This means
that drivers can just reuse those functions and don't need to move
everything into ->atomic_check callbacks. If drivers have no need
for additional constraint checking beyong their existing crtc
helper callbacks they don't need to do anything.
- The actual commit operation is properly stage: First we prepare
framebuffers, which can potentially still fail (due to memory
exhausting). This is important for the async case, where this must
be done synchronously to correctly return errors.
- The output configuration changes (done with crtc helper functions)
and the plane update (using atomic plane helpers) are correctly
interleaved: First we shut down any crtcs that need changing, then
we update planes and finally we enable everything again. Hardware
without GO bits must be more careful with ordering, which this
sequence enables.
- Also for hardware with shared output resources (like display PLLs)
we first must shut down the old configuration before we can enable
the new one. Otherwise we can hit an impossible intermediate state
where there's not enough PLLs (which is the point behind atomic
updates).
v2:
- Ensure that users of ->check update crtc_state->enable correctly.
- Update the legacy state in crtc/plane structures. Eventually we want
to remove that, but for now the drm core still expects this (especially
the plane->fb pointer).
v3: A few changes for better async handling:
- Reorder the software side state commit so that it happens all before
we touch the hardware. This way async support becomes very easy
since we can punt all the actual hw touching to a worker thread. And
as long as we synchronize with that thread (flushing or cancelling,
depending upon what the driver can handle) before we commit the next
software state there's no need for any locking in the worker thread
at all. Which greatly simplifies things.
And as long as we synchronize with all relevant threads we can have
a lot of them (e.g. per-crtc for per-crtc updates) running in
parallel.
- Expose pre/post plane commit steps separately. We need to expose the
actual hw commit step anyway for drivers to be able to implement
asynchronous commit workers. But if we expose pre/post and plane
commit steps individually we allow drivers to selectively use atomic
helpers.
- I've forgotten to call encoder/bridge ->mode_set functions, fix
this.
v4: Add debug output and fix a mixup between current and new state
that resulted in crtcs not getting updated correctly. And in an
Oops ...
v5:
- Be kind to driver writers in the vblank wait functions.. if thing
aren't working yet, and vblank irq will never come, then let's not
block forever.. especially under console-lock.
- Correctly clear connector_state->best_encoder when disabling.
Spotted while trying to understand a report from Rob Clark.
- Only steal encoder if it actually changed, otherwise hilarity ensues
if we steal from the current connector and so set the ->crtc pointer
unexpectedly to NULL. Reported by Rob Clark.
- Bail out in disable_outputs if an output currently doesn't have a
best_encoder - this means it's already disabled.
v6: Fixupe kerneldoc as reported by Paulo. And also fix up kerneldoc
in drm_crtc.h.
v7: Take ownership of the atomic state and clean it up with
drm_atomic_state_free().
v8 Various improvements all over:
- Polish code comments and kerneldoc.
- Improve debug output to make sure all failure cases are logged.
- Treat enabled crtc with no connectors as invalid input from userspace.
- Don't ignore the return value from mode_fixup().
v9:
- Improve debug output for crtc_state->mode_changed.
v10:
- Fixup the vblank waiting code to properly balance the vblank_get/put
calls.
- Better comments when checking/computing crtc->mode_changed
v11: Fixup the encoder stealing logic: We can't look at encoder->crtc
since that's not in the atomic state structures and might be updated
asynchronously in and async commit. Instead we need to inspect all the
connector states and check whether the encoder is currently in used
and if so, on which crtc.
v12: Review from Sean:
- A few spelling fixes.
- Flatten control flow indent by converting if blocks to early
continue/return in 2 places.
- Capture connectors_for_crtc return value in int num_connectors
instead of bool has_connectors and do an explicit int->bool
conversion with !!. I think the helper is more useful for drivers if
it returns the number of connectors (e.g. to detect cloning
configurations), so decided to keep that return value.
Cc: Sean Paul <seanpaul@chromium.org>
Cc: Paulo Zanoni <przanoni@gmail.com>
Cc: Rob Clark <robdclark@gmail.com>
Reviewed-by: Sean Paul <seanpaul@chromium.org>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-09-16 22:50:47 +07:00
|
|
|
/* last_vblank_count: for vblank waits before cleanup */
|
|
|
|
u32 last_vblank_count;
|
2014-11-05 06:14:14 +07:00
|
|
|
|
2014-10-29 17:13:47 +07:00
|
|
|
/* adjusted_mode: for use by helpers and drivers */
|
|
|
|
struct drm_display_mode adjusted_mode;
|
|
|
|
|
2014-10-28 02:28:44 +07:00
|
|
|
struct drm_display_mode mode;
|
|
|
|
|
2015-05-26 01:11:51 +07:00
|
|
|
/* blob property to expose current mode to atomic userspace */
|
|
|
|
struct drm_property_blob *mode_blob;
|
|
|
|
|
2014-10-28 02:28:44 +07:00
|
|
|
struct drm_pending_vblank_event *event;
|
|
|
|
|
|
|
|
struct drm_atomic_state *state;
|
|
|
|
};
|
2008-11-08 05:05:41 +07:00
|
|
|
|
|
|
|
/**
|
2014-10-27 22:54:27 +07:00
|
|
|
* struct drm_crtc_funcs - control CRTCs for a given device
|
2008-11-08 05:05:41 +07:00
|
|
|
*
|
|
|
|
* The drm_crtc_funcs structure is the central CRTC management structure
|
|
|
|
* in the DRM. Each CRTC controls one or more connectors (note that the name
|
|
|
|
* CRTC is simply historical, a CRTC may control LVDS, VGA, DVI, TV out, etc.
|
|
|
|
* connectors, not just CRTs).
|
|
|
|
*
|
|
|
|
* Each driver is responsible for filling out this structure at startup time,
|
|
|
|
* in addition to providing other modesetting features, like i2c and DDC
|
|
|
|
* bus accessors.
|
|
|
|
*/
|
|
|
|
struct drm_crtc_funcs {
|
2015-12-04 15:45:48 +07:00
|
|
|
/**
|
|
|
|
* @reset:
|
|
|
|
*
|
|
|
|
* Reset CRTC hardware and software state to off. This function isn't
|
|
|
|
* called by the core directly, only through drm_mode_config_reset().
|
|
|
|
* It's not a helper hook only for historical reasons.
|
|
|
|
*
|
|
|
|
* Atomic drivers can use drm_atomic_helper_crtc_reset() to reset
|
|
|
|
* atomic state using this hook.
|
|
|
|
*/
|
2011-01-24 22:11:08 +07:00
|
|
|
void (*reset)(struct drm_crtc *crtc);
|
2008-11-08 05:05:41 +07:00
|
|
|
|
2015-12-04 15:46:00 +07:00
|
|
|
/**
|
|
|
|
* @cursor_set:
|
|
|
|
*
|
|
|
|
* Update the cursor image. The cursor position is relative to the CRTC
|
|
|
|
* and can be partially or fully outside of the visible area.
|
|
|
|
*
|
|
|
|
* Note that contrary to all other KMS functions the legacy cursor entry
|
|
|
|
* points don't take a framebuffer object, but instead take directly a
|
|
|
|
* raw buffer object id from the driver's buffer manager (which is
|
|
|
|
* either GEM or TTM for current drivers).
|
|
|
|
*
|
|
|
|
* This entry point is deprecated, drivers should instead implement
|
|
|
|
* universal plane support and register a proper cursor plane using
|
|
|
|
* drm_crtc_init_with_planes().
|
|
|
|
*
|
|
|
|
* This callback is optional
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 on success or a negative error code on failure.
|
|
|
|
*/
|
2008-11-08 05:05:41 +07:00
|
|
|
int (*cursor_set)(struct drm_crtc *crtc, struct drm_file *file_priv,
|
|
|
|
uint32_t handle, uint32_t width, uint32_t height);
|
2015-12-04 15:46:00 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @cursor_set2:
|
|
|
|
*
|
|
|
|
* Update the cursor image, including hotspot information. The hotspot
|
|
|
|
* must not affect the cursor position in CRTC coordinates, but is only
|
|
|
|
* meant as a hint for virtualized display hardware to coordinate the
|
|
|
|
* guests and hosts cursor position. The cursor hotspot is relative to
|
|
|
|
* the cursor image. Otherwise this works exactly like @cursor_set.
|
|
|
|
*
|
|
|
|
* This entry point is deprecated, drivers should instead implement
|
|
|
|
* universal plane support and register a proper cursor plane using
|
|
|
|
* drm_crtc_init_with_planes().
|
|
|
|
*
|
|
|
|
* This callback is optional.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 on success or a negative error code on failure.
|
|
|
|
*/
|
2013-06-20 08:48:52 +07:00
|
|
|
int (*cursor_set2)(struct drm_crtc *crtc, struct drm_file *file_priv,
|
|
|
|
uint32_t handle, uint32_t width, uint32_t height,
|
|
|
|
int32_t hot_x, int32_t hot_y);
|
2015-12-04 15:46:00 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @cursor_move:
|
|
|
|
*
|
|
|
|
* Update the cursor position. The cursor does not need to be visible
|
|
|
|
* when this hook is called.
|
|
|
|
*
|
|
|
|
* This entry point is deprecated, drivers should instead implement
|
|
|
|
* universal plane support and register a proper cursor plane using
|
|
|
|
* drm_crtc_init_with_planes().
|
|
|
|
*
|
|
|
|
* This callback is optional.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 on success or a negative error code on failure.
|
|
|
|
*/
|
2008-11-08 05:05:41 +07:00
|
|
|
int (*cursor_move)(struct drm_crtc *crtc, int x, int y);
|
|
|
|
|
2015-12-04 15:46:00 +07:00
|
|
|
/**
|
|
|
|
* @gamma_set:
|
|
|
|
*
|
|
|
|
* Set gamma on the CRTC.
|
|
|
|
*
|
|
|
|
* This callback is optional.
|
|
|
|
*
|
|
|
|
* NOTE:
|
|
|
|
*
|
|
|
|
* Drivers that support gamma tables and also fbdev emulation through
|
|
|
|
* the provided helper library need to take care to fill out the gamma
|
|
|
|
* hooks for both. Currently there's a bit an unfortunate duplication
|
|
|
|
* going on, which should eventually be unified to just one set of
|
|
|
|
* hooks.
|
|
|
|
*/
|
2008-11-08 05:05:41 +07:00
|
|
|
void (*gamma_set)(struct drm_crtc *crtc, u16 *r, u16 *g, u16 *b,
|
2010-08-03 07:33:19 +07:00
|
|
|
uint32_t start, uint32_t size);
|
2015-12-04 15:45:48 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @destroy:
|
|
|
|
*
|
|
|
|
* Clean up plane resources. This is only called at driver unload time
|
|
|
|
* through drm_mode_config_cleanup() since a CRTC cannot be hotplugged
|
|
|
|
* in DRM.
|
|
|
|
*/
|
2008-11-08 05:05:41 +07:00
|
|
|
void (*destroy)(struct drm_crtc *crtc);
|
|
|
|
|
2015-12-04 15:46:00 +07:00
|
|
|
/**
|
|
|
|
* @set_config:
|
|
|
|
*
|
|
|
|
* This is the main legacy entry point to change the modeset state on a
|
|
|
|
* CRTC. All the details of the desired configuration are passed in a
|
|
|
|
* struct &drm_mode_set - see there for details.
|
|
|
|
*
|
|
|
|
* Drivers implementing atomic modeset should use
|
|
|
|
* drm_atomic_helper_set_config() to implement this hook.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 on success or a negative error code on failure.
|
|
|
|
*/
|
2008-11-08 05:05:41 +07:00
|
|
|
int (*set_config)(struct drm_mode_set *set);
|
2009-11-18 00:43:55 +07:00
|
|
|
|
2015-12-04 15:46:00 +07:00
|
|
|
/**
|
|
|
|
* @page_flip:
|
|
|
|
*
|
|
|
|
* Legacy entry point to schedule a flip to the given framebuffer.
|
|
|
|
*
|
|
|
|
* Page flipping is a synchronization mechanism that replaces the frame
|
|
|
|
* buffer being scanned out by the CRTC with a new frame buffer during
|
|
|
|
* vertical blanking, avoiding tearing (except when requested otherwise
|
|
|
|
* through the DRM_MODE_PAGE_FLIP_ASYNC flag). When an application
|
|
|
|
* requests a page flip the DRM core verifies that the new frame buffer
|
|
|
|
* is large enough to be scanned out by the CRTC in the currently
|
|
|
|
* configured mode and then calls the CRTC ->page_flip() operation with a
|
|
|
|
* pointer to the new frame buffer.
|
|
|
|
*
|
|
|
|
* The driver must wait for any pending rendering to the new framebuffer
|
|
|
|
* to complete before executing the flip. It should also wait for any
|
|
|
|
* pending rendering from other drivers if the underlying buffer is a
|
|
|
|
* shared dma-buf.
|
|
|
|
*
|
|
|
|
* An application can request to be notified when the page flip has
|
|
|
|
* completed. The drm core will supply a struct &drm_event in the event
|
|
|
|
* parameter in this case. This can be handled by the
|
|
|
|
* drm_crtc_send_vblank_event() function, which the driver should call on
|
|
|
|
* the provided event upon completion of the flip. Note that if
|
|
|
|
* the driver supports vblank signalling and timestamping the vblank
|
|
|
|
* counters and timestamps must agree with the ones returned from page
|
|
|
|
* flip events. With the current vblank helper infrastructure this can
|
|
|
|
* be achieved by holding a vblank reference while the page flip is
|
|
|
|
* pending, acquired through drm_crtc_vblank_get() and released with
|
|
|
|
* drm_crtc_vblank_put(). Drivers are free to implement their own vblank
|
|
|
|
* counter and timestamp tracking though, e.g. if they have accurate
|
|
|
|
* timestamp registers in hardware.
|
|
|
|
*
|
|
|
|
* FIXME:
|
|
|
|
*
|
|
|
|
* Up to that point drivers need to manage events themselves and can use
|
|
|
|
* even->base.list freely for that. Specifically they need to ensure
|
|
|
|
* that they don't send out page flip (or vblank) events for which the
|
|
|
|
* corresponding drm file has been closed already. The drm core
|
|
|
|
* unfortunately does not (yet) take care of that. Therefore drivers
|
|
|
|
* currently must clean up and release pending events in their
|
|
|
|
* ->preclose driver function.
|
|
|
|
*
|
|
|
|
* This callback is optional.
|
|
|
|
*
|
|
|
|
* NOTE:
|
|
|
|
*
|
|
|
|
* Very early versions of the KMS ABI mandated that the driver must
|
|
|
|
* block (but not reject) any rendering to the old framebuffer until the
|
|
|
|
* flip operation has completed and the old framebuffer is no longer
|
|
|
|
* visible. This requirement has been lifted, and userspace is instead
|
|
|
|
* expected to request delivery of an event and wait with recycling old
|
|
|
|
* buffers until such has been received.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 on success or a negative error code on failure. Note that if a
|
|
|
|
* ->page_flip() operation is already pending the callback should return
|
|
|
|
* -EBUSY. Pageflips on a disabled CRTC (either by setting a NULL mode
|
|
|
|
* or just runtime disabled through DPMS respectively the new atomic
|
|
|
|
* "ACTIVE" state) should result in an -EINVAL error code.
|
2009-11-18 00:43:55 +07:00
|
|
|
*/
|
|
|
|
int (*page_flip)(struct drm_crtc *crtc,
|
|
|
|
struct drm_framebuffer *fb,
|
2013-07-23 08:49:58 +07:00
|
|
|
struct drm_pending_vblank_event *event,
|
|
|
|
uint32_t flags);
|
2012-05-16 04:09:05 +07:00
|
|
|
|
2015-12-04 15:45:48 +07:00
|
|
|
/**
|
|
|
|
* @set_property:
|
|
|
|
*
|
|
|
|
* This is the legacy entry point to update a property attached to the
|
|
|
|
* CRTC.
|
|
|
|
*
|
|
|
|
* Drivers implementing atomic modeset should use
|
|
|
|
* drm_atomic_helper_crtc_set_property() to implement this hook.
|
|
|
|
*
|
|
|
|
* This callback is optional if the driver does not support any legacy
|
|
|
|
* driver-private properties.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 on success or a negative error code on failure.
|
|
|
|
*/
|
2012-05-16 04:09:05 +07:00
|
|
|
int (*set_property)(struct drm_crtc *crtc,
|
|
|
|
struct drm_property *property, uint64_t val);
|
2014-10-28 02:28:44 +07:00
|
|
|
|
2015-12-04 15:45:48 +07:00
|
|
|
/**
|
|
|
|
* @atomic_duplicate_state:
|
|
|
|
*
|
|
|
|
* Duplicate the current atomic state for this CRTC and return it.
|
|
|
|
* The core and helpers gurantee that any atomic state duplicated with
|
|
|
|
* this hook and still owned by the caller (i.e. not transferred to the
|
|
|
|
* driver by calling ->atomic_commit() from struct
|
|
|
|
* &drm_mode_config_funcs) will be cleaned up by calling the
|
|
|
|
* @atomic_destroy_state hook in this structure.
|
|
|
|
*
|
|
|
|
* Atomic drivers which don't subclass struct &drm_crtc should use
|
|
|
|
* drm_atomic_helper_crtc_duplicate_state(). Drivers that subclass the
|
|
|
|
* state structure to extend it with driver-private state should use
|
|
|
|
* __drm_atomic_helper_crtc_duplicate_state() to make sure shared state is
|
|
|
|
* duplicated in a consistent fashion across drivers.
|
|
|
|
*
|
|
|
|
* It is an error to call this hook before crtc->state has been
|
|
|
|
* initialized correctly.
|
|
|
|
*
|
|
|
|
* NOTE:
|
|
|
|
*
|
|
|
|
* If the duplicate state references refcounted resources this hook must
|
|
|
|
* acquire a reference for each of them. The driver must release these
|
|
|
|
* references again in @atomic_destroy_state.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* Duplicated atomic state or NULL when the allocation failed.
|
|
|
|
*/
|
2014-10-28 02:28:44 +07:00
|
|
|
struct drm_crtc_state *(*atomic_duplicate_state)(struct drm_crtc *crtc);
|
2015-12-04 15:45:48 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @atomic_destroy_state:
|
|
|
|
*
|
|
|
|
* Destroy a state duplicated with @atomic_duplicate_state and release
|
|
|
|
* or unreference all resources it references
|
|
|
|
*/
|
2014-10-28 02:28:44 +07:00
|
|
|
void (*atomic_destroy_state)(struct drm_crtc *crtc,
|
2014-07-26 02:30:38 +07:00
|
|
|
struct drm_crtc_state *state);
|
2015-12-04 15:45:48 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @atomic_set_property:
|
|
|
|
*
|
|
|
|
* Decode a driver-private property value and store the decoded value
|
|
|
|
* into the passed-in state structure. Since the atomic core decodes all
|
|
|
|
* standardized properties (even for extensions beyond the core set of
|
|
|
|
* properties which might not be implemented by all drivers) this
|
|
|
|
* requires drivers to subclass the state structure.
|
|
|
|
*
|
|
|
|
* Such driver-private properties should really only be implemented for
|
|
|
|
* truly hardware/vendor specific state. Instead it is preferred to
|
|
|
|
* standardize atomic extension and decode the properties used to expose
|
|
|
|
* such an extension in the core.
|
|
|
|
*
|
|
|
|
* Do not call this function directly, use
|
|
|
|
* drm_atomic_crtc_set_property() instead.
|
|
|
|
*
|
|
|
|
* This callback is optional if the driver does not support any
|
|
|
|
* driver-private atomic properties.
|
|
|
|
*
|
|
|
|
* NOTE:
|
|
|
|
*
|
|
|
|
* This function is called in the state assembly phase of atomic
|
|
|
|
* modesets, which can be aborted for any reason (including on
|
|
|
|
* userspace's request to just check whether a configuration would be
|
|
|
|
* possible). Drivers MUST NOT touch any persistent state (hardware or
|
|
|
|
* software) or data structures except the passed in @state parameter.
|
|
|
|
*
|
|
|
|
* Also since userspace controls in which order properties are set this
|
|
|
|
* function must not do any input validation (since the state update is
|
|
|
|
* incomplete and hence likely inconsistent). Instead any such input
|
|
|
|
* validation must be done in the various atomic_check callbacks.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 if the property has been found, -EINVAL if the property isn't
|
|
|
|
* implemented by the driver (which should never happen, the core only
|
|
|
|
* asks for properties attached to this CRTC). No other validation is
|
|
|
|
* allowed by the driver. The core already checks that the property
|
|
|
|
* value is within the range (integer, valid enum value, ...) the driver
|
|
|
|
* set when registering the property.
|
|
|
|
*/
|
2014-10-28 02:28:44 +07:00
|
|
|
int (*atomic_set_property)(struct drm_crtc *crtc,
|
|
|
|
struct drm_crtc_state *state,
|
|
|
|
struct drm_property *property,
|
|
|
|
uint64_t val);
|
2015-12-04 15:45:48 +07:00
|
|
|
/**
|
|
|
|
* @atomic_get_property:
|
|
|
|
*
|
|
|
|
* Reads out the decoded driver-private property. This is used to
|
2015-12-04 15:46:01 +07:00
|
|
|
* implement the GETCRTC IOCTL.
|
2015-12-04 15:45:48 +07:00
|
|
|
*
|
|
|
|
* Do not call this function directly, use
|
|
|
|
* drm_atomic_crtc_get_property() instead.
|
|
|
|
*
|
|
|
|
* This callback is optional if the driver does not support any
|
|
|
|
* driver-private atomic properties.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 on success, -EINVAL if the property isn't implemented by the
|
|
|
|
* driver (which should never happen, the core only asks for
|
|
|
|
* properties attached to this CRTC).
|
|
|
|
*/
|
2014-12-19 04:01:47 +07:00
|
|
|
int (*atomic_get_property)(struct drm_crtc *crtc,
|
|
|
|
const struct drm_crtc_state *state,
|
|
|
|
struct drm_property *property,
|
|
|
|
uint64_t *val);
|
2008-11-08 05:05:41 +07:00
|
|
|
};
|
|
|
|
|
|
|
|
/**
|
2014-10-27 22:54:27 +07:00
|
|
|
* struct drm_crtc - central CRTC control structure
|
2011-11-08 03:03:14 +07:00
|
|
|
* @dev: parent DRM device
|
2014-10-28 02:19:38 +07:00
|
|
|
* @port: OF node used by drm_of_find_possible_crtcs()
|
2011-11-08 03:03:14 +07:00
|
|
|
* @head: list management
|
2013-11-20 00:10:12 +07:00
|
|
|
* @mutex: per-CRTC locking
|
2011-11-08 03:03:14 +07:00
|
|
|
* @base: base KMS object for ID tracking etc.
|
2014-04-02 05:22:38 +07:00
|
|
|
* @primary: primary plane for this CRTC
|
|
|
|
* @cursor: cursor plane for this CRTC
|
2014-10-28 02:19:38 +07:00
|
|
|
* @cursor_x: current x position of the cursor, used for universal cursor planes
|
|
|
|
* @cursor_y: current y position of the cursor, used for universal cursor planes
|
2008-11-08 05:05:41 +07:00
|
|
|
* @enabled: is this CRTC enabled?
|
2011-11-08 03:03:14 +07:00
|
|
|
* @mode: current mode timings
|
|
|
|
* @hwmode: mode timings as programmed to hw regs
|
2008-11-08 05:05:41 +07:00
|
|
|
* @x: x position on screen
|
|
|
|
* @y: y position on screen
|
|
|
|
* @funcs: CRTC control functions
|
2011-11-08 03:03:14 +07:00
|
|
|
* @gamma_size: size of gamma ramp
|
|
|
|
* @gamma_store: gamma ramp values
|
|
|
|
* @helper_private: mid-layer private data
|
2012-05-16 04:09:05 +07:00
|
|
|
* @properties: property tracking for this CRTC
|
2014-10-28 02:28:44 +07:00
|
|
|
* @state: current atomic state for this CRTC
|
2014-10-28 02:19:38 +07:00
|
|
|
* @acquire_ctx: per-CRTC implicit acquire context used by atomic drivers for
|
2015-12-04 15:46:01 +07:00
|
|
|
* legacy IOCTLs
|
2008-11-08 05:05:41 +07:00
|
|
|
*
|
|
|
|
* Each CRTC may have one or more connectors associated with it. This structure
|
|
|
|
* allows the CRTC to be controlled.
|
|
|
|
*/
|
|
|
|
struct drm_crtc {
|
|
|
|
struct drm_device *dev;
|
2014-06-15 17:07:12 +07:00
|
|
|
struct device_node *port;
|
2008-11-08 05:05:41 +07:00
|
|
|
struct list_head head;
|
|
|
|
|
2015-12-08 23:41:53 +07:00
|
|
|
char *name;
|
|
|
|
|
2014-10-27 22:54:27 +07:00
|
|
|
/*
|
drm: add per-crtc locks
*drumroll*
The basic idea is to protect per-crtc state which can change without
touching the output configuration with separate mutexes, i.e. all the
input side state to a crtc like framebuffers, cursor settings or plane
configuration. Holding such a crtc lock gives a read-lock on all the
other crtc state which can be changed by e.g. a modeset.
All non-crtc state is still protected by the mode_config mutex.
Callers that need to change modeset state of a crtc (e.g. dpms or
set_mode) need to grab both the mode_config lock and nested within any
crtc locks.
Note that since there can only ever be one holder of the mode_config
lock we can grab the subordinate crtc locks in any order (if we need
to grab more than one of them). Lockdep can handle such nesting with
the mutex_lock_nest_lock call correctly.
With this functions that only touch connectors/encoders but not crtcs
only need to take the mode_config lock. The biggest such case is the
output probing, which means that we can now pageflip and move cursors
while the output probe code is reading an edid.
Most cases neatly fall into the three buckets:
- Only touches connectors and similar output state and so only needs
the mode_config lock.
- Touches the global configuration and so needs all locks.
- Only touches the crtc input side and so only needs the crtc lock.
But a few cases that need special consideration:
- Load detection which requires a crtc. The mode_config lock already
prevents a modeset change, so we can use any unused crtc as we like
to do load detection. The only thing to consider is that such
temporary state changes don't leak out to userspace through ioctls
that only take the crtc look (like a pageflip). Hence the load
detect code needs to grab the crtc of any output pipes it touches
(but only if it touches state used by the pageflip or cursor
ioctls).
- Atomic pageflip when moving planes. The first case is sane hw, where
planes have a fixed association with crtcs - nothing needs to be
done there. More insane^Wflexible hw needs to have plane->crtc
mapping which is separately protect with a lock that nests within
the crtc lock. If the plane is unused we can just assign it to the
current crtc and continue. But if a plane is already in use by
another crtc we can't just reassign it.
Two solution present themselves: Either go back to a slow-path which
takes all modeset locks, potentially incure quite a hefty delay. Or
simply disallowing such changes in one atomic pageflip - in general
the vblanks of two crtcs are not synced, so there's no sane way to
atomically flip such plane changes accross more than one crtc. I'd
heavily favour the later approach, going as far as mandating it as
part of the ABI of such a new a nuclear pageflip.
And if we _really_ want such semantics, we can always get them by
introducing another pageflip mutex between the mode_config.mutex and
the individual crtc locks. Pageflips crossing more than one crtc
would then need to take that lock first, to lock out concurrent
multi-crtc pageflips.
- Optimized global modeset operations: We could just take the
mode_config lock and then lazily lock all crtc which are affected by
a modeset operation. This has the advantage that pageflip could
continue unhampered on unaffected crtc. But if e.g. global resources
like plls need to be reassigned and so affect unrelated crtcs we can
still do that - nested locking works in any order.
This patch just adds the locks and takes them in drm_modeset_lock_all,
no real locking changes yet.
v2: Need to initialize the new lock in crtc_init and lock it righ
away, for otherwise the modeset_unlock_all below will try to unlock a
not-locked mutex.
Reviewed-by: Rob Clark <rob@ti.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2012-12-02 08:18:25 +07:00
|
|
|
* crtc mutex
|
|
|
|
*
|
|
|
|
* This provides a read lock for the overall crtc state (mode, dpms
|
|
|
|
* state, ...) and a write lock for everything which can be update
|
|
|
|
* without a full modeset (fb, cursor data, ...)
|
|
|
|
*/
|
2013-11-20 00:10:12 +07:00
|
|
|
struct drm_modeset_lock mutex;
|
drm: add per-crtc locks
*drumroll*
The basic idea is to protect per-crtc state which can change without
touching the output configuration with separate mutexes, i.e. all the
input side state to a crtc like framebuffers, cursor settings or plane
configuration. Holding such a crtc lock gives a read-lock on all the
other crtc state which can be changed by e.g. a modeset.
All non-crtc state is still protected by the mode_config mutex.
Callers that need to change modeset state of a crtc (e.g. dpms or
set_mode) need to grab both the mode_config lock and nested within any
crtc locks.
Note that since there can only ever be one holder of the mode_config
lock we can grab the subordinate crtc locks in any order (if we need
to grab more than one of them). Lockdep can handle such nesting with
the mutex_lock_nest_lock call correctly.
With this functions that only touch connectors/encoders but not crtcs
only need to take the mode_config lock. The biggest such case is the
output probing, which means that we can now pageflip and move cursors
while the output probe code is reading an edid.
Most cases neatly fall into the three buckets:
- Only touches connectors and similar output state and so only needs
the mode_config lock.
- Touches the global configuration and so needs all locks.
- Only touches the crtc input side and so only needs the crtc lock.
But a few cases that need special consideration:
- Load detection which requires a crtc. The mode_config lock already
prevents a modeset change, so we can use any unused crtc as we like
to do load detection. The only thing to consider is that such
temporary state changes don't leak out to userspace through ioctls
that only take the crtc look (like a pageflip). Hence the load
detect code needs to grab the crtc of any output pipes it touches
(but only if it touches state used by the pageflip or cursor
ioctls).
- Atomic pageflip when moving planes. The first case is sane hw, where
planes have a fixed association with crtcs - nothing needs to be
done there. More insane^Wflexible hw needs to have plane->crtc
mapping which is separately protect with a lock that nests within
the crtc lock. If the plane is unused we can just assign it to the
current crtc and continue. But if a plane is already in use by
another crtc we can't just reassign it.
Two solution present themselves: Either go back to a slow-path which
takes all modeset locks, potentially incure quite a hefty delay. Or
simply disallowing such changes in one atomic pageflip - in general
the vblanks of two crtcs are not synced, so there's no sane way to
atomically flip such plane changes accross more than one crtc. I'd
heavily favour the later approach, going as far as mandating it as
part of the ABI of such a new a nuclear pageflip.
And if we _really_ want such semantics, we can always get them by
introducing another pageflip mutex between the mode_config.mutex and
the individual crtc locks. Pageflips crossing more than one crtc
would then need to take that lock first, to lock out concurrent
multi-crtc pageflips.
- Optimized global modeset operations: We could just take the
mode_config lock and then lazily lock all crtc which are affected by
a modeset operation. This has the advantage that pageflip could
continue unhampered on unaffected crtc. But if e.g. global resources
like plls need to be reassigned and so affect unrelated crtcs we can
still do that - nested locking works in any order.
This patch just adds the locks and takes them in drm_modeset_lock_all,
no real locking changes yet.
v2: Need to initialize the new lock in crtc_init and lock it righ
away, for otherwise the modeset_unlock_all below will try to unlock a
not-locked mutex.
Reviewed-by: Rob Clark <rob@ti.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2012-12-02 08:18:25 +07:00
|
|
|
|
2008-11-08 05:05:41 +07:00
|
|
|
struct drm_mode_object base;
|
|
|
|
|
2014-04-02 05:22:38 +07:00
|
|
|
/* primary and cursor planes for CRTC */
|
|
|
|
struct drm_plane *primary;
|
|
|
|
struct drm_plane *cursor;
|
|
|
|
|
2014-06-10 22:28:10 +07:00
|
|
|
/* position of cursor plane on crtc */
|
|
|
|
int cursor_x;
|
|
|
|
int cursor_y;
|
|
|
|
|
2008-11-08 05:05:41 +07:00
|
|
|
bool enabled;
|
|
|
|
|
drm/vblank: Add support for precise vblank timestamping.
The DRI2 swap & sync implementation needs precise
vblank counts and precise timestamps corresponding
to those vblank counts. For conformance to the OpenML
OML_sync_control extension specification the DRM
timestamp associated with a vblank count should
correspond to the start of video scanout of the first
scanline of the video frame following the vblank
interval for that vblank count.
Therefore we need to carry around precise timestamps
for vblanks. Currently the DRM and KMS drivers generate
timestamps ad-hoc via do_gettimeofday() in some
places. The resulting timestamps are sometimes not
very precise due to interrupt handling delays, they
don't conform to OML_sync_control and some are wrong,
as they aren't taken synchronized to the vblank.
This patch implements support inside the drm core
for precise and robust timestamping. It consists
of the following interrelated pieces.
1. Vblank timestamp caching:
A per-crtc ringbuffer stores the most recent vblank
timestamps corresponding to vblank counts.
The ringbuffer can be read out lock-free via the
accessor function:
struct timeval timestamp;
vblankcount = drm_vblank_count_and_time(dev, crtcid, ×tamp).
The function returns the current vblank count and
the corresponding timestamp for start of video
scanout following the vblank interval. It can be
used anywhere between enclosing drm_vblank_get(dev, crtcid)
and drm_vblank_put(dev,crtcid) statements. It is used
inside the drmWaitVblank ioctl and in the vblank event
queueing and handling. It should be used by kms drivers for
timestamping of bufferswap completion.
The timestamp ringbuffer is reinitialized each time
vblank irq's get reenabled in drm_vblank_get()/
drm_update_vblank_count(). It is invalidated when
vblank irq's get disabled.
The ringbuffer is updated inside drm_handle_vblank()
at each vblank irq.
2. Calculation of precise vblank timestamps:
drm_get_last_vbltimestamp() is used to compute the
timestamp for the end of the most recent vblank (if
inside active scanout), or the expected end of the
current vblank interval (if called inside a vblank
interval). The function calls into a new optional kms
driver entry point dev->driver->get_vblank_timestamp()
which is supposed to provide the precise timestamp.
If a kms driver doesn't implement the entry point or
if the call fails, a simple do_gettimeofday() timestamp
is returned as crude approximation of the true vblank time.
A new drm module parameter drm.timestamp_precision_usec
allows to disable high precision timestamps (if set to
zero) or to specify the maximum acceptable error in
the timestamps in microseconds.
Kms drivers could implement their get_vblank_timestamp()
function in a gpu specific way, as long as returned
timestamps conform to OML_sync_control, e.g., by use
of gpu specific hardware timestamps.
Optionally, kms drivers can simply wrap and use the new
utility function drm_calc_vbltimestamp_from_scanoutpos().
This function calls a new optional kms driver function
dev->driver->get_scanout_position() which returns the
current horizontal and vertical video scanout position
of the crtc. The scanout position together with the
drm_display_timing of the current video mode is used
to calculate elapsed time relative to start of active scanout
for the current video frame. This elapsed time is subtracted
from the current do_gettimeofday() time to get the timestamp
corresponding to start of video scanout. Currently
non-interlaced, non-doublescan video modes, with or
without panel scaling are handled correctly. Interlaced/
doublescan modes are tbd in a future patch.
3. Filtering of redundant vblank irq's and removal of
some race-conditions in the vblank irq enable/disable path:
Some gpu's (e.g., Radeon R500/R600) send spurious vblank
irq's outside the vblank if vblank irq's get reenabled.
These get detected by use of the vblank timestamps and
filtered out to avoid miscounting of vblanks.
Some race-conditions between the vblank irq enable/disable
functions, the vblank irq handler and the gpu itself (updating
its hardware vblank counter in the "wrong" moment) are
fixed inside vblank_disable_and_save() and
drm_update_vblank_count() by use of the vblank timestamps and
a new spinlock dev->vblank_time_lock.
The time until vblank irq disable is now configurable via
a new drm module parameter drm.vblankoffdelay to allow
experimentation with timeouts that are much shorter than
the current 5 seconds and should allow longer vblank off
periods for better power savings.
Followup patches will use these new functions to
implement precise timestamping for the intel and radeon
kms drivers.
Signed-off-by: Mario Kleiner <mario.kleiner@tuebingen.mpg.de>
Signed-off-by: Dave Airlie <airlied@redhat.com>
2010-10-23 09:20:23 +07:00
|
|
|
/* Requested mode from modesetting. */
|
2008-11-08 05:05:41 +07:00
|
|
|
struct drm_display_mode mode;
|
|
|
|
|
drm/vblank: Add support for precise vblank timestamping.
The DRI2 swap & sync implementation needs precise
vblank counts and precise timestamps corresponding
to those vblank counts. For conformance to the OpenML
OML_sync_control extension specification the DRM
timestamp associated with a vblank count should
correspond to the start of video scanout of the first
scanline of the video frame following the vblank
interval for that vblank count.
Therefore we need to carry around precise timestamps
for vblanks. Currently the DRM and KMS drivers generate
timestamps ad-hoc via do_gettimeofday() in some
places. The resulting timestamps are sometimes not
very precise due to interrupt handling delays, they
don't conform to OML_sync_control and some are wrong,
as they aren't taken synchronized to the vblank.
This patch implements support inside the drm core
for precise and robust timestamping. It consists
of the following interrelated pieces.
1. Vblank timestamp caching:
A per-crtc ringbuffer stores the most recent vblank
timestamps corresponding to vblank counts.
The ringbuffer can be read out lock-free via the
accessor function:
struct timeval timestamp;
vblankcount = drm_vblank_count_and_time(dev, crtcid, ×tamp).
The function returns the current vblank count and
the corresponding timestamp for start of video
scanout following the vblank interval. It can be
used anywhere between enclosing drm_vblank_get(dev, crtcid)
and drm_vblank_put(dev,crtcid) statements. It is used
inside the drmWaitVblank ioctl and in the vblank event
queueing and handling. It should be used by kms drivers for
timestamping of bufferswap completion.
The timestamp ringbuffer is reinitialized each time
vblank irq's get reenabled in drm_vblank_get()/
drm_update_vblank_count(). It is invalidated when
vblank irq's get disabled.
The ringbuffer is updated inside drm_handle_vblank()
at each vblank irq.
2. Calculation of precise vblank timestamps:
drm_get_last_vbltimestamp() is used to compute the
timestamp for the end of the most recent vblank (if
inside active scanout), or the expected end of the
current vblank interval (if called inside a vblank
interval). The function calls into a new optional kms
driver entry point dev->driver->get_vblank_timestamp()
which is supposed to provide the precise timestamp.
If a kms driver doesn't implement the entry point or
if the call fails, a simple do_gettimeofday() timestamp
is returned as crude approximation of the true vblank time.
A new drm module parameter drm.timestamp_precision_usec
allows to disable high precision timestamps (if set to
zero) or to specify the maximum acceptable error in
the timestamps in microseconds.
Kms drivers could implement their get_vblank_timestamp()
function in a gpu specific way, as long as returned
timestamps conform to OML_sync_control, e.g., by use
of gpu specific hardware timestamps.
Optionally, kms drivers can simply wrap and use the new
utility function drm_calc_vbltimestamp_from_scanoutpos().
This function calls a new optional kms driver function
dev->driver->get_scanout_position() which returns the
current horizontal and vertical video scanout position
of the crtc. The scanout position together with the
drm_display_timing of the current video mode is used
to calculate elapsed time relative to start of active scanout
for the current video frame. This elapsed time is subtracted
from the current do_gettimeofday() time to get the timestamp
corresponding to start of video scanout. Currently
non-interlaced, non-doublescan video modes, with or
without panel scaling are handled correctly. Interlaced/
doublescan modes are tbd in a future patch.
3. Filtering of redundant vblank irq's and removal of
some race-conditions in the vblank irq enable/disable path:
Some gpu's (e.g., Radeon R500/R600) send spurious vblank
irq's outside the vblank if vblank irq's get reenabled.
These get detected by use of the vblank timestamps and
filtered out to avoid miscounting of vblanks.
Some race-conditions between the vblank irq enable/disable
functions, the vblank irq handler and the gpu itself (updating
its hardware vblank counter in the "wrong" moment) are
fixed inside vblank_disable_and_save() and
drm_update_vblank_count() by use of the vblank timestamps and
a new spinlock dev->vblank_time_lock.
The time until vblank irq disable is now configurable via
a new drm module parameter drm.vblankoffdelay to allow
experimentation with timeouts that are much shorter than
the current 5 seconds and should allow longer vblank off
periods for better power savings.
Followup patches will use these new functions to
implement precise timestamping for the intel and radeon
kms drivers.
Signed-off-by: Mario Kleiner <mario.kleiner@tuebingen.mpg.de>
Signed-off-by: Dave Airlie <airlied@redhat.com>
2010-10-23 09:20:23 +07:00
|
|
|
/* Programmed mode in hw, after adjustments for encoders,
|
|
|
|
* crtc, panel scaling etc. Needed for timestamping etc.
|
|
|
|
*/
|
|
|
|
struct drm_display_mode hwmode;
|
|
|
|
|
2008-11-08 05:05:41 +07:00
|
|
|
int x, y;
|
|
|
|
const struct drm_crtc_funcs *funcs;
|
|
|
|
|
|
|
|
/* CRTC gamma size for reporting to userspace */
|
|
|
|
uint32_t gamma_size;
|
|
|
|
uint16_t *gamma_store;
|
|
|
|
|
|
|
|
/* if you are using the helper */
|
2015-12-04 15:45:45 +07:00
|
|
|
const struct drm_crtc_helper_funcs *helper_private;
|
2012-05-16 04:09:05 +07:00
|
|
|
|
|
|
|
struct drm_object_properties properties;
|
2014-07-25 23:07:40 +07:00
|
|
|
|
2014-10-28 02:28:44 +07:00
|
|
|
struct drm_crtc_state *state;
|
|
|
|
|
2014-07-25 23:07:40 +07:00
|
|
|
/*
|
2015-12-04 15:46:01 +07:00
|
|
|
* For legacy crtc IOCTLs so that atomic drivers can get at the locking
|
2014-07-25 23:07:40 +07:00
|
|
|
* acquire context.
|
|
|
|
*/
|
|
|
|
struct drm_modeset_acquire_ctx *acquire_ctx;
|
2008-11-08 05:05:41 +07:00
|
|
|
};
|
|
|
|
|
2014-10-28 02:28:44 +07:00
|
|
|
/**
|
|
|
|
* struct drm_connector_state - mutable connector state
|
2014-11-27 21:49:39 +07:00
|
|
|
* @connector: backpointer to the connector
|
2014-07-26 02:30:38 +07:00
|
|
|
* @crtc: CRTC to connect connector to, NULL if disabled
|
drm: Atomic crtc/connector updates using crtc/plane helper interfaces
So this is finally the integration of the crtc and plane helper
interfaces into the atomic helper functions.
In the check function we now have a few steps:
- First we update the output routing and figure out which crtcs need a
full mode set. Suitable encoders are selected using ->best_encoder,
with the same semantics as the crtc helpers of implicitly disabling
all connectors currently using the encoder.
- Then we pull all other connectors into the state update which feed
from a crtc which changes. This must be done do catch mode changes
and similar updates - atomic updates are differences on top of the
current state.
- Then we call all the various ->mode_fixup to compute the adjusted
mode. Note that here we have a slight semantic difference compared
to the crtc helpers: We have not yet updated the encoder->crtc link
when calling the encoder's ->mode_fixup function. But that's a
requirement when converting to atomic since we want to prepare the
entire state completely contained with the over drm_atomic_state
structure. So this must be carefully checked when converting drivers
over to atomic helpers.
- Finally we do call the atomic_check functions on planes and crtcs.
The commit function is also quite a beast:
- The only step that can fail is done first, namely pinning the
framebuffers. After that we cross the point of no return, an async
commit would push all that into the worker thread.
- The disabling of encoders and connectors is a bit tricky, since
depending upon the final state we need to select different crtc
helper functions.
- Software tracking is a bit clarified compared to the crtc helpers:
We commit the software state before starting to touch the hardware,
like crtc helpers. But since we just swap them we still have the old
state (i.e. the current hw state) around, which is really handy to
write simple disable functions. So no more
drm_crtc_helper_disable_all_unused_functions kind of fun because
we're leaving unused crtcs/encoders behind. Everything gets shut
down in-order now, which is one of the key differences of the i915
helpers compared to crtc helpers and a really nice additional
guarantee.
- Like with the plane helpers the atomic commit function waits for one
vblank to pass before calling the framebuffer cleanup function.
Compared to Rob's helper approach there's a bunch of upsides:
- All the interfaces which can fail are called in the ->check hook
(i.e. ->best_match and the various ->mode_fixup hooks). This means
that drivers can just reuse those functions and don't need to move
everything into ->atomic_check callbacks. If drivers have no need
for additional constraint checking beyong their existing crtc
helper callbacks they don't need to do anything.
- The actual commit operation is properly stage: First we prepare
framebuffers, which can potentially still fail (due to memory
exhausting). This is important for the async case, where this must
be done synchronously to correctly return errors.
- The output configuration changes (done with crtc helper functions)
and the plane update (using atomic plane helpers) are correctly
interleaved: First we shut down any crtcs that need changing, then
we update planes and finally we enable everything again. Hardware
without GO bits must be more careful with ordering, which this
sequence enables.
- Also for hardware with shared output resources (like display PLLs)
we first must shut down the old configuration before we can enable
the new one. Otherwise we can hit an impossible intermediate state
where there's not enough PLLs (which is the point behind atomic
updates).
v2:
- Ensure that users of ->check update crtc_state->enable correctly.
- Update the legacy state in crtc/plane structures. Eventually we want
to remove that, but for now the drm core still expects this (especially
the plane->fb pointer).
v3: A few changes for better async handling:
- Reorder the software side state commit so that it happens all before
we touch the hardware. This way async support becomes very easy
since we can punt all the actual hw touching to a worker thread. And
as long as we synchronize with that thread (flushing or cancelling,
depending upon what the driver can handle) before we commit the next
software state there's no need for any locking in the worker thread
at all. Which greatly simplifies things.
And as long as we synchronize with all relevant threads we can have
a lot of them (e.g. per-crtc for per-crtc updates) running in
parallel.
- Expose pre/post plane commit steps separately. We need to expose the
actual hw commit step anyway for drivers to be able to implement
asynchronous commit workers. But if we expose pre/post and plane
commit steps individually we allow drivers to selectively use atomic
helpers.
- I've forgotten to call encoder/bridge ->mode_set functions, fix
this.
v4: Add debug output and fix a mixup between current and new state
that resulted in crtcs not getting updated correctly. And in an
Oops ...
v5:
- Be kind to driver writers in the vblank wait functions.. if thing
aren't working yet, and vblank irq will never come, then let's not
block forever.. especially under console-lock.
- Correctly clear connector_state->best_encoder when disabling.
Spotted while trying to understand a report from Rob Clark.
- Only steal encoder if it actually changed, otherwise hilarity ensues
if we steal from the current connector and so set the ->crtc pointer
unexpectedly to NULL. Reported by Rob Clark.
- Bail out in disable_outputs if an output currently doesn't have a
best_encoder - this means it's already disabled.
v6: Fixupe kerneldoc as reported by Paulo. And also fix up kerneldoc
in drm_crtc.h.
v7: Take ownership of the atomic state and clean it up with
drm_atomic_state_free().
v8 Various improvements all over:
- Polish code comments and kerneldoc.
- Improve debug output to make sure all failure cases are logged.
- Treat enabled crtc with no connectors as invalid input from userspace.
- Don't ignore the return value from mode_fixup().
v9:
- Improve debug output for crtc_state->mode_changed.
v10:
- Fixup the vblank waiting code to properly balance the vblank_get/put
calls.
- Better comments when checking/computing crtc->mode_changed
v11: Fixup the encoder stealing logic: We can't look at encoder->crtc
since that's not in the atomic state structures and might be updated
asynchronously in and async commit. Instead we need to inspect all the
connector states and check whether the encoder is currently in used
and if so, on which crtc.
v12: Review from Sean:
- A few spelling fixes.
- Flatten control flow indent by converting if blocks to early
continue/return in 2 places.
- Capture connectors_for_crtc return value in int num_connectors
instead of bool has_connectors and do an explicit int->bool
conversion with !!. I think the helper is more useful for drivers if
it returns the number of connectors (e.g. to detect cloning
configurations), so decided to keep that return value.
Cc: Sean Paul <seanpaul@chromium.org>
Cc: Paulo Zanoni <przanoni@gmail.com>
Cc: Rob Clark <robdclark@gmail.com>
Reviewed-by: Sean Paul <seanpaul@chromium.org>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-09-16 22:50:47 +07:00
|
|
|
* @best_encoder: can be used by helpers and drivers to select the encoder
|
2014-10-28 02:28:44 +07:00
|
|
|
* @state: backpointer to global drm_atomic_state
|
|
|
|
*/
|
|
|
|
struct drm_connector_state {
|
2014-11-27 21:49:39 +07:00
|
|
|
struct drm_connector *connector;
|
|
|
|
|
2014-11-22 03:28:31 +07:00
|
|
|
struct drm_crtc *crtc; /* do not write directly, use drm_atomic_set_crtc_for_connector() */
|
2014-10-28 02:28:44 +07:00
|
|
|
|
drm: Atomic crtc/connector updates using crtc/plane helper interfaces
So this is finally the integration of the crtc and plane helper
interfaces into the atomic helper functions.
In the check function we now have a few steps:
- First we update the output routing and figure out which crtcs need a
full mode set. Suitable encoders are selected using ->best_encoder,
with the same semantics as the crtc helpers of implicitly disabling
all connectors currently using the encoder.
- Then we pull all other connectors into the state update which feed
from a crtc which changes. This must be done do catch mode changes
and similar updates - atomic updates are differences on top of the
current state.
- Then we call all the various ->mode_fixup to compute the adjusted
mode. Note that here we have a slight semantic difference compared
to the crtc helpers: We have not yet updated the encoder->crtc link
when calling the encoder's ->mode_fixup function. But that's a
requirement when converting to atomic since we want to prepare the
entire state completely contained with the over drm_atomic_state
structure. So this must be carefully checked when converting drivers
over to atomic helpers.
- Finally we do call the atomic_check functions on planes and crtcs.
The commit function is also quite a beast:
- The only step that can fail is done first, namely pinning the
framebuffers. After that we cross the point of no return, an async
commit would push all that into the worker thread.
- The disabling of encoders and connectors is a bit tricky, since
depending upon the final state we need to select different crtc
helper functions.
- Software tracking is a bit clarified compared to the crtc helpers:
We commit the software state before starting to touch the hardware,
like crtc helpers. But since we just swap them we still have the old
state (i.e. the current hw state) around, which is really handy to
write simple disable functions. So no more
drm_crtc_helper_disable_all_unused_functions kind of fun because
we're leaving unused crtcs/encoders behind. Everything gets shut
down in-order now, which is one of the key differences of the i915
helpers compared to crtc helpers and a really nice additional
guarantee.
- Like with the plane helpers the atomic commit function waits for one
vblank to pass before calling the framebuffer cleanup function.
Compared to Rob's helper approach there's a bunch of upsides:
- All the interfaces which can fail are called in the ->check hook
(i.e. ->best_match and the various ->mode_fixup hooks). This means
that drivers can just reuse those functions and don't need to move
everything into ->atomic_check callbacks. If drivers have no need
for additional constraint checking beyong their existing crtc
helper callbacks they don't need to do anything.
- The actual commit operation is properly stage: First we prepare
framebuffers, which can potentially still fail (due to memory
exhausting). This is important for the async case, where this must
be done synchronously to correctly return errors.
- The output configuration changes (done with crtc helper functions)
and the plane update (using atomic plane helpers) are correctly
interleaved: First we shut down any crtcs that need changing, then
we update planes and finally we enable everything again. Hardware
without GO bits must be more careful with ordering, which this
sequence enables.
- Also for hardware with shared output resources (like display PLLs)
we first must shut down the old configuration before we can enable
the new one. Otherwise we can hit an impossible intermediate state
where there's not enough PLLs (which is the point behind atomic
updates).
v2:
- Ensure that users of ->check update crtc_state->enable correctly.
- Update the legacy state in crtc/plane structures. Eventually we want
to remove that, but for now the drm core still expects this (especially
the plane->fb pointer).
v3: A few changes for better async handling:
- Reorder the software side state commit so that it happens all before
we touch the hardware. This way async support becomes very easy
since we can punt all the actual hw touching to a worker thread. And
as long as we synchronize with that thread (flushing or cancelling,
depending upon what the driver can handle) before we commit the next
software state there's no need for any locking in the worker thread
at all. Which greatly simplifies things.
And as long as we synchronize with all relevant threads we can have
a lot of them (e.g. per-crtc for per-crtc updates) running in
parallel.
- Expose pre/post plane commit steps separately. We need to expose the
actual hw commit step anyway for drivers to be able to implement
asynchronous commit workers. But if we expose pre/post and plane
commit steps individually we allow drivers to selectively use atomic
helpers.
- I've forgotten to call encoder/bridge ->mode_set functions, fix
this.
v4: Add debug output and fix a mixup between current and new state
that resulted in crtcs not getting updated correctly. And in an
Oops ...
v5:
- Be kind to driver writers in the vblank wait functions.. if thing
aren't working yet, and vblank irq will never come, then let's not
block forever.. especially under console-lock.
- Correctly clear connector_state->best_encoder when disabling.
Spotted while trying to understand a report from Rob Clark.
- Only steal encoder if it actually changed, otherwise hilarity ensues
if we steal from the current connector and so set the ->crtc pointer
unexpectedly to NULL. Reported by Rob Clark.
- Bail out in disable_outputs if an output currently doesn't have a
best_encoder - this means it's already disabled.
v6: Fixupe kerneldoc as reported by Paulo. And also fix up kerneldoc
in drm_crtc.h.
v7: Take ownership of the atomic state and clean it up with
drm_atomic_state_free().
v8 Various improvements all over:
- Polish code comments and kerneldoc.
- Improve debug output to make sure all failure cases are logged.
- Treat enabled crtc with no connectors as invalid input from userspace.
- Don't ignore the return value from mode_fixup().
v9:
- Improve debug output for crtc_state->mode_changed.
v10:
- Fixup the vblank waiting code to properly balance the vblank_get/put
calls.
- Better comments when checking/computing crtc->mode_changed
v11: Fixup the encoder stealing logic: We can't look at encoder->crtc
since that's not in the atomic state structures and might be updated
asynchronously in and async commit. Instead we need to inspect all the
connector states and check whether the encoder is currently in used
and if so, on which crtc.
v12: Review from Sean:
- A few spelling fixes.
- Flatten control flow indent by converting if blocks to early
continue/return in 2 places.
- Capture connectors_for_crtc return value in int num_connectors
instead of bool has_connectors and do an explicit int->bool
conversion with !!. I think the helper is more useful for drivers if
it returns the number of connectors (e.g. to detect cloning
configurations), so decided to keep that return value.
Cc: Sean Paul <seanpaul@chromium.org>
Cc: Paulo Zanoni <przanoni@gmail.com>
Cc: Rob Clark <robdclark@gmail.com>
Reviewed-by: Sean Paul <seanpaul@chromium.org>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2014-09-16 22:50:47 +07:00
|
|
|
struct drm_encoder *best_encoder;
|
|
|
|
|
2014-10-28 02:28:44 +07:00
|
|
|
struct drm_atomic_state *state;
|
|
|
|
};
|
2008-11-08 05:05:41 +07:00
|
|
|
|
|
|
|
/**
|
2014-10-27 22:54:27 +07:00
|
|
|
* struct drm_connector_funcs - control connectors on a given device
|
2014-10-28 02:28:44 +07:00
|
|
|
*
|
2008-11-08 05:05:41 +07:00
|
|
|
* Each CRTC may have one or more connectors attached to it. The functions
|
|
|
|
* below allow the core DRM code to control connectors, enumerate available modes,
|
|
|
|
* etc.
|
|
|
|
*/
|
|
|
|
struct drm_connector_funcs {
|
2015-12-04 15:45:58 +07:00
|
|
|
/**
|
|
|
|
* @dpms:
|
|
|
|
*
|
|
|
|
* Legacy entry point to set the per-connector DPMS state. Legacy DPMS
|
|
|
|
* is exposed as a standard property on the connector, but diverted to
|
|
|
|
* this callback in the drm core. Note that atomic drivers don't
|
|
|
|
* implement the 4 level DPMS support on the connector any more, but
|
|
|
|
* instead only have an on/off "ACTIVE" property on the CRTC object.
|
|
|
|
*
|
|
|
|
* Drivers implementing atomic modeset should use
|
|
|
|
* drm_atomic_helper_connector_dpms() to implement this hook.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 on success or a negative error code on failure.
|
|
|
|
*/
|
2015-07-21 16:34:55 +07:00
|
|
|
int (*dpms)(struct drm_connector *connector, int mode);
|
2015-12-04 15:45:48 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @reset:
|
|
|
|
*
|
|
|
|
* Reset connector hardware and software state to off. This function isn't
|
|
|
|
* called by the core directly, only through drm_mode_config_reset().
|
|
|
|
* It's not a helper hook only for historical reasons.
|
|
|
|
*
|
|
|
|
* Atomic drivers can use drm_atomic_helper_connector_reset() to reset
|
|
|
|
* atomic state using this hook.
|
|
|
|
*/
|
2011-01-24 22:11:08 +07:00
|
|
|
void (*reset)(struct drm_connector *connector);
|
2010-09-14 17:07:23 +07:00
|
|
|
|
2015-12-04 15:45:58 +07:00
|
|
|
/**
|
|
|
|
* @detect:
|
|
|
|
*
|
|
|
|
* Check to see if anything is attached to the connector. The parameter
|
|
|
|
* force is set to false whilst polling, true when checking the
|
|
|
|
* connector due to a user request. force can be used by the driver to
|
|
|
|
* avoid expensive, destructive operations during automated probing.
|
|
|
|
*
|
|
|
|
* FIXME:
|
|
|
|
*
|
|
|
|
* Note that this hook is only called by the probe helper. It's not in
|
|
|
|
* the helper library vtable purely for historical reasons. The only DRM
|
|
|
|
* core entry point to probe connector state is @fill_modes.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* drm_connector_status indicating the connector's status.
|
2010-09-14 17:07:23 +07:00
|
|
|
*/
|
2010-09-10 05:51:02 +07:00
|
|
|
enum drm_connector_status (*detect)(struct drm_connector *connector,
|
2010-09-14 17:07:23 +07:00
|
|
|
bool force);
|
2015-12-04 15:45:58 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @force:
|
|
|
|
*
|
|
|
|
* This function is called to update internal encoder state when the
|
|
|
|
* connector is forced to a certain state by userspace, either through
|
|
|
|
* the sysfs interfaces or on the kernel cmdline. In that case the
|
|
|
|
* @detect callback isn't called.
|
|
|
|
*
|
|
|
|
* FIXME:
|
|
|
|
*
|
|
|
|
* Note that this hook is only called by the probe helper. It's not in
|
|
|
|
* the helper library vtable purely for historical reasons. The only DRM
|
|
|
|
* core entry point to probe connector state is @fill_modes.
|
|
|
|
*/
|
|
|
|
void (*force)(struct drm_connector *connector);
|
|
|
|
|
|
|
|
/**
|
|
|
|
* @fill_modes:
|
|
|
|
*
|
|
|
|
* Entry point for output detection and basic mode validation. The
|
|
|
|
* driver should reprobe the output if needed (e.g. when hotplug
|
|
|
|
* handling is unreliable), add all detected modes to connector->modes
|
|
|
|
* and filter out any the device can't support in any configuration. It
|
|
|
|
* also needs to filter out any modes wider or higher than the
|
|
|
|
* parameters max_width and max_height indicate.
|
|
|
|
*
|
|
|
|
* The drivers must also prune any modes no longer valid from
|
|
|
|
* connector->modes. Furthermore it must update connector->status and
|
|
|
|
* connector->edid. If no EDID has been received for this output
|
|
|
|
* connector->edid must be NULL.
|
|
|
|
*
|
|
|
|
* Drivers using the probe helpers should use
|
|
|
|
* drm_helper_probe_single_connector_modes() or
|
|
|
|
* drm_helper_probe_single_connector_modes_nomerge() to implement this
|
|
|
|
* function.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* The number of modes detected and filled into connector->modes.
|
|
|
|
*/
|
drm: initial KMS config fixes
When mode setting is first initialized, the driver will call into
drm_helper_initial_config() to set up an initial output and framebuffer
configuration. This routine is responsible for probing the available
connectors, encoders, and crtcs, looking for modes and putting together
something reasonable (where reasonable is defined as "allows kernel
messages to be visible on as many displays as possible").
However, the code was a bit too aggressive in setting default modes when
none were found on a given connector. Even if some connectors had modes,
any connectors found lacking modes would have the default 800x600 mode added
to their mode list, which in some cases could cause problems later down the
line. In my case, the LVDS was perfectly available, but the initial config
code added 800x600 modes to both of the detected but unavailable HDMI
connectors (which are on my non-existent docking station). This ended up
preventing later code from setting a mode on my LVDS, which is bad.
This patch fixes that behavior by making the initial config code walk
through the connectors first, counting the available modes, before it decides
to add any default modes to a possibly connected output. It also fixes the
logic in drm_target_preferred() that was causing zeroed out modes to be set
as the preferred mode for a given connector, even if no modes were available.
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Signed-off-by: Eric Anholt <eric@anholt.net>
Signed-off-by: Dave Airlie <airlied@linux.ie>
2009-01-13 03:05:32 +07:00
|
|
|
int (*fill_modes)(struct drm_connector *connector, uint32_t max_width, uint32_t max_height);
|
2015-12-04 15:45:48 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @set_property:
|
|
|
|
*
|
|
|
|
* This is the legacy entry point to update a property attached to the
|
|
|
|
* connector.
|
|
|
|
*
|
|
|
|
* Drivers implementing atomic modeset should use
|
|
|
|
* drm_atomic_helper_connector_set_property() to implement this hook.
|
|
|
|
*
|
|
|
|
* This callback is optional if the driver does not support any legacy
|
|
|
|
* driver-private properties.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 on success or a negative error code on failure.
|
|
|
|
*/
|
2008-11-08 05:05:41 +07:00
|
|
|
int (*set_property)(struct drm_connector *connector, struct drm_property *property,
|
|
|
|
uint64_t val);
|
2015-12-04 15:45:48 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @destroy:
|
|
|
|
*
|
|
|
|
* Clean up connector resources. This is called at driver unload time
|
|
|
|
* through drm_mode_config_cleanup(). It can also be called at runtime
|
|
|
|
* when a connector is being hot-unplugged for drivers that support
|
|
|
|
* connector hotplugging (e.g. DisplayPort MST).
|
|
|
|
*/
|
2008-11-08 05:05:41 +07:00
|
|
|
void (*destroy)(struct drm_connector *connector);
|
2014-10-28 02:28:44 +07:00
|
|
|
|
2015-12-04 15:45:48 +07:00
|
|
|
/**
|
|
|
|
* @atomic_duplicate_state:
|
|
|
|
*
|
|
|
|
* Duplicate the current atomic state for this connector and return it.
|
|
|
|
* The core and helpers gurantee that any atomic state duplicated with
|
|
|
|
* this hook and still owned by the caller (i.e. not transferred to the
|
|
|
|
* driver by calling ->atomic_commit() from struct
|
|
|
|
* &drm_mode_config_funcs) will be cleaned up by calling the
|
|
|
|
* @atomic_destroy_state hook in this structure.
|
|
|
|
*
|
|
|
|
* Atomic drivers which don't subclass struct &drm_connector_state should use
|
|
|
|
* drm_atomic_helper_connector_duplicate_state(). Drivers that subclass the
|
|
|
|
* state structure to extend it with driver-private state should use
|
|
|
|
* __drm_atomic_helper_connector_duplicate_state() to make sure shared state is
|
|
|
|
* duplicated in a consistent fashion across drivers.
|
|
|
|
*
|
|
|
|
* It is an error to call this hook before connector->state has been
|
|
|
|
* initialized correctly.
|
|
|
|
*
|
|
|
|
* NOTE:
|
|
|
|
*
|
|
|
|
* If the duplicate state references refcounted resources this hook must
|
|
|
|
* acquire a reference for each of them. The driver must release these
|
|
|
|
* references again in @atomic_destroy_state.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* Duplicated atomic state or NULL when the allocation failed.
|
|
|
|
*/
|
2014-10-28 02:28:44 +07:00
|
|
|
struct drm_connector_state *(*atomic_duplicate_state)(struct drm_connector *connector);
|
2015-12-04 15:45:48 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @atomic_destroy_state:
|
|
|
|
*
|
|
|
|
* Destroy a state duplicated with @atomic_duplicate_state and release
|
|
|
|
* or unreference all resources it references
|
|
|
|
*/
|
2014-10-28 02:28:44 +07:00
|
|
|
void (*atomic_destroy_state)(struct drm_connector *connector,
|
2014-07-26 02:30:38 +07:00
|
|
|
struct drm_connector_state *state);
|
2015-12-04 15:45:48 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @atomic_set_property:
|
|
|
|
*
|
|
|
|
* Decode a driver-private property value and store the decoded value
|
|
|
|
* into the passed-in state structure. Since the atomic core decodes all
|
|
|
|
* standardized properties (even for extensions beyond the core set of
|
|
|
|
* properties which might not be implemented by all drivers) this
|
|
|
|
* requires drivers to subclass the state structure.
|
|
|
|
*
|
|
|
|
* Such driver-private properties should really only be implemented for
|
|
|
|
* truly hardware/vendor specific state. Instead it is preferred to
|
|
|
|
* standardize atomic extension and decode the properties used to expose
|
|
|
|
* such an extension in the core.
|
|
|
|
*
|
|
|
|
* Do not call this function directly, use
|
|
|
|
* drm_atomic_connector_set_property() instead.
|
|
|
|
*
|
|
|
|
* This callback is optional if the driver does not support any
|
|
|
|
* driver-private atomic properties.
|
|
|
|
*
|
|
|
|
* NOTE:
|
|
|
|
*
|
|
|
|
* This function is called in the state assembly phase of atomic
|
|
|
|
* modesets, which can be aborted for any reason (including on
|
|
|
|
* userspace's request to just check whether a configuration would be
|
|
|
|
* possible). Drivers MUST NOT touch any persistent state (hardware or
|
|
|
|
* software) or data structures except the passed in @state parameter.
|
|
|
|
*
|
|
|
|
* Also since userspace controls in which order properties are set this
|
|
|
|
* function must not do any input validation (since the state update is
|
|
|
|
* incomplete and hence likely inconsistent). Instead any such input
|
|
|
|
* validation must be done in the various atomic_check callbacks.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 if the property has been found, -EINVAL if the property isn't
|
|
|
|
* implemented by the driver (which shouldn't ever happen, the core only
|
|
|
|
* asks for properties attached to this connector). No other validation
|
|
|
|
* is allowed by the driver. The core already checks that the property
|
|
|
|
* value is within the range (integer, valid enum value, ...) the driver
|
|
|
|
* set when registering the property.
|
|
|
|
*/
|
2014-10-28 02:28:44 +07:00
|
|
|
int (*atomic_set_property)(struct drm_connector *connector,
|
|
|
|
struct drm_connector_state *state,
|
|
|
|
struct drm_property *property,
|
|
|
|
uint64_t val);
|
2015-12-04 15:45:48 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @atomic_get_property:
|
|
|
|
*
|
|
|
|
* Reads out the decoded driver-private property. This is used to
|
2015-12-04 15:46:01 +07:00
|
|
|
* implement the GETCONNECTOR IOCTL.
|
2015-12-04 15:45:48 +07:00
|
|
|
*
|
|
|
|
* Do not call this function directly, use
|
|
|
|
* drm_atomic_connector_get_property() instead.
|
|
|
|
*
|
|
|
|
* This callback is optional if the driver does not support any
|
|
|
|
* driver-private atomic properties.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 on success, -EINVAL if the property isn't implemented by the
|
|
|
|
* driver (which shouldn't ever happen, the core only asks for
|
|
|
|
* properties attached to this connector).
|
|
|
|
*/
|
2014-12-19 04:01:47 +07:00
|
|
|
int (*atomic_get_property)(struct drm_connector *connector,
|
|
|
|
const struct drm_connector_state *state,
|
|
|
|
struct drm_property *property,
|
|
|
|
uint64_t *val);
|
2008-11-08 05:05:41 +07:00
|
|
|
};
|
|
|
|
|
2011-11-08 03:03:16 +07:00
|
|
|
/**
|
2014-10-27 22:54:27 +07:00
|
|
|
* struct drm_encoder_funcs - encoder controls
|
2011-11-08 03:03:16 +07:00
|
|
|
*
|
|
|
|
* Encoders sit between CRTCs and connectors.
|
|
|
|
*/
|
2008-11-08 05:05:41 +07:00
|
|
|
struct drm_encoder_funcs {
|
2015-12-04 15:45:48 +07:00
|
|
|
/**
|
|
|
|
* @reset:
|
|
|
|
*
|
|
|
|
* Reset encoder hardware and software state to off. This function isn't
|
|
|
|
* called by the core directly, only through drm_mode_config_reset().
|
|
|
|
* It's not a helper hook only for historical reasons.
|
|
|
|
*/
|
2011-01-24 22:11:08 +07:00
|
|
|
void (*reset)(struct drm_encoder *encoder);
|
2015-12-04 15:45:48 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @destroy:
|
|
|
|
*
|
|
|
|
* Clean up encoder resources. This is only called at driver unload time
|
|
|
|
* through drm_mode_config_cleanup() since an encoder cannot be
|
|
|
|
* hotplugged in DRM.
|
|
|
|
*/
|
2008-11-08 05:05:41 +07:00
|
|
|
void (*destroy)(struct drm_encoder *encoder);
|
|
|
|
};
|
|
|
|
|
2012-01-12 13:00:57 +07:00
|
|
|
#define DRM_CONNECTOR_MAX_ENCODER 3
|
2008-11-08 05:05:41 +07:00
|
|
|
|
|
|
|
/**
|
2014-10-27 22:54:27 +07:00
|
|
|
* struct drm_encoder - central DRM encoder structure
|
2011-11-08 03:03:17 +07:00
|
|
|
* @dev: parent DRM device
|
|
|
|
* @head: list management
|
|
|
|
* @base: base KMS object
|
2014-05-14 20:58:20 +07:00
|
|
|
* @name: encoder name
|
2011-11-08 03:03:17 +07:00
|
|
|
* @encoder_type: one of the %DRM_MODE_ENCODER_<foo> types in drm_mode.h
|
|
|
|
* @possible_crtcs: bitmask of potential CRTC bindings
|
|
|
|
* @possible_clones: bitmask of potential sibling encoders for cloning
|
|
|
|
* @crtc: currently bound CRTC
|
2013-08-15 03:47:37 +07:00
|
|
|
* @bridge: bridge associated to the encoder
|
2011-11-08 03:03:17 +07:00
|
|
|
* @funcs: control functions
|
|
|
|
* @helper_private: mid-layer private data
|
|
|
|
*
|
|
|
|
* CRTCs drive pixels to encoders, which convert them into signals
|
|
|
|
* appropriate for a given connector or set of connectors.
|
2008-11-08 05:05:41 +07:00
|
|
|
*/
|
|
|
|
struct drm_encoder {
|
|
|
|
struct drm_device *dev;
|
|
|
|
struct list_head head;
|
|
|
|
|
|
|
|
struct drm_mode_object base;
|
2014-05-14 20:58:20 +07:00
|
|
|
char *name;
|
2008-11-08 05:05:41 +07:00
|
|
|
int encoder_type;
|
|
|
|
uint32_t possible_crtcs;
|
|
|
|
uint32_t possible_clones;
|
|
|
|
|
|
|
|
struct drm_crtc *crtc;
|
2013-08-15 03:47:37 +07:00
|
|
|
struct drm_bridge *bridge;
|
2008-11-08 05:05:41 +07:00
|
|
|
const struct drm_encoder_funcs *funcs;
|
2015-12-04 15:45:45 +07:00
|
|
|
const struct drm_encoder_helper_funcs *helper_private;
|
2008-11-08 05:05:41 +07:00
|
|
|
};
|
|
|
|
|
2010-05-07 13:42:51 +07:00
|
|
|
/* should we poll this connector for connects and disconnects */
|
|
|
|
/* hot plug detectable */
|
|
|
|
#define DRM_CONNECTOR_POLL_HPD (1 << 0)
|
|
|
|
/* poll for connections */
|
|
|
|
#define DRM_CONNECTOR_POLL_CONNECT (1 << 1)
|
|
|
|
/* can cleanly poll for disconnections without flickering the screen */
|
|
|
|
/* DACs should rarely do this without a lot of testing */
|
|
|
|
#define DRM_CONNECTOR_POLL_DISCONNECT (1 << 2)
|
|
|
|
|
2011-09-05 13:23:20 +07:00
|
|
|
#define MAX_ELD_BYTES 128
|
|
|
|
|
2008-11-08 05:05:41 +07:00
|
|
|
/**
|
2014-10-27 22:54:27 +07:00
|
|
|
* struct drm_connector - central DRM connector control structure
|
2011-11-08 03:03:18 +07:00
|
|
|
* @dev: parent DRM device
|
|
|
|
* @kdev: kernel device for sysfs attributes
|
|
|
|
* @attr: sysfs attributes
|
|
|
|
* @head: list management
|
|
|
|
* @base: base KMS object
|
2014-05-14 20:58:19 +07:00
|
|
|
* @name: connector name
|
2011-11-08 03:03:18 +07:00
|
|
|
* @connector_type: one of the %DRM_MODE_CONNECTOR_<foo> types from drm_mode.h
|
|
|
|
* @connector_type_id: index into connector type enum
|
2008-11-08 05:05:41 +07:00
|
|
|
* @interlace_allowed: can this connector handle interlaced modes?
|
|
|
|
* @doublescan_allowed: can this connector handle doublescan?
|
2014-10-28 02:19:38 +07:00
|
|
|
* @stereo_allowed: can this connector handle stereo modes?
|
2011-11-08 03:03:18 +07:00
|
|
|
* @modes: modes available on this connector (from fill_modes() + user)
|
|
|
|
* @status: one of the drm_connector_status enums (connected, not, or unknown)
|
|
|
|
* @probed_modes: list of modes derived directly from the display
|
|
|
|
* @display_info: information about attached display (e.g. from EDID)
|
2008-11-08 05:05:41 +07:00
|
|
|
* @funcs: connector control functions
|
2011-11-08 03:03:18 +07:00
|
|
|
* @edid_blob_ptr: DRM property containing EDID if present
|
2012-05-16 04:09:01 +07:00
|
|
|
* @properties: property tracking for this connector
|
2014-10-28 02:19:38 +07:00
|
|
|
* @path_blob_ptr: DRM blob property data for the DP MST path property
|
2011-11-08 03:03:18 +07:00
|
|
|
* @polled: a %DRM_CONNECTOR_POLL_<foo> value for core driven polling
|
|
|
|
* @dpms: current dpms state
|
|
|
|
* @helper_private: mid-layer private data
|
2014-10-28 02:19:38 +07:00
|
|
|
* @cmdline_mode: mode line parsed from the kernel cmdline for this connector
|
2011-11-08 03:03:18 +07:00
|
|
|
* @force: a %DRM_FORCE_<foo> state for forced mode sets
|
2014-10-28 02:19:38 +07:00
|
|
|
* @override_edid: has the EDID been overwritten through debugfs for testing?
|
2011-11-08 03:03:18 +07:00
|
|
|
* @encoder_ids: valid encoders for this connector
|
|
|
|
* @encoder: encoder driving this connector, if any
|
|
|
|
* @eld: EDID-like data, if present
|
|
|
|
* @dvi_dual: dual link DVI, if found
|
|
|
|
* @max_tmds_clock: max clock rate, if found
|
|
|
|
* @latency_present: AV delay info from ELD, if found
|
|
|
|
* @video_latency: video latency info from ELD, if found
|
|
|
|
* @audio_latency: audio latency info from ELD, if found
|
|
|
|
* @null_edid_counter: track sinks that give us all zeros for the EDID
|
2014-10-28 02:19:38 +07:00
|
|
|
* @bad_edid_counter: track sinks that give us an EDID with invalid checksum
|
2015-05-08 21:15:41 +07:00
|
|
|
* @edid_corrupt: indicates whether the last read EDID was corrupt
|
2014-10-28 02:19:38 +07:00
|
|
|
* @debugfs_entry: debugfs directory for this connector
|
2014-10-28 02:28:44 +07:00
|
|
|
* @state: current atomic state for this connector
|
2014-10-20 13:29:33 +07:00
|
|
|
* @has_tile: is this connector connected to a tiled monitor
|
|
|
|
* @tile_group: tile group for the connected monitor
|
|
|
|
* @tile_is_single_monitor: whether the tile is one monitor housing
|
|
|
|
* @num_h_tile: number of horizontal tiles in the tile group
|
|
|
|
* @num_v_tile: number of vertical tiles in the tile group
|
|
|
|
* @tile_h_loc: horizontal location of this tile
|
|
|
|
* @tile_v_loc: vertical location of this tile
|
|
|
|
* @tile_h_size: horizontal size of this tile.
|
|
|
|
* @tile_v_size: vertical size of this tile.
|
2008-11-08 05:05:41 +07:00
|
|
|
*
|
|
|
|
* Each connector may be connected to one or more CRTCs, or may be clonable by
|
|
|
|
* another connector if they can share a CRTC. Each connector also has a specific
|
|
|
|
* position in the broader display (referred to as a 'screen' though it could
|
|
|
|
* span multiple monitors).
|
|
|
|
*/
|
|
|
|
struct drm_connector {
|
|
|
|
struct drm_device *dev;
|
2013-10-11 11:07:25 +07:00
|
|
|
struct device *kdev;
|
2008-11-08 05:05:41 +07:00
|
|
|
struct device_attribute *attr;
|
|
|
|
struct list_head head;
|
|
|
|
|
|
|
|
struct drm_mode_object base;
|
|
|
|
|
2014-05-14 20:58:19 +07:00
|
|
|
char *name;
|
2008-11-08 05:05:41 +07:00
|
|
|
int connector_type;
|
|
|
|
int connector_type_id;
|
|
|
|
bool interlace_allowed;
|
|
|
|
bool doublescan_allowed;
|
2013-09-25 22:45:29 +07:00
|
|
|
bool stereo_allowed;
|
2008-11-08 05:05:41 +07:00
|
|
|
struct list_head modes; /* list of modes on this connector */
|
|
|
|
|
|
|
|
enum drm_connector_status status;
|
|
|
|
|
|
|
|
/* these are modes added by probing with DDC or the BIOS */
|
|
|
|
struct list_head probed_modes;
|
|
|
|
|
|
|
|
struct drm_display_info display_info;
|
|
|
|
const struct drm_connector_funcs *funcs;
|
|
|
|
|
|
|
|
struct drm_property_blob *edid_blob_ptr;
|
2012-05-16 04:09:01 +07:00
|
|
|
struct drm_object_properties properties;
|
2008-11-08 05:05:41 +07:00
|
|
|
|
2014-06-05 11:01:31 +07:00
|
|
|
struct drm_property_blob *path_blob_ptr;
|
|
|
|
|
2014-10-20 13:30:50 +07:00
|
|
|
struct drm_property_blob *tile_blob_ptr;
|
|
|
|
|
2010-05-07 13:42:51 +07:00
|
|
|
uint8_t polled; /* DRM_CONNECTOR_POLL_* */
|
|
|
|
|
2009-05-31 10:42:28 +07:00
|
|
|
/* requested DPMS state */
|
|
|
|
int dpms;
|
|
|
|
|
2015-12-04 15:45:45 +07:00
|
|
|
const struct drm_connector_helper_funcs *helper_private;
|
2008-11-08 05:05:41 +07:00
|
|
|
|
drm/kms: start adding command line interface using fb.
[note this requires an fb patch posted to linux-fbdev-devel already]
This uses the normal video= command line option to control the kms
output setup at boot time. It is used to override the autodetection
done by kms.
video= normally takes a framebuffer as the first parameter, in kms
it will take a connector name, DVI-I-1, or LVDS-1 etc. If no output
connector is specified the mode string will apply to all connectors.
The mode specification used will match down the probed modes, and if
no mode is found it will add a CVT mode that matches.
video=1024x768 - all connectors match a 1024x768 mode or add a CVT on
video=VGA-1:1024x768, VGA-1 connector gets mode only.
The same strings as used in current fb modedb.c are used, except I've
added three more letters, e, D, d, e = enable, D = enable Digital,
d = disable, which allow a connector to be forced into a certain state.
Signed-off-by: Dave Airlie <airlied@redhat.com>
2009-09-23 11:44:08 +07:00
|
|
|
/* forced on connector */
|
2014-08-06 15:08:32 +07:00
|
|
|
struct drm_cmdline_mode cmdline_mode;
|
drm/kms: start adding command line interface using fb.
[note this requires an fb patch posted to linux-fbdev-devel already]
This uses the normal video= command line option to control the kms
output setup at boot time. It is used to override the autodetection
done by kms.
video= normally takes a framebuffer as the first parameter, in kms
it will take a connector name, DVI-I-1, or LVDS-1 etc. If no output
connector is specified the mode string will apply to all connectors.
The mode specification used will match down the probed modes, and if
no mode is found it will add a CVT mode that matches.
video=1024x768 - all connectors match a 1024x768 mode or add a CVT on
video=VGA-1:1024x768, VGA-1 connector gets mode only.
The same strings as used in current fb modedb.c are used, except I've
added three more letters, e, D, d, e = enable, D = enable Digital,
d = disable, which allow a connector to be forced into a certain state.
Signed-off-by: Dave Airlie <airlied@redhat.com>
2009-09-23 11:44:08 +07:00
|
|
|
enum drm_connector_force force;
|
2014-06-18 23:52:33 +07:00
|
|
|
bool override_edid;
|
2008-11-08 05:05:41 +07:00
|
|
|
uint32_t encoder_ids[DRM_CONNECTOR_MAX_ENCODER];
|
|
|
|
struct drm_encoder *encoder; /* currently active encoder */
|
2011-06-14 13:13:55 +07:00
|
|
|
|
2011-09-05 13:23:20 +07:00
|
|
|
/* EDID bits */
|
|
|
|
uint8_t eld[MAX_ELD_BYTES];
|
|
|
|
bool dvi_dual;
|
|
|
|
int max_tmds_clock; /* in MHz */
|
|
|
|
bool latency_present[2];
|
|
|
|
int video_latency[2]; /* [0]: progressive, [1]: interlaced */
|
|
|
|
int audio_latency[2];
|
2011-06-14 13:13:55 +07:00
|
|
|
int null_edid_counter; /* needed to workaround some HW bugs where we get all 0s */
|
2012-08-09 22:25:51 +07:00
|
|
|
unsigned bad_edid_counter;
|
2014-06-18 23:52:32 +07:00
|
|
|
|
2015-04-22 01:09:41 +07:00
|
|
|
/* Flag for raw EDID header corruption - used in Displayport
|
|
|
|
* compliance testing - * Displayport Link CTS Core 1.2 rev1.1 4.2.2.6
|
|
|
|
*/
|
|
|
|
bool edid_corrupt;
|
|
|
|
|
2014-06-18 23:52:32 +07:00
|
|
|
struct dentry *debugfs_entry;
|
2014-10-28 02:28:44 +07:00
|
|
|
|
|
|
|
struct drm_connector_state *state;
|
2014-10-20 13:29:33 +07:00
|
|
|
|
|
|
|
/* DisplayID bits */
|
|
|
|
bool has_tile;
|
|
|
|
struct drm_tile_group *tile_group;
|
|
|
|
bool tile_is_single_monitor;
|
|
|
|
|
|
|
|
uint8_t num_h_tile, num_v_tile;
|
|
|
|
uint8_t tile_h_loc, tile_v_loc;
|
|
|
|
uint16_t tile_h_size, tile_v_size;
|
2014-10-28 02:28:44 +07:00
|
|
|
};
|
|
|
|
|
|
|
|
/**
|
|
|
|
* struct drm_plane_state - mutable plane state
|
2014-11-27 21:49:39 +07:00
|
|
|
* @plane: backpointer to the plane
|
2014-10-28 02:28:44 +07:00
|
|
|
* @crtc: currently bound CRTC, NULL if disabled
|
2014-07-26 02:30:38 +07:00
|
|
|
* @fb: currently bound framebuffer
|
2014-10-29 17:34:56 +07:00
|
|
|
* @fence: optional fence to wait for before scanning out @fb
|
2014-10-28 02:28:44 +07:00
|
|
|
* @crtc_x: left position of visible portion of plane on crtc
|
|
|
|
* @crtc_y: upper position of visible portion of plane on crtc
|
|
|
|
* @crtc_w: width of visible portion of plane on crtc
|
|
|
|
* @crtc_h: height of visible portion of plane on crtc
|
|
|
|
* @src_x: left position of visible portion of plane within
|
|
|
|
* plane (in 16.16)
|
|
|
|
* @src_y: upper position of visible portion of plane within
|
|
|
|
* plane (in 16.16)
|
|
|
|
* @src_w: width of visible portion of plane (in 16.16)
|
|
|
|
* @src_h: height of visible portion of plane (in 16.16)
|
|
|
|
* @state: backpointer to global drm_atomic_state
|
|
|
|
*/
|
|
|
|
struct drm_plane_state {
|
2014-11-27 21:49:39 +07:00
|
|
|
struct drm_plane *plane;
|
|
|
|
|
2014-11-22 03:28:31 +07:00
|
|
|
struct drm_crtc *crtc; /* do not write directly, use drm_atomic_set_crtc_for_plane() */
|
|
|
|
struct drm_framebuffer *fb; /* do not write directly, use drm_atomic_set_fb_for_plane() */
|
2014-10-29 17:34:56 +07:00
|
|
|
struct fence *fence;
|
2014-10-28 02:28:44 +07:00
|
|
|
|
|
|
|
/* Signed dest location allows it to be partially off screen */
|
|
|
|
int32_t crtc_x, crtc_y;
|
|
|
|
uint32_t crtc_w, crtc_h;
|
|
|
|
|
|
|
|
/* Source values are 16.16 fixed point */
|
|
|
|
uint32_t src_x, src_y;
|
|
|
|
uint32_t src_h, src_w;
|
|
|
|
|
2015-01-22 07:35:40 +07:00
|
|
|
/* Plane rotation */
|
|
|
|
unsigned int rotation;
|
|
|
|
|
2014-10-28 02:28:44 +07:00
|
|
|
struct drm_atomic_state *state;
|
2008-11-08 05:05:41 +07:00
|
|
|
};
|
|
|
|
|
2014-10-28 02:28:44 +07:00
|
|
|
|
2011-11-15 05:51:27 +07:00
|
|
|
/**
|
2014-10-27 22:54:27 +07:00
|
|
|
* struct drm_plane_funcs - driver plane control functions
|
2011-11-15 05:51:27 +07:00
|
|
|
*/
|
|
|
|
struct drm_plane_funcs {
|
2015-12-04 15:45:48 +07:00
|
|
|
/**
|
|
|
|
* @update_plane:
|
|
|
|
*
|
|
|
|
* This is the legacy entry point to enable and configure the plane for
|
|
|
|
* the given CRTC and framebuffer. It is never called to disable the
|
|
|
|
* plane, i.e. the passed-in crtc and fb paramters are never NULL.
|
|
|
|
*
|
|
|
|
* The source rectangle in frame buffer memory coordinates is given by
|
|
|
|
* the src_x, src_y, src_w and src_h parameters (as 16.16 fixed point
|
|
|
|
* values). Devices that don't support subpixel plane coordinates can
|
|
|
|
* ignore the fractional part.
|
|
|
|
*
|
|
|
|
* The destination rectangle in CRTC coordinates is given by the
|
|
|
|
* crtc_x, crtc_y, crtc_w and crtc_h parameters (as integer values).
|
|
|
|
* Devices scale the source rectangle to the destination rectangle. If
|
|
|
|
* scaling is not supported, and the source rectangle size doesn't match
|
|
|
|
* the destination rectangle size, the driver must return a
|
|
|
|
* -<errorname>EINVAL</errorname> error.
|
|
|
|
*
|
|
|
|
* Drivers implementing atomic modeset should use
|
|
|
|
* drm_atomic_helper_update_plane() to implement this hook.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 on success or a negative error code on failure.
|
|
|
|
*/
|
2011-11-15 05:51:27 +07:00
|
|
|
int (*update_plane)(struct drm_plane *plane,
|
|
|
|
struct drm_crtc *crtc, struct drm_framebuffer *fb,
|
|
|
|
int crtc_x, int crtc_y,
|
|
|
|
unsigned int crtc_w, unsigned int crtc_h,
|
|
|
|
uint32_t src_x, uint32_t src_y,
|
|
|
|
uint32_t src_w, uint32_t src_h);
|
2015-12-04 15:45:48 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @disable_plane:
|
|
|
|
*
|
|
|
|
* This is the legacy entry point to disable the plane. The DRM core
|
2015-12-04 15:46:01 +07:00
|
|
|
* calls this method in response to a DRM_IOCTL_MODE_SETPLANE IOCTL call
|
2015-12-04 15:45:48 +07:00
|
|
|
* with the frame buffer ID set to 0. Disabled planes must not be
|
|
|
|
* processed by the CRTC.
|
|
|
|
*
|
|
|
|
* Drivers implementing atomic modeset should use
|
|
|
|
* drm_atomic_helper_disable_plane() to implement this hook.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 on success or a negative error code on failure.
|
|
|
|
*/
|
2011-11-15 05:51:27 +07:00
|
|
|
int (*disable_plane)(struct drm_plane *plane);
|
2015-12-04 15:45:48 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @destroy:
|
|
|
|
*
|
|
|
|
* Clean up plane resources. This is only called at driver unload time
|
|
|
|
* through drm_mode_config_cleanup() since a plane cannot be hotplugged
|
|
|
|
* in DRM.
|
|
|
|
*/
|
2011-11-15 05:51:27 +07:00
|
|
|
void (*destroy)(struct drm_plane *plane);
|
2015-12-04 15:45:48 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @reset:
|
|
|
|
*
|
|
|
|
* Reset plane hardware and software state to off. This function isn't
|
|
|
|
* called by the core directly, only through drm_mode_config_reset().
|
|
|
|
* It's not a helper hook only for historical reasons.
|
|
|
|
*
|
|
|
|
* Atomic drivers can use drm_atomic_helper_plane_reset() to reset
|
|
|
|
* atomic state using this hook.
|
|
|
|
*/
|
2014-07-29 20:32:37 +07:00
|
|
|
void (*reset)(struct drm_plane *plane);
|
2012-05-17 15:23:27 +07:00
|
|
|
|
2015-12-04 15:45:48 +07:00
|
|
|
/**
|
|
|
|
* @set_property:
|
|
|
|
*
|
|
|
|
* This is the legacy entry point to update a property attached to the
|
|
|
|
* plane.
|
|
|
|
*
|
|
|
|
* Drivers implementing atomic modeset should use
|
|
|
|
* drm_atomic_helper_plane_set_property() to implement this hook.
|
|
|
|
*
|
|
|
|
* This callback is optional if the driver does not support any legacy
|
|
|
|
* driver-private properties.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 on success or a negative error code on failure.
|
|
|
|
*/
|
2012-05-17 15:23:27 +07:00
|
|
|
int (*set_property)(struct drm_plane *plane,
|
|
|
|
struct drm_property *property, uint64_t val);
|
2014-10-28 02:28:44 +07:00
|
|
|
|
2015-12-04 15:45:48 +07:00
|
|
|
/**
|
|
|
|
* @atomic_duplicate_state:
|
|
|
|
*
|
|
|
|
* Duplicate the current atomic state for this plane and return it.
|
|
|
|
* The core and helpers gurantee that any atomic state duplicated with
|
|
|
|
* this hook and still owned by the caller (i.e. not transferred to the
|
|
|
|
* driver by calling ->atomic_commit() from struct
|
|
|
|
* &drm_mode_config_funcs) will be cleaned up by calling the
|
|
|
|
* @atomic_destroy_state hook in this structure.
|
|
|
|
*
|
|
|
|
* Atomic drivers which don't subclass struct &drm_plane_state should use
|
|
|
|
* drm_atomic_helper_plane_duplicate_state(). Drivers that subclass the
|
|
|
|
* state structure to extend it with driver-private state should use
|
|
|
|
* __drm_atomic_helper_plane_duplicate_state() to make sure shared state is
|
|
|
|
* duplicated in a consistent fashion across drivers.
|
|
|
|
*
|
|
|
|
* It is an error to call this hook before plane->state has been
|
|
|
|
* initialized correctly.
|
|
|
|
*
|
|
|
|
* NOTE:
|
|
|
|
*
|
|
|
|
* If the duplicate state references refcounted resources this hook must
|
|
|
|
* acquire a reference for each of them. The driver must release these
|
|
|
|
* references again in @atomic_destroy_state.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* Duplicated atomic state or NULL when the allocation failed.
|
|
|
|
*/
|
2014-10-28 02:28:44 +07:00
|
|
|
struct drm_plane_state *(*atomic_duplicate_state)(struct drm_plane *plane);
|
2015-12-04 15:45:48 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @atomic_destroy_state:
|
|
|
|
*
|
|
|
|
* Destroy a state duplicated with @atomic_duplicate_state and release
|
|
|
|
* or unreference all resources it references
|
|
|
|
*/
|
2014-10-28 02:28:44 +07:00
|
|
|
void (*atomic_destroy_state)(struct drm_plane *plane,
|
2014-07-26 02:30:38 +07:00
|
|
|
struct drm_plane_state *state);
|
2015-12-04 15:45:48 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @atomic_set_property:
|
|
|
|
*
|
|
|
|
* Decode a driver-private property value and store the decoded value
|
|
|
|
* into the passed-in state structure. Since the atomic core decodes all
|
|
|
|
* standardized properties (even for extensions beyond the core set of
|
|
|
|
* properties which might not be implemented by all drivers) this
|
|
|
|
* requires drivers to subclass the state structure.
|
|
|
|
*
|
|
|
|
* Such driver-private properties should really only be implemented for
|
|
|
|
* truly hardware/vendor specific state. Instead it is preferred to
|
|
|
|
* standardize atomic extension and decode the properties used to expose
|
|
|
|
* such an extension in the core.
|
|
|
|
*
|
|
|
|
* Do not call this function directly, use
|
|
|
|
* drm_atomic_plane_set_property() instead.
|
|
|
|
*
|
|
|
|
* This callback is optional if the driver does not support any
|
|
|
|
* driver-private atomic properties.
|
|
|
|
*
|
|
|
|
* NOTE:
|
|
|
|
*
|
|
|
|
* This function is called in the state assembly phase of atomic
|
|
|
|
* modesets, which can be aborted for any reason (including on
|
|
|
|
* userspace's request to just check whether a configuration would be
|
|
|
|
* possible). Drivers MUST NOT touch any persistent state (hardware or
|
|
|
|
* software) or data structures except the passed in @state parameter.
|
|
|
|
*
|
|
|
|
* Also since userspace controls in which order properties are set this
|
|
|
|
* function must not do any input validation (since the state update is
|
|
|
|
* incomplete and hence likely inconsistent). Instead any such input
|
|
|
|
* validation must be done in the various atomic_check callbacks.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 if the property has been found, -EINVAL if the property isn't
|
|
|
|
* implemented by the driver (which shouldn't ever happen, the core only
|
|
|
|
* asks for properties attached to this plane). No other validation is
|
|
|
|
* allowed by the driver. The core already checks that the property
|
|
|
|
* value is within the range (integer, valid enum value, ...) the driver
|
|
|
|
* set when registering the property.
|
|
|
|
*/
|
2014-10-28 02:28:44 +07:00
|
|
|
int (*atomic_set_property)(struct drm_plane *plane,
|
|
|
|
struct drm_plane_state *state,
|
|
|
|
struct drm_property *property,
|
|
|
|
uint64_t val);
|
2015-12-04 15:45:48 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @atomic_get_property:
|
|
|
|
*
|
|
|
|
* Reads out the decoded driver-private property. This is used to
|
2015-12-04 15:46:01 +07:00
|
|
|
* implement the GETPLANE IOCTL.
|
2015-12-04 15:45:48 +07:00
|
|
|
*
|
|
|
|
* Do not call this function directly, use
|
|
|
|
* drm_atomic_plane_get_property() instead.
|
|
|
|
*
|
|
|
|
* This callback is optional if the driver does not support any
|
|
|
|
* driver-private atomic properties.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 on success, -EINVAL if the property isn't implemented by the
|
|
|
|
* driver (which should never happen, the core only asks for
|
|
|
|
* properties attached to this plane).
|
|
|
|
*/
|
2014-12-19 04:01:47 +07:00
|
|
|
int (*atomic_get_property)(struct drm_plane *plane,
|
|
|
|
const struct drm_plane_state *state,
|
|
|
|
struct drm_property *property,
|
|
|
|
uint64_t *val);
|
2011-11-15 05:51:27 +07:00
|
|
|
};
|
|
|
|
|
2014-04-02 05:22:30 +07:00
|
|
|
enum drm_plane_type {
|
|
|
|
DRM_PLANE_TYPE_OVERLAY,
|
|
|
|
DRM_PLANE_TYPE_PRIMARY,
|
|
|
|
DRM_PLANE_TYPE_CURSOR,
|
|
|
|
};
|
|
|
|
|
2015-12-04 15:45:48 +07:00
|
|
|
|
2011-11-15 05:51:27 +07:00
|
|
|
/**
|
2014-10-27 22:54:27 +07:00
|
|
|
* struct drm_plane - central DRM plane control structure
|
2011-11-15 05:51:27 +07:00
|
|
|
* @dev: DRM device this plane belongs to
|
|
|
|
* @head: for list management
|
|
|
|
* @base: base mode object
|
|
|
|
* @possible_crtcs: pipes this plane can be bound to
|
|
|
|
* @format_types: array of formats supported by this plane
|
|
|
|
* @format_count: number of formats supported
|
2015-03-09 15:41:07 +07:00
|
|
|
* @format_default: driver hasn't supplied supported formats for the plane
|
2011-11-15 05:51:27 +07:00
|
|
|
* @crtc: currently bound CRTC
|
|
|
|
* @fb: currently bound fb
|
2014-10-28 02:19:38 +07:00
|
|
|
* @old_fb: Temporary tracking of the old fb while a modeset is ongoing. Used by
|
|
|
|
* drm_mode_set_config_internal() to implement correct refcounting.
|
2011-11-15 05:51:27 +07:00
|
|
|
* @funcs: helper functions
|
2012-05-17 15:23:27 +07:00
|
|
|
* @properties: property tracking for this plane
|
2014-04-02 05:22:30 +07:00
|
|
|
* @type: type of plane (overlay, primary, cursor)
|
2014-10-28 02:28:44 +07:00
|
|
|
* @state: current atomic state for this plane
|
2011-11-15 05:51:27 +07:00
|
|
|
*/
|
|
|
|
struct drm_plane {
|
|
|
|
struct drm_device *dev;
|
|
|
|
struct list_head head;
|
|
|
|
|
2015-12-08 23:41:54 +07:00
|
|
|
char *name;
|
|
|
|
|
2014-11-11 16:12:00 +07:00
|
|
|
struct drm_modeset_lock mutex;
|
|
|
|
|
2011-11-15 05:51:27 +07:00
|
|
|
struct drm_mode_object base;
|
|
|
|
|
|
|
|
uint32_t possible_crtcs;
|
|
|
|
uint32_t *format_types;
|
2015-08-12 21:54:28 +07:00
|
|
|
unsigned int format_count;
|
2015-03-09 15:41:07 +07:00
|
|
|
bool format_default;
|
2011-11-15 05:51:27 +07:00
|
|
|
|
|
|
|
struct drm_crtc *crtc;
|
|
|
|
struct drm_framebuffer *fb;
|
|
|
|
|
2014-07-27 18:42:42 +07:00
|
|
|
struct drm_framebuffer *old_fb;
|
|
|
|
|
2011-11-15 05:51:27 +07:00
|
|
|
const struct drm_plane_funcs *funcs;
|
2012-05-17 15:23:27 +07:00
|
|
|
|
|
|
|
struct drm_object_properties properties;
|
2014-04-02 05:22:30 +07:00
|
|
|
|
|
|
|
enum drm_plane_type type;
|
2014-10-28 02:28:44 +07:00
|
|
|
|
2015-12-04 15:45:45 +07:00
|
|
|
const struct drm_plane_helper_funcs *helper_private;
|
2014-11-05 06:14:14 +07:00
|
|
|
|
2014-10-28 02:28:44 +07:00
|
|
|
struct drm_plane_state *state;
|
2011-11-15 05:51:27 +07:00
|
|
|
};
|
|
|
|
|
2013-08-15 03:47:37 +07:00
|
|
|
/**
|
2014-10-27 22:54:27 +07:00
|
|
|
* struct drm_bridge_funcs - drm_bridge control functions
|
2015-01-20 23:38:44 +07:00
|
|
|
* @attach: Called during drm_bridge_attach
|
2013-08-15 03:47:37 +07:00
|
|
|
*/
|
|
|
|
struct drm_bridge_funcs {
|
2015-01-20 23:38:44 +07:00
|
|
|
int (*attach)(struct drm_bridge *bridge);
|
2015-12-04 15:45:47 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @mode_fixup:
|
|
|
|
*
|
|
|
|
* This callback is used to validate and adjust a mode. The paramater
|
|
|
|
* mode is the display mode that should be fed to the next element in
|
|
|
|
* the display chain, either the final &drm_connector or the next
|
|
|
|
* &drm_bridge. The parameter adjusted_mode is the input mode the bridge
|
|
|
|
* requires. It can be modified by this callback and does not need to
|
|
|
|
* match mode.
|
|
|
|
*
|
|
|
|
* This is the only hook that allows a bridge to reject a modeset. If
|
|
|
|
* this function passes all other callbacks must succeed for this
|
|
|
|
* configuration.
|
|
|
|
*
|
|
|
|
* NOTE:
|
|
|
|
*
|
|
|
|
* This function is called in the check phase of atomic modesets, which
|
|
|
|
* can be aborted for any reason (including on userspace's request to
|
|
|
|
* just check whether a configuration would be possible). Drivers MUST
|
|
|
|
* NOT touch any persistent state (hardware or software) or data
|
2015-12-04 15:45:48 +07:00
|
|
|
* structures except the passed in @state parameter.
|
2015-12-04 15:45:47 +07:00
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* True if an acceptable configuration is possible, false if the modeset
|
|
|
|
* operation should be rejected.
|
|
|
|
*/
|
2013-08-15 03:47:37 +07:00
|
|
|
bool (*mode_fixup)(struct drm_bridge *bridge,
|
|
|
|
const struct drm_display_mode *mode,
|
|
|
|
struct drm_display_mode *adjusted_mode);
|
2015-12-04 15:45:47 +07:00
|
|
|
/**
|
|
|
|
* @disable:
|
|
|
|
*
|
|
|
|
* This callback should disable the bridge. It is called right before
|
|
|
|
* the preceding element in the display pipe is disabled. If the
|
|
|
|
* preceding element is a bridge this means it's called before that
|
|
|
|
* bridge's ->disable() function. If the preceding element is a
|
|
|
|
* &drm_encoder it's called right before the encoder's ->disable(),
|
|
|
|
* ->prepare() or ->dpms() hook from struct &drm_encoder_helper_funcs.
|
|
|
|
*
|
|
|
|
* The bridge can assume that the display pipe (i.e. clocks and timing
|
|
|
|
* signals) feeding it is still running when this callback is called.
|
|
|
|
*/
|
2013-08-15 03:47:37 +07:00
|
|
|
void (*disable)(struct drm_bridge *bridge);
|
2015-12-04 15:45:47 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @post_disable:
|
|
|
|
*
|
|
|
|
* This callback should disable the bridge. It is called right after
|
|
|
|
* the preceding element in the display pipe is disabled. If the
|
|
|
|
* preceding element is a bridge this means it's called after that
|
|
|
|
* bridge's ->post_disable() function. If the preceding element is a
|
|
|
|
* &drm_encoder it's called right after the encoder's ->disable(),
|
|
|
|
* ->prepare() or ->dpms() hook from struct &drm_encoder_helper_funcs.
|
|
|
|
*
|
|
|
|
* The bridge must assume that the display pipe (i.e. clocks and timing
|
|
|
|
* singals) feeding it is no longer running when this callback is
|
|
|
|
* called.
|
|
|
|
*/
|
2013-08-15 03:47:37 +07:00
|
|
|
void (*post_disable)(struct drm_bridge *bridge);
|
2015-12-04 15:45:47 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @mode_set:
|
|
|
|
*
|
|
|
|
* This callback should set the given mode on the bridge. It is called
|
|
|
|
* after the ->mode_set() callback for the preceding element in the
|
|
|
|
* display pipeline has been called already. The display pipe (i.e.
|
|
|
|
* clocks and timing signals) is off when this function is called.
|
|
|
|
*/
|
2013-08-15 03:47:37 +07:00
|
|
|
void (*mode_set)(struct drm_bridge *bridge,
|
|
|
|
struct drm_display_mode *mode,
|
|
|
|
struct drm_display_mode *adjusted_mode);
|
2015-12-04 15:45:47 +07:00
|
|
|
/**
|
|
|
|
* @pre_enable:
|
|
|
|
*
|
|
|
|
* This callback should enable the bridge. It is called right before
|
|
|
|
* the preceding element in the display pipe is enabled. If the
|
|
|
|
* preceding element is a bridge this means it's called before that
|
|
|
|
* bridge's ->pre_enable() function. If the preceding element is a
|
|
|
|
* &drm_encoder it's called right before the encoder's ->enable(),
|
|
|
|
* ->commit() or ->dpms() hook from struct &drm_encoder_helper_funcs.
|
|
|
|
*
|
|
|
|
* The display pipe (i.e. clocks and timing signals) feeding this bridge
|
|
|
|
* will not yet be running when this callback is called. The bridge must
|
|
|
|
* not enable the display link feeding the next bridge in the chain (if
|
|
|
|
* there is one) when this callback is called.
|
|
|
|
*/
|
2013-08-15 03:47:37 +07:00
|
|
|
void (*pre_enable)(struct drm_bridge *bridge);
|
2015-12-04 15:45:47 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @enable:
|
|
|
|
*
|
|
|
|
* This callback should enable the bridge. It is called right after
|
|
|
|
* the preceding element in the display pipe is enabled. If the
|
|
|
|
* preceding element is a bridge this means it's called after that
|
|
|
|
* bridge's ->enable() function. If the preceding element is a
|
|
|
|
* &drm_encoder it's called right after the encoder's ->enable(),
|
|
|
|
* ->commit() or ->dpms() hook from struct &drm_encoder_helper_funcs.
|
|
|
|
*
|
|
|
|
* The bridge can assume that the display pipe (i.e. clocks and timing
|
|
|
|
* signals) feeding it is running when this callback is called. This
|
|
|
|
* callback must enable the display link feeding the next bridge in the
|
|
|
|
* chain if there is one.
|
|
|
|
*/
|
2013-08-15 03:47:37 +07:00
|
|
|
void (*enable)(struct drm_bridge *bridge);
|
|
|
|
};
|
|
|
|
|
|
|
|
/**
|
2014-10-27 22:54:27 +07:00
|
|
|
* struct drm_bridge - central DRM bridge control structure
|
2013-08-15 03:47:37 +07:00
|
|
|
* @dev: DRM device this bridge belongs to
|
drm: bridge: Allow daisy chaining of bridges
Allow drm_bridge objects to link to each other in order to form an encoder
chain. The requirement for creating a chain of bridges comes because the
MSM drm driver uses up its encoder and bridge objects for blocks within
the SoC itself. There isn't anything left to use if the SoC display output
is connected to an external encoder IC. Having an additional bridge
connected to the existing bridge helps here. In general, it is possible for
platforms to have multiple devices between the encoder and the
connector/panel that require some sort of configuration.
We create drm bridge helper functions corresponding to each op in
'drm_bridge_funcs'. These helpers call the corresponding
'drm_bridge_funcs' op for the entire chain of bridges. These helpers are
used internally by drm_atomic_helper.c and drm_crtc_helper.c.
The drm_bridge_enable/pre_enable helpers execute enable/pre_enable ops of
the bridge closet to the encoder, and proceed until the last bridge in the
chain is enabled. The same holds for drm_bridge_mode_set/mode_fixup
helpers. The drm_bridge_disable/post_disable helpers disable the last
bridge in the chain first, and proceed until the first bridge in the chain
is disabled.
drm_bridge_attach() remains the same. As before, the driver calling this
function should make sure it has set the links correctly. The order in
which the bridges are connected to each other determines the order in which
the calls are made. One requirement is that every bridge in the chain
should point the parent encoder object. This is required since bridge
drivers expect a valid encoder pointer in drm_bridge. For example, consider
a chain where an encoder's output is connected to bridge1, and bridge1's
output is connected to bridge2:
/* Like before, attach bridge to an encoder */
bridge1->encoder = encoder;
ret = drm_bridge_attach(dev, bridge1);
..
/*
* set the first bridge's 'next' bridge to bridge2, set its encoder
* as bridge1's encoder
*/
bridge1->next = bridge2
bridge2->encoder = bridge1->encoder;
ret = drm_bridge_attach(dev, bridge2);
...
...
This method of bridge chaining isn't intrusive and existing drivers that
use drm_bridge will behave the same way as before. The bridge helpers also
cleans up the atomic and crtc helper files a bit.
Reviewed-by: Jani Nikula <jani.nikula@linux.intel.com>
Reviewed-by: Rob Clark <robdclark@gmail.com>
Reviewed-by: Daniel Vetter <daniel@ffwll.ch>
Signed-off-by: Archit Taneja <architt@codeaurora.org>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2015-05-21 12:33:16 +07:00
|
|
|
* @encoder: encoder to which this bridge is connected
|
|
|
|
* @next: the next bridge in the encoder chain
|
2015-01-20 23:38:44 +07:00
|
|
|
* @of_node: device node pointer to the bridge
|
|
|
|
* @list: to keep track of all added bridges
|
2013-08-15 03:47:37 +07:00
|
|
|
* @funcs: control functions
|
|
|
|
* @driver_private: pointer to the bridge driver's internal context
|
|
|
|
*/
|
|
|
|
struct drm_bridge {
|
|
|
|
struct drm_device *dev;
|
2015-01-20 23:38:44 +07:00
|
|
|
struct drm_encoder *encoder;
|
drm: bridge: Allow daisy chaining of bridges
Allow drm_bridge objects to link to each other in order to form an encoder
chain. The requirement for creating a chain of bridges comes because the
MSM drm driver uses up its encoder and bridge objects for blocks within
the SoC itself. There isn't anything left to use if the SoC display output
is connected to an external encoder IC. Having an additional bridge
connected to the existing bridge helps here. In general, it is possible for
platforms to have multiple devices between the encoder and the
connector/panel that require some sort of configuration.
We create drm bridge helper functions corresponding to each op in
'drm_bridge_funcs'. These helpers call the corresponding
'drm_bridge_funcs' op for the entire chain of bridges. These helpers are
used internally by drm_atomic_helper.c and drm_crtc_helper.c.
The drm_bridge_enable/pre_enable helpers execute enable/pre_enable ops of
the bridge closet to the encoder, and proceed until the last bridge in the
chain is enabled. The same holds for drm_bridge_mode_set/mode_fixup
helpers. The drm_bridge_disable/post_disable helpers disable the last
bridge in the chain first, and proceed until the first bridge in the chain
is disabled.
drm_bridge_attach() remains the same. As before, the driver calling this
function should make sure it has set the links correctly. The order in
which the bridges are connected to each other determines the order in which
the calls are made. One requirement is that every bridge in the chain
should point the parent encoder object. This is required since bridge
drivers expect a valid encoder pointer in drm_bridge. For example, consider
a chain where an encoder's output is connected to bridge1, and bridge1's
output is connected to bridge2:
/* Like before, attach bridge to an encoder */
bridge1->encoder = encoder;
ret = drm_bridge_attach(dev, bridge1);
..
/*
* set the first bridge's 'next' bridge to bridge2, set its encoder
* as bridge1's encoder
*/
bridge1->next = bridge2
bridge2->encoder = bridge1->encoder;
ret = drm_bridge_attach(dev, bridge2);
...
...
This method of bridge chaining isn't intrusive and existing drivers that
use drm_bridge will behave the same way as before. The bridge helpers also
cleans up the atomic and crtc helper files a bit.
Reviewed-by: Jani Nikula <jani.nikula@linux.intel.com>
Reviewed-by: Rob Clark <robdclark@gmail.com>
Reviewed-by: Daniel Vetter <daniel@ffwll.ch>
Signed-off-by: Archit Taneja <architt@codeaurora.org>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2015-05-21 12:33:16 +07:00
|
|
|
struct drm_bridge *next;
|
2015-01-20 23:38:44 +07:00
|
|
|
#ifdef CONFIG_OF
|
|
|
|
struct device_node *of_node;
|
|
|
|
#endif
|
|
|
|
struct list_head list;
|
2013-08-15 03:47:37 +07:00
|
|
|
|
|
|
|
const struct drm_bridge_funcs *funcs;
|
|
|
|
void *driver_private;
|
|
|
|
};
|
|
|
|
|
2014-07-26 02:30:38 +07:00
|
|
|
/**
|
2015-03-11 21:23:09 +07:00
|
|
|
* struct drm_atomic_state - the global state object for atomic updates
|
2014-07-26 02:30:38 +07:00
|
|
|
* @dev: parent DRM device
|
2014-12-19 04:01:56 +07:00
|
|
|
* @allow_modeset: allow full modeset
|
2015-12-04 15:46:01 +07:00
|
|
|
* @legacy_cursor_update: hint to enforce legacy cursor IOCTL semantics
|
2014-07-26 02:30:38 +07:00
|
|
|
* @planes: pointer to array of plane pointers
|
|
|
|
* @plane_states: pointer to array of plane states pointers
|
|
|
|
* @crtcs: pointer to array of CRTC pointers
|
|
|
|
* @crtc_states: pointer to array of CRTC states pointers
|
2014-11-20 00:38:08 +07:00
|
|
|
* @num_connector: size of the @connectors and @connector_states arrays
|
2014-07-26 02:30:38 +07:00
|
|
|
* @connectors: pointer to array of connector pointers
|
|
|
|
* @connector_states: pointer to array of connector states pointers
|
|
|
|
* @acquire_ctx: acquire context for this atomic modeset state update
|
|
|
|
*/
|
|
|
|
struct drm_atomic_state {
|
|
|
|
struct drm_device *dev;
|
2014-12-19 04:01:56 +07:00
|
|
|
bool allow_modeset : 1;
|
2015-01-22 22:36:23 +07:00
|
|
|
bool legacy_cursor_update : 1;
|
2014-07-26 02:30:38 +07:00
|
|
|
struct drm_plane **planes;
|
|
|
|
struct drm_plane_state **plane_states;
|
|
|
|
struct drm_crtc **crtcs;
|
|
|
|
struct drm_crtc_state **crtc_states;
|
2014-11-20 00:38:08 +07:00
|
|
|
int num_connector;
|
2014-07-26 02:30:38 +07:00
|
|
|
struct drm_connector **connectors;
|
|
|
|
struct drm_connector_state **connector_states;
|
|
|
|
|
|
|
|
struct drm_modeset_acquire_ctx *acquire_ctx;
|
|
|
|
};
|
|
|
|
|
|
|
|
|
2008-11-08 05:05:41 +07:00
|
|
|
/**
|
2014-10-27 22:54:27 +07:00
|
|
|
* struct drm_mode_set - new values for a CRTC config change
|
2011-11-08 03:03:19 +07:00
|
|
|
* @fb: framebuffer to use for new config
|
|
|
|
* @crtc: CRTC whose configuration we're about to change
|
|
|
|
* @mode: mode timings to use
|
|
|
|
* @x: position of this CRTC relative to @fb
|
|
|
|
* @y: position of this CRTC relative to @fb
|
|
|
|
* @connectors: array of connectors to drive with this CRTC if possible
|
|
|
|
* @num_connectors: size of @connectors array
|
2008-11-08 05:05:41 +07:00
|
|
|
*
|
|
|
|
* Represents a single crtc the connectors that it drives with what mode
|
|
|
|
* and from which framebuffer it scans out from.
|
|
|
|
*
|
|
|
|
* This is used to set modes.
|
|
|
|
*/
|
|
|
|
struct drm_mode_set {
|
|
|
|
struct drm_framebuffer *fb;
|
|
|
|
struct drm_crtc *crtc;
|
|
|
|
struct drm_display_mode *mode;
|
|
|
|
|
|
|
|
uint32_t x;
|
|
|
|
uint32_t y;
|
|
|
|
|
|
|
|
struct drm_connector **connectors;
|
|
|
|
size_t num_connectors;
|
|
|
|
};
|
|
|
|
|
|
|
|
/**
|
2011-11-08 03:03:20 +07:00
|
|
|
* struct drm_mode_config_funcs - basic driver provided mode setting functions
|
|
|
|
*
|
|
|
|
* Some global (i.e. not per-CRTC, connector, etc) mode setting functions that
|
|
|
|
* involve drivers.
|
2008-11-08 05:05:41 +07:00
|
|
|
*/
|
|
|
|
struct drm_mode_config_funcs {
|
2015-12-04 15:46:02 +07:00
|
|
|
/**
|
|
|
|
* @fb_create:
|
|
|
|
*
|
|
|
|
* Create a new framebuffer object. The core does basic checks on the
|
|
|
|
* requested metadata, but most of that is left to the driver. See
|
|
|
|
* struct &drm_mode_fb_cmd2 for details.
|
|
|
|
*
|
2015-12-08 15:49:19 +07:00
|
|
|
* If the parameters are deemed valid and the backing storage objects in
|
|
|
|
* the underlying memory manager all exist, then the driver allocates
|
|
|
|
* a new &drm_framebuffer structure, subclassed to contain
|
|
|
|
* driver-specific information (like the internal native buffer object
|
|
|
|
* references). It also needs to fill out all relevant metadata, which
|
|
|
|
* should be done by calling drm_helper_mode_fill_fb_struct().
|
|
|
|
*
|
|
|
|
* The initialization is finalized by calling drm_framebuffer_init(),
|
|
|
|
* which registers the framebuffer and makes it accessible to other
|
|
|
|
* threads.
|
|
|
|
*
|
2015-12-04 15:46:02 +07:00
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* A new framebuffer with an initial reference count of 1 or a negative
|
|
|
|
* error code encoded with ERR_PTR().
|
|
|
|
*/
|
2011-11-08 03:03:20 +07:00
|
|
|
struct drm_framebuffer *(*fb_create)(struct drm_device *dev,
|
|
|
|
struct drm_file *file_priv,
|
2015-11-12 00:11:29 +07:00
|
|
|
const struct drm_mode_fb_cmd2 *mode_cmd);
|
2015-12-04 15:46:02 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @output_poll_changed:
|
|
|
|
*
|
|
|
|
* Callback used by helpers to inform the driver of output configuration
|
|
|
|
* changes.
|
|
|
|
*
|
|
|
|
* Drivers implementing fbdev emulation with the helpers can call
|
|
|
|
* drm_fb_helper_hotplug_changed from this hook to inform the fbdev
|
|
|
|
* helper of output changes.
|
|
|
|
*
|
|
|
|
* FIXME:
|
|
|
|
*
|
|
|
|
* Except that there's no vtable for device-level helper callbacks
|
|
|
|
* there's no reason this is a core function.
|
|
|
|
*/
|
2010-05-07 13:42:51 +07:00
|
|
|
void (*output_poll_changed)(struct drm_device *dev);
|
2014-07-26 02:30:38 +07:00
|
|
|
|
2015-12-04 15:46:02 +07:00
|
|
|
/**
|
|
|
|
* @atomic_check:
|
|
|
|
*
|
|
|
|
* This is the only hook to validate an atomic modeset update. This
|
|
|
|
* function must reject any modeset and state changes which the hardware
|
|
|
|
* or driver doesn't support. This includes but is of course not limited
|
|
|
|
* to:
|
|
|
|
*
|
|
|
|
* - Checking that the modes, framebuffers, scaling and placement
|
|
|
|
* requirements and so on are within the limits of the hardware.
|
|
|
|
*
|
|
|
|
* - Checking that any hidden shared resources are not oversubscribed.
|
|
|
|
* This can be shared PLLs, shared lanes, overall memory bandwidth,
|
|
|
|
* display fifo space (where shared between planes or maybe even
|
|
|
|
* CRTCs).
|
|
|
|
*
|
|
|
|
* - Checking that virtualized resources exported to userspace are not
|
|
|
|
* oversubscribed. For various reasons it can make sense to expose
|
|
|
|
* more planes, crtcs or encoders than which are physically there. One
|
|
|
|
* example is dual-pipe operations (which generally should be hidden
|
|
|
|
* from userspace if when lockstepped in hardware, exposed otherwise),
|
|
|
|
* where a plane might need 1 hardware plane (if it's just on one
|
|
|
|
* pipe), 2 hardware planes (when it spans both pipes) or maybe even
|
|
|
|
* shared a hardware plane with a 2nd plane (if there's a compatible
|
|
|
|
* plane requested on the area handled by the other pipe).
|
|
|
|
*
|
|
|
|
* - Check that any transitional state is possible and that if
|
|
|
|
* requested, the update can indeed be done in the vblank period
|
|
|
|
* without temporarily disabling some functions.
|
|
|
|
*
|
|
|
|
* - Check any other constraints the driver or hardware might have.
|
|
|
|
*
|
|
|
|
* - This callback also needs to correctly fill out the &drm_crtc_state
|
|
|
|
* in this update to make sure that drm_atomic_crtc_needs_modeset()
|
|
|
|
* reflects the nature of the possible update and returns true if and
|
|
|
|
* only if the update cannot be applied without tearing within one
|
|
|
|
* vblank on that CRTC. The core uses that information to reject
|
|
|
|
* updates which require a full modeset (i.e. blanking the screen, or
|
|
|
|
* at least pausing updates for a substantial amount of time) if
|
|
|
|
* userspace has disallowed that in its request.
|
|
|
|
*
|
|
|
|
* - The driver also does not need to repeat basic input validation
|
|
|
|
* like done for the corresponding legacy entry points. The core does
|
|
|
|
* that before calling this hook.
|
|
|
|
*
|
|
|
|
* See the documentation of @atomic_commit for an exhaustive list of
|
|
|
|
* error conditions which don't have to be checked at the
|
|
|
|
* ->atomic_check() stage?
|
|
|
|
*
|
|
|
|
* See the documentation for struct &drm_atomic_state for how exactly
|
|
|
|
* an atomic modeset update is described.
|
|
|
|
*
|
|
|
|
* Drivers using the atomic helpers can implement this hook using
|
|
|
|
* drm_atomic_helper_check(), or one of the exported sub-functions of
|
|
|
|
* it.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 on success or one of the below negative error codes:
|
|
|
|
*
|
|
|
|
* - -EINVAL, if any of the above constraints are violated.
|
|
|
|
*
|
|
|
|
* - -EDEADLK, when returned from an attempt to acquire an additional
|
|
|
|
* &drm_modeset_lock through drm_modeset_lock().
|
|
|
|
*
|
|
|
|
* - -ENOMEM, if allocating additional state sub-structures failed due
|
|
|
|
* to lack of memory.
|
|
|
|
*
|
|
|
|
* - -EINTR, -EAGAIN or -ERESTARTSYS, if the IOCTL should be restarted.
|
|
|
|
* This can either be due to a pending signal, or because the driver
|
|
|
|
* needs to completely bail out to recover from an exceptional
|
|
|
|
* situation like a GPU hang. From a userspace point all errors are
|
|
|
|
* treated equally.
|
|
|
|
*/
|
2014-07-26 02:30:38 +07:00
|
|
|
int (*atomic_check)(struct drm_device *dev,
|
2015-12-04 15:46:02 +07:00
|
|
|
struct drm_atomic_state *state);
|
|
|
|
|
|
|
|
/**
|
|
|
|
* @atomic_commit:
|
|
|
|
*
|
|
|
|
* This is the only hook to commit an atomic modeset update. The core
|
|
|
|
* guarantees that @atomic_check has been called successfully before
|
|
|
|
* calling this function, and that nothing has been changed in the
|
|
|
|
* interim.
|
|
|
|
*
|
|
|
|
* See the documentation for struct &drm_atomic_state for how exactly
|
|
|
|
* an atomic modeset update is described.
|
|
|
|
*
|
|
|
|
* Drivers using the atomic helpers can implement this hook using
|
|
|
|
* drm_atomic_helper_commit(), or one of the exported sub-functions of
|
|
|
|
* it.
|
|
|
|
*
|
|
|
|
* Asynchronous commits (as indicated with the async parameter) must
|
|
|
|
* do any preparatory work which might result in an unsuccessful commit
|
|
|
|
* in the context of this callback. The only exceptions are hardware
|
|
|
|
* errors resulting in -EIO. But even in that case the driver must
|
|
|
|
* ensure that the display pipe is at least running, to avoid
|
|
|
|
* compositors crashing when pageflips don't work. Anything else,
|
|
|
|
* specifically committing the update to the hardware, should be done
|
|
|
|
* without blocking the caller. For updates which do not require a
|
|
|
|
* modeset this must be guaranteed.
|
|
|
|
*
|
|
|
|
* The driver must wait for any pending rendering to the new
|
|
|
|
* framebuffers to complete before executing the flip. It should also
|
|
|
|
* wait for any pending rendering from other drivers if the underlying
|
|
|
|
* buffer is a shared dma-buf. Asynchronous commits must not wait for
|
|
|
|
* rendering in the context of this callback.
|
|
|
|
*
|
|
|
|
* An application can request to be notified when the atomic commit has
|
|
|
|
* completed. These events are per-CRTC and can be distinguished by the
|
|
|
|
* CRTC index supplied in &drm_event to userspace.
|
|
|
|
*
|
|
|
|
* The drm core will supply a struct &drm_event in the event
|
|
|
|
* member of each CRTC's &drm_crtc_state structure. This can be handled by the
|
|
|
|
* drm_crtc_send_vblank_event() function, which the driver should call on
|
|
|
|
* the provided event upon completion of the atomic commit. Note that if
|
|
|
|
* the driver supports vblank signalling and timestamping the vblank
|
|
|
|
* counters and timestamps must agree with the ones returned from page
|
|
|
|
* flip events. With the current vblank helper infrastructure this can
|
|
|
|
* be achieved by holding a vblank reference while the page flip is
|
|
|
|
* pending, acquired through drm_crtc_vblank_get() and released with
|
|
|
|
* drm_crtc_vblank_put(). Drivers are free to implement their own vblank
|
|
|
|
* counter and timestamp tracking though, e.g. if they have accurate
|
|
|
|
* timestamp registers in hardware.
|
|
|
|
*
|
|
|
|
* NOTE:
|
|
|
|
*
|
|
|
|
* Drivers are not allowed to shut down any display pipe successfully
|
|
|
|
* enabled through an atomic commit on their own. Doing so can result in
|
|
|
|
* compositors crashing if a page flip is suddenly rejected because the
|
|
|
|
* pipe is off.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* 0 on success or one of the below negative error codes:
|
|
|
|
*
|
|
|
|
* - -EBUSY, if an asynchronous updated is requested and there is
|
|
|
|
* an earlier updated pending. Drivers are allowed to support a queue
|
|
|
|
* of outstanding updates, but currently no driver supports that.
|
|
|
|
* Note that drivers must wait for preceding updates to complete if a
|
|
|
|
* synchronous update is requested, they are not allowed to fail the
|
|
|
|
* commit in that case.
|
|
|
|
*
|
|
|
|
* - -ENOMEM, if the driver failed to allocate memory. Specifically
|
|
|
|
* this can happen when trying to pin framebuffers, which must only
|
|
|
|
* be done when committing the state.
|
|
|
|
*
|
|
|
|
* - -ENOSPC, as a refinement of the more generic -ENOMEM to indicate
|
|
|
|
* that the driver has run out of vram, iommu space or similar GPU
|
|
|
|
* address space needed for framebuffer.
|
|
|
|
*
|
|
|
|
* - -EIO, if the hardware completely died.
|
|
|
|
*
|
|
|
|
* - -EINTR, -EAGAIN or -ERESTARTSYS, if the IOCTL should be restarted.
|
|
|
|
* This can either be due to a pending signal, or because the driver
|
|
|
|
* needs to completely bail out to recover from an exceptional
|
|
|
|
* situation like a GPU hang. From a userspace point of view all errors are
|
|
|
|
* treated equally.
|
|
|
|
*
|
|
|
|
* This list is exhaustive. Specifically this hook is not allowed to
|
|
|
|
* return -EINVAL (any invalid requests should be caught in
|
|
|
|
* @atomic_check) or -EDEADLK (this function must not acquire
|
|
|
|
* additional modeset locks).
|
|
|
|
*/
|
2014-07-26 02:30:38 +07:00
|
|
|
int (*atomic_commit)(struct drm_device *dev,
|
2015-12-04 15:46:02 +07:00
|
|
|
struct drm_atomic_state *state,
|
2014-07-26 02:30:38 +07:00
|
|
|
bool async);
|
2015-12-04 15:46:02 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @atomic_state_alloc:
|
|
|
|
*
|
|
|
|
* This optional hook can be used by drivers that want to subclass struct
|
|
|
|
* &drm_atomic_state to be able to track their own driver-private global
|
|
|
|
* state easily. If this hook is implemented, drivers must also
|
|
|
|
* implement @atomic_state_clear and @atomic_state_free.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
*
|
|
|
|
* A new &drm_atomic_state on success or NULL on failure.
|
|
|
|
*/
|
2015-05-18 15:06:40 +07:00
|
|
|
struct drm_atomic_state *(*atomic_state_alloc)(struct drm_device *dev);
|
2015-12-04 15:46:02 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @atomic_state_clear:
|
|
|
|
*
|
|
|
|
* This hook must clear any driver private state duplicated into the
|
|
|
|
* passed-in &drm_atomic_state. This hook is called when the caller
|
|
|
|
* encountered a &drm_modeset_lock deadlock and needs to drop all
|
|
|
|
* already acquired locks as part of the deadlock avoidance dance
|
|
|
|
* implemented in drm_modeset_lock_backoff().
|
|
|
|
*
|
|
|
|
* Any duplicated state must be invalidated since a concurrent atomic
|
|
|
|
* update might change it, and the drm atomic interfaces always apply
|
|
|
|
* updates as relative changes to the current state.
|
|
|
|
*
|
|
|
|
* Drivers that implement this must call drm_atomic_state_default_clear()
|
|
|
|
* to clear common state.
|
|
|
|
*/
|
2015-05-18 15:06:40 +07:00
|
|
|
void (*atomic_state_clear)(struct drm_atomic_state *state);
|
2015-12-04 15:46:02 +07:00
|
|
|
|
|
|
|
/**
|
|
|
|
* @atomic_state_free:
|
|
|
|
*
|
|
|
|
* This hook needs driver private resources and the &drm_atomic_state
|
|
|
|
* itself. Note that the core first calls drm_atomic_state_clear() to
|
|
|
|
* avoid code duplicate between the clear and free hooks.
|
|
|
|
*
|
|
|
|
* Drivers that implement this must call drm_atomic_state_default_free()
|
|
|
|
* to release common resources.
|
|
|
|
*/
|
2015-05-18 15:06:40 +07:00
|
|
|
void (*atomic_state_free)(struct drm_atomic_state *state);
|
2008-11-08 05:05:41 +07:00
|
|
|
};
|
|
|
|
|
|
|
|
/**
|
2014-10-28 02:19:38 +07:00
|
|
|
* struct drm_mode_config - Mode configuration control structure
|
2011-11-08 03:03:22 +07:00
|
|
|
* @mutex: mutex protecting KMS related lists and structures
|
2014-10-28 02:19:38 +07:00
|
|
|
* @connection_mutex: ww mutex protecting connector state and routing
|
|
|
|
* @acquire_ctx: global implicit acquire context used by atomic drivers for
|
2015-12-04 15:46:01 +07:00
|
|
|
* legacy IOCTLs
|
2011-11-08 03:03:22 +07:00
|
|
|
* @idr_mutex: mutex for KMS ID allocation and management
|
|
|
|
* @crtc_idr: main KMS ID tracking object
|
2014-10-28 02:19:38 +07:00
|
|
|
* @fb_lock: mutex to protect fb state and lists
|
2011-11-08 03:03:22 +07:00
|
|
|
* @num_fb: number of fbs available
|
|
|
|
* @fb_list: list of framebuffers available
|
|
|
|
* @num_connector: number of connectors on this device
|
|
|
|
* @connector_list: list of connector objects
|
|
|
|
* @num_encoder: number of encoders on this device
|
|
|
|
* @encoder_list: list of encoder objects
|
2014-10-28 02:19:38 +07:00
|
|
|
* @num_overlay_plane: number of overlay planes on this device
|
|
|
|
* @num_total_plane: number of universal (i.e. with primary/curso) planes on this device
|
|
|
|
* @plane_list: list of plane objects
|
2011-11-08 03:03:22 +07:00
|
|
|
* @num_crtc: number of CRTCs on this device
|
|
|
|
* @crtc_list: list of CRTC objects
|
2014-10-28 02:19:38 +07:00
|
|
|
* @property_list: list of property objects
|
2011-11-08 03:03:22 +07:00
|
|
|
* @min_width: minimum pixel width on this device
|
|
|
|
* @min_height: minimum pixel height on this device
|
|
|
|
* @max_width: maximum pixel width on this device
|
|
|
|
* @max_height: maximum pixel height on this device
|
|
|
|
* @funcs: core driver provided mode setting functions
|
|
|
|
* @fb_base: base address of the framebuffer
|
2014-10-28 02:19:38 +07:00
|
|
|
* @poll_enabled: track polling support for this device
|
|
|
|
* @poll_running: track polling status for this device
|
2011-11-08 03:03:22 +07:00
|
|
|
* @output_poll_work: delayed work for polling in process context
|
2014-10-28 02:19:38 +07:00
|
|
|
* @property_blob_list: list of all the blob property objects
|
2015-04-21 01:22:54 +07:00
|
|
|
* @blob_lock: mutex for blob property allocation and management
|
2011-11-08 03:03:22 +07:00
|
|
|
* @*_property: core property tracking
|
2014-10-28 02:19:38 +07:00
|
|
|
* @preferred_depth: preferred RBG pixel depth, used by fb helpers
|
|
|
|
* @prefer_shadow: hint to userspace to prefer shadow-fb rendering
|
|
|
|
* @async_page_flip: does this device support async flips on the primary plane?
|
|
|
|
* @cursor_width: hint to userspace for max cursor width
|
|
|
|
* @cursor_height: hint to userspace for max cursor height
|
2008-11-08 05:05:41 +07:00
|
|
|
*
|
2011-11-08 03:03:22 +07:00
|
|
|
* Core mode resource tracking structure. All CRTC, encoders, and connectors
|
|
|
|
* enumerated by the driver are added here, as are global properties. Some
|
|
|
|
* global restrictions are also here, e.g. dimension restrictions.
|
2008-11-08 05:05:41 +07:00
|
|
|
*/
|
|
|
|
struct drm_mode_config {
|
2009-01-19 14:21:45 +07:00
|
|
|
struct mutex mutex; /* protects configuration (mode lists etc.) */
|
2013-11-20 00:10:12 +07:00
|
|
|
struct drm_modeset_lock connection_mutex; /* protects connector->encoder and encoder->crtc links */
|
|
|
|
struct drm_modeset_acquire_ctx *acquire_ctx; /* for legacy _lock_all() / _unlock_all() */
|
2009-01-19 14:21:45 +07:00
|
|
|
struct mutex idr_mutex; /* for IDR management */
|
2008-11-08 05:05:41 +07:00
|
|
|
struct idr crtc_idr; /* use this idr for all IDs, fb, crtc, connector, modes - just makes life easier */
|
2014-10-20 13:17:17 +07:00
|
|
|
struct idr tile_idr; /* use this idr for all IDs, fb, crtc, connector, modes - just makes life easier */
|
2008-11-08 05:05:41 +07:00
|
|
|
/* this is limited to one for now */
|
drm: revamp locking around fb creation/destruction
Well, at least step 1. The goal here is that framebuffer objects can
survive outside of the mode_config lock, with just a reference held
as protection. The first step to get there is to introduce a special
fb_lock which protects fb lookup, creation and destruction, to make
them appear atomic.
This new fb_lock can nest within the mode_config lock. But the idea is
(once the reference counting part is completed) that we only quickly
take that fb_lock to lookup a framebuffer and grab a reference,
without any other locks involved.
vmwgfx is the only driver which does framebuffer lookups itself, also
wrap those calls to drm_mode_object_find with the new lock.
Also protect the fb_list walking in i915 and omapdrm with the new lock.
As a slight complication there's also the list of user-created fbs
attached to the file private. The problem now is that at fclose() time
we need to walk that list, eventually do a modeset call to remove the
fb from active usage (and are required to be able to take the
mode_config lock), but in the end we need to grab the new fb_lock to
remove the fb from the list. The easiest solution is to add another
mutex to protect this per-file list.
Currently that new fbs_lock nests within the modeset locks and so
appears redudant. But later patches will switch around this sequence
so that taking the modeset locks in the fb destruction path is
optional in the fastpath. Ultimately the goal is that addfb and rmfb
do not require the mode_config lock, since otherwise they have the
potential to introduce stalls in the pageflip sequence of a compositor
(if the compositor e.g. switches to a fullscreen client or if it
enables a plane). But that requires a few more steps and hoops to jump
through.
Note that framebuffer creation/destruction is now double-protected -
once by the fb_lock and in parts by the idr_lock. The later would be
unnecessariy if framebuffers would have their own idr allocator. But
that's material for another patch (series).
v2: Properly initialize the fb->filp_head list in _init, otherwise the
newly added WARN to check whether the fb isn't on a fpriv list any
more will fail for driver-private objects.
v3: Fixup two error-case unlock bugs spotted by Richard Wilbur.
Reviewed-by: Rob Clark <rob@ti.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2012-12-11 03:19:18 +07:00
|
|
|
|
2014-10-28 02:19:38 +07:00
|
|
|
struct mutex fb_lock; /* proctects global and per-file fb lists */
|
2008-11-08 05:05:41 +07:00
|
|
|
int num_fb;
|
|
|
|
struct list_head fb_list;
|
drm: revamp locking around fb creation/destruction
Well, at least step 1. The goal here is that framebuffer objects can
survive outside of the mode_config lock, with just a reference held
as protection. The first step to get there is to introduce a special
fb_lock which protects fb lookup, creation and destruction, to make
them appear atomic.
This new fb_lock can nest within the mode_config lock. But the idea is
(once the reference counting part is completed) that we only quickly
take that fb_lock to lookup a framebuffer and grab a reference,
without any other locks involved.
vmwgfx is the only driver which does framebuffer lookups itself, also
wrap those calls to drm_mode_object_find with the new lock.
Also protect the fb_list walking in i915 and omapdrm with the new lock.
As a slight complication there's also the list of user-created fbs
attached to the file private. The problem now is that at fclose() time
we need to walk that list, eventually do a modeset call to remove the
fb from active usage (and are required to be able to take the
mode_config lock), but in the end we need to grab the new fb_lock to
remove the fb from the list. The easiest solution is to add another
mutex to protect this per-file list.
Currently that new fbs_lock nests within the modeset locks and so
appears redudant. But later patches will switch around this sequence
so that taking the modeset locks in the fb destruction path is
optional in the fastpath. Ultimately the goal is that addfb and rmfb
do not require the mode_config lock, since otherwise they have the
potential to introduce stalls in the pageflip sequence of a compositor
(if the compositor e.g. switches to a fullscreen client or if it
enables a plane). But that requires a few more steps and hoops to jump
through.
Note that framebuffer creation/destruction is now double-protected -
once by the fb_lock and in parts by the idr_lock. The later would be
unnecessariy if framebuffers would have their own idr allocator. But
that's material for another patch (series).
v2: Properly initialize the fb->filp_head list in _init, otherwise the
newly added WARN to check whether the fb isn't on a fpriv list any
more will fail for driver-private objects.
v3: Fixup two error-case unlock bugs spotted by Richard Wilbur.
Reviewed-by: Rob Clark <rob@ti.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2012-12-11 03:19:18 +07:00
|
|
|
|
2008-11-08 05:05:41 +07:00
|
|
|
int num_connector;
|
|
|
|
struct list_head connector_list;
|
|
|
|
int num_encoder;
|
|
|
|
struct list_head encoder_list;
|
2014-04-02 05:22:30 +07:00
|
|
|
|
|
|
|
/*
|
|
|
|
* Track # of overlay planes separately from # of total planes. By
|
|
|
|
* default we only advertise overlay planes to userspace; if userspace
|
|
|
|
* sets the "universal plane" capability bit, we'll go ahead and
|
|
|
|
* expose all planes.
|
|
|
|
*/
|
|
|
|
int num_overlay_plane;
|
|
|
|
int num_total_plane;
|
2011-11-15 05:51:27 +07:00
|
|
|
struct list_head plane_list;
|
2008-11-08 05:05:41 +07:00
|
|
|
|
|
|
|
int num_crtc;
|
|
|
|
struct list_head crtc_list;
|
|
|
|
|
|
|
|
struct list_head property_list;
|
|
|
|
|
|
|
|
int min_width, min_height;
|
|
|
|
int max_width, max_height;
|
2012-05-17 18:27:23 +07:00
|
|
|
const struct drm_mode_config_funcs *funcs;
|
2009-02-02 12:55:45 +07:00
|
|
|
resource_size_t fb_base;
|
2008-11-08 05:05:41 +07:00
|
|
|
|
2010-05-07 13:42:51 +07:00
|
|
|
/* output poll support */
|
|
|
|
bool poll_enabled;
|
2012-10-24 01:23:36 +07:00
|
|
|
bool poll_running;
|
2015-01-21 14:45:21 +07:00
|
|
|
bool delayed_event;
|
2010-07-21 03:09:02 +07:00
|
|
|
struct delayed_work output_poll_work;
|
2010-05-07 13:42:51 +07:00
|
|
|
|
2015-04-21 01:22:54 +07:00
|
|
|
struct mutex blob_lock;
|
|
|
|
|
2008-11-08 05:05:41 +07:00
|
|
|
/* pointers to standard properties */
|
|
|
|
struct list_head property_blob_list;
|
|
|
|
struct drm_property *edid_property;
|
|
|
|
struct drm_property *dpms_property;
|
2014-06-05 11:01:31 +07:00
|
|
|
struct drm_property *path_property;
|
2014-10-20 13:30:50 +07:00
|
|
|
struct drm_property *tile_property;
|
2014-04-02 07:16:57 +07:00
|
|
|
struct drm_property *plane_type_property;
|
2014-08-05 12:56:54 +07:00
|
|
|
struct drm_property *rotation_property;
|
2014-12-19 04:01:53 +07:00
|
|
|
struct drm_property *prop_src_x;
|
|
|
|
struct drm_property *prop_src_y;
|
|
|
|
struct drm_property *prop_src_w;
|
|
|
|
struct drm_property *prop_src_h;
|
|
|
|
struct drm_property *prop_crtc_x;
|
|
|
|
struct drm_property *prop_crtc_y;
|
|
|
|
struct drm_property *prop_crtc_w;
|
|
|
|
struct drm_property *prop_crtc_h;
|
|
|
|
struct drm_property *prop_fb_id;
|
|
|
|
struct drm_property *prop_crtc_id;
|
2015-01-22 22:36:21 +07:00
|
|
|
struct drm_property *prop_active;
|
2015-05-26 01:11:52 +07:00
|
|
|
struct drm_property *prop_mode_id;
|
2008-11-08 05:05:41 +07:00
|
|
|
|
|
|
|
/* DVI-I properties */
|
|
|
|
struct drm_property *dvi_i_subconnector_property;
|
|
|
|
struct drm_property *dvi_i_select_subconnector_property;
|
|
|
|
|
|
|
|
/* TV properties */
|
|
|
|
struct drm_property *tv_subconnector_property;
|
|
|
|
struct drm_property *tv_select_subconnector_property;
|
|
|
|
struct drm_property *tv_mode_property;
|
|
|
|
struct drm_property *tv_left_margin_property;
|
|
|
|
struct drm_property *tv_right_margin_property;
|
|
|
|
struct drm_property *tv_top_margin_property;
|
|
|
|
struct drm_property *tv_bottom_margin_property;
|
2009-08-02 09:19:20 +07:00
|
|
|
struct drm_property *tv_brightness_property;
|
|
|
|
struct drm_property *tv_contrast_property;
|
|
|
|
struct drm_property *tv_flicker_reduction_property;
|
2009-08-12 07:30:10 +07:00
|
|
|
struct drm_property *tv_overscan_property;
|
|
|
|
struct drm_property *tv_saturation_property;
|
|
|
|
struct drm_property *tv_hue_property;
|
2008-11-08 05:05:41 +07:00
|
|
|
|
|
|
|
/* Optional properties */
|
|
|
|
struct drm_property *scaling_mode_property;
|
2014-06-11 12:16:48 +07:00
|
|
|
struct drm_property *aspect_ratio_property;
|
2009-12-04 06:25:47 +07:00
|
|
|
struct drm_property *dirty_info_property;
|
2011-09-29 22:20:42 +07:00
|
|
|
|
2014-11-10 07:18:15 +07:00
|
|
|
/* properties for virtual machine layout */
|
|
|
|
struct drm_property *suggested_x_property;
|
|
|
|
struct drm_property *suggested_y_property;
|
|
|
|
|
2011-09-29 22:20:42 +07:00
|
|
|
/* dumb ioctl parameters */
|
|
|
|
uint32_t preferred_depth, prefer_shadow;
|
2013-07-23 08:50:00 +07:00
|
|
|
|
|
|
|
/* whether async page flip is supported or not */
|
|
|
|
bool async_page_flip;
|
2014-02-13 00:48:23 +07:00
|
|
|
|
2015-02-05 21:41:52 +07:00
|
|
|
/* whether the driver supports fb modifiers */
|
|
|
|
bool allow_fb_modifiers;
|
|
|
|
|
2014-02-13 00:48:23 +07:00
|
|
|
/* cursor size */
|
|
|
|
uint32_t cursor_width, cursor_height;
|
2008-11-08 05:05:41 +07:00
|
|
|
};
|
|
|
|
|
2014-11-26 08:29:46 +07:00
|
|
|
/**
|
|
|
|
* drm_for_each_plane_mask - iterate over planes specified by bitmask
|
|
|
|
* @plane: the loop cursor
|
|
|
|
* @dev: the DRM device
|
|
|
|
* @plane_mask: bitmask of plane indices
|
|
|
|
*
|
|
|
|
* Iterate over all planes specified by bitmask.
|
|
|
|
*/
|
|
|
|
#define drm_for_each_plane_mask(plane, dev, plane_mask) \
|
|
|
|
list_for_each_entry((plane), &(dev)->mode_config.plane_list, head) \
|
drm: fix potential dangling else problems in for_each_ macros
We have serious dangling else bugs waiting to happen in our for_each_
style macros with ifs. Consider, for example,
#define drm_for_each_plane_mask(plane, dev, plane_mask) \
list_for_each_entry((plane), &(dev)->mode_config.plane_list, head) \
if ((plane_mask) & (1 << drm_plane_index(plane)))
If this is used in context:
if (condition)
drm_for_each_plane_mask(plane, dev, plane_mask);
else
foo();
foo() will be called for each plane *not* in plane_mask, if condition
holds, and not at all if condition doesn't hold.
Fix this by reversing the conditions in the macros, and adding an else
branch for the "for each" block, so that other if/else blocks can't
interfere. Provide a "for_each_if" helper macro to make it easier to get
this right.
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1448392916-2281-1-git-send-email-jani.nikula@intel.com
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2015-11-25 02:21:55 +07:00
|
|
|
for_each_if ((plane_mask) & (1 << drm_plane_index(plane)))
|
2014-11-26 08:29:46 +07:00
|
|
|
|
|
|
|
|
2008-11-08 05:05:41 +07:00
|
|
|
#define obj_to_crtc(x) container_of(x, struct drm_crtc, base)
|
|
|
|
#define obj_to_connector(x) container_of(x, struct drm_connector, base)
|
|
|
|
#define obj_to_encoder(x) container_of(x, struct drm_encoder, base)
|
|
|
|
#define obj_to_mode(x) container_of(x, struct drm_display_mode, base)
|
|
|
|
#define obj_to_fb(x) container_of(x, struct drm_framebuffer, base)
|
|
|
|
#define obj_to_property(x) container_of(x, struct drm_property, base)
|
|
|
|
#define obj_to_blob(x) container_of(x, struct drm_property_blob, base)
|
2011-11-15 05:51:27 +07:00
|
|
|
#define obj_to_plane(x) container_of(x, struct drm_plane, base)
|
2008-11-08 05:05:41 +07:00
|
|
|
|
2012-02-06 16:58:17 +07:00
|
|
|
struct drm_prop_enum_list {
|
|
|
|
int type;
|
|
|
|
char *name;
|
|
|
|
};
|
2008-11-08 05:05:41 +07:00
|
|
|
|
drm: Pass 'name' to drm_crtc_init_with_planes()
Done with coccinelle for the most part. However, it thinks '...' is
part of the semantic patch, so I put an 'int DOTDOTDOT' placeholder
in its place and got rid of it with sed afterwards.
I didn't convert drm_crtc_init() since passing the varargs through
would mean either cpp macros or va_list, and I figured we don't
care about these legacy functions enough to warrant the extra pain.
@@
identifier dev, crtc, primary, cursor, funcs;
@@
int drm_crtc_init_with_planes(struct drm_device *dev,
struct drm_crtc *crtc,
struct drm_plane *primary, struct drm_plane *cursor,
const struct drm_crtc_funcs *funcs
+ ,const char *name, int DOTDOTDOT
)
{ ... }
@@
identifier dev, crtc, primary, cursor, funcs;
@@
int drm_crtc_init_with_planes(struct drm_device *dev,
struct drm_crtc *crtc,
struct drm_plane *primary, struct drm_plane *cursor,
const struct drm_crtc_funcs *funcs
+ ,const char *name, int DOTDOTDOT
);
@@
expression E1, E2, E3, E4, E5;
@@
drm_crtc_init_with_planes(E1, E2, E3, E4, E5
+ ,NULL
)
v2: Split crtc and plane changes apart
Pass NULL for no-name instead of ""
Leave drm_crtc_init() alone
v3: Add ', or NULL...' to @name kernel doc (Jani)
Annotate the function with __printf() attribute (Jani)
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/1449670771-2751-1-git-send-email-ville.syrjala@linux.intel.com
2015-12-09 21:19:31 +07:00
|
|
|
extern __printf(6, 7)
|
|
|
|
int drm_crtc_init_with_planes(struct drm_device *dev,
|
|
|
|
struct drm_crtc *crtc,
|
|
|
|
struct drm_plane *primary,
|
|
|
|
struct drm_plane *cursor,
|
|
|
|
const struct drm_crtc_funcs *funcs,
|
|
|
|
const char *name, ...);
|
2008-11-08 05:05:41 +07:00
|
|
|
extern void drm_crtc_cleanup(struct drm_crtc *crtc);
|
2014-01-03 04:27:33 +07:00
|
|
|
extern unsigned int drm_crtc_index(struct drm_crtc *crtc);
|
|
|
|
|
|
|
|
/**
|
|
|
|
* drm_crtc_mask - find the mask of a registered CRTC
|
|
|
|
* @crtc: CRTC to find mask for
|
|
|
|
*
|
|
|
|
* Given a registered CRTC, return the mask bit of that CRTC for an
|
|
|
|
* encoder's possible_crtcs field.
|
|
|
|
*/
|
|
|
|
static inline uint32_t drm_crtc_mask(struct drm_crtc *crtc)
|
|
|
|
{
|
|
|
|
return 1 << drm_crtc_index(crtc);
|
|
|
|
}
|
2008-11-08 05:05:41 +07:00
|
|
|
|
2013-08-08 09:34:48 +07:00
|
|
|
extern void drm_connector_ida_init(void);
|
|
|
|
extern void drm_connector_ida_destroy(void);
|
2012-03-13 17:35:48 +07:00
|
|
|
extern int drm_connector_init(struct drm_device *dev,
|
|
|
|
struct drm_connector *connector,
|
|
|
|
const struct drm_connector_funcs *funcs,
|
|
|
|
int connector_type);
|
2014-05-29 22:57:41 +07:00
|
|
|
int drm_connector_register(struct drm_connector *connector);
|
|
|
|
void drm_connector_unregister(struct drm_connector *connector);
|
2008-11-08 05:05:41 +07:00
|
|
|
|
|
|
|
extern void drm_connector_cleanup(struct drm_connector *connector);
|
2014-07-29 18:47:11 +07:00
|
|
|
extern unsigned int drm_connector_index(struct drm_connector *connector);
|
2012-02-20 21:16:40 +07:00
|
|
|
/* helper to unplug all connectors from sysfs for device */
|
|
|
|
extern void drm_connector_unplug_all(struct drm_device *dev);
|
2008-11-08 05:05:41 +07:00
|
|
|
|
2015-01-20 23:38:44 +07:00
|
|
|
extern int drm_bridge_add(struct drm_bridge *bridge);
|
|
|
|
extern void drm_bridge_remove(struct drm_bridge *bridge);
|
|
|
|
extern struct drm_bridge *of_drm_find_bridge(struct device_node *np);
|
|
|
|
extern int drm_bridge_attach(struct drm_device *dev, struct drm_bridge *bridge);
|
2013-08-15 03:47:37 +07:00
|
|
|
|
drm: bridge: Allow daisy chaining of bridges
Allow drm_bridge objects to link to each other in order to form an encoder
chain. The requirement for creating a chain of bridges comes because the
MSM drm driver uses up its encoder and bridge objects for blocks within
the SoC itself. There isn't anything left to use if the SoC display output
is connected to an external encoder IC. Having an additional bridge
connected to the existing bridge helps here. In general, it is possible for
platforms to have multiple devices between the encoder and the
connector/panel that require some sort of configuration.
We create drm bridge helper functions corresponding to each op in
'drm_bridge_funcs'. These helpers call the corresponding
'drm_bridge_funcs' op for the entire chain of bridges. These helpers are
used internally by drm_atomic_helper.c and drm_crtc_helper.c.
The drm_bridge_enable/pre_enable helpers execute enable/pre_enable ops of
the bridge closet to the encoder, and proceed until the last bridge in the
chain is enabled. The same holds for drm_bridge_mode_set/mode_fixup
helpers. The drm_bridge_disable/post_disable helpers disable the last
bridge in the chain first, and proceed until the first bridge in the chain
is disabled.
drm_bridge_attach() remains the same. As before, the driver calling this
function should make sure it has set the links correctly. The order in
which the bridges are connected to each other determines the order in which
the calls are made. One requirement is that every bridge in the chain
should point the parent encoder object. This is required since bridge
drivers expect a valid encoder pointer in drm_bridge. For example, consider
a chain where an encoder's output is connected to bridge1, and bridge1's
output is connected to bridge2:
/* Like before, attach bridge to an encoder */
bridge1->encoder = encoder;
ret = drm_bridge_attach(dev, bridge1);
..
/*
* set the first bridge's 'next' bridge to bridge2, set its encoder
* as bridge1's encoder
*/
bridge1->next = bridge2
bridge2->encoder = bridge1->encoder;
ret = drm_bridge_attach(dev, bridge2);
...
...
This method of bridge chaining isn't intrusive and existing drivers that
use drm_bridge will behave the same way as before. The bridge helpers also
cleans up the atomic and crtc helper files a bit.
Reviewed-by: Jani Nikula <jani.nikula@linux.intel.com>
Reviewed-by: Rob Clark <robdclark@gmail.com>
Reviewed-by: Daniel Vetter <daniel@ffwll.ch>
Signed-off-by: Archit Taneja <architt@codeaurora.org>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2015-05-21 12:33:16 +07:00
|
|
|
bool drm_bridge_mode_fixup(struct drm_bridge *bridge,
|
|
|
|
const struct drm_display_mode *mode,
|
|
|
|
struct drm_display_mode *adjusted_mode);
|
|
|
|
void drm_bridge_disable(struct drm_bridge *bridge);
|
|
|
|
void drm_bridge_post_disable(struct drm_bridge *bridge);
|
|
|
|
void drm_bridge_mode_set(struct drm_bridge *bridge,
|
|
|
|
struct drm_display_mode *mode,
|
|
|
|
struct drm_display_mode *adjusted_mode);
|
|
|
|
void drm_bridge_pre_enable(struct drm_bridge *bridge);
|
|
|
|
void drm_bridge_enable(struct drm_bridge *bridge);
|
|
|
|
|
drm: Pass 'name' to drm_encoder_init()
Done with coccinelle for the most part. However, it thinks '...' is
part of the semantic patch, so I put an 'int DOTDOTDOT' placeholder
in its place and got rid of it with sed afterwards.
@@
identifier dev, encoder, funcs;
@@
int drm_encoder_init(struct drm_device *dev,
struct drm_encoder *encoder,
const struct drm_encoder_funcs *funcs,
int encoder_type
+ ,const char *name, int DOTDOTDOT
)
{ ... }
@@
identifier dev, encoder, funcs;
@@
int drm_encoder_init(struct drm_device *dev,
struct drm_encoder *encoder,
const struct drm_encoder_funcs *funcs,
int encoder_type
+ ,const char *name, int DOTDOTDOT
);
@@
expression E1, E2, E3, E4;
@@
drm_encoder_init(E1, E2, E3, E4
+ ,NULL
)
v2: Add ', or NULL...' to @name kernel doc (Jani)
Annotate the function with __printf() attribute (Jani)
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/1449670818-2966-1-git-send-email-ville.syrjala@linux.intel.com
2015-12-09 21:20:18 +07:00
|
|
|
extern __printf(5, 6)
|
|
|
|
int drm_encoder_init(struct drm_device *dev,
|
|
|
|
struct drm_encoder *encoder,
|
|
|
|
const struct drm_encoder_funcs *funcs,
|
|
|
|
int encoder_type, const char *name, ...);
|
2008-11-08 05:05:41 +07:00
|
|
|
|
2014-01-13 20:33:20 +07:00
|
|
|
/**
|
|
|
|
* drm_encoder_crtc_ok - can a given crtc drive a given encoder?
|
|
|
|
* @encoder: encoder to test
|
|
|
|
* @crtc: crtc to test
|
|
|
|
*
|
|
|
|
* Return false if @encoder can't be driven by @crtc, true otherwise.
|
|
|
|
*/
|
|
|
|
static inline bool drm_encoder_crtc_ok(struct drm_encoder *encoder,
|
|
|
|
struct drm_crtc *crtc)
|
|
|
|
{
|
|
|
|
return !!(encoder->possible_crtcs & drm_crtc_mask(crtc));
|
|
|
|
}
|
|
|
|
|
drm: Pass 'name' to drm_universal_plane_init()
Done with coccinelle for the most part. It choked on
msm/mdp/mdp5/mdp5_plane.c like so:
"BAD:!!!!! enum drm_plane_type type;"
No idea how to deal with that, so I just fixed that up
by hand.
Also it thinks '...' is part of the semantic patch, so I put an
'int DOTDOTDOT' placeholder in its place and got rid of it with
sed afterwards.
I didn't convert drm_plane_init() since passing the varargs through
would mean either cpp macros or va_list, and I figured we don't
care about these legacy functions enough to warrant the extra pain.
@@
typedef uint32_t;
identifier dev, plane, possible_crtcs, funcs, formats, format_count, type;
@@
int drm_universal_plane_init(struct drm_device *dev,
struct drm_plane *plane,
unsigned long possible_crtcs,
const struct drm_plane_funcs *funcs,
const uint32_t *formats,
unsigned int format_count,
enum drm_plane_type type
+ ,const char *name, int DOTDOTDOT
)
{ ... }
@@
identifier dev, plane, possible_crtcs, funcs, formats, format_count, type;
@@
int drm_universal_plane_init(struct drm_device *dev,
struct drm_plane *plane,
unsigned long possible_crtcs,
const struct drm_plane_funcs *funcs,
const uint32_t *formats,
unsigned int format_count,
enum drm_plane_type type
+ ,const char *name, int DOTDOTDOT
);
@@
expression E1, E2, E3, E4, E5, E6, E7;
@@
drm_universal_plane_init(E1, E2, E3, E4, E5, E6, E7
+ ,NULL
)
v2: Split crtc and plane changes apart
Pass NUL for no-name instead of ""
Leave drm_plane_init() alone
v3: Add ', or NULL...' to @name kernel doc (Jani)
Annotate the function with __printf() attribute (Jani)
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/1449670795-2853-1-git-send-email-ville.syrjala@linux.intel.com
2015-12-09 21:19:55 +07:00
|
|
|
extern __printf(8, 9)
|
|
|
|
int drm_universal_plane_init(struct drm_device *dev,
|
|
|
|
struct drm_plane *plane,
|
|
|
|
unsigned long possible_crtcs,
|
|
|
|
const struct drm_plane_funcs *funcs,
|
|
|
|
const uint32_t *formats,
|
|
|
|
unsigned int format_count,
|
|
|
|
enum drm_plane_type type,
|
|
|
|
const char *name, ...);
|
2011-11-15 05:51:27 +07:00
|
|
|
extern int drm_plane_init(struct drm_device *dev,
|
|
|
|
struct drm_plane *plane,
|
|
|
|
unsigned long possible_crtcs,
|
|
|
|
const struct drm_plane_funcs *funcs,
|
2015-08-12 21:54:28 +07:00
|
|
|
const uint32_t *formats, unsigned int format_count,
|
2014-04-02 05:22:36 +07:00
|
|
|
bool is_primary);
|
2011-11-15 05:51:27 +07:00
|
|
|
extern void drm_plane_cleanup(struct drm_plane *plane);
|
2014-07-29 18:47:11 +07:00
|
|
|
extern unsigned int drm_plane_index(struct drm_plane *plane);
|
2015-04-10 07:36:21 +07:00
|
|
|
extern struct drm_plane * drm_plane_from_index(struct drm_device *dev, int idx);
|
2013-06-03 20:10:40 +07:00
|
|
|
extern void drm_plane_force_disable(struct drm_plane *plane);
|
2015-03-05 07:25:43 +07:00
|
|
|
extern int drm_plane_check_pixel_format(const struct drm_plane *plane,
|
|
|
|
u32 format);
|
2014-12-02 06:40:09 +07:00
|
|
|
extern void drm_crtc_get_hv_timing(const struct drm_display_mode *mode,
|
|
|
|
int *hdisplay, int *vdisplay);
|
2014-04-02 05:22:34 +07:00
|
|
|
extern int drm_crtc_check_viewport(const struct drm_crtc *crtc,
|
|
|
|
int x, int y,
|
|
|
|
const struct drm_display_mode *mode,
|
|
|
|
const struct drm_framebuffer *fb);
|
2011-11-15 05:51:27 +07:00
|
|
|
|
2008-11-08 05:05:41 +07:00
|
|
|
extern void drm_encoder_cleanup(struct drm_encoder *encoder);
|
|
|
|
|
2013-06-07 22:43:07 +07:00
|
|
|
extern const char *drm_get_connector_status_name(enum drm_connector_status status);
|
2014-02-11 06:32:44 +07:00
|
|
|
extern const char *drm_get_subpixel_order_name(enum subpixel_order order);
|
2013-06-07 22:43:07 +07:00
|
|
|
extern const char *drm_get_dpms_name(int val);
|
|
|
|
extern const char *drm_get_dvi_i_subconnector_name(int val);
|
|
|
|
extern const char *drm_get_dvi_i_select_name(int val);
|
|
|
|
extern const char *drm_get_tv_subconnector_name(int val);
|
|
|
|
extern const char *drm_get_tv_select_name(int val);
|
2009-02-13 02:37:56 +07:00
|
|
|
extern void drm_fb_release(struct drm_file *file_priv);
|
2015-05-22 19:34:51 +07:00
|
|
|
extern void drm_property_destroy_user_blobs(struct drm_device *dev,
|
|
|
|
struct drm_file *file_priv);
|
2012-09-18 21:58:47 +07:00
|
|
|
extern bool drm_probe_ddc(struct i2c_adapter *adapter);
|
2008-11-08 05:05:41 +07:00
|
|
|
extern struct edid *drm_get_edid(struct drm_connector *connector,
|
|
|
|
struct i2c_adapter *adapter);
|
2013-09-27 19:08:27 +07:00
|
|
|
extern struct edid *drm_edid_duplicate(const struct edid *edid);
|
2008-11-08 05:05:41 +07:00
|
|
|
extern int drm_add_edid_modes(struct drm_connector *connector, struct edid *edid);
|
|
|
|
extern void drm_mode_config_init(struct drm_device *dev);
|
2011-01-24 22:11:08 +07:00
|
|
|
extern void drm_mode_config_reset(struct drm_device *dev);
|
2008-11-08 05:05:41 +07:00
|
|
|
extern void drm_mode_config_cleanup(struct drm_device *dev);
|
2014-01-23 21:52:20 +07:00
|
|
|
|
2014-06-05 11:01:31 +07:00
|
|
|
extern int drm_mode_connector_set_path_property(struct drm_connector *connector,
|
2014-05-13 16:38:36 +07:00
|
|
|
const char *path);
|
2014-10-20 13:30:50 +07:00
|
|
|
int drm_mode_connector_set_tile_property(struct drm_connector *connector);
|
2008-11-08 05:05:41 +07:00
|
|
|
extern int drm_mode_connector_update_edid_property(struct drm_connector *connector,
|
2014-05-13 16:38:36 +07:00
|
|
|
const struct edid *edid);
|
2014-05-30 22:34:01 +07:00
|
|
|
|
2014-07-22 17:09:10 +07:00
|
|
|
extern int drm_display_info_set_bus_formats(struct drm_display_info *info,
|
|
|
|
const u32 *formats,
|
|
|
|
unsigned int num_formats);
|
|
|
|
|
2014-05-30 22:34:01 +07:00
|
|
|
static inline bool drm_property_type_is(struct drm_property *property,
|
|
|
|
uint32_t type)
|
|
|
|
{
|
|
|
|
/* instanceof for props.. handles extended type vs original types: */
|
|
|
|
if (property->flags & DRM_MODE_PROP_EXTENDED_TYPE)
|
|
|
|
return (property->flags & DRM_MODE_PROP_EXTENDED_TYPE) == type;
|
|
|
|
return property->flags & type;
|
|
|
|
}
|
|
|
|
|
|
|
|
static inline bool drm_property_type_valid(struct drm_property *property)
|
|
|
|
{
|
|
|
|
if (property->flags & DRM_MODE_PROP_EXTENDED_TYPE)
|
|
|
|
return !(property->flags & DRM_MODE_PROP_LEGACY_TYPE);
|
|
|
|
return !!(property->flags & DRM_MODE_PROP_LEGACY_TYPE);
|
|
|
|
}
|
|
|
|
|
2012-05-16 04:09:02 +07:00
|
|
|
extern int drm_object_property_set_value(struct drm_mode_object *obj,
|
|
|
|
struct drm_property *property,
|
|
|
|
uint64_t val);
|
|
|
|
extern int drm_object_property_get_value(struct drm_mode_object *obj,
|
|
|
|
struct drm_property *property,
|
|
|
|
uint64_t *value);
|
2008-11-08 05:05:41 +07:00
|
|
|
extern int drm_framebuffer_init(struct drm_device *dev,
|
|
|
|
struct drm_framebuffer *fb,
|
|
|
|
const struct drm_framebuffer_funcs *funcs);
|
2012-12-03 03:53:40 +07:00
|
|
|
extern struct drm_framebuffer *drm_framebuffer_lookup(struct drm_device *dev,
|
|
|
|
uint32_t id);
|
2012-09-06 04:48:38 +07:00
|
|
|
extern void drm_framebuffer_unreference(struct drm_framebuffer *fb);
|
|
|
|
extern void drm_framebuffer_reference(struct drm_framebuffer *fb);
|
|
|
|
extern void drm_framebuffer_remove(struct drm_framebuffer *fb);
|
2008-11-08 05:05:41 +07:00
|
|
|
extern void drm_framebuffer_cleanup(struct drm_framebuffer *fb);
|
drm: revamp framebuffer cleanup interfaces
We have two classes of framebuffer
- Created by the driver (atm only for fbdev), and the driver holds
onto the last reference count until destruction.
- Created by userspace and associated with a given fd. These
framebuffers will be reaped when their assoiciated fb is closed.
Now these two cases are set up differently, the framebuffers are on
different lists and hence destruction needs to clean up different
things. Also, for userspace framebuffers we remove them from any
current usage, whereas for internal framebuffers it is assumed that
the driver has done this already.
Long story short, we need two different ways to cleanup such drivers.
Three functions are involved in total:
- drm_framebuffer_remove: Convenience function which removes the fb
from all active usage and then drops the passed-in reference.
- drm_framebuffer_unregister_private: Will remove driver-private
framebuffers from relevant lists and drop the corresponding
references. Should be called for driver-private framebuffers before
dropping the last reference (or like for a lot of the drivers where
the fbdev is embedded someplace else, before doing the cleanup
manually).
- drm_framebuffer_cleanup: Final cleanup for both classes of fbs,
should be called by the driver's ->destroy callback once the last
reference is gone.
This patch just rolls out the new interfaces and updates all drivers
(by adding calls to drm_framebuffer_unregister_private at all the
right places)- no functional changes yet. Follow-on patches will move
drm core code around and update the lifetime management for
framebuffers, so that we are no longer required to keep framebuffers
alive by locking mode_config.mutex.
I've also updated the kerneldoc already.
vmwgfx seems to again be a bit special, at least I haven't figured out
how the fbdev support in that driver works. It smells like it's
external though.
v2: The i915 driver creates another private framebuffer in the
load-detect code. Adjust its cleanup code, too.
Reviewed-by: Rob Clark <rob@ti.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2012-12-11 02:42:17 +07:00
|
|
|
extern void drm_framebuffer_unregister_private(struct drm_framebuffer *fb);
|
2008-11-08 05:05:41 +07:00
|
|
|
|
2012-05-16 04:09:02 +07:00
|
|
|
extern void drm_object_attach_property(struct drm_mode_object *obj,
|
|
|
|
struct drm_property *property,
|
|
|
|
uint64_t init_val);
|
2008-11-08 05:05:41 +07:00
|
|
|
extern struct drm_property *drm_property_create(struct drm_device *dev, int flags,
|
|
|
|
const char *name, int num_values);
|
2012-02-06 16:58:17 +07:00
|
|
|
extern struct drm_property *drm_property_create_enum(struct drm_device *dev, int flags,
|
|
|
|
const char *name,
|
2012-05-17 15:23:26 +07:00
|
|
|
const struct drm_prop_enum_list *props,
|
|
|
|
int num_values);
|
|
|
|
struct drm_property *drm_property_create_bitmask(struct drm_device *dev,
|
|
|
|
int flags, const char *name,
|
2012-02-06 16:58:17 +07:00
|
|
|
const struct drm_prop_enum_list *props,
|
2014-07-08 12:01:52 +07:00
|
|
|
int num_props,
|
|
|
|
uint64_t supported_bits);
|
2012-02-06 16:58:18 +07:00
|
|
|
struct drm_property *drm_property_create_range(struct drm_device *dev, int flags,
|
|
|
|
const char *name,
|
|
|
|
uint64_t min, uint64_t max);
|
2012-09-13 10:22:31 +07:00
|
|
|
struct drm_property *drm_property_create_signed_range(struct drm_device *dev,
|
|
|
|
int flags, const char *name,
|
|
|
|
int64_t min, int64_t max);
|
2014-05-30 22:37:03 +07:00
|
|
|
struct drm_property *drm_property_create_object(struct drm_device *dev,
|
|
|
|
int flags, const char *name, uint32_t type);
|
2015-01-21 14:47:38 +07:00
|
|
|
struct drm_property *drm_property_create_bool(struct drm_device *dev, int flags,
|
|
|
|
const char *name);
|
2015-04-21 01:22:55 +07:00
|
|
|
struct drm_property_blob *drm_property_create_blob(struct drm_device *dev,
|
|
|
|
size_t length,
|
|
|
|
const void *data);
|
|
|
|
struct drm_property_blob *drm_property_lookup_blob(struct drm_device *dev,
|
|
|
|
uint32_t id);
|
|
|
|
struct drm_property_blob *drm_property_reference_blob(struct drm_property_blob *blob);
|
|
|
|
void drm_property_unreference_blob(struct drm_property_blob *blob);
|
2008-11-08 05:05:41 +07:00
|
|
|
extern void drm_property_destroy(struct drm_device *dev, struct drm_property *property);
|
|
|
|
extern int drm_property_add_enum(struct drm_property *property, int index,
|
|
|
|
uint64_t value, const char *name);
|
|
|
|
extern int drm_mode_create_dvi_i_properties(struct drm_device *dev);
|
2014-10-13 17:45:57 +07:00
|
|
|
extern int drm_mode_create_tv_properties(struct drm_device *dev,
|
|
|
|
unsigned int num_modes,
|
2015-08-31 19:09:26 +07:00
|
|
|
const char * const modes[]);
|
2008-11-08 05:05:41 +07:00
|
|
|
extern int drm_mode_create_scaling_mode_property(struct drm_device *dev);
|
2014-06-11 12:16:48 +07:00
|
|
|
extern int drm_mode_create_aspect_ratio_property(struct drm_device *dev);
|
2009-12-04 06:25:47 +07:00
|
|
|
extern int drm_mode_create_dirty_info_property(struct drm_device *dev);
|
2014-11-10 07:18:15 +07:00
|
|
|
extern int drm_mode_create_suggested_offset_properties(struct drm_device *dev);
|
2014-12-19 04:01:56 +07:00
|
|
|
extern bool drm_property_change_valid_get(struct drm_property *property,
|
|
|
|
uint64_t value, struct drm_mode_object **ref);
|
|
|
|
extern void drm_property_change_valid_put(struct drm_property *property,
|
|
|
|
struct drm_mode_object *ref);
|
2008-11-08 05:05:41 +07:00
|
|
|
|
|
|
|
extern int drm_mode_connector_attach_encoder(struct drm_connector *connector,
|
|
|
|
struct drm_encoder *encoder);
|
2012-02-01 17:38:23 +07:00
|
|
|
extern int drm_mode_crtc_set_gamma_size(struct drm_crtc *crtc,
|
2008-11-08 05:05:41 +07:00
|
|
|
int gamma_size);
|
2009-09-16 03:57:31 +07:00
|
|
|
extern struct drm_mode_object *drm_mode_object_find(struct drm_device *dev,
|
|
|
|
uint32_t id, uint32_t type);
|
2014-05-30 22:37:03 +07:00
|
|
|
|
2008-11-08 05:05:41 +07:00
|
|
|
/* IOCTLs */
|
|
|
|
extern int drm_mode_getresources(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
2011-11-15 05:51:27 +07:00
|
|
|
extern int drm_mode_getplane_res(struct drm_device *dev, void *data,
|
|
|
|
struct drm_file *file_priv);
|
2008-11-08 05:05:41 +07:00
|
|
|
extern int drm_mode_getcrtc(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
|
|
|
extern int drm_mode_getconnector(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
2012-12-11 19:47:23 +07:00
|
|
|
extern int drm_mode_set_config_internal(struct drm_mode_set *set);
|
2008-11-08 05:05:41 +07:00
|
|
|
extern int drm_mode_setcrtc(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
2011-11-15 05:51:27 +07:00
|
|
|
extern int drm_mode_getplane(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
|
|
|
extern int drm_mode_setplane(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
2008-11-08 05:05:41 +07:00
|
|
|
extern int drm_mode_cursor_ioctl(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
2013-06-20 08:48:52 +07:00
|
|
|
extern int drm_mode_cursor2_ioctl(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
2008-11-08 05:05:41 +07:00
|
|
|
extern int drm_mode_addfb(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
2011-11-15 05:51:28 +07:00
|
|
|
extern int drm_mode_addfb2(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
|
|
|
extern uint32_t drm_mode_legacy_fb_format(uint32_t bpp, uint32_t depth);
|
2008-11-08 05:05:41 +07:00
|
|
|
extern int drm_mode_rmfb(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
|
|
|
extern int drm_mode_getfb(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
2009-12-04 06:25:47 +07:00
|
|
|
extern int drm_mode_dirtyfb_ioctl(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
2008-11-08 05:05:41 +07:00
|
|
|
|
|
|
|
extern int drm_mode_getproperty_ioctl(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
|
|
|
extern int drm_mode_getblob_ioctl(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
2015-05-22 19:34:51 +07:00
|
|
|
extern int drm_mode_createblob_ioctl(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
|
|
|
extern int drm_mode_destroyblob_ioctl(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
2008-11-08 05:05:41 +07:00
|
|
|
extern int drm_mode_connector_property_set_ioctl(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
|
|
|
extern int drm_mode_getencoder(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
|
|
|
extern int drm_mode_gamma_get_ioctl(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
|
|
|
extern int drm_mode_gamma_set_ioctl(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
2012-12-20 21:41:44 +07:00
|
|
|
extern u8 drm_match_cea_mode(const struct drm_display_mode *to_match);
|
2014-04-01 17:56:59 +07:00
|
|
|
extern enum hdmi_picture_aspect drm_get_cea_aspect_ratio(const u8 video_code);
|
2009-03-26 18:26:23 +07:00
|
|
|
extern bool drm_detect_hdmi_monitor(struct edid *edid);
|
2010-09-19 13:27:28 +07:00
|
|
|
extern bool drm_detect_monitor_audio(struct edid *edid);
|
2013-01-17 21:31:30 +07:00
|
|
|
extern bool drm_rgb_quant_range_selectable(struct edid *edid);
|
2009-11-18 00:43:55 +07:00
|
|
|
extern int drm_mode_page_flip_ioctl(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
2009-09-03 08:33:48 +07:00
|
|
|
extern int drm_add_modes_noedid(struct drm_connector *connector,
|
|
|
|
int hdisplay, int vdisplay);
|
2013-10-11 15:01:08 +07:00
|
|
|
extern void drm_set_preferred_mode(struct drm_connector *connector,
|
|
|
|
int hpref, int vpref);
|
2010-02-05 16:21:19 +07:00
|
|
|
|
2011-07-29 21:28:57 +07:00
|
|
|
extern int drm_edid_header_is_valid(const u8 *raw_edid);
|
2015-04-22 01:09:41 +07:00
|
|
|
extern bool drm_edid_block_valid(u8 *raw_edid, int block, bool print_bad_edid,
|
|
|
|
bool *edid_corrupt);
|
2010-02-05 16:21:19 +07:00
|
|
|
extern bool drm_edid_is_valid(struct edid *edid);
|
2014-10-20 13:17:17 +07:00
|
|
|
|
|
|
|
extern struct drm_tile_group *drm_mode_create_tile_group(struct drm_device *dev,
|
|
|
|
char topology[8]);
|
|
|
|
extern struct drm_tile_group *drm_mode_get_tile_group(struct drm_device *dev,
|
|
|
|
char topology[8]);
|
|
|
|
extern void drm_mode_put_tile_group(struct drm_device *dev,
|
|
|
|
struct drm_tile_group *tg);
|
2010-05-07 12:02:30 +07:00
|
|
|
struct drm_display_mode *drm_mode_find_dmt(struct drm_device *dev,
|
2012-04-14 03:33:31 +07:00
|
|
|
int hsize, int vsize, int fresh,
|
|
|
|
bool rb);
|
2011-02-07 09:16:14 +07:00
|
|
|
|
|
|
|
extern int drm_mode_create_dumb_ioctl(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
|
|
|
extern int drm_mode_mmap_dumb_ioctl(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
|
|
|
extern int drm_mode_destroy_dumb_ioctl(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
2012-05-16 04:09:02 +07:00
|
|
|
extern int drm_mode_obj_get_properties_ioctl(struct drm_device *dev, void *data,
|
|
|
|
struct drm_file *file_priv);
|
|
|
|
extern int drm_mode_obj_set_property_ioctl(struct drm_device *dev, void *data,
|
|
|
|
struct drm_file *file_priv);
|
2014-08-20 20:45:00 +07:00
|
|
|
extern int drm_mode_plane_set_obj_prop(struct drm_plane *plane,
|
|
|
|
struct drm_property *property,
|
|
|
|
uint64_t value);
|
2014-12-19 04:01:56 +07:00
|
|
|
extern int drm_mode_atomic_ioctl(struct drm_device *dev,
|
|
|
|
void *data, struct drm_file *file_priv);
|
2011-11-30 03:02:54 +07:00
|
|
|
|
|
|
|
extern void drm_fb_get_bpp_depth(uint32_t format, unsigned int *depth,
|
|
|
|
int *bpp);
|
2012-04-06 01:35:15 +07:00
|
|
|
extern int drm_format_num_planes(uint32_t format);
|
2012-04-06 01:35:16 +07:00
|
|
|
extern int drm_format_plane_cpp(uint32_t format, int plane);
|
2012-04-06 01:35:17 +07:00
|
|
|
extern int drm_format_horz_chroma_subsampling(uint32_t format);
|
|
|
|
extern int drm_format_vert_chroma_subsampling(uint32_t format);
|
2013-06-07 22:43:07 +07:00
|
|
|
extern const char *drm_get_format_name(uint32_t format);
|
2014-07-08 12:01:53 +07:00
|
|
|
extern struct drm_property *drm_mode_create_rotation_property(struct drm_device *dev,
|
|
|
|
unsigned int supported_rotations);
|
2014-07-08 12:01:56 +07:00
|
|
|
extern unsigned int drm_rotation_simplify(unsigned int rotation,
|
|
|
|
unsigned int supported_rotations);
|
2012-04-06 01:35:15 +07:00
|
|
|
|
2012-08-15 19:59:49 +07:00
|
|
|
/* Helpers */
|
2013-10-06 03:36:52 +07:00
|
|
|
|
|
|
|
static inline struct drm_plane *drm_plane_find(struct drm_device *dev,
|
|
|
|
uint32_t id)
|
|
|
|
{
|
|
|
|
struct drm_mode_object *mo;
|
|
|
|
mo = drm_mode_object_find(dev, id, DRM_MODE_OBJECT_PLANE);
|
|
|
|
return mo ? obj_to_plane(mo) : NULL;
|
|
|
|
}
|
|
|
|
|
2012-08-15 19:59:49 +07:00
|
|
|
static inline struct drm_crtc *drm_crtc_find(struct drm_device *dev,
|
|
|
|
uint32_t id)
|
|
|
|
{
|
|
|
|
struct drm_mode_object *mo;
|
|
|
|
mo = drm_mode_object_find(dev, id, DRM_MODE_OBJECT_CRTC);
|
|
|
|
return mo ? obj_to_crtc(mo) : NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
static inline struct drm_encoder *drm_encoder_find(struct drm_device *dev,
|
|
|
|
uint32_t id)
|
|
|
|
{
|
|
|
|
struct drm_mode_object *mo;
|
|
|
|
mo = drm_mode_object_find(dev, id, DRM_MODE_OBJECT_ENCODER);
|
|
|
|
return mo ? obj_to_encoder(mo) : NULL;
|
|
|
|
}
|
|
|
|
|
2013-10-06 03:36:52 +07:00
|
|
|
static inline struct drm_connector *drm_connector_find(struct drm_device *dev,
|
|
|
|
uint32_t id)
|
|
|
|
{
|
|
|
|
struct drm_mode_object *mo;
|
|
|
|
mo = drm_mode_object_find(dev, id, DRM_MODE_OBJECT_CONNECTOR);
|
|
|
|
return mo ? obj_to_connector(mo) : NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
static inline struct drm_property *drm_property_find(struct drm_device *dev,
|
|
|
|
uint32_t id)
|
|
|
|
{
|
|
|
|
struct drm_mode_object *mo;
|
|
|
|
mo = drm_mode_object_find(dev, id, DRM_MODE_OBJECT_PROPERTY);
|
|
|
|
return mo ? obj_to_property(mo) : NULL;
|
|
|
|
}
|
|
|
|
|
2014-04-02 05:22:30 +07:00
|
|
|
/* Plane list iterator for legacy (overlay only) planes. */
|
2015-07-10 04:44:24 +07:00
|
|
|
#define drm_for_each_legacy_plane(plane, dev) \
|
|
|
|
list_for_each_entry(plane, &(dev)->mode_config.plane_list, head) \
|
drm: fix potential dangling else problems in for_each_ macros
We have serious dangling else bugs waiting to happen in our for_each_
style macros with ifs. Consider, for example,
#define drm_for_each_plane_mask(plane, dev, plane_mask) \
list_for_each_entry((plane), &(dev)->mode_config.plane_list, head) \
if ((plane_mask) & (1 << drm_plane_index(plane)))
If this is used in context:
if (condition)
drm_for_each_plane_mask(plane, dev, plane_mask);
else
foo();
foo() will be called for each plane *not* in plane_mask, if condition
holds, and not at all if condition doesn't hold.
Fix this by reversing the conditions in the macros, and adding an else
branch for the "for each" block, so that other if/else blocks can't
interfere. Provide a "for_each_if" helper macro to make it easier to get
this right.
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1448392916-2281-1-git-send-email-jani.nikula@intel.com
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2015-11-25 02:21:55 +07:00
|
|
|
for_each_if (plane->type == DRM_PLANE_TYPE_OVERLAY)
|
2014-04-02 05:22:30 +07:00
|
|
|
|
drm: Add modeset object iterators
And roll them out across drm_* files. The point here isn't code
prettification (it helps with that too) but that some of these lists
aren't static any more. And having macros will gives us a convenient
place to put locking checks into.
I didn't add an iterator for props since that's only used by a
list_for_each_entry_safe in the driver teardown code.
Search&replace was done with the below cocci spatch. Note that there's
a bunch more places that didn't match and which would need some manual
changes, but I've intentially left these out for this mostly automated
patch.
iterator name drm_for_each_crtc;
struct drm_crtc *crtc;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
+ drm_for_each_crtc (crtc, dev) {
...
}
@@
iterator name drm_for_each_encoder;
struct drm_encoder *encoder;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(encoder, &dev->mode_config.encoder_list, head) {
+ drm_for_each_encoder (encoder, dev) {
...
}
@@
iterator name drm_for_each_fb;
struct drm_framebuffer *fb;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(fb, &dev->mode_config.fb_list, head) {
+ drm_for_each_fb (fb, dev) {
...
}
@@
iterator name drm_for_each_connector;
struct drm_connector *connector;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(connector, &dev->mode_config.connector_list, head) {
+ drm_for_each_connector (connector, dev) {
...
}
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2015-07-10 04:44:25 +07:00
|
|
|
#define drm_for_each_plane(plane, dev) \
|
|
|
|
list_for_each_entry(plane, &(dev)->mode_config.plane_list, head)
|
|
|
|
|
|
|
|
#define drm_for_each_crtc(crtc, dev) \
|
|
|
|
list_for_each_entry(crtc, &(dev)->mode_config.crtc_list, head)
|
|
|
|
|
2015-07-10 04:44:28 +07:00
|
|
|
static inline void
|
|
|
|
assert_drm_connector_list_read_locked(struct drm_mode_config *mode_config)
|
|
|
|
{
|
2015-07-10 04:44:33 +07:00
|
|
|
/*
|
|
|
|
* The connector hotadd/remove code currently grabs both locks when
|
|
|
|
* updating lists. Hence readers need only hold either of them to be
|
|
|
|
* safe and the check amounts to
|
|
|
|
*
|
|
|
|
* WARN_ON(not_holding(A) && not_holding(B)).
|
|
|
|
*/
|
|
|
|
WARN_ON(!mutex_is_locked(&mode_config->mutex) &&
|
|
|
|
!drm_modeset_is_locked(&mode_config->connection_mutex));
|
2015-07-10 04:44:28 +07:00
|
|
|
}
|
|
|
|
|
drm: Add modeset object iterators
And roll them out across drm_* files. The point here isn't code
prettification (it helps with that too) but that some of these lists
aren't static any more. And having macros will gives us a convenient
place to put locking checks into.
I didn't add an iterator for props since that's only used by a
list_for_each_entry_safe in the driver teardown code.
Search&replace was done with the below cocci spatch. Note that there's
a bunch more places that didn't match and which would need some manual
changes, but I've intentially left these out for this mostly automated
patch.
iterator name drm_for_each_crtc;
struct drm_crtc *crtc;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
+ drm_for_each_crtc (crtc, dev) {
...
}
@@
iterator name drm_for_each_encoder;
struct drm_encoder *encoder;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(encoder, &dev->mode_config.encoder_list, head) {
+ drm_for_each_encoder (encoder, dev) {
...
}
@@
iterator name drm_for_each_fb;
struct drm_framebuffer *fb;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(fb, &dev->mode_config.fb_list, head) {
+ drm_for_each_fb (fb, dev) {
...
}
@@
iterator name drm_for_each_connector;
struct drm_connector *connector;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(connector, &dev->mode_config.connector_list, head) {
+ drm_for_each_connector (connector, dev) {
...
}
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2015-07-10 04:44:25 +07:00
|
|
|
#define drm_for_each_connector(connector, dev) \
|
2015-07-10 04:44:28 +07:00
|
|
|
for (assert_drm_connector_list_read_locked(&(dev)->mode_config), \
|
|
|
|
connector = list_first_entry(&(dev)->mode_config.connector_list, \
|
|
|
|
struct drm_connector, head); \
|
|
|
|
&connector->head != (&(dev)->mode_config.connector_list); \
|
|
|
|
connector = list_next_entry(connector, head))
|
drm: Add modeset object iterators
And roll them out across drm_* files. The point here isn't code
prettification (it helps with that too) but that some of these lists
aren't static any more. And having macros will gives us a convenient
place to put locking checks into.
I didn't add an iterator for props since that's only used by a
list_for_each_entry_safe in the driver teardown code.
Search&replace was done with the below cocci spatch. Note that there's
a bunch more places that didn't match and which would need some manual
changes, but I've intentially left these out for this mostly automated
patch.
iterator name drm_for_each_crtc;
struct drm_crtc *crtc;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
+ drm_for_each_crtc (crtc, dev) {
...
}
@@
iterator name drm_for_each_encoder;
struct drm_encoder *encoder;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(encoder, &dev->mode_config.encoder_list, head) {
+ drm_for_each_encoder (encoder, dev) {
...
}
@@
iterator name drm_for_each_fb;
struct drm_framebuffer *fb;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(fb, &dev->mode_config.fb_list, head) {
+ drm_for_each_fb (fb, dev) {
...
}
@@
iterator name drm_for_each_connector;
struct drm_connector *connector;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(connector, &dev->mode_config.connector_list, head) {
+ drm_for_each_connector (connector, dev) {
...
}
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2015-07-10 04:44:25 +07:00
|
|
|
|
|
|
|
#define drm_for_each_encoder(encoder, dev) \
|
|
|
|
list_for_each_entry(encoder, &(dev)->mode_config.encoder_list, head)
|
|
|
|
|
|
|
|
#define drm_for_each_fb(fb, dev) \
|
2015-07-10 04:44:30 +07:00
|
|
|
for (WARN_ON(!mutex_is_locked(&(dev)->mode_config.fb_lock)), \
|
|
|
|
fb = list_first_entry(&(dev)->mode_config.fb_list, \
|
|
|
|
struct drm_framebuffer, head); \
|
|
|
|
&fb->head != (&(dev)->mode_config.fb_list); \
|
|
|
|
fb = list_next_entry(fb, head))
|
drm: Add modeset object iterators
And roll them out across drm_* files. The point here isn't code
prettification (it helps with that too) but that some of these lists
aren't static any more. And having macros will gives us a convenient
place to put locking checks into.
I didn't add an iterator for props since that's only used by a
list_for_each_entry_safe in the driver teardown code.
Search&replace was done with the below cocci spatch. Note that there's
a bunch more places that didn't match and which would need some manual
changes, but I've intentially left these out for this mostly automated
patch.
iterator name drm_for_each_crtc;
struct drm_crtc *crtc;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
+ drm_for_each_crtc (crtc, dev) {
...
}
@@
iterator name drm_for_each_encoder;
struct drm_encoder *encoder;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(encoder, &dev->mode_config.encoder_list, head) {
+ drm_for_each_encoder (encoder, dev) {
...
}
@@
iterator name drm_for_each_fb;
struct drm_framebuffer *fb;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(fb, &dev->mode_config.fb_list, head) {
+ drm_for_each_fb (fb, dev) {
...
}
@@
iterator name drm_for_each_connector;
struct drm_connector *connector;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(connector, &dev->mode_config.connector_list, head) {
+ drm_for_each_connector (connector, dev) {
...
}
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2015-07-10 04:44:25 +07:00
|
|
|
|
2008-11-08 05:05:41 +07:00
|
|
|
#endif /* __DRM_CRTC_H__ */
|