Re: [PATCH] media: Fix indentation issues introduced by subdev-wide state struct

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 15/02/2023 18:05, Laurent Pinchart wrote:
Commit 0d346d2a6f54 ("media: v4l2-subdev: add subdev-wide state struct")
applied a large media tree-wide change produced by coccinelle. It was so
large that a set of identical indentation issues went unnoticed during
review. Fix them.

While at it, and because it's easy to review both changes together, add
a trailing comma for the last (and only) struct member initialization of
the related structures, to avoid future changes should new fields need
to be initialized.

Fixes: 0d346d2a6f54 ("media: v4l2-subdev: add subdev-wide state struct")
Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
---
  drivers/media/pci/saa7134/saa7134-empress.c        |  4 ++--
  drivers/media/platform/atmel/atmel-isi.c           |  4 ++--
  drivers/media/platform/intel/pxa_camera.c          |  4 ++--
  drivers/media/platform/marvell/mcam-core.c         |  4 ++--
  drivers/media/platform/renesas/renesas-ceu.c       |  4 ++--
  .../platform/rockchip/rkisp1/rkisp1-resizer.c      | 14 +++++++-------
  drivers/media/platform/via/via-camera.c            |  4 ++--
  drivers/staging/media/atomisp/pci/atomisp_cmd.c    |  8 ++++----
  .../media/deprecated/atmel/atmel-isc-base.c        |  4 ++--
  9 files changed, 25 insertions(+), 25 deletions(-)


Reviewed-by: Tomi Valkeinen <tomi.valkeinen@xxxxxxxxxxxxxxxx>

 Tomi




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux