Re: [PATCH] drm/exynos: use drm_plane_force_disable

[Inki Dae <inki.dae@samsung.com>]

On 2015년 04월 10일 05:25, Gustavo Padovan wrote:
> Hi Joonyoung,
> 
> 2015-04-09 Joonyoung Shim <jy0922.shim@samsung.com>:
> 
>> Don't call directly disable callback of plane helper, we need to
>> disconnect the plane from the fb and crtc after disable callback.
>>
>> Signed-off-by: Joonyoung Shim <jy0922.shim@samsung.com>
>> ---
>>  drivers/gpu/drm/exynos/exynos_drm_crtc.c    | 5 +----
>>  drivers/gpu/drm/exynos/exynos_drm_encoder.c | 2 +-
>>  2 files changed, 2 insertions(+), 5 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/exynos/exynos_drm_crtc.c b/drivers/gpu/drm/exynos/exynos_drm_crtc.c
>> index 519c569..50c830e 100644
>> --- a/drivers/gpu/drm/exynos/exynos_drm_crtc.c
>> +++ b/drivers/gpu/drm/exynos/exynos_drm_crtc.c
>> @@ -48,7 +48,6 @@ static void exynos_drm_crtc_disable(struct drm_crtc *crtc)
>>  {
>>  	struct exynos_drm_crtc *exynos_crtc = to_exynos_crtc(crtc);
>>  	struct drm_plane *plane;
>> -	int ret;
>>  
>>  	if (!exynos_crtc->enabled)
>>  		return;
>> @@ -69,9 +68,7 @@ static void exynos_drm_crtc_disable(struct drm_crtc *crtc)
>>  		if (plane->crtc != crtc)
>>  			continue;
>>  
>> -		ret = plane->funcs->disable_plane(plane);
>> -		if (ret)
>> -			DRM_ERROR("Failed to disable plane %d\n", ret);
>> +		drm_plane_force_disable(plane);
>>  	}
> 
> Which tree did you based this code? I've removed all this code in atomic.
> These two pieces of code makes no sense in atomic modesetting, disable would
> be called from the drm atomic core there.

Mr. Gustavo,

Could you check if you have other codes not posted? At least, it seems
that the codes you posted are same as above.

Thanks,
Inki Dae

> 
> 	Gustavo
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
> 

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel