Commit 322c1c03 authored by Dave Airlie's avatar Dave Airlie

Merge tag 'topic/core-stuff-2014-07-18' of git://anongit.freedesktop.org/drm-intel into drm-next

misc core patches.

* tag 'topic/core-stuff-2014-07-18' of git://anongit.freedesktop.org/drm-intel:
  drm: Check for connection_mutex in drm_select_eld
  drm/dp-mst-helper: Don't use uninitialized fields of the sideband message header
  drm/dp-mst-helper: Avoid reading uninitialized value
  drm/plane-helper: Use proper plane init function
  drivers/gpu/drm/drm_buffer.c: remove unnecessary null test before kfree
  drm: Fix function names in kerneldoc
parents c51f7167 008f4045
...@@ -80,11 +80,7 @@ int drm_buffer_alloc(struct drm_buffer **buf, int size) ...@@ -80,11 +80,7 @@ int drm_buffer_alloc(struct drm_buffer **buf, int size)
error_out: error_out:
/* Only last element can be null pointer so check for it first. */ for (; idx >= 0; --idx)
if ((*buf)->data[idx])
kfree((*buf)->data[idx]);
for (--idx; idx >= 0; --idx)
kfree((*buf)->data[idx]); kfree((*buf)->data[idx]);
kfree(*buf); kfree(*buf);
......
...@@ -3388,7 +3388,7 @@ struct drm_property *drm_property_create(struct drm_device *dev, int flags, ...@@ -3388,7 +3388,7 @@ struct drm_property *drm_property_create(struct drm_device *dev, int flags,
EXPORT_SYMBOL(drm_property_create); EXPORT_SYMBOL(drm_property_create);
/** /**
* drm_property_create - create a new enumeration property type * drm_property_create_enum - create a new enumeration property type
* @dev: drm device * @dev: drm device
* @flags: flags specifying the property type * @flags: flags specifying the property type
* @name: name of the property * @name: name of the property
...@@ -3434,7 +3434,7 @@ struct drm_property *drm_property_create_enum(struct drm_device *dev, int flags, ...@@ -3434,7 +3434,7 @@ struct drm_property *drm_property_create_enum(struct drm_device *dev, int flags,
EXPORT_SYMBOL(drm_property_create_enum); EXPORT_SYMBOL(drm_property_create_enum);
/** /**
* drm_property_create - create a new bitmask property type * drm_property_create_bitmask - create a new bitmask property type
* @dev: drm device * @dev: drm device
* @flags: flags specifying the property type * @flags: flags specifying the property type
* @name: name of the property * @name: name of the property
...@@ -3496,7 +3496,7 @@ static struct drm_property *property_create_range(struct drm_device *dev, ...@@ -3496,7 +3496,7 @@ static struct drm_property *property_create_range(struct drm_device *dev,
} }
/** /**
* drm_property_create - create a new ranged property type * drm_property_create_range - create a new ranged property type
* @dev: drm device * @dev: drm device
* @flags: flags specifying the property type * @flags: flags specifying the property type
* @name: name of the property * @name: name of the property
......
...@@ -1290,6 +1290,8 @@ static int process_single_tx_qlock(struct drm_dp_mst_topology_mgr *mgr, ...@@ -1290,6 +1290,8 @@ static int process_single_tx_qlock(struct drm_dp_mst_topology_mgr *mgr,
int len, space, idx, tosend; int len, space, idx, tosend;
int ret; int ret;
memset(&hdr, 0, sizeof(struct drm_dp_sideband_msg_hdr));
if (txmsg->state == DRM_DP_SIDEBAND_TX_QUEUED) { if (txmsg->state == DRM_DP_SIDEBAND_TX_QUEUED) {
txmsg->seqno = -1; txmsg->seqno = -1;
txmsg->state = DRM_DP_SIDEBAND_TX_START_SEND; txmsg->state = DRM_DP_SIDEBAND_TX_START_SEND;
...@@ -1644,7 +1646,7 @@ int drm_dp_update_payload_part2(struct drm_dp_mst_topology_mgr *mgr) ...@@ -1644,7 +1646,7 @@ int drm_dp_update_payload_part2(struct drm_dp_mst_topology_mgr *mgr)
{ {
struct drm_dp_mst_port *port; struct drm_dp_mst_port *port;
int i; int i;
int ret; int ret = 0;
mutex_lock(&mgr->payload_lock); mutex_lock(&mgr->payload_lock);
for (i = 0; i < mgr->max_payloads; i++) { for (i = 0; i < mgr->max_payloads; i++) {
......
...@@ -3305,6 +3305,7 @@ struct drm_connector *drm_select_eld(struct drm_encoder *encoder, ...@@ -3305,6 +3305,7 @@ struct drm_connector *drm_select_eld(struct drm_encoder *encoder,
struct drm_device *dev = encoder->dev; struct drm_device *dev = encoder->dev;
WARN_ON(!mutex_is_locked(&dev->mode_config.mutex)); WARN_ON(!mutex_is_locked(&dev->mode_config.mutex));
WARN_ON(!drm_modeset_is_locked(&dev->mode_config.connection_mutex));
list_for_each_entry(connector, &dev->mode_config.connector_list, head) list_for_each_entry(connector, &dev->mode_config.connector_list, head)
if (connector->encoder == encoder && connector->eld[0]) if (connector->encoder == encoder && connector->eld[0])
......
...@@ -335,9 +335,10 @@ struct drm_plane *drm_primary_helper_create_plane(struct drm_device *dev, ...@@ -335,9 +335,10 @@ struct drm_plane *drm_primary_helper_create_plane(struct drm_device *dev,
} }
/* possible_crtc's will be filled in later by crtc_init */ /* possible_crtc's will be filled in later by crtc_init */
ret = drm_plane_init(dev, primary, 0, &drm_primary_helper_funcs, ret = drm_universal_plane_init(dev, primary, 0,
formats, num_formats, &drm_primary_helper_funcs,
DRM_PLANE_TYPE_PRIMARY); formats, num_formats,
DRM_PLANE_TYPE_PRIMARY);
if (ret) { if (ret) {
kfree(primary); kfree(primary);
primary = NULL; primary = NULL;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment