Re: drivers: staging: vme: Fixed some code style warnings

2016-09-14 Thread Andrew Kanner
‎Thanks, I understood my fault, but haven't done this changes yet. I can't understand if I should reply to original message with v2 patch or send a new email with it?   Исходное сообщение   От: Markus Böhme Отправлено: среда, 14 сентября 2016 г., 15:56 Кому: Andrew Kanner; gre

drivers: staging: vme: Fixed some code style warnings

2016-09-12 Thread Andrew Kanner
Signed-off-by: Andrew Kanner <andrew.kan...@gmail.com> --- drivers/staging/vme/devices/vme_pio2_core.c | 12 ++-- drivers/staging/vme/devices/vme_user.c | 2 +- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/vme/devices/vme_pio2_core.c b/d

[PATCH v2] drivers: staging: vme: convert to octal notation for permission bits

2016-09-16 Thread Andrew Kanner
Ran checkpatch.pl -f vme_pio2_core.c Fixed: WARNING: Symbolic permissions are not preferred. Consider using octal permissions (0444) Signed-off-by: Andrew Kanner <andrew.kan...@gmail.com> --- drivers/staging/vme/devices/vme_pio2_core.c | 12 ++-- 1 file changed, 6 insertions

[PATCH] staging: lustre: fixed shadowed variable in socklnd_cb.c

2016-11-03 Thread Andrew Kanner
Changed variable 'tx' name in local scope Fixed: sparse warning: socklnd_cb.c:2476:41: warning: symbol 'tx' shadows an earlier one socklnd_cb.c:2435:25: originally declared here Signed-off-by: Andrew Kanner <andrew.kan...@gmail.com> --- drivers/staging/lustre/lnet/klnds/socklnd/socklnd_cb

Re: [PATCH] staging: lustre: fixed shadowed variable in socklnd_cb.c

2016-11-03 Thread Andrew Kanner
2016-11-04 1:12 GMT+03:00 Dilger, Andreas <andreas.dil...@intel.com>: > On Nov 3, 2016, at 15:54, Andrew Kanner <andrew.kan...@gmail.com> wrote: >> >> Changed variable 'tx' name in local scope >> Fixed: sparse warning: >> socklnd_cb.c:2476:41: warn

[PATCH v2] staging: lustre: fixed shadowed variable in socklnd_cb.c

2016-11-03 Thread Andrew Kanner
Removed redundant declaration of variable 'tx' in local scope Fixed: sparse warning: socklnd_cb.c:2476:41: warning: symbol 'tx' shadows an earlier one socklnd_cb.c:2435:25: originally declared here Signed-off-by: Andrew Kanner <andrew.kan...@gmail.com> --- drivers/staging/lustre/lnet