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

Jeongtae Park posted 7 patches 2 years, 2 months ago
Only 0 patches received!
There is a newer version of this series
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(-)
[PATCH v2 0/7] cxl: Fix checkpatch issues
Posted by Jeongtae Park 2 years, 2 months ago
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/20230921025110.3717583-1-jtp.park@samsung.com/

---
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
-- 
2.34.1
Re: [PATCH v2 0/7] cxl: Fix checkpatch issues
Posted by Greg Kroah-Hartman 2 years, 2 months ago
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/20230921025110.3717583-1-jtp.park@samsung.com/
> 
> ---
> 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
Re: [PATCH v2 0/7] cxl: Fix checkpatch issues
Posted by Jeongtae Park 2 years, 2 months ago
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/20230921025110.3717583-1-jtp.park@samsung.com/
> > 
> > ---
> > 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