diff options
author | Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> | 2015-01-02 15:55:02 -0500 |
---|---|---|
committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2015-01-26 11:23:32 -0500 |
commit | 5a413dfbaf3f446a154d65d1d9811a6d311482a5 (patch) | |
tree | 13203c79b51a4bbe47f65aecb920a5aa86e61671 | |
parent | f114040e3ea6e07372334ade75d1ee0775c355e1 (diff) |
drm/armada: armada_output: Remove some unused functions
Removes some functions that are not used anywhere:
armada_drm_encoder_mode_fixup() armada_drm_encoder_commit() armada_drm_encoder_prepare()
This was partially found by using a static code analysis program called cppcheck.
Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
-rw-r--r-- | drivers/gpu/drm/armada/armada_output.c | 16 | ||||
-rw-r--r-- | drivers/gpu/drm/armada/armada_output.h | 6 |
2 files changed, 0 insertions, 22 deletions
diff --git a/drivers/gpu/drm/armada/armada_output.c b/drivers/gpu/drm/armada/armada_output.c index abbc309fe539..5a9823178291 100644 --- a/drivers/gpu/drm/armada/armada_output.c +++ b/drivers/gpu/drm/armada/armada_output.c | |||
@@ -72,22 +72,6 @@ static const struct drm_connector_funcs armada_drm_conn_funcs = { | |||
72 | .set_property = armada_drm_connector_set_property, | 72 | .set_property = armada_drm_connector_set_property, |
73 | }; | 73 | }; |
74 | 74 | ||
75 | void armada_drm_encoder_prepare(struct drm_encoder *encoder) | ||
76 | { | ||
77 | encoder_helper_funcs(encoder)->dpms(encoder, DRM_MODE_DPMS_OFF); | ||
78 | } | ||
79 | |||
80 | void armada_drm_encoder_commit(struct drm_encoder *encoder) | ||
81 | { | ||
82 | encoder_helper_funcs(encoder)->dpms(encoder, DRM_MODE_DPMS_ON); | ||
83 | } | ||
84 | |||
85 | bool armada_drm_encoder_mode_fixup(struct drm_encoder *encoder, | ||
86 | const struct drm_display_mode *mode, struct drm_display_mode *adjusted) | ||
87 | { | ||
88 | return true; | ||
89 | } | ||
90 | |||
91 | /* Shouldn't this be a generic helper function? */ | 75 | /* Shouldn't this be a generic helper function? */ |
92 | int armada_drm_slave_encoder_mode_valid(struct drm_connector *conn, | 76 | int armada_drm_slave_encoder_mode_valid(struct drm_connector *conn, |
93 | struct drm_display_mode *mode) | 77 | struct drm_display_mode *mode) |
diff --git a/drivers/gpu/drm/armada/armada_output.h b/drivers/gpu/drm/armada/armada_output.h index 4126d43b5057..d3a6ce79d094 100644 --- a/drivers/gpu/drm/armada/armada_output.h +++ b/drivers/gpu/drm/armada/armada_output.h | |||
@@ -21,12 +21,6 @@ struct armada_output_type { | |||
21 | 21 | ||
22 | struct drm_encoder *armada_drm_connector_encoder(struct drm_connector *conn); | 22 | struct drm_encoder *armada_drm_connector_encoder(struct drm_connector *conn); |
23 | 23 | ||
24 | void armada_drm_encoder_prepare(struct drm_encoder *encoder); | ||
25 | void armada_drm_encoder_commit(struct drm_encoder *encoder); | ||
26 | |||
27 | bool armada_drm_encoder_mode_fixup(struct drm_encoder *encoder, | ||
28 | const struct drm_display_mode *mode, struct drm_display_mode *adj); | ||
29 | |||
30 | int armada_drm_slave_encoder_mode_valid(struct drm_connector *conn, | 24 | int armada_drm_slave_encoder_mode_valid(struct drm_connector *conn, |
31 | struct drm_display_mode *mode); | 25 | struct drm_display_mode *mode); |
32 | 26 | ||