From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Ricardo Ribalda <ribalda@chromium.org>
Cc: "Martin Tuma" <martin.tuma@digiteqautomotive.com>,
"Mauro Carvalho Chehab" <mchehab@kernel.org>,
"Hans Verkuil" <hverkuil-cisco@xs4all.nl>,
"Hugues Fruchet" <hugues.fruchet@foss.st.com>,
"Alain Volmat" <alain.volmat@foss.st.com>,
"Maxime Coquelin" <mcoquelin.stm32@gmail.com>,
"Alexandre Torgue" <alexandre.torgue@foss.st.com>,
"Paul Kocialkowski" <paul.kocialkowski@bootlin.com>,
"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
"Chen-Yu Tsai" <wens@csie.org>,
"Jernej Skrabec" <jernej.skrabec@gmail.com>,
"Samuel Holland" <samuel@sholland.org>,
"Sakari Ailus" <sakari.ailus@linux.intel.com>,
"Thierry Reding" <thierry.reding@gmail.com>,
"Jonathan Hunter" <jonathanh@nvidia.com>,
"Sowjanya Komatineni" <skomatineni@nvidia.com>,
"Luca Ceresoli" <luca.ceresoli@bootlin.com>,
"Matthias Brugger" <matthias.bgg@gmail.com>,
"AngeloGioacchino Del Regno"
<angelogioacchino.delregno@collabora.com>,
"Hans Verkuil" <hverkuil@xs4all.nl>,
"Sergey Kozlov" <serjk@netup.ru>,
"Abylay Ospan" <aospan@netup.ru>,
"Ezequiel Garcia" <ezequiel@vanguardiasur.com.ar>,
"Dmitry Osipenko" <digetx@gmail.com>,
"Stanimir Varbanov" <stanimir.k.varbanov@gmail.com>,
"Vikash Garodia" <quic_vgarodia@quicinc.com>,
"Bryan O'Donoghue" <bryan.odonoghue@linaro.org>,
"Bjorn Andersson" <andersson@kernel.org>,
"Konrad Dybcio" <konrad.dybcio@linaro.org>,
"Benjamin Mugnier" <benjamin.mugnier@foss.st.com>,
"Sylvain Petinot" <sylvain.petinot@foss.st.com>,
"Jacopo Mondi" <jacopo+renesas@jmondi.org>,
"Kieran Bingham" <kieran.bingham+renesas@ideasonboard.com>,
"Niklas Söderlund" <niklas.soderlund+renesas@ragnatech.se>,
"Pavel Machek" <pavel@ucw.cz>,
linux-media@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-stm32@st-md-mailman.stormreply.com,
linux-arm-kernel@lists.infradead.org,
linux-staging@lists.linux.dev, linux-sunxi@lists.linux.dev,
linux-tegra@vger.kernel.org, linux-mediatek@lists.infradead.org,
linux-arm-msm@vger.kernel.org
Subject: Re: [PATCH 00/35] media: Fix coccinelle warning/errors
Date: Mon, 15 Apr 2024 22:53:48 +0300 [thread overview]
Message-ID: <20240415195348.GD22954@pendragon.ideasonboard.com> (raw)
In-Reply-To: <20240415-fix-cocci-v1-0-477afb23728b@chromium.org>
Hi Ricardo,
I'm afraid I won't have time to review any of this for the time being.
Unless you would like me to put uvcvideo reviews on hold ;-)
Jokes aside, my first reaction was that this feels like a bit of a waste
of maintainer's time :-S
On Mon, Apr 15, 2024 at 07:34:17PM +0000, Ricardo Ribalda wrote:
> After this set is applied, these are the only warnings left:
> drivers/media/pci/ivtv/ivtv-fileops.c:223:4-10: preceding lock on line 267
> drivers/media/pci/ivtv/ivtv-fileops.c:230:3-9: preceding lock on line 267
> drivers/media/pci/ivtv/ivtv-fileops.c:236:4-10: preceding lock on line 267
> drivers/media/pci/ivtv/ivtv-fileops.c:245:3-9: preceding lock on line 267
> drivers/media/pci/ivtv/ivtv-fileops.c:251:3-9: preceding lock on line 267
> drivers/media/pci/ivtv/ivtv-fileops.c:257:3-9: preceding lock on line 267
> drivers/media/pci/ivtv/ivtv-fileops.c:272:3-9: preceding lock on line 267
> drivers/media/pci/ivtv/ivtv-fileops.c:598:4-10: preceding lock on line 627
> drivers/media/pci/ivtv/ivtv-fileops.c:598:4-10: preceding lock on line 689
> drivers/media/pci/ivtv/ivtv-fileops.c:606:3-9: preceding lock on line 627
> drivers/media/pci/ivtv/ivtv-fileops.c:606:3-9: preceding lock on line 689
> drivers/media/pci/ivtv/ivtv-fileops.c:648:3-9: preceding lock on line 627
> drivers/media/pci/ivtv/ivtv-fileops.c:648:3-9: preceding lock on line 689
> drivers/media/pci/ivtv/ivtv-fileops.c:692:4-10: preceding lock on line 689
> drivers/media/dvb-core/dvb_frontend.c:2897:1-7: preceding lock on line 2776
> drivers/media/dvb-core/dvb_frontend.c:2897:1-7: preceding lock on line 2786
> drivers/media/dvb-core/dvb_frontend.c:2897:1-7: preceding lock on line 2809
> drivers/media/dvb-frontends/stv090x.c:799:1-7: preceding lock on line 768
> drivers/media/usb/go7007/go7007-i2c.c:125:1-7: preceding lock on line 61
> drivers/media/rc/imon.c:1167:1-7: preceding lock on line 1153
> drivers/media/pci/cx18/cx18-scb.h:261:22-29: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_cmds.h:77:5-9: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_cmds.h:85:5-16: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_cmds.h:154:5-9: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_cmds.h:171:5-9: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_cmds.h:180:5-9: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_cmds.h:189:5-9: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_cmds.h:201:5-9: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_cmds.h:220:5-9: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_cmds.h:230:5-16: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_helper.h:764:5-15: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_helper.h:1008:43-60: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_helper.h:1014:36-46: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_helper.h:1041:5-15: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_helper.h:1088:39-51: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_helper.h:1093:5-22: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_helper.h:1144:4-8: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_helper.h:1239:4-8: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_helper.h:1267:5-9: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_helper.h:1272:4-13: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/common/siano/smscoreapi.h:619:5-13: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/common/siano/smscoreapi.h:669:6-13: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/common/siano/smscoreapi.h:1049:4-8: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/common/siano/smscoreapi.h:1055:4-8: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/dvb-frontends/mxl5xx_defs.h:171:4-8: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/dvb-frontends/mxl5xx_defs.h:182:4-8: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/allegro-dvt/nal-hevc.h:102:14-22: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/xilinx/xilinx-dma.h:100:19-22: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/staging/media/atomisp/pci/atomisp_tpg.h:30:18-22: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
>
> CI tested:
> https://gitlab.freedesktop.org/linux-media/media-staging/-/commit/055b5211c68e721c3a7090be5373cf44859da1a7/pipelines?ref=ribalda%2Ftest-cocci
>
> Signed-off-by: Ricardo Ribalda <ribalda@chromium.org>
> ---
> Ricardo Ribalda (35):
> media: pci: mgb4: Refactor struct resources
> media: stb0899: Remove unreacheable code
> media: uvcvideo: Refactor iterators
> media: uvcvideo: Use max() macro
> media: go7007: Use min and max macros
> media: stm32-dcmipp: Remove redundant printk
> media: staging: sun6i-isp: Remove redundant printk
> media: dvb-frontends: tda18271c2dd: Remove casting during div
> media: v4l: async: refactor v4l2_async_create_ancillary_links
> staging: media: tegra-video: Use swap macro
> media: s2255: Use refcount_t instead of atomic_t for num_channels
> media: platform: mtk-mdp3: Use refcount_t for job_count
> media: common: saa7146: Use min macro
> media: dvb-frontends: drx39xyj: Use min macro
> media: netup_unidvb: Use min macro
> media: au0828: Use min macro
> media: flexcop-usb: Use min macro
> media: gspca: cpia1: Use min macro
> media: stk1160: Use min macro
> media: tegra-vde: Refactor timeout handling
> media: venus: Use div64_u64
> media: i2c: st-mipid02: Use the correct div function
> media: dvb-frontends: tda10048: Use the right div
> media: tc358746: Use the correct div_ function
> media: venus: Use the correct div_ function
> media: venus: Refator return path
> media: dib0700: Refator return path
> media: usb: cx231xx: Refator return path
> media: i2c: rdacm20: Refator return path
> media: i2c: et8ek8: Refator return path
> media: cx231xx: Refator return path
> media: si4713: Refator return path
> media: ttpci: Refator return path
> media: hdpvr: Refator return path
> media: venus: Refator return path
>
> drivers/media/common/saa7146/saa7146_hlp.c | 8 +++----
> drivers/media/dvb-frontends/drx39xyj/drxj.c | 9 +++-----
> drivers/media/dvb-frontends/stb0899_drv.c | 5 -----
> drivers/media/dvb-frontends/tda10048.c | 3 +--
> drivers/media/dvb-frontends/tda18271c2dd.c | 4 ++--
> drivers/media/i2c/et8ek8/et8ek8_driver.c | 4 +++-
> drivers/media/i2c/rdacm20.c | 5 ++++-
> drivers/media/i2c/st-mipid02.c | 2 +-
> drivers/media/i2c/tc358746.c | 3 +--
> drivers/media/pci/mgb4/mgb4_core.c | 4 ++--
> drivers/media/pci/mgb4/mgb4_regs.c | 2 +-
> drivers/media/pci/netup_unidvb/netup_unidvb_i2c.c | 2 +-
> drivers/media/pci/ttpci/budget-core.c | 5 ++++-
> .../media/platform/mediatek/mdp3/mtk-mdp3-cmdq.c | 10 ++++-----
> .../media/platform/mediatek/mdp3/mtk-mdp3-core.c | 6 ++---
> .../media/platform/mediatek/mdp3/mtk-mdp3-core.h | 2 +-
> .../media/platform/mediatek/mdp3/mtk-mdp3-m2m.c | 6 ++---
> drivers/media/platform/nvidia/tegra-vde/h264.c | 6 ++---
> drivers/media/platform/qcom/venus/vdec.c | 15 +++++++------
> drivers/media/platform/qcom/venus/venc.c | 19 +++++++++-------
> .../platform/st/stm32/stm32-dcmipp/dcmipp-core.c | 5 +----
> drivers/media/radio/si4713/radio-usb-si4713.c | 8 +++++--
> drivers/media/usb/au0828/au0828-video.c | 5 +----
> drivers/media/usb/b2c2/flexcop-usb.c | 5 +----
> drivers/media/usb/cx231xx/cx231xx-i2c.c | 16 +++++++++----
> drivers/media/usb/cx231xx/cx231xx-video.c | 10 +++++++--
> drivers/media/usb/dvb-usb/dib0700_core.c | 4 +++-
> drivers/media/usb/go7007/go7007-fw.c | 4 ++--
> drivers/media/usb/gspca/cpia1.c | 6 ++---
> drivers/media/usb/hdpvr/hdpvr-control.c | 4 +++-
> drivers/media/usb/s2255/s2255drv.c | 20 ++++++++---------
> drivers/media/usb/stk1160/stk1160-video.c | 10 ++-------
> drivers/media/usb/uvc/uvc_ctrl.c | 26 ++++++++++++----------
> drivers/media/v4l2-core/v4l2-async.c | 8 +++----
> drivers/staging/media/sunxi/sun6i-isp/sun6i_isp.c | 1 -
> drivers/staging/media/tegra-video/tegra20.c | 9 ++------
> 36 files changed, 132 insertions(+), 129 deletions(-)
> ---
> base-commit: 71b3ed53b08d87212fbbe51bdc3bf44eb8c462f8
> change-id: 20240415-fix-cocci-2df3ef22a6f7
>
> Best regards,
--
Regards,
Laurent Pinchart
WARNING: multiple messages have this Message-ID (diff)
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Ricardo Ribalda <ribalda@chromium.org>
Cc: "Martin Tuma" <martin.tuma@digiteqautomotive.com>,
"Mauro Carvalho Chehab" <mchehab@kernel.org>,
"Hans Verkuil" <hverkuil-cisco@xs4all.nl>,
"Hugues Fruchet" <hugues.fruchet@foss.st.com>,
"Alain Volmat" <alain.volmat@foss.st.com>,
"Maxime Coquelin" <mcoquelin.stm32@gmail.com>,
"Alexandre Torgue" <alexandre.torgue@foss.st.com>,
"Paul Kocialkowski" <paul.kocialkowski@bootlin.com>,
"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
"Chen-Yu Tsai" <wens@csie.org>,
"Jernej Skrabec" <jernej.skrabec@gmail.com>,
"Samuel Holland" <samuel@sholland.org>,
"Sakari Ailus" <sakari.ailus@linux.intel.com>,
"Thierry Reding" <thierry.reding@gmail.com>,
"Jonathan Hunter" <jonathanh@nvidia.com>,
"Sowjanya Komatineni" <skomatineni@nvidia.com>,
"Luca Ceresoli" <luca.ceresoli@bootlin.com>,
"Matthias Brugger" <matthias.bgg@gmail.com>,
"AngeloGioacchino Del Regno"
<angelogioacchino.delregno@collabora.com>,
"Hans Verkuil" <hverkuil@xs4all.nl>,
"Sergey Kozlov" <serjk@netup.ru>,
"Abylay Ospan" <aospan@netup.ru>,
"Ezequiel Garcia" <ezequiel@vanguardiasur.com.ar>,
"Dmitry Osipenko" <digetx@gmail.com>,
"Stanimir Varbanov" <stanimir.k.varbanov@gmail.com>,
"Vikash Garodia" <quic_vgarodia@quicinc.com>,
"Bryan O'Donoghue" <bryan.odonoghue@linaro.org>,
"Bjorn Andersson" <andersson@kernel.org>,
"Konrad Dybcio" <konrad.dybcio@linaro.org>,
"Benjamin Mugnier" <benjamin.mugnier@foss.st.com>,
"Sylvain Petinot" <sylvain.petinot@foss.st.com>,
"Jacopo Mondi" <jacopo+renesas@jmondi.org>,
"Kieran Bingham" <kieran.bingham+renesas@ideasonboard.com>,
"Niklas Söderlund" <niklas.soderlund+renesas@ragnatech.se>,
"Pavel Machek" <pavel@ucw.cz>,
linux-media@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-stm32@st-md-mailman.stormreply.com,
linux-arm-kernel@lists.infradead.org,
linux-staging@lists.linux.dev, linux-sunxi@lists.linux.dev,
linux-tegra@vger.kernel.org, linux-mediatek@lists.infradead.org,
linux-arm-msm@vger.kernel.org
Subject: Re: [PATCH 00/35] media: Fix coccinelle warning/errors
Date: Mon, 15 Apr 2024 22:53:48 +0300 [thread overview]
Message-ID: <20240415195348.GD22954@pendragon.ideasonboard.com> (raw)
In-Reply-To: <20240415-fix-cocci-v1-0-477afb23728b@chromium.org>
Hi Ricardo,
I'm afraid I won't have time to review any of this for the time being.
Unless you would like me to put uvcvideo reviews on hold ;-)
Jokes aside, my first reaction was that this feels like a bit of a waste
of maintainer's time :-S
On Mon, Apr 15, 2024 at 07:34:17PM +0000, Ricardo Ribalda wrote:
> After this set is applied, these are the only warnings left:
> drivers/media/pci/ivtv/ivtv-fileops.c:223:4-10: preceding lock on line 267
> drivers/media/pci/ivtv/ivtv-fileops.c:230:3-9: preceding lock on line 267
> drivers/media/pci/ivtv/ivtv-fileops.c:236:4-10: preceding lock on line 267
> drivers/media/pci/ivtv/ivtv-fileops.c:245:3-9: preceding lock on line 267
> drivers/media/pci/ivtv/ivtv-fileops.c:251:3-9: preceding lock on line 267
> drivers/media/pci/ivtv/ivtv-fileops.c:257:3-9: preceding lock on line 267
> drivers/media/pci/ivtv/ivtv-fileops.c:272:3-9: preceding lock on line 267
> drivers/media/pci/ivtv/ivtv-fileops.c:598:4-10: preceding lock on line 627
> drivers/media/pci/ivtv/ivtv-fileops.c:598:4-10: preceding lock on line 689
> drivers/media/pci/ivtv/ivtv-fileops.c:606:3-9: preceding lock on line 627
> drivers/media/pci/ivtv/ivtv-fileops.c:606:3-9: preceding lock on line 689
> drivers/media/pci/ivtv/ivtv-fileops.c:648:3-9: preceding lock on line 627
> drivers/media/pci/ivtv/ivtv-fileops.c:648:3-9: preceding lock on line 689
> drivers/media/pci/ivtv/ivtv-fileops.c:692:4-10: preceding lock on line 689
> drivers/media/dvb-core/dvb_frontend.c:2897:1-7: preceding lock on line 2776
> drivers/media/dvb-core/dvb_frontend.c:2897:1-7: preceding lock on line 2786
> drivers/media/dvb-core/dvb_frontend.c:2897:1-7: preceding lock on line 2809
> drivers/media/dvb-frontends/stv090x.c:799:1-7: preceding lock on line 768
> drivers/media/usb/go7007/go7007-i2c.c:125:1-7: preceding lock on line 61
> drivers/media/rc/imon.c:1167:1-7: preceding lock on line 1153
> drivers/media/pci/cx18/cx18-scb.h:261:22-29: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_cmds.h:77:5-9: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_cmds.h:85:5-16: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_cmds.h:154:5-9: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_cmds.h:171:5-9: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_cmds.h:180:5-9: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_cmds.h:189:5-9: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_cmds.h:201:5-9: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_cmds.h:220:5-9: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_cmds.h:230:5-16: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_helper.h:764:5-15: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_helper.h:1008:43-60: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_helper.h:1014:36-46: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_helper.h:1041:5-15: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_helper.h:1088:39-51: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_helper.h:1093:5-22: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_helper.h:1144:4-8: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_helper.h:1239:4-8: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_helper.h:1267:5-9: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/qcom/venus/hfi_helper.h:1272:4-13: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/common/siano/smscoreapi.h:619:5-13: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/common/siano/smscoreapi.h:669:6-13: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/common/siano/smscoreapi.h:1049:4-8: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/common/siano/smscoreapi.h:1055:4-8: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/dvb-frontends/mxl5xx_defs.h:171:4-8: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/dvb-frontends/mxl5xx_defs.h:182:4-8: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/allegro-dvt/nal-hevc.h:102:14-22: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/media/platform/xilinx/xilinx-dma.h:100:19-22: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
> drivers/staging/media/atomisp/pci/atomisp_tpg.h:30:18-22: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
>
> CI tested:
> https://gitlab.freedesktop.org/linux-media/media-staging/-/commit/055b5211c68e721c3a7090be5373cf44859da1a7/pipelines?ref=ribalda%2Ftest-cocci
>
> Signed-off-by: Ricardo Ribalda <ribalda@chromium.org>
> ---
> Ricardo Ribalda (35):
> media: pci: mgb4: Refactor struct resources
> media: stb0899: Remove unreacheable code
> media: uvcvideo: Refactor iterators
> media: uvcvideo: Use max() macro
> media: go7007: Use min and max macros
> media: stm32-dcmipp: Remove redundant printk
> media: staging: sun6i-isp: Remove redundant printk
> media: dvb-frontends: tda18271c2dd: Remove casting during div
> media: v4l: async: refactor v4l2_async_create_ancillary_links
> staging: media: tegra-video: Use swap macro
> media: s2255: Use refcount_t instead of atomic_t for num_channels
> media: platform: mtk-mdp3: Use refcount_t for job_count
> media: common: saa7146: Use min macro
> media: dvb-frontends: drx39xyj: Use min macro
> media: netup_unidvb: Use min macro
> media: au0828: Use min macro
> media: flexcop-usb: Use min macro
> media: gspca: cpia1: Use min macro
> media: stk1160: Use min macro
> media: tegra-vde: Refactor timeout handling
> media: venus: Use div64_u64
> media: i2c: st-mipid02: Use the correct div function
> media: dvb-frontends: tda10048: Use the right div
> media: tc358746: Use the correct div_ function
> media: venus: Use the correct div_ function
> media: venus: Refator return path
> media: dib0700: Refator return path
> media: usb: cx231xx: Refator return path
> media: i2c: rdacm20: Refator return path
> media: i2c: et8ek8: Refator return path
> media: cx231xx: Refator return path
> media: si4713: Refator return path
> media: ttpci: Refator return path
> media: hdpvr: Refator return path
> media: venus: Refator return path
>
> drivers/media/common/saa7146/saa7146_hlp.c | 8 +++----
> drivers/media/dvb-frontends/drx39xyj/drxj.c | 9 +++-----
> drivers/media/dvb-frontends/stb0899_drv.c | 5 -----
> drivers/media/dvb-frontends/tda10048.c | 3 +--
> drivers/media/dvb-frontends/tda18271c2dd.c | 4 ++--
> drivers/media/i2c/et8ek8/et8ek8_driver.c | 4 +++-
> drivers/media/i2c/rdacm20.c | 5 ++++-
> drivers/media/i2c/st-mipid02.c | 2 +-
> drivers/media/i2c/tc358746.c | 3 +--
> drivers/media/pci/mgb4/mgb4_core.c | 4 ++--
> drivers/media/pci/mgb4/mgb4_regs.c | 2 +-
> drivers/media/pci/netup_unidvb/netup_unidvb_i2c.c | 2 +-
> drivers/media/pci/ttpci/budget-core.c | 5 ++++-
> .../media/platform/mediatek/mdp3/mtk-mdp3-cmdq.c | 10 ++++-----
> .../media/platform/mediatek/mdp3/mtk-mdp3-core.c | 6 ++---
> .../media/platform/mediatek/mdp3/mtk-mdp3-core.h | 2 +-
> .../media/platform/mediatek/mdp3/mtk-mdp3-m2m.c | 6 ++---
> drivers/media/platform/nvidia/tegra-vde/h264.c | 6 ++---
> drivers/media/platform/qcom/venus/vdec.c | 15 +++++++------
> drivers/media/platform/qcom/venus/venc.c | 19 +++++++++-------
> .../platform/st/stm32/stm32-dcmipp/dcmipp-core.c | 5 +----
> drivers/media/radio/si4713/radio-usb-si4713.c | 8 +++++--
> drivers/media/usb/au0828/au0828-video.c | 5 +----
> drivers/media/usb/b2c2/flexcop-usb.c | 5 +----
> drivers/media/usb/cx231xx/cx231xx-i2c.c | 16 +++++++++----
> drivers/media/usb/cx231xx/cx231xx-video.c | 10 +++++++--
> drivers/media/usb/dvb-usb/dib0700_core.c | 4 +++-
> drivers/media/usb/go7007/go7007-fw.c | 4 ++--
> drivers/media/usb/gspca/cpia1.c | 6 ++---
> drivers/media/usb/hdpvr/hdpvr-control.c | 4 +++-
> drivers/media/usb/s2255/s2255drv.c | 20 ++++++++---------
> drivers/media/usb/stk1160/stk1160-video.c | 10 ++-------
> drivers/media/usb/uvc/uvc_ctrl.c | 26 ++++++++++++----------
> drivers/media/v4l2-core/v4l2-async.c | 8 +++----
> drivers/staging/media/sunxi/sun6i-isp/sun6i_isp.c | 1 -
> drivers/staging/media/tegra-video/tegra20.c | 9 ++------
> 36 files changed, 132 insertions(+), 129 deletions(-)
> ---
> base-commit: 71b3ed53b08d87212fbbe51bdc3bf44eb8c462f8
> change-id: 20240415-fix-cocci-2df3ef22a6f7
>
> Best regards,
--
Regards,
Laurent Pinchart
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2024-04-15 19:54 UTC|newest]
Thread overview: 160+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-15 19:34 [PATCH 00/35] media: Fix coccinelle warning/errors Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-15 19:34 ` [PATCH 01/35] media: pci: mgb4: Refactor struct resources Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-16 11:30 ` Martin Tůma
2024-04-16 11:30 ` Martin Tůma
2024-04-15 19:34 ` [PATCH 02/35] media: stb0899: Remove unreacheable code Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-16 7:32 ` Dan Carpenter
2024-04-16 7:32 ` Dan Carpenter
2024-04-15 19:34 ` [PATCH 03/35] media: uvcvideo: Refactor iterators Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-16 4:39 ` Sergey Senozhatsky
2024-04-16 4:39 ` Sergey Senozhatsky
2024-04-16 5:08 ` Sergey Senozhatsky
2024-04-16 5:08 ` Sergey Senozhatsky
2024-04-16 7:03 ` Markus Elfring
2024-04-16 7:03 ` Markus Elfring
2024-04-16 7:15 ` Greg Kroah-Hartman
2024-04-16 7:15 ` Greg Kroah-Hartman
2024-04-18 11:04 ` Laurent Pinchart
2024-04-18 11:04 ` Laurent Pinchart
2024-04-15 19:34 ` [PATCH 04/35] media: uvcvideo: Use max() macro Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-15 21:06 ` Kieran Bingham
2024-04-15 21:06 ` Kieran Bingham
2024-04-16 4:41 ` Sergey Senozhatsky
2024-04-16 4:41 ` Sergey Senozhatsky
2024-04-18 10:54 ` Laurent Pinchart
2024-04-18 10:54 ` Laurent Pinchart
2024-04-15 19:34 ` [PATCH 05/35] media: go7007: Use min and max macros Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-15 19:34 ` [PATCH 06/35] media: stm32-dcmipp: Remove redundant printk Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-16 7:30 ` Dan Carpenter
2024-04-16 7:30 ` Dan Carpenter
2024-04-15 19:34 ` [PATCH 07/35] media: staging: sun6i-isp: " Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-15 20:43 ` Jernej Škrabec
2024-04-15 20:43 ` Jernej Škrabec
2024-04-16 7:37 ` Dan Carpenter
2024-04-16 7:37 ` Dan Carpenter
2024-04-15 19:34 ` [PATCH 08/35] media: dvb-frontends: tda18271c2dd: Remove casting during div Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-15 19:34 ` [PATCH 09/35] media: v4l: async: refactor v4l2_async_create_ancillary_links Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-16 7:44 ` Dan Carpenter
2024-04-16 7:44 ` Dan Carpenter
2024-04-15 19:34 ` [PATCH 10/35] staging: media: tegra-video: Use swap macro Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-16 2:03 ` Luca Ceresoli
2024-04-16 2:03 ` Luca Ceresoli
2024-04-15 19:34 ` [PATCH 11/35] media: s2255: Use refcount_t instead of atomic_t for num_channels Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-23 12:43 ` Bryan O'Donoghue
2024-04-23 12:43 ` Bryan O'Donoghue
2024-04-23 13:22 ` Dan Carpenter
2024-04-23 13:22 ` Dan Carpenter
2024-04-15 19:34 ` [PATCH 12/35] media: platform: mtk-mdp3: Use refcount_t for job_count Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-23 12:46 ` Bryan O'Donoghue
2024-04-23 12:46 ` Bryan O'Donoghue
2024-04-15 19:34 ` [PATCH 13/35] media: common: saa7146: Use min macro Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-23 12:49 ` Bryan O'Donoghue
2024-04-23 12:49 ` Bryan O'Donoghue
2024-04-15 19:34 ` [PATCH 14/35] media: dvb-frontends: drx39xyj: " Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-23 16:40 ` Bryan O'Donoghue
2024-04-23 16:40 ` Bryan O'Donoghue
2024-04-15 19:34 ` [PATCH 15/35] media: netup_unidvb: " Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-15 19:34 ` [PATCH 16/35] media: au0828: " Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-15 19:34 ` [PATCH 17/35] media: flexcop-usb: " Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-15 19:34 ` [PATCH 18/35] media: gspca: cpia1: " Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-15 19:34 ` [PATCH 19/35] media: stk1160: " Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-23 16:42 ` Bryan O'Donoghue
2024-04-23 16:42 ` Bryan O'Donoghue
2024-04-15 19:34 ` [PATCH 20/35] media: tegra-vde: Refactor timeout handling Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-15 19:34 ` [PATCH 21/35] media: venus: Use div64_u64 Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-16 7:57 ` Dan Carpenter
2024-04-16 7:57 ` Dan Carpenter
2024-04-15 19:34 ` [PATCH 22/35] media: i2c: st-mipid02: Use the correct div function Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-17 12:55 ` Benjamin Mugnier
2024-04-17 12:55 ` Benjamin Mugnier
2024-04-15 19:34 ` [PATCH 23/35] media: dvb-frontends: tda10048: Use the right div Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-16 10:10 ` Dan Carpenter
2024-04-16 10:10 ` Dan Carpenter
2024-04-16 10:27 ` Dan Carpenter
2024-04-16 10:27 ` Dan Carpenter
2024-04-23 9:55 ` Dan Carpenter
2024-04-23 9:55 ` Dan Carpenter
2024-04-29 11:39 ` Ricardo Ribalda
2024-04-29 11:39 ` Ricardo Ribalda
2024-04-16 10:39 ` Ricardo Ribalda
2024-04-16 10:39 ` Ricardo Ribalda
2024-04-16 10:56 ` Dan Carpenter
2024-04-16 10:56 ` Dan Carpenter
2024-04-15 19:34 ` [PATCH 24/35] media: tc358746: Use the correct div_ function Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-15 19:34 ` [PATCH 25/35] media: venus: " Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-17 8:36 ` Dan Carpenter
2024-04-17 8:36 ` Dan Carpenter
2024-04-15 19:34 ` [PATCH 26/35] media: venus: Refator return path Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-23 16:49 ` Bryan O'Donoghue
2024-04-23 16:49 ` Bryan O'Donoghue
2024-04-15 19:34 ` [PATCH 27/35] media: dib0700: " Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-15 19:34 ` [PATCH 28/35] media: usb: cx231xx: " Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-15 19:34 ` [PATCH 29/35] media: i2c: rdacm20: " Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-15 19:34 ` [PATCH 30/35] media: i2c: et8ek8: " Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-15 19:34 ` [PATCH 31/35] media: cx231xx: " Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-15 19:34 ` [PATCH 32/35] media: si4713: " Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-15 19:34 ` [PATCH 33/35] media: ttpci: " Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-15 19:34 ` [PATCH 34/35] media: hdpvr: " Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-15 19:34 ` [PATCH 35/35] media: venus: " Ricardo Ribalda
2024-04-15 19:34 ` Ricardo Ribalda
2024-04-15 19:53 ` Laurent Pinchart [this message]
2024-04-15 19:53 ` [PATCH 00/35] media: Fix coccinelle warning/errors Laurent Pinchart
2024-04-15 19:58 ` Ricardo Ribalda
2024-04-15 19:58 ` Ricardo Ribalda
2024-04-15 20:33 ` Niklas Söderlund
2024-04-15 20:33 ` Niklas Söderlund
2024-04-15 21:16 ` Ricardo Ribalda
2024-04-15 21:16 ` Ricardo Ribalda
2024-04-15 21:38 ` Niklas Söderlund
2024-04-15 21:38 ` Niklas Söderlund
2024-04-16 7:08 ` Laurent Pinchart
2024-04-16 7:08 ` Laurent Pinchart
2024-04-16 7:59 ` Julia Lawall
2024-04-16 7:59 ` Julia Lawall
2024-04-16 8:47 ` Dan Carpenter
2024-04-16 8:47 ` Dan Carpenter
2024-04-17 15:51 ` Laurent Pinchart
2024-04-17 15:51 ` Laurent Pinchart
2024-04-17 16:19 ` Ricardo Ribalda
2024-04-17 16:19 ` Ricardo Ribalda
2024-04-18 10:53 ` Laurent Pinchart
2024-04-18 10:53 ` Laurent Pinchart
2024-04-18 14:51 ` Ricardo Ribalda
2024-04-18 14:51 ` Ricardo Ribalda
2024-04-18 15:46 ` Laurent Pinchart
2024-04-18 15:46 ` Laurent Pinchart
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240415195348.GD22954@pendragon.ideasonboard.com \
--to=laurent.pinchart@ideasonboard.com \
--cc=alain.volmat@foss.st.com \
--cc=alexandre.torgue@foss.st.com \
--cc=andersson@kernel.org \
--cc=angelogioacchino.delregno@collabora.com \
--cc=aospan@netup.ru \
--cc=benjamin.mugnier@foss.st.com \
--cc=bryan.odonoghue@linaro.org \
--cc=digetx@gmail.com \
--cc=ezequiel@vanguardiasur.com.ar \
--cc=gregkh@linuxfoundation.org \
--cc=hugues.fruchet@foss.st.com \
--cc=hverkuil-cisco@xs4all.nl \
--cc=hverkuil@xs4all.nl \
--cc=jacopo+renesas@jmondi.org \
--cc=jernej.skrabec@gmail.com \
--cc=jonathanh@nvidia.com \
--cc=kieran.bingham+renesas@ideasonboard.com \
--cc=konrad.dybcio@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-staging@lists.linux.dev \
--cc=linux-stm32@st-md-mailman.stormreply.com \
--cc=linux-sunxi@lists.linux.dev \
--cc=linux-tegra@vger.kernel.org \
--cc=luca.ceresoli@bootlin.com \
--cc=martin.tuma@digiteqautomotive.com \
--cc=matthias.bgg@gmail.com \
--cc=mchehab@kernel.org \
--cc=mcoquelin.stm32@gmail.com \
--cc=niklas.soderlund+renesas@ragnatech.se \
--cc=paul.kocialkowski@bootlin.com \
--cc=pavel@ucw.cz \
--cc=quic_vgarodia@quicinc.com \
--cc=ribalda@chromium.org \
--cc=sakari.ailus@linux.intel.com \
--cc=samuel@sholland.org \
--cc=serjk@netup.ru \
--cc=skomatineni@nvidia.com \
--cc=stanimir.k.varbanov@gmail.com \
--cc=sylvain.petinot@foss.st.com \
--cc=thierry.reding@gmail.com \
--cc=wens@csie.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.