Message-ID:
 <PAXPR04MB8254A8FDD7B30CEEF4117CFDE722A@PAXPR04MB8254.eurprd04.prod.outlook.com>
Date: Tue, 5 Aug 2025 01:34:44 +0000
From: Ming Qian <ming.qian@....com>
To: Jacopo Mondi <jacopo.mondi@...asonboard.com>, Mauro Carvalho Chehab
	<mchehab@...nel.org>, Devarsh Thakkar <devarsht@...com>, Benoit Parrot
	<bparrot@...com>, Hans Verkuil <hverkuil@...nel.org>, Mike Isely
	<isely@...ox.com>, Laurent Pinchart <laurent.pinchart@...asonboard.com>, Hans
 de Goede <hansg@...nel.org>, Parthiban Veerasooran
	<parthiban.veerasooran@...rochip.com>, Christian Gromm
	<christian.gromm@...rochip.com>, Greg Kroah-Hartman
	<gregkh@...uxfoundation.org>, Alex Shi <alexs@...nel.org>, Yanteng Si
	<si.yanteng@...ux.dev>, Dongliang Mu <dzm91@...t.edu.cn>, Jonathan Corbet
	<corbet@....net>, Tomasz Figa <tfiga@...omium.org>, Marek Szyprowski
	<m.szyprowski@...sung.com>, Andy Walls <awalls@...metrocast.net>, Michael
 Tretter <m.tretter@...gutronix.de>, Pengutronix Kernel Team
	<kernel@...gutronix.de>, Bin Liu <bin.liu@...iatek.com>, Matthias Brugger
	<matthias.bgg@...il.com>, AngeloGioacchino Del Regno
	<angelogioacchino.delregno@...labora.com>, Dmitry Osipenko
	<digetx@...il.com>, Thierry Reding <thierry.reding@...il.com>, Jonathan
 Hunter <jonathanh@...dia.com>, Mirela Rabulea <mirela.rabulea@....com>, Shawn
 Guo <shawnguo@...nel.org>, Sascha Hauer <s.hauer@...gutronix.de>, Fabio
 Estevam <festevam@...il.com>, Kieran Bingham
	<kieran.bingham+renesas@...asonboard.com>, Michal Simek
	<michal.simek@....com>, Eagle Zhou <eagle.zhou@....com>, "Xavier Roumegue
 (OSS)" <xavier.roumegue@....nxp.com>, Philipp Zabel <p.zabel@...gutronix.de>,
	Vikash Garodia <quic_vgarodia@...cinc.com>, Dikshita Agarwal
	<quic_dikshita@...cinc.com>, Abhinav Kumar <abhinav.kumar@...ux.dev>, Bryan
 O'Donoghue <bryan.odonoghue@...aro.org>, Sylwester Nawrocki
	<sylvester.nawrocki@...il.com>, Jernej Skrabec <jernej.skrabec@...il.com>,
	Chen-Yu Tsai <wens@...e.org>, Samuel Holland <samuel@...lland.org>, Daniel
 Almeida <daniel.almeida@...labora.com>, Neil Armstrong
	<neil.armstrong@...aro.org>, Kevin Hilman <khilman@...libre.com>, Jerome
 Brunet <jbrunet@...libre.com>, Martin Blumenstingl
	<martin.blumenstingl@...glemail.com>, Nas Chung <nas.chung@...psnmedia.com>,
	Jackson Lee <jackson.lee@...psnmedia.com>, Minghsiu Tsai
	<minghsiu.tsai@...iatek.com>, Houlong Wei <houlong.wei@...iatek.com>,
	Andrew-CT Chen <andrew-ct.chen@...iatek.com>, Tiffany Lin
	<tiffany.lin@...iatek.com>, Yunfei Dong <yunfei.dong@...iatek.com>, Geert
 Uytterhoeven <geert+renesas@...der.be>, Magnus Damm <magnus.damm@...il.com>,
	Mikhail Ulyanov <mikhail.ulyanov@...entembedded.com>, Jacob Chen
	<jacob-chen@...wrt.com>, Ezequiel Garcia <ezequiel@...guardiasur.com.ar>,
	Heiko Stuebner <heiko@...ech.de>, Detlev Casanova
	<detlev.casanova@...labora.com>, Krzysztof Kozlowski <krzk@...nel.org>, Alim
 Akhtar <alim.akhtar@...sung.com>, Sylwester Nawrocki
	<s.nawrocki@...sung.com>, Łukasz Stelmach
	<l.stelmach@...sung.com>, Andrzej Pietrasiewicz <andrzejtp2010@...il.com>,
	Jacek Anaszewski <jacek.anaszewski@...il.com>, Andrzej Hajda
	<andrzej.hajda@...el.com>, Fabien Dessenne <fabien.dessenne@...s.st.com>,
	Hugues Fruchet <hugues.fruchet@...s.st.com>, Jean-Christophe Trotin
	<jean-christophe.trotin@...s.st.com>, Maxime Coquelin
	<mcoquelin.stm32@...il.com>, Alexandre Torgue <alexandre.torgue@...s.st.com>,
	Nicolas Dufresne <nicolas.dufresne@...labora.com>, Benjamin Gaignard
	<benjamin.gaignard@...labora.com>, Steve Longerbeam <slongerbeam@...il.com>,
	Maxime Ripard <mripard@...nel.org>, Paul Kocialkowski <paulk@...-base.io>,
	Niklas Söderlund <niklas.soderlund@...natech.se>, Robert
 Foss <rfoss@...nel.org>, Todor Tomov <todor.too@...il.com>, Vladimir
 Zapolskiy <vladimir.zapolskiy@...aro.org>, Corentin Labbe
	<clabbe@...libre.com>, Sakari Ailus <sakari.ailus@...ux.intel.com>, Bingbu
 Cao <bingbu.cao@...el.com>, Tianshu Qiu <tian.shu.qiu@...el.com>, Stanislaw
 Gruszka <stanislaw.gruszka@...ux.intel.com>
