[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CANiDSCszoUyZW85qaCrkkfNR73dXBrAURayWD8_jpV6Er6fOsg@mail.gmail.com>
Date: Tue, 18 Jun 2024 14:36:27 +0200
From: Ricardo Ribalda <ribalda@...omium.org>
To: Vikash Garodia <quic_vgarodia@...cinc.com>
Cc: Michael Tretter <m.tretter@...gutronix.de>,
Pengutronix Kernel Team <kernel@...gutronix.de>, Mauro Carvalho Chehab <mchehab@...nel.org>,
Laurent Pinchart <laurent.pinchart@...asonboard.com>, Michal Simek <michal.simek@....com>,
Andy Walls <awalls@...metrocast.net>, Stanimir Varbanov <stanimir.k.varbanov@...il.com>,
"Bryan O'Donoghue" <bryan.odonoghue@...aro.org>, Bjorn Andersson <andersson@...nel.org>,
Konrad Dybcio <konrad.dybcio@...aro.org>, linux-media@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-arm-msm@...r.kernel.org, Hans Verkuil <hverkuil-cisco@...all.nl>
Subject: Re: [PATCH v3 00/18] media: Fix the last set of coccinelle warnings
Hi Vikash
On Tue, 18 Jun 2024 at 14:34, Vikash Garodia <quic_vgarodia@...cinc.com> wrote:
>
> Hi Ricardo,
>
> On 5/28/2024 2:38 AM, Ricardo Ribalda wrote:
> > With this set we are done with all the cocci warning/errors.
> >
> > Signed-off-by: Ricardo Ribalda <ribalda@...omium.org>
> > ---
> > Changes in v3:
> > - Do not rename structure fields. (Thanks Bryan)
> > - Link to v2: https://lore.kernel.org/r/20240507-cocci-flexarray-v2-0-7aea262cf065@chromium.org
> >
> > Changes in v2:
> > - allegro: Replace hard coded 1 with a define. (Thanks Michael)
> > - Link to v1: https://lore.kernel.org/r/20240507-cocci-flexarray-v1-0-4a421c21fd06@chromium.org
> >
> > ---
> > Ricardo Ribalda (18):
> > media: allegro: nal-hevc: Replace array[1] with arrray[N]
> > media: xilinx: Refactor struct xvip_dma
> > media: dvb-frontend/mxl5xx: Refactor struct MBIN_FILE_T
> > media: dvb-frontend/mxl5xx: Use flex array for MBIN_SEGMENT_T
> > media: pci: cx18: Use flex arrays for struct cx18_scb
> > media: siano: Refactor struct sms_msg_data
> > media: siano: Remove unused structures
> > media: siano: Use flex arrays for sms_firmware
> > media: venus: Remove unused structs
> > media: venus: Use flex array for hfi_session_release_buffer_pkt
> > media: venus: Refactor struct hfi_uncompressed_plane_info
> > media: venus: Refactor struct hfi_session_get_property_pkt
> > media: venus: Refactor struct hfi_uncompressed_format_supported
> > media: venus: Refactor hfi_session_empty_buffer_uncompressed_plane0_pkt
> > media: venus: Refactor hfi_session_empty_buffer_compressed_pkt
> > media: venus: Refactor hfi_sys_get_property_pkt
> > media: venus: Refactor hfi_session_fill_buffer_pkt
> > media: venus: Refactor hfi_buffer_alloc_mode_supported
> >
> > drivers/media/common/siano/smscoreapi.c | 10 +++---
> > drivers/media/common/siano/smscoreapi.h | 18 ++---------
> > drivers/media/common/siano/smsdvb-main.c | 4 +--
> > drivers/media/common/siano/smsendian.c | 8 +++--
> > drivers/media/dvb-frontends/mxl5xx.c | 2 +-
> > drivers/media/dvb-frontends/mxl5xx_defs.h | 4 +--
> > drivers/media/pci/cx18/cx18-scb.h | 2 +-
> > drivers/media/platform/allegro-dvt/nal-hevc.h | 7 ++--
> > drivers/media/platform/qcom/venus/hfi_cmds.c | 8 ++---
> > drivers/media/platform/qcom/venus/hfi_cmds.h | 38 ++++------------------
> > drivers/media/platform/qcom/venus/hfi_helper.h | 45 ++------------------------
> > drivers/media/platform/qcom/venus/hfi_parser.c | 2 +-
> > drivers/media/platform/xilinx/xilinx-dma.c | 4 +--
> > drivers/media/platform/xilinx/xilinx-dma.h | 2 +-
> > 14 files changed, 39 insertions(+), 115 deletions(-)
> > ---
> > base-commit: 48259b90973718d2277db27b5e510f0fe957eaa0
> > change-id: 20240507-cocci-flexarray-9a807a8e108e
> >
> > Best regards,
>
> Could you also mention if you have run any video usecase on any of the boards
> with these changes ?
I have only compile tested this series. If you have access to any of
the hardware it would be great if you could test it out :)
Regards!
>
> Regards,
> Vikash
--
Ricardo Ribalda
Powered by blists - more mailing lists