Re: [PATCH 2/2] Staging: most: aim-sound: sound.c: removed unnecessary parentheses

2017-12-28 Thread Ozgur
gt;  return channel; >  } Hello, I think this patch is not good, the code will not work. Please should understand in && operator and () why used with C. Ozgur > -- > 2.15.1 ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH 0/2] Staging: most: aim-sound: sound.c: coding style

2017-12-28 Thread Ozgur
29.12.2017, 02:07, "Philippe Loctaux" : > Fixes checkpatch coding style issues. Hello, how can i see is changed/patched code lines? Ozgur > Philippe Loctaux (2): >   Staging: most: aim-sound: sound.c: fixed an alignment issue >   Staging: most: aim-sound: sound.

Re: [PATCH] drivers: staging: fbtft: fix checkpatch error and udelay

2016-12-19 Thread Ozgur Karatas
d give a some errors. So, the code have to "udelay(20)". udelay(20); I read to Documentation/timers/timers-howto.txt and this line incorrect, usleep_range need must be add defined U_DELAY and fixed. udelay(U_DELAY, U_DELAY + 10); finally: $ checkpatch.pl drivers/staging/fbtft