mirror of https://gitee.com/openkylin/linux.git
drm: Merge __drm_atomic_helper_disable_all() into drm_atomic_helper_disable_all()
The only difference between the functions is the clean_old_fbs argument, whose
use was removed in the patch referenced below. So remove the internal
copy and drop the guts back into drm_atomic_helper_disable_all()
Changes in v2:
- Instead of just removing the unused arg, merge the functions
Fixes: e00fb8564e
("drm: Stop updating plane->crtc/fb/old_fb on atomic drivers")
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: Harry Wentland <harry.wentland@amd.com>
Cc: Sinclair Yeh <syeh@vmware.com> # This email bounces :(
Cc: Maxime Ripard <maxime.ripard@bootlin.com>
Cc: Sean Paul <sean@poorly.run>
Cc: David Airlie <airlied@linux.ie>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: dri-devel@lists.freedesktop.org
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Sean Paul <seanpaul@chromium.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20190212173245.51980-1-sean@poorly.run
This commit is contained in:
parent
6649a95d35
commit
37406a60fa
|
@ -3030,9 +3030,31 @@ int __drm_atomic_helper_set_config(struct drm_mode_set *set,
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int __drm_atomic_helper_disable_all(struct drm_device *dev,
|
/**
|
||||||
struct drm_modeset_acquire_ctx *ctx,
|
* drm_atomic_helper_disable_all - disable all currently active outputs
|
||||||
bool clean_old_fbs)
|
* @dev: DRM device
|
||||||
|
* @ctx: lock acquisition context
|
||||||
|
*
|
||||||
|
* Loops through all connectors, finding those that aren't turned off and then
|
||||||
|
* turns them off by setting their DPMS mode to OFF and deactivating the CRTC
|
||||||
|
* that they are connected to.
|
||||||
|
*
|
||||||
|
* This is used for example in suspend/resume to disable all currently active
|
||||||
|
* functions when suspending. If you just want to shut down everything at e.g.
|
||||||
|
* driver unload, look at drm_atomic_helper_shutdown().
|
||||||
|
*
|
||||||
|
* Note that if callers haven't already acquired all modeset locks this might
|
||||||
|
* return -EDEADLK, which must be handled by calling drm_modeset_backoff().
|
||||||
|
*
|
||||||
|
* Returns:
|
||||||
|
* 0 on success or a negative error code on failure.
|
||||||
|
*
|
||||||
|
* See also:
|
||||||
|
* drm_atomic_helper_suspend(), drm_atomic_helper_resume() and
|
||||||
|
* drm_atomic_helper_shutdown().
|
||||||
|
*/
|
||||||
|
int drm_atomic_helper_disable_all(struct drm_device *dev,
|
||||||
|
struct drm_modeset_acquire_ctx *ctx)
|
||||||
{
|
{
|
||||||
struct drm_atomic_state *state;
|
struct drm_atomic_state *state;
|
||||||
struct drm_connector_state *conn_state;
|
struct drm_connector_state *conn_state;
|
||||||
|
@ -3090,35 +3112,6 @@ static int __drm_atomic_helper_disable_all(struct drm_device *dev,
|
||||||
drm_atomic_state_put(state);
|
drm_atomic_state_put(state);
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* drm_atomic_helper_disable_all - disable all currently active outputs
|
|
||||||
* @dev: DRM device
|
|
||||||
* @ctx: lock acquisition context
|
|
||||||
*
|
|
||||||
* Loops through all connectors, finding those that aren't turned off and then
|
|
||||||
* turns them off by setting their DPMS mode to OFF and deactivating the CRTC
|
|
||||||
* that they are connected to.
|
|
||||||
*
|
|
||||||
* This is used for example in suspend/resume to disable all currently active
|
|
||||||
* functions when suspending. If you just want to shut down everything at e.g.
|
|
||||||
* driver unload, look at drm_atomic_helper_shutdown().
|
|
||||||
*
|
|
||||||
* Note that if callers haven't already acquired all modeset locks this might
|
|
||||||
* return -EDEADLK, which must be handled by calling drm_modeset_backoff().
|
|
||||||
*
|
|
||||||
* Returns:
|
|
||||||
* 0 on success or a negative error code on failure.
|
|
||||||
*
|
|
||||||
* See also:
|
|
||||||
* drm_atomic_helper_suspend(), drm_atomic_helper_resume() and
|
|
||||||
* drm_atomic_helper_shutdown().
|
|
||||||
*/
|
|
||||||
int drm_atomic_helper_disable_all(struct drm_device *dev,
|
|
||||||
struct drm_modeset_acquire_ctx *ctx)
|
|
||||||
{
|
|
||||||
return __drm_atomic_helper_disable_all(dev, ctx, false);
|
|
||||||
}
|
|
||||||
EXPORT_SYMBOL(drm_atomic_helper_disable_all);
|
EXPORT_SYMBOL(drm_atomic_helper_disable_all);
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
@ -3139,7 +3132,7 @@ void drm_atomic_helper_shutdown(struct drm_device *dev)
|
||||||
|
|
||||||
DRM_MODESET_LOCK_ALL_BEGIN(dev, ctx, 0, ret);
|
DRM_MODESET_LOCK_ALL_BEGIN(dev, ctx, 0, ret);
|
||||||
|
|
||||||
ret = __drm_atomic_helper_disable_all(dev, &ctx, true);
|
ret = drm_atomic_helper_disable_all(dev, &ctx);
|
||||||
if (ret)
|
if (ret)
|
||||||
DRM_ERROR("Disabling all crtc's during unload failed with %i\n", ret);
|
DRM_ERROR("Disabling all crtc's during unload failed with %i\n", ret);
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue