| Message ID | 20240618-cocci-flexarray-v5-0-6a8294942f48@chromium.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | media: Fix the last set of coccinelle warnings | expand |
On Tue, Jun 18, 2024 at 01:37:43PM GMT, Ricardo Ribalda wrote: > With this set we are done with all the cocci warning/errors. > > Signed-off-by: Ricardo Ribalda <ribalda@chromium.org> > --- > Changes in v5: Thanks Vikash > - Remove patches already merged > - Fix typos and remove hfi_resource_ocmem_requirement > - Link to v4: https://lore.kernel.org/r/20240606-cocci-flexarray-v4-0-3379ee5eed28@chromium.org > Just for the clarfification: does the driver build without errors if only a part of the patchset is applied? I have the feeling that first patches introduce a build error which is fixed in a later patch. If that's the case, it's a no-go.
Hi On Wed, 19 Jun 2024 at 08:41, Dmitry Baryshkov <dmitry.baryshkov@linaro.org> wrote: > > On Tue, Jun 18, 2024 at 01:37:43PM GMT, Ricardo Ribalda wrote: > > With this set we are done with all the cocci warning/errors. > > > > Signed-off-by: Ricardo Ribalda <ribalda@chromium.org> > > --- > > Changes in v5: Thanks Vikash > > - Remove patches already merged > > - Fix typos and remove hfi_resource_ocmem_requirement > > - Link to v4: https://lore.kernel.org/r/20240606-cocci-flexarray-v4-0-3379ee5eed28@chromium.org > > > > Just for the clarfification: does the driver build without errors > if only a part of the patchset is applied? I have the feeling that first > patches introduce a build error which is fixed in a later patch. If > that's the case, it's a no-go. I believe it should build without errors for every patch: It has passed the bisectability test https://gitlab.freedesktop.org/linux-media/users/ribalda/-/jobs/62071294 > > -- > With best wishes > Dmitry -- Ricardo Ribalda
With this set we are done with all the cocci warning/errors. Signed-off-by: Ricardo Ribalda <ribalda@chromium.org> --- Changes in v5: Thanks Vikash - Remove patches already merged - Fix typos and remove hfi_resource_ocmem_requirement - Link to v4: https://lore.kernel.org/r/20240606-cocci-flexarray-v4-0-3379ee5eed28@chromium.org Changes in v4: - Remove patches already merged - Combine dvb-frontend/mxl5xx patches and use flex on both (Thanks Hans) - Link to v3: https://lore.kernel.org/r/20240527-cocci-flexarray-v3-0-cda09c535816@chromium.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 (10): 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/platform/qcom/venus/hfi_cmds.c | 8 ++-- drivers/media/platform/qcom/venus/hfi_cmds.h | 38 +++---------------- drivers/media/platform/qcom/venus/hfi_helper.h | 52 ++------------------------ drivers/media/platform/qcom/venus/hfi_parser.c | 2 +- 4 files changed, 14 insertions(+), 86 deletions(-) --- base-commit: 24147897507cd3a7d63745d1518a638bf4132238 change-id: 20240507-cocci-flexarray-9a807a8e108e Best regards,