drm/msm/mdp: Use atomic helper to set crtc property
Assign drm_atomic_helper_crtc_set_property helper to MDP4 and MDP5 crtcs' set_property ops. This replaces the custom funcs that returned an error even for standard crtc properties. Signed-off-by: Archit Taneja <architt@codeaurora.org> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> Signed-off-by: Rob Clark <robdclark@gmail.com>
This commit is contained in:
		
							parent
							
								
									f31d12af95
								
							
						
					
					
						commit
						4103eef98c
					
				| @ -361,13 +361,6 @@ static void mdp4_crtc_atomic_flush(struct drm_crtc *crtc, | ||||
| 	request_pending(crtc, PENDING_FLIP); | ||||
| } | ||||
| 
 | ||||
| static int mdp4_crtc_set_property(struct drm_crtc *crtc, | ||||
| 		struct drm_property *property, uint64_t val) | ||||
| { | ||||
| 	// XXX
 | ||||
| 	return -EINVAL; | ||||
| } | ||||
| 
 | ||||
| #define CURSOR_WIDTH 64 | ||||
| #define CURSOR_HEIGHT 64 | ||||
| 
 | ||||
| @ -499,7 +492,7 @@ static const struct drm_crtc_funcs mdp4_crtc_funcs = { | ||||
| 	.set_config = drm_atomic_helper_set_config, | ||||
| 	.destroy = mdp4_crtc_destroy, | ||||
| 	.page_flip = drm_atomic_helper_page_flip, | ||||
| 	.set_property = mdp4_crtc_set_property, | ||||
| 	.set_property = drm_atomic_helper_crtc_set_property, | ||||
| 	.cursor_set = mdp4_crtc_cursor_set, | ||||
| 	.cursor_move = mdp4_crtc_cursor_move, | ||||
| 	.reset = drm_atomic_helper_crtc_reset, | ||||
|  | ||||
| @ -468,13 +468,6 @@ static void mdp5_crtc_atomic_flush(struct drm_crtc *crtc, | ||||
| 	request_pending(crtc, PENDING_FLIP); | ||||
| } | ||||
| 
 | ||||
| static int mdp5_crtc_set_property(struct drm_crtc *crtc, | ||||
| 		struct drm_property *property, uint64_t val) | ||||
| { | ||||
| 	// XXX
 | ||||
| 	return -EINVAL; | ||||
| } | ||||
| 
 | ||||
| static void get_roi(struct drm_crtc *crtc, uint32_t *roi_w, uint32_t *roi_h) | ||||
| { | ||||
| 	struct mdp5_crtc *mdp5_crtc = to_mdp5_crtc(crtc); | ||||
| @ -625,7 +618,7 @@ static const struct drm_crtc_funcs mdp5_crtc_funcs = { | ||||
| 	.set_config = drm_atomic_helper_set_config, | ||||
| 	.destroy = mdp5_crtc_destroy, | ||||
| 	.page_flip = drm_atomic_helper_page_flip, | ||||
| 	.set_property = mdp5_crtc_set_property, | ||||
| 	.set_property = drm_atomic_helper_crtc_set_property, | ||||
| 	.reset = drm_atomic_helper_crtc_reset, | ||||
| 	.atomic_duplicate_state = drm_atomic_helper_crtc_duplicate_state, | ||||
| 	.atomic_destroy_state = drm_atomic_helper_crtc_destroy_state, | ||||
|  | ||||
		Loading…
	
		Reference in New Issue
	
	Block a user