<html> <head> <meta content="text/html; charset=utf-8" http-equiv="Content-Type"> </head> <body bgcolor="#FFFFFF" text="#000000"> Thanks for these style fixes. Patches 1 and 2 seem good at first sight. Not acking because I haven't tested the patches and the implementation of kmalloc_array() appears to slightly differ from kmalloc(). This difference *should* only affect allocations larger than the ones in patch 1 and 2, but I'd like to see these changes tested before I can sleep soundly! Patches 4 and 5 are: Reviewed-by: Roy Spliet <a class="moz-txt-link-rfc2396E" href="mailto:nouveau@spliet.org"><nouveau@spliet.org></a> <div class="moz-cite-prefix">Op 21-09-16 om 08:23 schreef SF Markus Elfring: </div> <blockquote cite="mid:5bcff2ec-f3bd-ab9c-e13d-3a4f5cf7c73b@users.sourceforge.net" type="cite"> <pre wrap="">From: Markus Elfring <a class="moz-txt-link-rfc2396E" href="mailto:elfring@users.sourceforge.net"><elfring@users.sourceforge.net></a> Date: Wed, 21 Sep 2016 09:09:09 +0200 A few update suggestions were taken into account from static source code analysis. Markus Elfring (5): Use kmalloc_array() in nvbios_iccsense_parse() Use kmalloc_array() in gt215_link_train() Delete unnecessary braces Adjust a kzalloc() call in gt215_ram_new() Add space after an "if" drivers/gpu/drm/nouveau/nvkm/subdev/bios/iccsense.c | 4 +++- drivers/gpu/drm/nouveau/nvkm/subdev/fb/ramgt215.c | 21 +++++++++------------ 2 files changed, 12 insertions(+), 13 deletions(-) </pre> </blockquote> </body> </html>