Skip to content

Staging: emxx_udc: Fix checkpatch.pl warnings and remove extra code

Haneen Mohammed outreachy 11 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v10 Latest posting 2/24 patches 3 repliesIn review
  2. v10 posting 1/21 patch 0 repliesSuperseded
  3. v94 patches 1 replySuperseded
Earlier versions · 8
VersionPostedPatchesRepliesStatus
v85 patches4 repliesSuperseded
v70 of 5 patches0 repliesSuperseded
v60 of 4 patches0 repliesSuperseded
v5 posting 2/20 of 6 patches1 replySuperseded
v5 posting 1/26 patches5 repliesSuperseded
v46 patches5 repliesSuperseded
v21 patch0 repliesSuperseded
v11 patch1 replySuperseded

Changes in v10 from the cover letter

- Refresh against updated staging tree
- Separate a patch "removal of typedef'
- Added patch related to ERR( msg

Patches in v10 posting 2/2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/4Staging: emxx_udc: Remove argument test from functiondrivers/staging/emxx_udc/emxx_udc.c+0 -50 replies
2/4Staging: emxx_udc: Replace custom printk macro ERR with dev_err or pr_errdrivers/staging/emxx_udc/emxx_udc.c+41 -390 replies
3/4Staging: emxx_udc: Remove custom printk macro ERRdrivers/staging/emxx_udc/emxx_udc.h+0 -10 replies
4/4Staging: emxx_udc: Clean dev_err() loggingdrivers/staging/emxx_udc/emxx_udc.c+8 -103 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v10 yet.

Read the v10 thread

Cover letter [PATCH v10 0/4] Staging: emxx_udc: Fix checkpatch.pl warnings and remove extra code

This patchset fixes the following issues:
Remove argument test from function
Fix checkpatch.pl warning: printk(KERNEL_ERR not prefered. clean dev_err logging

Changes in v10:
- Refresh against updated staging tree
- Separate a patch "removal of typedef'
- Added patch related to ERR( msg
Changes in v9:
- Edit patches commit msg
- Merged two patches into one "the removal of the use of macro"
- Justify the use of pr_err
Changes in v8:
- Edit first patch subject line.
Changes in v7:
- Add missing first patch in v6: removal of typedefs and volatile. Changes in v6:
- Merged two patches into one: remove typedef and volatile.
- Justified the use of pr_err instead of dev_err. Changes in v5:
- Fix spelling mistakes
- Justify the removal of volatile
- Replaced pr_err with dev_err when possible
Changes in v4:
- Patchset broken into smaller pathces
Changes in v3:
- Create patchset from two patches
Changes in v2:
- First patch reversion

Haneen Mohammed (4):
Staging: emxx_udc: Remove argument test from function
Staging: emxx_udc: Replace custom printk macro ERR with dev_err or
pr_err
Staging: emxx_udc: Remove custom printk macro ERR
Staging: emxx_udc: Clean dev_err() logging

 drivers/staging/emxx_udc/emxx_udc.c | 83 +++++++++++++++++--------------------
 drivers/staging/emxx_udc/emxx_udc.h |  1 -
 2 files changed, 39 insertions(+), 45 deletions(-)
Signature · 1 line
1.9.1