[v2,0/7] cxl: Fix checkpatch issues

Message

Jeongtae Park Sept. 22, 2023, 11:34 a.m. UTC

Changes since v1: [1]
- Seperate a fix for whitespace error/warnings in memdev (Dave)
- Drop a fix for a traceevent macro (Dave)
- Add reviewed-by tags

[1]: https://lore.kernel.org/lkml/[e-mail]/

---
Cover letter same as v1

This series fixes various checkpatch errors and warnings.
I've been looking at the CXL driver recently and noticed that there are
trivial mistake codes, so I checked all files with the script and fixed
some warnings/errors.

Jeongtae Park (7):
  cxl/trace: Fix improper SPDX comment style for header file
  cxl/region: Fix a checkpatch warning
  cxl/mem: Fix a checkpatch error
  cxl: Fix a checkpatch error
  cxl: Fix block comment style
  cxl/memdev: Fix a whitespace error
  cxl/memdev: Fix a whitespace warning

 drivers/cxl/core/memdev.c | 12 ++++++------
 drivers/cxl/core/region.c |  7 +++----
 drivers/cxl/core/trace.h  |  2 +-
 drivers/cxl/cxl.h         |  4 ++--
 drivers/cxl/cxlmem.h      |  2 +-
 5 files changed, 13 insertions(+), 14 deletions(-)


base-commit: fe77cc2e5a6a7c85f5c6ef8a39d7694ffc7f41c9

Comments

Greg Kroah-Hartman Oct. 5, 2023, 8:08 a.m. UTC | #1

On Fri, Sep 22, 2023 at 08:34:43PM +0900, Jeongtae Park wrote:
> Changes since v1: [1]
> - Seperate a fix for whitespace error/warnings in memdev (Dave)
> - Drop a fix for a traceevent macro (Dave)
> - Add reviewed-by tags
> 
> [1]: https://lore.kernel.org/lkml/[e-mail]/
> 
> ---
> Cover letter same as v1

None of these emails were properly threaded (look at lore.kernel.org for
proof.)

Please fix your email client to do this properly, your v1 of this series
was threaded correctly, so something changed here :(

thanks,

greg k-h

Jeongtae Park Oct. 5, 2023, 8:42 a.m. UTC | #2

On Thu, Oct 05, 2023 at 10:08:42AM +0200, Greg Kroah-Hartman wrote:
> On Fri, Sep 22, 2023 at 08:34:43PM +0900, Jeongtae Park wrote:
> > Changes since v1: [1]
> > - Seperate a fix for whitespace error/warnings in memdev (Dave)
> > - Drop a fix for a traceevent macro (Dave)
> > - Add reviewed-by tags
> > 
> > [1]: https://lore.kernel.org/lkml/[e-mail]/
> > 
> > ---
> > Cover letter same as v1
> 
> None of these emails were properly threaded (look at lore.kernel.org for
> proof.)
> 
> Please fix your email client to do this properly, your v1 of this series
> was threaded correctly, so something changed here :(

Hello Greg,
Sorry about the problem, I guess it's a internal system problem but
I didn't figure out the root cause. So, I'll always test internally
before sending the patch series until resolving the internal problem.

Thanks!

> 
> thanks,
> 
> greg k-h