Re: [PATCH] drm/amd/display: remove some unneeded code

2017-11-06 Thread Alex Deucher
On Mon, Nov 6, 2017 at 6:59 AM, Christian König wrote: > Am 06.11.2017 um 12:44 schrieb Dan Carpenter: >> >> We assign "v_init = asic_blank_start;" a few lines earlier so there is >> no need to do it again inside the if statements. Also "v_init" is >> unsigned so it can't be less than zero. >> >>

Re: [PATCH] drm/amd/display: remove some unneeded code

2017-11-06 Thread Christian König
Am 06.11.2017 um 12:44 schrieb Dan Carpenter: We assign "v_init = asic_blank_start;" a few lines earlier so there is no need to do it again inside the if statements. Also "v_init" is unsigned so it can't be less than zero. Signed-off-by: Dan Carpenter Acked-by: Christian König diff --git

[PATCH] drm/amd/display: remove some unneeded code

2017-11-06 Thread Dan Carpenter
We assign "v_init = asic_blank_start;" a few lines earlier so there is no need to do it again inside the if statements. Also "v_init" is unsigned so it can't be less than zero. Signed-off-by: Dan Carpenter diff --git a/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_timing_generator.c b/drivers/gpu