On 26/01/18 01:38 PM, Christian König wrote:
Instead of "fix indentation" better write "fix coding style" and add some commit message to each patch. Something like "No functional change..." for the style changes should be ok.

Additional to that please move patch #11 to the top of the list and triple check in patch #10 that this is indeed safe.

With that done the series is Reviewed-by: Christian König <christian.koe...@amd.com>.

I'll do those changes on Monday and resubmit en masse. This will give time for other dri/ttm folk to review and I can avoid too much churn if anyone else has issues.

I agree that #10 is a bit tricky because retval had a default value which hopefully I captured with the assignment towards the end of the function. It just seemed kinda awkward to have ret and retval :-)

Thanks,
Tom


Regards,
Christian.

Am 26.01.2018 um 19:28 schrieb Tom St Denis:
This series includes mostly no-functional-changes to simplify
or cleanup various routines.

Patch #11 includes an fix to functional behaviour.

_______________________________________________
amd-gfx mailing list
amd-...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx


_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to