diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2017-07-25 04:01:18 -0400 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2017-08-08 08:45:16 -0400 |
commit | b6715570c10de945a8f24693a1a76be6896587f9 (patch) | |
tree | 404d9bf3f64f9591565eafd0db3a57d18dcbb825 /drivers | |
parent | 144a7999d6334be5237d5926ab19c56bc24d0204 (diff) |
drm: Nuke drm_atomic_helper_crtc_set_property
It's dead code because this is now handled in the core.
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Cc: Boris Brezillon <boris.brezillon@free-electrons.com>
Cc: Daniel Vetter <daniel.vetter@intel.com>
Cc: Jani Nikula <jani.nikula@linux.intel.com>
Cc: Sean Paul <seanpaul@chromium.org>
Cc: David Airlie <airlied@linux.ie>
Cc: Ben Skeggs <bskeggs@redhat.com>
Cc: Tomi Valkeinen <tomi.valkeinen@ti.com>
Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Cc: Alexey Brodkin <abrodkin@synopsys.com>
Cc: Shawn Guo <shawn.guo@linaro.org>
Cc: Eric Engestrom <eric@engestrom.ch>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
Cc: Rob Clark <robdclark@gmail.com>
Cc: Philippe Cornu <philippe.cornu@st.com>
Cc: Masahiro Yamada <yamada.masahiro@socionext.com>
Cc: Sushmita Susheelendra <ssusheel@codeaurora.org>
Cc: Archit Taneja <architt@codeaurora.org>
Cc: intel-gfx@lists.freedesktop.org
Cc: nouveau@lists.freedesktop.org
Cc: Philipp Zabel <p.zabel@pengutronix.de>
Cc: Maxime Ripard <maxime.ripard@free-electrons.com>
Cc: Thomas Hellstrom <thellstrom@vmware.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20170725080122.20548-5-daniel.vetter@ffwll.ch
Reviewed-by: Archit Taneja <architt@codeaurora.org>
Acked-by: Philippe Cornu <philippe.cornu@st.com>
Tested-by: Philippe Cornu <philippe.cornu@st.com>
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Acked-by: Thomas Hellstrom <thellstrom@vmware.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c | 1 | ||||
-rw-r--r-- | drivers/gpu/drm/drm_atomic_helper.c | 55 | ||||
-rw-r--r-- | drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c | 1 | ||||
-rw-r--r-- | drivers/gpu/drm/i915/intel_display.c | 1 | ||||
-rw-r--r-- | drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c | 1 | ||||
-rw-r--r-- | drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c | 2 | ||||
-rw-r--r-- | drivers/gpu/drm/nouveau/nv50_display.c | 1 | ||||
-rw-r--r-- | drivers/gpu/drm/omapdrm/omap_crtc.c | 1 |
8 files changed, 0 insertions, 63 deletions
diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c index 4fbbeab5c5d4..d73281095fac 100644 --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c | |||
@@ -431,7 +431,6 @@ static const struct drm_crtc_funcs atmel_hlcdc_crtc_funcs = { | |||
431 | .atomic_destroy_state = atmel_hlcdc_crtc_destroy_state, | 431 | .atomic_destroy_state = atmel_hlcdc_crtc_destroy_state, |
432 | .enable_vblank = atmel_hlcdc_crtc_enable_vblank, | 432 | .enable_vblank = atmel_hlcdc_crtc_enable_vblank, |
433 | .disable_vblank = atmel_hlcdc_crtc_disable_vblank, | 433 | .disable_vblank = atmel_hlcdc_crtc_disable_vblank, |
434 | .set_property = drm_atomic_helper_crtc_set_property, | ||
435 | .gamma_set = drm_atomic_helper_legacy_gamma_set, | 434 | .gamma_set = drm_atomic_helper_legacy_gamma_set, |
436 | }; | 435 | }; |
437 | 436 | ||
diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c index 41a8ff3d8b7e..38a7edb4779b 100644 --- a/drivers/gpu/drm/drm_atomic_helper.c +++ b/drivers/gpu/drm/drm_atomic_helper.c | |||
@@ -2954,61 +2954,6 @@ out: | |||
2954 | EXPORT_SYMBOL(drm_atomic_helper_resume); | 2954 | EXPORT_SYMBOL(drm_atomic_helper_resume); |
2955 | 2955 | ||
2956 | /** | 2956 | /** |
2957 | * drm_atomic_helper_crtc_set_property - helper for crtc properties | ||
2958 | * @crtc: DRM crtc | ||
2959 | * @property: DRM property | ||
2960 | * @val: value of property | ||
2961 | * | ||
2962 | * Provides a default crtc set_property handler using the atomic driver | ||
2963 | * interface. | ||
2964 | * | ||
2965 | * RETURNS: | ||
2966 | * Zero on success, error code on failure | ||
2967 | */ | ||
2968 | int | ||
2969 | drm_atomic_helper_crtc_set_property(struct drm_crtc *crtc, | ||
2970 | struct drm_property *property, | ||
2971 | uint64_t val) | ||
2972 | { | ||
2973 | struct drm_atomic_state *state; | ||
2974 | struct drm_crtc_state *crtc_state; | ||
2975 | int ret = 0; | ||
2976 | |||
2977 | state = drm_atomic_state_alloc(crtc->dev); | ||
2978 | if (!state) | ||
2979 | return -ENOMEM; | ||
2980 | |||
2981 | /* ->set_property is always called with all locks held. */ | ||
2982 | state->acquire_ctx = crtc->dev->mode_config.acquire_ctx; | ||
2983 | retry: | ||
2984 | crtc_state = drm_atomic_get_crtc_state(state, crtc); | ||
2985 | if (IS_ERR(crtc_state)) { | ||
2986 | ret = PTR_ERR(crtc_state); | ||
2987 | goto fail; | ||
2988 | } | ||
2989 | |||
2990 | ret = drm_atomic_crtc_set_property(crtc, crtc_state, | ||
2991 | property, val); | ||
2992 | if (ret) | ||
2993 | goto fail; | ||
2994 | |||
2995 | ret = drm_atomic_commit(state); | ||
2996 | fail: | ||
2997 | if (ret == -EDEADLK) | ||
2998 | goto backoff; | ||
2999 | |||
3000 | drm_atomic_state_put(state); | ||
3001 | return ret; | ||
3002 | |||
3003 | backoff: | ||
3004 | drm_atomic_state_clear(state); | ||
3005 | drm_atomic_legacy_backoff(state); | ||
3006 | |||
3007 | goto retry; | ||
3008 | } | ||
3009 | EXPORT_SYMBOL(drm_atomic_helper_crtc_set_property); | ||
3010 | |||
3011 | /** | ||
3012 | * drm_atomic_helper_plane_set_property - helper for plane properties | 2957 | * drm_atomic_helper_plane_set_property - helper for plane properties |
3013 | * @plane: DRM plane | 2958 | * @plane: DRM plane |
3014 | * @property: DRM property | 2959 | * @property: DRM property |
diff --git a/drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c b/drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c index 9882ebd8751c..6e5fd4f29ccb 100644 --- a/drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c +++ b/drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c | |||
@@ -567,7 +567,6 @@ static const struct drm_crtc_funcs ade_crtc_funcs = { | |||
567 | .set_config = drm_atomic_helper_set_config, | 567 | .set_config = drm_atomic_helper_set_config, |
568 | .page_flip = drm_atomic_helper_page_flip, | 568 | .page_flip = drm_atomic_helper_page_flip, |
569 | .reset = drm_atomic_helper_crtc_reset, | 569 | .reset = drm_atomic_helper_crtc_reset, |
570 | .set_property = drm_atomic_helper_crtc_set_property, | ||
571 | .atomic_duplicate_state = drm_atomic_helper_crtc_duplicate_state, | 570 | .atomic_duplicate_state = drm_atomic_helper_crtc_duplicate_state, |
572 | .atomic_destroy_state = drm_atomic_helper_crtc_destroy_state, | 571 | .atomic_destroy_state = drm_atomic_helper_crtc_destroy_state, |
573 | .enable_vblank = ade_crtc_enable_vblank, | 572 | .enable_vblank = ade_crtc_enable_vblank, |
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index 8998d6a27288..f6fbbceb0bac 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c | |||
@@ -13323,7 +13323,6 @@ static int intel_atomic_commit(struct drm_device *dev, | |||
13323 | static const struct drm_crtc_funcs intel_crtc_funcs = { | 13323 | static const struct drm_crtc_funcs intel_crtc_funcs = { |
13324 | .gamma_set = drm_atomic_helper_legacy_gamma_set, | 13324 | .gamma_set = drm_atomic_helper_legacy_gamma_set, |
13325 | .set_config = drm_atomic_helper_set_config, | 13325 | .set_config = drm_atomic_helper_set_config, |
13326 | .set_property = drm_atomic_helper_crtc_set_property, | ||
13327 | .destroy = intel_crtc_destroy, | 13326 | .destroy = intel_crtc_destroy, |
13328 | .page_flip = drm_atomic_helper_page_flip, | 13327 | .page_flip = drm_atomic_helper_page_flip, |
13329 | .atomic_duplicate_state = intel_crtc_duplicate_state, | 13328 | .atomic_duplicate_state = intel_crtc_duplicate_state, |
diff --git a/drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c b/drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c index 3c7a9d343e05..47fa2aba1983 100644 --- a/drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c +++ b/drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c | |||
@@ -484,7 +484,6 @@ static const struct drm_crtc_funcs mdp4_crtc_funcs = { | |||
484 | .set_config = drm_atomic_helper_set_config, | 484 | .set_config = drm_atomic_helper_set_config, |
485 | .destroy = mdp4_crtc_destroy, | 485 | .destroy = mdp4_crtc_destroy, |
486 | .page_flip = drm_atomic_helper_page_flip, | 486 | .page_flip = drm_atomic_helper_page_flip, |
487 | .set_property = drm_atomic_helper_crtc_set_property, | ||
488 | .cursor_set = mdp4_crtc_cursor_set, | 487 | .cursor_set = mdp4_crtc_cursor_set, |
489 | .cursor_move = mdp4_crtc_cursor_move, | 488 | .cursor_move = mdp4_crtc_cursor_move, |
490 | .reset = drm_atomic_helper_crtc_reset, | 489 | .reset = drm_atomic_helper_crtc_reset, |
diff --git a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c index 4322a502555a..5e3bc7224eee 100644 --- a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c +++ b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c | |||
@@ -919,7 +919,6 @@ static const struct drm_crtc_funcs mdp5_crtc_funcs = { | |||
919 | .set_config = drm_atomic_helper_set_config, | 919 | .set_config = drm_atomic_helper_set_config, |
920 | .destroy = mdp5_crtc_destroy, | 920 | .destroy = mdp5_crtc_destroy, |
921 | .page_flip = drm_atomic_helper_page_flip, | 921 | .page_flip = drm_atomic_helper_page_flip, |
922 | .set_property = drm_atomic_helper_crtc_set_property, | ||
923 | .reset = mdp5_crtc_reset, | 922 | .reset = mdp5_crtc_reset, |
924 | .atomic_duplicate_state = mdp5_crtc_duplicate_state, | 923 | .atomic_duplicate_state = mdp5_crtc_duplicate_state, |
925 | .atomic_destroy_state = mdp5_crtc_destroy_state, | 924 | .atomic_destroy_state = mdp5_crtc_destroy_state, |
@@ -932,7 +931,6 @@ static const struct drm_crtc_funcs mdp5_crtc_no_lm_cursor_funcs = { | |||
932 | .set_config = drm_atomic_helper_set_config, | 931 | .set_config = drm_atomic_helper_set_config, |
933 | .destroy = mdp5_crtc_destroy, | 932 | .destroy = mdp5_crtc_destroy, |
934 | .page_flip = drm_atomic_helper_page_flip, | 933 | .page_flip = drm_atomic_helper_page_flip, |
935 | .set_property = drm_atomic_helper_crtc_set_property, | ||
936 | .reset = mdp5_crtc_reset, | 934 | .reset = mdp5_crtc_reset, |
937 | .atomic_duplicate_state = mdp5_crtc_duplicate_state, | 935 | .atomic_duplicate_state = mdp5_crtc_duplicate_state, |
938 | .atomic_destroy_state = mdp5_crtc_destroy_state, | 936 | .atomic_destroy_state = mdp5_crtc_destroy_state, |
diff --git a/drivers/gpu/drm/nouveau/nv50_display.c b/drivers/gpu/drm/nouveau/nv50_display.c index eda1c9dcef8c..762fb5b9b83b 100644 --- a/drivers/gpu/drm/nouveau/nv50_display.c +++ b/drivers/gpu/drm/nouveau/nv50_display.c | |||
@@ -2318,7 +2318,6 @@ nv50_head_func = { | |||
2318 | .destroy = nv50_head_destroy, | 2318 | .destroy = nv50_head_destroy, |
2319 | .set_config = drm_atomic_helper_set_config, | 2319 | .set_config = drm_atomic_helper_set_config, |
2320 | .page_flip = drm_atomic_helper_page_flip, | 2320 | .page_flip = drm_atomic_helper_page_flip, |
2321 | .set_property = drm_atomic_helper_crtc_set_property, | ||
2322 | .atomic_duplicate_state = nv50_head_atomic_duplicate_state, | 2321 | .atomic_duplicate_state = nv50_head_atomic_duplicate_state, |
2323 | .atomic_destroy_state = nv50_head_atomic_destroy_state, | 2322 | .atomic_destroy_state = nv50_head_atomic_destroy_state, |
2324 | }; | 2323 | }; |
diff --git a/drivers/gpu/drm/omapdrm/omap_crtc.c b/drivers/gpu/drm/omapdrm/omap_crtc.c index 09e05e002703..400d0d2f6790 100644 --- a/drivers/gpu/drm/omapdrm/omap_crtc.c +++ b/drivers/gpu/drm/omapdrm/omap_crtc.c | |||
@@ -604,7 +604,6 @@ static const struct drm_crtc_funcs omap_crtc_funcs = { | |||
604 | .destroy = omap_crtc_destroy, | 604 | .destroy = omap_crtc_destroy, |
605 | .page_flip = drm_atomic_helper_page_flip, | 605 | .page_flip = drm_atomic_helper_page_flip, |
606 | .gamma_set = drm_atomic_helper_legacy_gamma_set, | 606 | .gamma_set = drm_atomic_helper_legacy_gamma_set, |
607 | .set_property = drm_atomic_helper_crtc_set_property, | ||
608 | .atomic_duplicate_state = omap_crtc_duplicate_state, | 607 | .atomic_duplicate_state = omap_crtc_duplicate_state, |
609 | .atomic_destroy_state = drm_atomic_helper_crtc_destroy_state, | 608 | .atomic_destroy_state = drm_atomic_helper_crtc_destroy_state, |
610 | .atomic_set_property = omap_crtc_atomic_set_property, | 609 | .atomic_set_property = omap_crtc_atomic_set_property, |