Re: [PATCH 00/13] nvdimm: Use more common kernel coding style

From: Jeff Moyer
Date: Thu Sep 12 2019 - 10:00:14 EST


Joe Perches <joe@xxxxxxxxxxx> writes:

> Rather than have a local coding style, use the typical kernel style.

The coding style isn't that different from the core kernel, and it's
still quite readable. I'd rather avoid the churn and the risk of
introducing regressions. This will also make backports to stable more
of a pain, so it isn't without cost. Dan, is this really something you
want to do?

-Jeff

>
> Joe Perches (13):
> nvdimm: Use more typical whitespace
> nvdimm: Move logical continuations to previous line
> nvdimm: Use octal permissions
> nvdimm: Use a more common kernel spacing style
> nvdimm: Use "unsigned int" in preference to "unsigned"
> nvdimm: Add and remove blank lines
> nvdimm: Use typical kernel brace styles
> nvdimm: Use typical kernel style indentation
> nvdimm: btt.h: Neaten #defines to improve readability
> nvdimm: namespace_devs: Move assignment operators
> nvdimm: Use more common logic testing styles and bare ; positions
> nvdimm: namespace_devs: Change progess typo to progress
> nvdimm: Miscellaneous neatening
>
> drivers/nvdimm/badrange.c | 22 +-
> drivers/nvdimm/blk.c | 39 ++--
> drivers/nvdimm/btt.c | 249 +++++++++++----------
> drivers/nvdimm/btt.h | 56 ++---
> drivers/nvdimm/btt_devs.c | 68 +++---
> drivers/nvdimm/bus.c | 138 ++++++------
> drivers/nvdimm/claim.c | 50 ++---
> drivers/nvdimm/core.c | 42 ++--
> drivers/nvdimm/dax_devs.c | 3 +-
> drivers/nvdimm/dimm.c | 3 +-
> drivers/nvdimm/dimm_devs.c | 107 ++++-----
> drivers/nvdimm/e820.c | 2 +-
> drivers/nvdimm/label.c | 213 +++++++++---------
> drivers/nvdimm/label.h | 6 +-
> drivers/nvdimm/namespace_devs.c | 472 +++++++++++++++++++++-------------------
> drivers/nvdimm/nd-core.h | 31 +--
> drivers/nvdimm/nd.h | 94 ++++----
> drivers/nvdimm/nd_virtio.c | 20 +-
> drivers/nvdimm/of_pmem.c | 6 +-
> drivers/nvdimm/pfn_devs.c | 136 ++++++------
> drivers/nvdimm/pmem.c | 57 ++---
> drivers/nvdimm/pmem.h | 2 +-
> drivers/nvdimm/region.c | 20 +-
> drivers/nvdimm/region_devs.c | 160 +++++++-------
> drivers/nvdimm/security.c | 138 ++++++------
> drivers/nvdimm/virtio_pmem.c | 10 +-
> 26 files changed, 1115 insertions(+), 1029 deletions(-)