Re:Re: [PATCH] drm/amdgpu: Return more error codes in amdgpu_connector_set_property()

[赵军奎]

发件人：Markus Elfring <Markus.Elfring at web.de>
发送日期：2020-04-19 17:34:47
收件人：Bernard Zhao <bernard at vivo.com>,amd-gfx at lists.freedesktop.org,dri-devel at lists.freedesktop.org,Alex Deucher <alexander.deucher at amd.com>,"Christian König" <christian.koenig at amd.com>,Chunming Zhou <David1.Zhou at amd.com>
抄送人：Andrzej Pietrasiewicz <andrzej.p at collabora.com>,Daniel Vetter <daniel at ffwll.ch>,David Airlie <airlied at linux.ie>,Dhinakaran Pandiyan <dhinakaran.pandiyan at intel.com>,"José Roberto de Souza" <jose.souza at intel.com>,Lyude Paul <lyude at redhat.com>,Neil Armstrong <narmstrong at baylibre.com>,Sam Ravnborg <sam at ravnborg.org>,linux-kernel at vger.kernel.org,opensource.kernel at vivo.com
主题：Re: [PATCH] drm/amdgpu: Return more error codes in amdgpu_connector_set_property()>> The "if(!encoder)" branch return the same value 0 of the success
>> branch, maybe return -EINVAL is more better.
>
>I suggest to improve the commit message.

Sure.

>
>* Would you like to adjust the patch subject?
>
>* How do you think about to add the tag “Fixes”
>  because of adjustments for the exception handling?

This is a good idea, the code will be more in line with the Linux specifications.
I will adjust this patch commit message and modify the code. I will submit it again.

>
>Regards,
>Markus