RE: [PATCH v2] drm/fsl-dcu: Exchange fail_encoder and fail_connector

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2016-06-16 21:02, Meng Yi wrote:
> The lable fail_connector should placed before fail_encoder since encoder was
> initialized before connector. which should also be called after
> connector initialization failed.
> 
> Hi Stefan,
> 
> What do you think?

The current error handling is wrong, I agree. Just looked into it a bit
closer, I think the right thing to do would be to use
drm_mode_config_cleanup in any case, which will call all the destroy
functions. I will send out a patch which removes the current error
handling and uses drm_mode_config_cleanup.

--
Stefan


> 
>> ---
>> Change in V2:
>> -add prefix "drm/fsl-dcu" to subject
>> ---
>>  drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_kms.c | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_kms.c
>> b/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_kms.c
>> index c564ec6..759527e 100644
>> --- a/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_kms.c
>> +++ b/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_kms.c
>> @@ -51,9 +51,9 @@ int fsl_dcu_drm_modeset_init(struct fsl_dcu_drm_device
>> *fsl_dev)
>>  	drm_kms_helper_poll_init(fsl_dev->drm);
>>
>>  	return 0;
>> -fail_encoder:
>> -	fsl_dev->crtc.funcs->destroy(&fsl_dev->crtc);
>>  fail_connector:
>>  	fsl_dev->encoder.funcs->destroy(&fsl_dev->encoder);
>> +fail_encoder:
>> +	fsl_dev->crtc.funcs->destroy(&fsl_dev->crtc);
>>  	return ret;
>>  }
>> --
>> 2.1.0.27.g96db324
> 
> Best Regards,
> Meng Yi
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux