Skip to content

Commit

Permalink
Revert "drm: cleanup: drm_modeset_lock_all_ctx() --> DRM_MODESET_LOCK…
Browse files Browse the repository at this point in the history
…_ALL_BEGIN()"

This reverts commit 21dde40.

This patchset breaks on intel platforms and was previously NACK'd by
Ville.

Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Cc: Fernando Ramos <greenfoo@u92.eu>
Acked-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Sean Paul <seanpaul@chromium.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20211002154542.15800-16-sean@poorly.run
  • Loading branch information
Sean Paul committed Oct 4, 2021
1 parent 369de54 commit 61bae13
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions drivers/gpu/drm/drm_client_modeset.c
Original file line number Diff line number Diff line change
Expand Up @@ -574,7 +574,6 @@ static bool drm_client_firmware_config(struct drm_client_dev *client,
int num_connectors_detected = 0;
int num_tiled_conns = 0;
struct drm_modeset_acquire_ctx ctx;
int err;

if (!drm_drv_uses_atomic_modeset(dev))
return false;
Expand All @@ -586,7 +585,10 @@ static bool drm_client_firmware_config(struct drm_client_dev *client,
if (!save_enabled)
return false;

DRM_MODESET_LOCK_ALL_BEGIN(dev, ctx, 0, err);
drm_modeset_acquire_init(&ctx, 0);

while (drm_modeset_lock_all_ctx(dev, &ctx) != 0)
drm_modeset_backoff(&ctx);

memcpy(save_enabled, enabled, count);
mask = GENMASK(count - 1, 0);
Expand Down Expand Up @@ -741,7 +743,8 @@ static bool drm_client_firmware_config(struct drm_client_dev *client,
ret = false;
}

DRM_MODESET_LOCK_ALL_END(dev, ctx, err);
drm_modeset_drop_locks(&ctx);
drm_modeset_acquire_fini(&ctx);

kfree(save_enabled);
return ret;
Expand Down

0 comments on commit 61bae13

Please sign in to comment.