Re: [PATCH v3 0/2] Fixing check patch styling issues

From: Dave Jiang
Date: Mon May 08 2023 - 12:04:05 EST

On 5/8/23 1:25 AM, Raghu Halharvi wrote:
v3 changes:
- Update the cover letter and commit message with full author
- Correct the "typo error" in commit message(Fabio)

Hi Raghu,
Please make sure you pick up all the review tags given by reviewers in the previous round and also add them to the respective patches.

Using the b4 tool can do that for you.

v2 changes:
Thanks Alison, Ira for your comments, modified the v1 patches as
Dropped the patch containing tab changes in port.c

v1 cover letter:
The following patches are cleanup or fixing the styling issues found
using checkpatch

In cxl/core/mbox.c, in case of null check failure, returning errno or
-ENOMEM in this case is good enough, removing the redundant dev_err

In cxl/core/region.c, the else is not required after the return
statement, cleaned it up.

Verified the build and sanity by booting the guest VM using the freshly
built components.

Raghu Halharvi (2):
cxl/mbox: Remove redundant dev_err() after failed mem alloc
cxl/region: Remove else after return statement

drivers/cxl/core/mbox.c | 4 +---
drivers/cxl/core/region.c | 8 ++++----
2 files changed, 5 insertions(+), 7 deletions(-)