CC: "linux-media@...r.kernel.org" <linux-media@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-staging@...ts.linux.dev" <linux-staging@...ts.linux.dev>,
	"linux-doc@...r.kernel.org" <linux-doc@...r.kernel.org>,
	"linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>, "linux-mediatek@...ts.infradead.org"
	<linux-mediatek@...ts.infradead.org>, "linux-tegra@...r.kernel.org"
	<linux-tegra@...r.kernel.org>, "imx@...ts.linux.dev" <imx@...ts.linux.dev>,
	"linux-renesas-soc@...r.kernel.org" <linux-renesas-soc@...r.kernel.org>,
	"linux-arm-msm@...r.kernel.org" <linux-arm-msm@...r.kernel.org>,
	"linux-samsung-soc@...r.kernel.org" <linux-samsung-soc@...r.kernel.org>,
	"linux-sunxi@...ts.linux.dev" <linux-sunxi@...ts.linux.dev>,
	"linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>,
	"linux-amlogic@...ts.infradead.org" <linux-amlogic@...ts.infradead.org>,
	"linux-rockchip@...ts.infradead.org" <linux-rockchip@...ts.infradead.org>,
	"linux-stm32@...md-mailman.stormreply.com"
	<linux-stm32@...md-mailman.stormreply.com>,
	"mjpeg-users@...ts.sourceforge.net" <mjpeg-users@...ts.sourceforge.net>
Subject: RE: [EXT] [PATCH 13/65] media: amphion: Make some vpu_v4l2 functions
 static



>-----Original Message-----
>From: Jacopo Mondi <jacopo.mondi@...asonboard.com>
>Sent: Saturday, August 2, 2025 5:23 PM
>To: Mauro Carvalho Chehab <mchehab@...nel.org>; Devarsh Thakkar
><devarsht@...com>; Benoit Parrot <bparrot@...com>; Hans Verkuil
><hverkuil@...nel.org>; Mike Isely <isely@...ox.com>; Laurent Pinchart
><laurent.pinchart@...asonboard.com>; Hans de Goede <hansg@...nel.org>;
>Parthiban Veerasooran <parthiban.veerasooran@...rochip.com>; Christian
>Gromm <christian.gromm@...rochip.com>; Greg Kroah-Hartman
><gregkh@...uxfoundation.org>; Alex Shi <alexs@...nel.org>; Yanteng Si
><si.yanteng@...ux.dev>; Dongliang Mu <dzm91@...t.edu.cn>; Jonathan
>Corbet <corbet@....net>; Tomasz Figa <tfiga@...omium.org>; Marek
>Szyprowski <m.szyprowski@...sung.com>; Andy Walls
><awalls@...metrocast.net>; Michael Tretter <m.tretter@...gutronix.de>;
>Pengutronix Kernel Team <kernel@...gutronix.de>; Bin Liu
><bin.liu@...iatek.com>; Matthias Brugger <matthias.bgg@...il.com>;
>AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>;
>Dmitry Osipenko <digetx@...il.com>; Thierry Reding
><thierry.reding@...il.com>; Jonathan Hunter <jonathanh@...dia.com>;
>Mirela Rabulea <mirela.rabulea@....com>; Shawn Guo
><shawnguo@...nel.org>; Sascha Hauer <s.hauer@...gutronix.de>; Fabio
>Estevam <festevam@...il.com>; Kieran Bingham
><kieran.bingham+renesas@...asonboard.com>; Michal Simek
><michal.simek@....com>; Ming Qian <ming.qian@....com>; Eagle Zhou
><eagle.zhou@....com>; Xavier Roumegue (OSS)
><xavier.roumegue@....nxp.com>; Philipp Zabel <p.zabel@...gutronix.de>;
>Vikash Garodia <quic_vgarodia@...cinc.com>; Dikshita Agarwal
><quic_dikshita@...cinc.com>; Abhinav Kumar <abhinav.kumar@...ux.dev>;
>Bryan O'Donoghue <bryan.odonoghue@...aro.org>; Sylwester Nawrocki
><sylvester.nawrocki@...il.com>; Jernej Skrabec <jernej.skrabec@...il.com>;
>Chen-Yu Tsai <wens@...e.org>; Samuel Holland <samuel@...lland.org>;
>Daniel Almeida <daniel.almeida@...labora.com>; Neil Armstrong
><neil.armstrong@...aro.org>; Kevin Hilman <khilman@...libre.com>; Jerome
>Brunet <jbrunet@...libre.com>; Martin Blumenstingl
><martin.blumenstingl@...glemail.com>; Nas Chung
><nas.chung@...psnmedia.com>; Jackson Lee
><jackson.lee@...psnmedia.com>; Minghsiu Tsai
><minghsiu.tsai@...iatek.com>; Houlong Wei <houlong.wei@...iatek.com>;
>Andrew-CT Chen <andrew-ct.chen@...iatek.com>; Tiffany Lin
><tiffany.lin@...iatek.com>; Yunfei Dong <yunfei.dong@...iatek.com>;
>Geert Uytterhoeven <geert+renesas@...der.be>; Magnus Damm
><magnus.damm@...il.com>; Mikhail Ulyanov
><mikhail.ulyanov@...entembedded.com>; Jacob Chen <jacob-
>chen@...wrt.com>; Ezequiel Garcia <ezequiel@...guardiasur.com.ar>; Heiko
>Stuebner <heiko@...ech.de>; Detlev Casanova
><detlev.casanova@...labora.com>; Krzysztof Kozlowski <krzk@...nel.org>;
>Alim Akhtar <alim.akhtar@...sung.com>; Sylwester Nawrocki
><s.nawrocki@...sung.com>; Łukasz Stelmach <l.stelmach@...sung.com>;
>Andrzej Pietrasiewicz <andrzejtp2010@...il.com>; Jacek Anaszewski
><jacek.anaszewski@...il.com>; Andrzej Hajda <andrzej.hajda@...el.com>;
>Fabien Dessenne <fabien.dessenne@...s.st.com>; Hugues Fruchet
><hugues.fruchet@...s.st.com>; Jean-Christophe Trotin <jean-
>christophe.trotin@...s.st.com>; Maxime Coquelin
><mcoquelin.stm32@...il.com>; Alexandre Torgue
><alexandre.torgue@...s.st.com>; Nicolas Dufresne
><nicolas.dufresne@...labora.com>; Benjamin Gaignard
><benjamin.gaignard@...labora.com>; Steve Longerbeam
><slongerbeam@...il.com>; Maxime Ripard <mripard@...nel.org>; Paul
>Kocialkowski <paulk@...-base.io>; Niklas Söderlund
><niklas.soderlund@...natech.se>; Robert Foss <rfoss@...nel.org>; Todor
>Tomov <todor.too@...il.com>; Vladimir Zapolskiy
><vladimir.zapolskiy@...aro.org>; Corentin Labbe <clabbe@...libre.com>;
>Sakari Ailus <sakari.ailus@...ux.intel.com>; Bingbu Cao
><bingbu.cao@...el.com>; Tianshu Qiu <tian.shu.qiu@...el.com>; Stanislaw
>Gruszka <stanislaw.gruszka@...ux.intel.com>
>Cc: linux-media@...r.kernel.org; linux-kernel@...r.kernel.org; linux-
>staging@...ts.linux.dev; linux-doc@...r.kernel.org; linux-arm-
>kernel@...ts.infradead.org; linux-mediatek@...ts.infradead.org; linux-
>tegra@...r.kernel.org; imx@...ts.linux.dev; linux-renesas-soc@...r.kernel.org;
>linux-arm-msm@...r.kernel.org; linux-samsung-soc@...r.kernel.org; linux-
>sunxi@...ts.linux.dev; linux-usb@...r.kernel.org; linux-
>amlogic@...ts.infradead.org; linux-rockchip@...ts.infradead.org; linux-
>stm32@...md-mailman.stormreply.com; mjpeg-users@...ts.sourceforge.net;
>Jacopo Mondi <jacopo.mondi@...asonboard.com>
>Subject: [EXT] [PATCH 13/65] media: amphion: Make some vpu_v4l2 functions
>static
>
>Caution: This is an external email. Please take care when clicking links or
>opening attachments. When in doubt, report the message using the 'Report
>this email' button
>
>
>From: Laurent Pinchart <laurent.pinchart@...asonboard.com>
>
>Some functions defined in vpu_v4l2.c are never used outside of that
>compilation unit. Make them static.
>
>Signed-off-by: Laurent Pinchart <laurent.pinchart@...asonboard.com>
>Signed-off-by: Jacopo Mondi <jacopo.mondi@...asonboard.com>

