Checkpatch emits several warnings when parsing the function ks_sdio_interrupt(). Function lends itself to being refactored. Patch 01 renames identifier 'retval' to 'ret' in line with the rest of the driver. Patch 02 inverts if statement conditional and reduces the level of indentation in subsequent code by one level. Patch 03 fixes white space issues, clearing a couple of checkpatch warnings and trivial style issues. Code is build-tested only (x86_64 and PowerPC). Tobin C. Harding (3): staging: ks7010: rename identifier retval to ret staging: ks7010: invert conditional, reduce indentation staging: ks7010: fix whitespace issues drivers/staging/ks7010/ks7010_sdio.c | 118 ++++++++++++++++------------------- 1 file changed, 55 insertions(+), 63 deletions(-) -- 2.7.4 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel