Messages in this thread | | | From | Alex Deucher <> | Date | Tue, 4 Aug 2020 18:19:58 -0400 | Subject | Re: [PATCH] drm/amd/display: Fix wrong return value in dm_update_plane_state() |
| |
On Mon, Aug 3, 2020 at 4:21 AM Tianjia Zhang <tianjia.zhang@linux.alibaba.com> wrote: > > On an error exit path, a negative error code should be returned > instead of a positive return value. > > Fixes: 9e869063b0021 ("drm/amd/display: Move iteration out of dm_update_planes") > Cc: Leo Li <sunpeng.li@amd.com> > Signed-off-by: Tianjia Zhang <tianjia.zhang@linux.alibaba.com>
Applied. Thanks!
Alex
> --- > drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c > index 710edc70e37e..5810416e2ddc 100644 > --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c > +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c > @@ -8182,8 +8182,7 @@ static int dm_update_plane_state(struct dc *dc, > dm_old_plane_state->dc_state, > dm_state->context)) { > > - ret = EINVAL; > - return ret; > + return -EINVAL; > } > > > -- > 2.26.2 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel
| |