Function lends itself to being refactored. There are two checkpatch issues generated from this function. Patch 01 removes a redundant nested conditional check. Patch 02 inverts two conditionals, and reduces the subsequent code indentation. Tobin C. Harding (2): staging: ks7010: remove redundant check staging: ks7010: invert if statement conditionals drivers/staging/ks7010/ks_hostif.c | 21 ++++++++++----------- 1 file changed, 10 insertions(+), 11 deletions(-) -- 2.7.4 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel