diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-12-04 03:45:55 -0500 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-12-08 10:13:49 -0500 |
commit | 813b0f3e00d402279718403aeb621c965f97180e (patch) | |
tree | 6838d93af043348740f8ce71edb05ad407b593c6 | |
parent | 2c3d77155fb0e0ca80d33b29bfeee894ecf1f247 (diff) |
drm: Remove crtc/connector->save/restore hooks
They're not how system suspend/resume should be done with atomic
(there's new helpers for that developed by Thierry Reding), and for
legacy drivers this really should be a helper hook and not a core one.
But there's not even helper code to use them, and only 2 drivers
(which now have their own private hooks) set them. Ditch them.
Saves me typing some kerneldoc, too ;-)
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1449218769-16577-15-git-send-email-daniel.vetter@ffwll.ch
Reviewed-by: Thierry Reding <treding@nvidia.com>
-rw-r--r-- | include/drm/drm_crtc.h | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h index 1bcfa094af16..e75b06b61143 100644 --- a/include/drm/drm_crtc.h +++ b/include/drm/drm_crtc.h | |||
@@ -320,8 +320,6 @@ struct drm_crtc_state { | |||
320 | 320 | ||
321 | /** | 321 | /** |
322 | * struct drm_crtc_funcs - control CRTCs for a given device | 322 | * struct drm_crtc_funcs - control CRTCs for a given device |
323 | * @save: save CRTC state | ||
324 | * @restore: restore CRTC state | ||
325 | * @cursor_set: setup the cursor | 323 | * @cursor_set: setup the cursor |
326 | * @cursor_set2: setup the cursor with hotspot, superseeds @cursor_set if set | 324 | * @cursor_set2: setup the cursor with hotspot, superseeds @cursor_set if set |
327 | * @cursor_move: move the cursor | 325 | * @cursor_move: move the cursor |
@@ -339,11 +337,6 @@ struct drm_crtc_state { | |||
339 | * bus accessors. | 337 | * bus accessors. |
340 | */ | 338 | */ |
341 | struct drm_crtc_funcs { | 339 | struct drm_crtc_funcs { |
342 | /* Save CRTC state */ | ||
343 | void (*save)(struct drm_crtc *crtc); /* suspend? */ | ||
344 | /* Restore CRTC state */ | ||
345 | void (*restore)(struct drm_crtc *crtc); /* resume? */ | ||
346 | |||
347 | /** | 340 | /** |
348 | * @reset: | 341 | * @reset: |
349 | * | 342 | * |
@@ -625,8 +618,6 @@ struct drm_connector_state { | |||
625 | /** | 618 | /** |
626 | * struct drm_connector_funcs - control connectors on a given device | 619 | * struct drm_connector_funcs - control connectors on a given device |
627 | * @dpms: set power state | 620 | * @dpms: set power state |
628 | * @save: save connector state | ||
629 | * @restore: restore connector state | ||
630 | * @detect: is this connector active? | 621 | * @detect: is this connector active? |
631 | * @fill_modes: fill mode list for this connector | 622 | * @fill_modes: fill mode list for this connector |
632 | * @force: notify the driver that the connector is forced on | 623 | * @force: notify the driver that the connector is forced on |
@@ -637,8 +628,6 @@ struct drm_connector_state { | |||
637 | */ | 628 | */ |
638 | struct drm_connector_funcs { | 629 | struct drm_connector_funcs { |
639 | int (*dpms)(struct drm_connector *connector, int mode); | 630 | int (*dpms)(struct drm_connector *connector, int mode); |
640 | void (*save)(struct drm_connector *connector); | ||
641 | void (*restore)(struct drm_connector *connector); | ||
642 | 631 | ||
643 | /** | 632 | /** |
644 | * @reset: | 633 | * @reset: |