Skip to content

Coding style cleanups after the fwnode patchset

Mauro Carvalho Chehab linux-media 2 versions In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest3 patches 1 replyIn review
  2. v13 patches 11 repliesSuperseded

Changes in v2 from the cover letter

- fixed a weird line break at patch 1/3;
- kept the order of arguments inside a function at patch 3/3.

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3media: v4l2-core: cleanup coding style at V4L2 async/fwnodedrivers/media/v4l2-core/v4l2-async.c drivers/media/v4l2-core/v4l2-fwnode.c include/media/v4l2-async.h +2 more+179 -1411 reply
2/3media: v4l2-fwnode: cleanup functions that parse endpointsdrivers/media/v4l2-core/v4l2-fwnode.c include/media/v4l2-fwnode.h+37 -460 replies
3/3media: v4l2-fwnode: simplify v4l2_fwnode_reference_parse_int_props() calldrivers/media/v4l2-core/v4l2-fwnode.c+12 -110 replies

Reviews trailers per patch

2 of 3 patches have a Reviewed-by or Acked-by.

PatchSakari Ailus
1/3A-b
2/3
3/3A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [PATCH v2 0/3] Coding style cleanups after the fwnode patchset

The fwnode patchset added a several new warnings, as identified by checkpatch.pl --strict.

Those are at the core stuff, and makes harder to review patches there.

In order to fully address checkpatch.pl, we still need to:

- Add SPDX headers;
- Rename functions.

Let's do those on some future change.

-

v2:
- fixed a weird line break at patch 1/3;
- kept the order of arguments inside a function at patch 3/3.

Mauro Carvalho Chehab (3):
media: v4l2-core: cleanup coding style at V4L2 async/fwnode
media: v4l2-fwnode: cleanup functions that parse endpoints
media: v4l2-fwnode: simplify v4l2_fwnode_reference_parse_int_props()
call

 drivers/media/v4l2-core/v4l2-async.c  |  45 +++---
 drivers/media/v4l2-core/v4l2-fwnode.c | 188 ++++++++++++++------------
 include/media/v4l2-async.h            |  12 +-
 include/media/v4l2-fwnode.h           |  45 +++---
 include/media/v4l2-mediabus.h         |  32 +++--
 5 files changed, 176 insertions(+), 146 deletions(-)
Signature · 1 line
2.17.1