linux-raid.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 00/34] address various checkpatch.pl requirements
@ 2023-03-06 21:27 heinzm
  2023-03-06 21:27 ` [PATCH 01/34] md: fix required/prohibited spaces [ERROR] heinzm
                   ` (34 more replies)
  0 siblings, 35 replies; 42+ messages in thread
From: heinzm @ 2023-03-06 21:27 UTC (permalink / raw)
  To: linux-raid; +Cc: ncroxon, xni, dkeefe

From: heinzm <heinzm@redhat.com>

This patch series addresses checkpatch.pl reuirements.

It is grouped into patches addressing errors first then warnings.
Each patch fixes flaws in one semantical respect (e.g. fix spaces).

Series passed upstream regression tests succesfully.

Signed-off-by: Heinz Mauelshagen <heinzm@redhat.com>
Reviewed-by: Nigel Croxon <ncroxon@redhat.com>
Reviewed-by: Xiao Ni <xni@redhat.com>
Tested-by: Nigel Croxon <ncroxon@redhat.com>
Tested-by: Xiao Ni <xni@redhat.com>

Heinz Mauelshagen (34):
  md: fix required/prohibited spaces [ERROR]
  md: fix 'foo*' and 'foo * bar' [ERROR]
  md: fix EXPORT_SYMBOL() to follow its functions immediately [ERROR]
  md: adjust braces on functions/structures [ERROR]
  md: correct code indent [ERROR]
  md: move trailing statements to next line [ERROR]
  md: consistent spacing around operators [ERROR]
  md: don't initialize statics/globals to 0/false [ERROR]
  md: else should follow close curly brace [ERROR]
  md: remove trailing whitespace [ERROR]
  md: do not use assignment in if condition [ERROR]
  md: add missing blank line after declaration [WARNING]
  md: space prohibited between function and opening parenthesis [WARNING]
  md: prefer seq_put[cs]() to seq_printf() |WARNING]
  md: avoid multiple line dereference [WARNING}
  md: fix block comments [WARNING]
  md: add missing function identifier names to function definition arguments [WARNING]
  md: avoid redundant braces in single line statements [WARNING]
  md: place constant on the right side of a test [WARNING]
  md: avoid pointless filenames in files [WARNING]
  md: avoid useless else after break or return [WARNING]
  md: don't indent labels [WARNING]
  md: fix code indent for conditional statements [WARNING]
  md: prefer octal permissions [WARNING]
  md: remove bogus IS_ENABLED() macro [WARNING]
  md autodetect: correct placement of __initdata [WARNING]
  md: prefer using "%s...", __func__ [WARNING]
  md pq: adjust __attribute__ [WARNING]
  md: prefer 'unsigned int' [WARNING]
  md: prefer kmap_local_page() instead of deprecated kmap_atomic() [WARNING]
  md raid5: prefer 'int' instead of 'signed' [WARNING]
  md: prefer kvmalloc_array() with multiply [WARNING]
  md: avoid splitting quoted strings [WARNING]
  md: avoid return in void functions [WARNING]

 drivers/md/md-autodetect.c |  18 +-
 drivers/md/md-bitmap.c     | 159 ++++----
 drivers/md/md-bitmap.h     |   2 +-
 drivers/md/md-cluster.c    |  51 ++-
 drivers/md/md-faulty.c     |  43 ++-
 drivers/md/md-linear.c     |  33 +-
 drivers/md/md-linear.h     |   3 +-
 drivers/md/md-multipath.c  |  52 +--
 drivers/md/md.c            | 755 +++++++++++++++++++------------------
 drivers/md/md.h            |  17 +-
 drivers/md/raid0.c         |  70 ++--
 drivers/md/raid1-10.c      |   3 +-
 drivers/md/raid1.c         |  91 +++--
 drivers/md/raid10.c        | 174 +++++----
 drivers/md/raid5-cache.c   |  41 +-
 drivers/md/raid5-ppl.c     |  16 +-
 drivers/md/raid5.c         | 293 +++++++-------
 include/linux/raid/pq.h    |  33 +-
 include/linux/raid/xor.h   |  34 +-
 19 files changed, 1032 insertions(+), 856 deletions(-)

-- 
2.39.2


^ permalink raw reply	[flat|nested] 42+ messages in thread

end of thread, other threads:[~2023-03-14 20:43 UTC | newest]

Thread overview: 42+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-03-06 21:27 [PATCH 00/34] address various checkpatch.pl requirements heinzm
2023-03-06 21:27 ` [PATCH 01/34] md: fix required/prohibited spaces [ERROR] heinzm
2023-03-06 21:27 ` [PATCH 02/34] md: fix 'foo*' and 'foo * bar' [ERROR] heinzm
2023-03-06 21:27 ` [PATCH 03/34] md: fix EXPORT_SYMBOL() to follow its functions immediately [ERROR] heinzm
2023-03-06 21:27 ` [PATCH 04/34] md: adjust braces on functions/structures [ERROR] heinzm
2023-03-06 21:27 ` [PATCH 05/34] md: correct code indent [ERROR] heinzm
2023-03-06 21:27 ` [PATCH 06/34] md: move trailing statements to next line [ERROR] heinzm
2023-03-07 20:23   ` kernel test robot
2023-03-06 21:27 ` [PATCH 07/34] md: consistent spacing around operators [ERROR] heinzm
2023-03-06 21:27 ` [PATCH 08/34] md: don't initilize statics/globals to 0/false [ERROR] heinzm
2023-03-07  9:42   ` Paul Menzel
2023-03-06 21:27 ` [PATCH 09/34] md: else should follow close curly brace [ERROR] heinzm
2023-03-07  9:45   ` Paul Menzel
2023-03-06 21:27 ` [PATCH 10/34] md: remove trailing whitespace [ERROR] heinzm
2023-03-06 21:27 ` [PATCH 11/34] md: do not use assignment in if condition [ERROR] heinzm
2023-03-06 21:27 ` [PATCH 12/34] md: add missing blank line after declaration [WARNING] heinzm
2023-03-06 21:27 ` [PATCH 13/34] md: space prohibited between function and opening parenthesis [WARNING] heinzm
2023-03-06 21:27 ` [PATCH 14/34] md: prefer seq_put[cs]() to seq_printf() |WARNING] heinzm
2023-03-06 21:27 ` [PATCH 15/34] md: avoid multiple line dereference [WARNING} heinzm
2023-03-06 21:27 ` [PATCH 16/34] md: fix block comments [WARNING] heinzm
2023-03-06 21:27 ` [PATCH 17/34] md: add missing function identifier names to function definition arguments [WARNING] heinzm
2023-03-06 21:27 ` [PATCH 18/34] md: avoid redundant braces in single line statements [WARNING] heinzm
2023-03-06 21:27 ` [PATCH 19/34] md: place constant on the right side of a test [WARNING] heinzm
2023-03-06 21:27 ` [PATCH 20/34] md: avoid pointless filenames in files [WARNING] heinzm
2023-03-06 21:27 ` [PATCH 21/34] md: avoid useless else after break or return [WARNING] heinzm
2023-03-06 21:27 ` [PATCH 22/34] md: don't indent labels [WARNING] heinzm
2023-03-06 21:27 ` [PATCH 23/34] md: fix code indent for conditional statements [WARNING] heinzm
2023-03-06 21:27 ` [PATCH 24/34] md: prefer octal permissions [WARNING] heinzm
2023-03-06 21:27 ` [PATCH 25/34] md: remove bogus IS_ENABLED() macro [WARNING] heinzm
2023-03-06 21:27 ` [PATCH 26/34] md autodetect: correct placement of __initdata [WARNING] heinzm
2023-03-06 21:27 ` [PATCH 27/34] md: prefer using "%s...", __func__ [WARNING] heinzm
2023-03-06 21:27 ` [PATCH 28/34] md pq: adjust __attribute__ [WARNING] heinzm
2023-03-06 21:27 ` [PATCH 29/34] md: prefer 'unsigned int' [WARNING] heinzm
2023-03-06 21:27 ` [PATCH 30/34] md: prefer kmap_local_page() instead of deprecated kmap_atomic() [WARNING] heinzm
2023-03-06 21:27 ` [PATCH 31/34] md raid5: prefer 'int' instead of 'signed' [WARNING] heinzm
2023-03-06 21:27 ` [PATCH 32/34] md: prefer kvmalloc_array() with multiply [WARNING] heinzm
2023-03-06 21:27 ` [PATCH 33/34] md: avoid splitting quoted strings [WARNING] heinzm
2023-03-06 21:27 ` [PATCH 34/34] md: avoid return in void functions [WARNING] heinzm
2023-03-07  1:18 ` [PATCH 00/34] address various checkpatch.pl requirements Guoqing Jiang
     [not found]   ` <CAM23Vxqf-XMdoobeEyyk1MC=PzkWM=5w88jM8R-joxrrT82ukw@mail.gmail.com>
2023-03-08  1:30     ` Guoqing Jiang
2023-03-13 21:20       ` Song Liu
     [not found]         ` <CAM23Vxpzbt50iyJWQPxDnf51iO1E+cgQJnLFfKCZ3o5Xtro5aQ@mail.gmail.com>
2023-03-14 20:42           ` Song Liu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).