mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 10:40:15 +02:00 
			
		
		
		
	drm: Replace drm_modeset_lock/unlock_all with DRM_MODESET_LOCK_ALL_* helpers
As suggested by the TODO list for the kernel DRM subsystem, replace the deprecated functions that take/drop modeset locks with new helpers. Signed-off-by: Michal Orzel <michalorzel.eng@gmail.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch> Link: https://patchwork.freedesktop.org/patch/msgid/1588093804-30446-1-git-send-email-michalorzel.eng@gmail.com
This commit is contained in:
		
							parent
							
								
									8976eeee8d
								
							
						
					
					
						commit
						9bcaa3fe58
					
				
					 1 changed files with 6 additions and 4 deletions
				
			
		| 
						 | 
					@ -402,12 +402,13 @@ int drm_mode_obj_get_properties_ioctl(struct drm_device *dev, void *data,
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
	struct drm_mode_obj_get_properties *arg = data;
 | 
						struct drm_mode_obj_get_properties *arg = data;
 | 
				
			||||||
	struct drm_mode_object *obj;
 | 
						struct drm_mode_object *obj;
 | 
				
			||||||
 | 
						struct drm_modeset_acquire_ctx ctx;
 | 
				
			||||||
	int ret = 0;
 | 
						int ret = 0;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	if (!drm_core_check_feature(dev, DRIVER_MODESET))
 | 
						if (!drm_core_check_feature(dev, DRIVER_MODESET))
 | 
				
			||||||
		return -EOPNOTSUPP;
 | 
							return -EOPNOTSUPP;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	drm_modeset_lock_all(dev);
 | 
						DRM_MODESET_LOCK_ALL_BEGIN(dev, ctx, 0, ret);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	obj = drm_mode_object_find(dev, file_priv, arg->obj_id, arg->obj_type);
 | 
						obj = drm_mode_object_find(dev, file_priv, arg->obj_id, arg->obj_type);
 | 
				
			||||||
	if (!obj) {
 | 
						if (!obj) {
 | 
				
			||||||
| 
						 | 
					@ -427,7 +428,7 @@ int drm_mode_obj_get_properties_ioctl(struct drm_device *dev, void *data,
 | 
				
			||||||
out_unref:
 | 
					out_unref:
 | 
				
			||||||
	drm_mode_object_put(obj);
 | 
						drm_mode_object_put(obj);
 | 
				
			||||||
out:
 | 
					out:
 | 
				
			||||||
	drm_modeset_unlock_all(dev);
 | 
						DRM_MODESET_LOCK_ALL_END(ctx, ret);
 | 
				
			||||||
	return ret;
 | 
						return ret;
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					@ -449,12 +450,13 @@ static int set_property_legacy(struct drm_mode_object *obj,
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
	struct drm_device *dev = prop->dev;
 | 
						struct drm_device *dev = prop->dev;
 | 
				
			||||||
	struct drm_mode_object *ref;
 | 
						struct drm_mode_object *ref;
 | 
				
			||||||
 | 
						struct drm_modeset_acquire_ctx ctx;
 | 
				
			||||||
	int ret = -EINVAL;
 | 
						int ret = -EINVAL;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	if (!drm_property_change_valid_get(prop, prop_value, &ref))
 | 
						if (!drm_property_change_valid_get(prop, prop_value, &ref))
 | 
				
			||||||
		return -EINVAL;
 | 
							return -EINVAL;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	drm_modeset_lock_all(dev);
 | 
						DRM_MODESET_LOCK_ALL_BEGIN(dev, ctx, 0, ret);
 | 
				
			||||||
	switch (obj->type) {
 | 
						switch (obj->type) {
 | 
				
			||||||
	case DRM_MODE_OBJECT_CONNECTOR:
 | 
						case DRM_MODE_OBJECT_CONNECTOR:
 | 
				
			||||||
		ret = drm_connector_set_obj_prop(obj, prop, prop_value);
 | 
							ret = drm_connector_set_obj_prop(obj, prop, prop_value);
 | 
				
			||||||
| 
						 | 
					@ -468,7 +470,7 @@ static int set_property_legacy(struct drm_mode_object *obj,
 | 
				
			||||||
		break;
 | 
							break;
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
	drm_property_change_valid_put(prop, ref);
 | 
						drm_property_change_valid_put(prop, ref);
 | 
				
			||||||
	drm_modeset_unlock_all(dev);
 | 
						DRM_MODESET_LOCK_ALL_END(ctx, ret);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	return ret;
 | 
						return ret;
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in a new issue