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