[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <08fab56b-c938-4509-940e-eab503f87740@notapiano>
Date: Fri, 21 Jul 2023 10:52:23 -0400
From: Nícolas F. R. A. Prado
<nfraprado@...labora.com>
To: Hans Verkuil <hverkuil-cisco@...all.nl>
Cc: Matthias Brugger <matthias.bgg@...il.com>,
AngeloGioacchino Del Regno
<angelogioacchino.delregno@...labora.com>, kernel@...labora.com,
Andrew-CT Chen <andrew-ct.chen@...iatek.com>,
Mauro Carvalho Chehab <mchehab@...nel.org>,
Tiffany Lin <tiffany.lin@...iatek.com>,
Yunfei Dong <yunfei.dong@...iatek.com>,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
linux-media@...r.kernel.org, linux-mediatek@...ts.infradead.org
Subject: Re: [PATCH v5 5/7] media: mediatek: vcodec: Read HW active status
from syscon
On Fri, Jul 21, 2023 at 01:44:10PM +0200, Hans Verkuil wrote:
> On 30/06/2023 17:14, Nícolas F. R. A. Prado wrote:
[..]
> > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h
> > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h
> > @@ -489,6 +489,7 @@ struct mtk_vcodec_dev {
> > void __iomem *reg_base[NUM_MAX_VCODEC_REG_BASE];
> > const struct mtk_vcodec_dec_pdata *vdec_pdata;
> > const struct mtk_vcodec_enc_pdata *venc_pdata;
> > + struct regmap *vdecsys_regmap;
>
> You forgot to add the kerneldoc documentation for this new field.
>
> If you just give me the documentation of this field then I can modify the
> patch. That's actually easier for me.
Sorry about that. Seems like I'm not getting a kerneldoc warning due to that,
I'll look into why so I can catch this next time.
This is the documentation:
@vdecsys_regmap: VDEC_SYS register space passed through syscon
Or if a patch that applies on top would make it easier to fixup:
cat 0001-media-mediatek-vcodec-Add-missing-kerneldoc-for-vdec.patch
>From ee6a6d619dbe60c8f4947188a9b9bbeafc3616f7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?N=C3=ADcolas=20F=2E=20R=2E=20A=2E=20Prado?=
<nfraprado@...labora.com>
Date: Fri, 21 Jul 2023 16:33:54 +0200
Subject: [PATCH] media: mediatek: vcodec: Add missing kerneldoc for
vdecsys_regmap
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Signed-off-by: Nícolas F. R. A. Prado <nfraprado@...labora.com>
---
drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h
index 0b430936f67d..c38eb62bc72a 100644
--- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h
+++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h
@@ -441,6 +441,7 @@ struct mtk_vcodec_enc_pdata {
* @reg_base: Mapped address of MTK Vcodec registers.
* @vdec_pdata: decoder IC-specific data
* @venc_pdata: encoder IC-specific data
+ * @vdecsys_regmap: VDEC_SYS register space passed through syscon
*
* @fw_handler: used to communicate with the firmware.
* @id_counter: used to identify current opened instance
--
2.30.2
Thanks,
Nícolas
Powered by blists - more mailing lists