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

Harry Wentland harry.wentland at amd.com
Wed Feb 15 14:47:05 UTC 2017


On 2017-02-15 06:44 AM, Emil Velikov wrote:
> On 14 February 2017 at 20:36, Harry Wentland <harry.wentland at amd.com> 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] the code somewhat more readable".
>

Thanks, Emil. We actually have that hooked up at pre-submission in our 
internal repo but don't block submission on warnings and allow a couple 
people to push anyways despite errors because we seem to get some 
unnecessary errors on complex macros that can't really be handled 
differently.

Need to think on how to improve that. It's a pain being the bad guy 
constantly and people don't always see the importance of good coding and 
commit msg style.

Harry

> -Emil
>


More information about the amd-gfx mailing list