Reviewed-by: Ming Qian <ming.qian@....nxp.com>

>---
> drivers/media/platform/amphion/vpu_v4l2.c | 12 +++++++++---
>drivers/media/platform/amphion/vpu_v4l2.h |  8 --------
> 2 files changed, 9 insertions(+), 11 deletions(-)
>
>diff --git a/drivers/media/platform/amphion/vpu_v4l2.c
>b/drivers/media/platform/amphion/vpu_v4l2.c
>index
>74668fa362e24fd34829b500e99c8455a9413fc1..306d94e0f8e79faaacfa35b28e
>5786860f7bd1ca 100644
>--- a/drivers/media/platform/amphion/vpu_v4l2.c
>+++ b/drivers/media/platform/amphion/vpu_v4l2.c
>@@ -24,6 +24,11 @@
> #include "vpu_msgs.h"
> #include "vpu_helpers.h"
>
>+static char *vpu_type_name(u32 type)
>+{
>+       return V4L2_TYPE_IS_OUTPUT(type) ? "output" : "capture"; }
>+
> void vpu_inst_lock(struct vpu_inst *inst)  {
>        mutex_lock(&inst->lock);
>@@ -42,7 +47,7 @@ dma_addr_t vpu_get_vb_phy_addr(struct vb2_buffer
>*vb, u32 plane_no)
>                        vb->planes[plane_no].data_offset;  }
>
>-unsigned int vpu_get_vb_length(struct vb2_buffer *vb, u32 plane_no)
>+static unsigned int vpu_get_vb_length(struct vb2_buffer *vb, u32
>+plane_no)
> {
>        if (plane_no >= vb->num_planes)
>                return 0;
>@@ -81,7 +86,7 @@ void vpu_v4l2_set_error(struct vpu_inst *inst)
>        vpu_inst_unlock(inst);
> }
>
>-int vpu_notify_eos(struct vpu_inst *inst)
>+static int vpu_notify_eos(struct vpu_inst *inst)
> {
>        static const struct v4l2_event ev = {
>                .id = 0,
>@@ -573,7 +578,8 @@ static void vpu_vb2_buf_finish(struct vb2_buffer *vb)
>                call_void_vop(inst, on_queue_empty, q->type);  }
>
>-void vpu_vb2_buffers_return(struct vpu_inst *inst, unsigned int type, enum
>vb2_buffer_state state)
>+static void vpu_vb2_buffers_return(struct vpu_inst *inst, unsigned int type,
>+                                  enum vb2_buffer_state state)
> {
>        struct vb2_v4l2_buffer *buf;
>
>diff --git a/drivers/media/platform/amphion/vpu_v4l2.h
>b/drivers/media/platform/amphion/vpu_v4l2.h
>index
>56f2939fa84d07b7ea07e889ab5ad3bb7ca9ab5c..4a87b06ae520306ede356b6f
>3309d8b4a67c204a 100644
>--- a/drivers/media/platform/amphion/vpu_v4l2.h
>+++ b/drivers/media/platform/amphion/vpu_v4l2.h
>@@ -26,15 +26,12 @@ void vpu_skip_frame(struct vpu_inst *inst, int count);
>struct vb2_v4l2_buffer *vpu_find_buf_by_sequence(struct vpu_inst *inst, u32
>type, u32 sequence);  struct vb2_v4l2_buffer *vpu_find_buf_by_idx(struct
>vpu_inst *inst, u32 type, u32 idx);  void vpu_v4l2_set_error(struct vpu_inst
>*inst); -int vpu_notify_eos(struct vpu_inst *inst);  int
>vpu_notify_source_change(struct vpu_inst *inst);  int
>vpu_set_last_buffer_dequeued(struct vpu_inst *inst, bool eos); -void
>vpu_vb2_buffers_return(struct vpu_inst *inst, unsigned int type, enum
>vb2_buffer_state state);  int vpu_get_num_buffers(struct vpu_inst *inst, u32
>type);  bool vpu_is_source_empty(struct vpu_inst *inst);
>
> dma_addr_t vpu_get_vb_phy_addr(struct vb2_buffer *vb, u32 plane_no); -
>unsigned int vpu_get_vb_length(struct vb2_buffer *vb, u32 plane_no);  static
>inline struct vpu_format *vpu_get_format(struct vpu_inst *inst, u32 type)  {
>        if (V4L2_TYPE_IS_OUTPUT(type))
>@@ -43,11 +40,6 @@ static inline struct vpu_format *vpu_get_format(struct
>vpu_inst *inst, u32 type)
>                return &inst->cap_format;  }
>
>-static inline char *vpu_type_name(u32 type) -{
>-       return V4L2_TYPE_IS_OUTPUT(type) ? "output" : "capture";
>-}
>-
> static inline int vpu_vb_is_codecconfig(struct vb2_v4l2_buffer *vbuf)  {  #ifdef
>V4L2_BUF_FLAG_CODECCONFIG
>
>--
>2.49.0

Powered by blists - more mailing lists