Re: [PATCH] drm/amd/display: Fix 64-bit division, yet again

2017-02-15 Thread Emil Velikov
On 15 February 2017 at 14:47, Harry Wentland wrote: > On 2017-02-15 06:44 AM, Emil Velikov wrote: >> >> On 14 February 2017 at 20:36, Harry Wentland >> wrote: >>> >>> Also make the code somewhat more readable. >>> >> I'd suggest reaching to the

Re: [PATCH] drm/amd/display: Fix 64-bit division, yet again

2017-02-15 Thread Harry Wentland
On 2017-02-15 06:44 AM, Emil Velikov wrote: On 14 February 2017 at 20:36, Harry Wentland wrote: Also make the code somewhat more readable. I'd suggest reaching to the team to integrate scripts/checkpatch.pl in the pre-commit hook. It will help you improve the coding

Re: [PATCH] drm/amd/display: Fix 64-bit division, yet again

2017-02-15 Thread Emil Velikov
On 14 February 2017 at 20:36, Harry Wentland wrote: > Also make the code somewhat more readable. > I'd suggest reaching to the team to integrate scripts/checkpatch.pl in the pre-commit hook. It will help you improve the coding standard and, as you mentioned, it "make[s]

[PATCH] drm/amd/display: Fix 64-bit division, yet again

2017-02-14 Thread Harry Wentland
Also make the code somewhat more readable. Signed-off-by: Harry Wentland --- drivers/gpu/drm/amd/display/modules/freesync/freesync.c | 15 +++ 1 file changed, 11 insertions(+), 4 deletions(-) diff --git