From: Hans Verkuil <hverkuil-cisco@xs4all.nl> To: Moudy Ho <moudy.ho@mediatek.com>, Mauro Carvalho Chehab <mchehab@kernel.org>, Rob Herring <robh+dt@kernel.org>, Matthias Brugger <matthias.bgg@gmail.com>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org> Cc: Chun-Kuang Hu <chunkuang.hu@kernel.org>, Rob Landley <rob@landley.net>, Laurent Pinchart <laurent.pinchart@ideasonboard.com>, linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, Alexandre Courbot <acourbot@chromium.org>, tfiga@chromium.org, drinkcat@chromium.org, pihsun@chromium.org, hsinyi@google.com, Benjamin Gaignard <benjamin.gaignard@collabora.com>, AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>, Project_Global_Chrome_Upstream_Group@mediatek.com, cellopoint.kai@gmail.com, Ping-Hsun Wu <ping-hsun.wu@mediatek.com>, daoyuan huang <daoyuan.huang@mediatek.com> Subject: Re: [PATCH v25 4/4] media: platform: mtk-mdp3: add MediaTek MDP3 driver Date: Thu, 18 Aug 2022 13:02:23 +0200 [thread overview] Message-ID: <b0b4ff87-355e-1910-c6d2-a9690f3d7543@xs4all.nl> (raw) In-Reply-To: <20220817095629.29911-5-moudy.ho@mediatek.com> Hi Moudy, I noticed one more thing (and it is probably better to post a v26 after all): On 17/08/2022 11:56, Moudy Ho wrote: > This patch adds driver for MediaTek's Media Data Path ver.3 (MDP3). > It provides the following functions: > color transform, format conversion, resize, crop, rotate, flip > and additional image quality enhancement. > > The MDP3 driver is mainly used for Google Chromebook products to > import the new architecture to set the HW settings as shown below: > User -> V4L2 framework > -> MDP3 driver -> SCP (setting calculations) > -> MDP3 driver -> CMDQ (GCE driver) -> HW > > Each modules' related operation control is sited in mtk-mdp3-comp.c > Each modules' register table is defined in file with "mdp_reg_" prefix > GCE related API, operation control sited in mtk-mdp3-cmdq.c > V4L2 m2m device functions are implemented in mtk-mdp3-m2m.c > Probe, power, suspend/resume, system level functions are defined in > mtk-mdp3-core.c > > v4l2-compliance 1.22.1, 32 bits, 32-bit time_t First of all, the v4l2-compliance output belongs to the cover letter, not to a commit log for a patch. More importantly, I can tell that v4l2-compliance was a prepackaged version, but you need to compile it from the git repo yourself: git clone git://linuxtv.org/v4l-utils.git cd v4l-utils ./bootstrap.sh ./configure make sudo make install Running v4l2-compliance should start with this (or something close): v4l2-compliance 1.23.0-4941, 64 bits, 64-bit time_t v4l2-compliance SHA: 71112d214762 2022-07-28 15:31:13 I need to see the SHA to confirm that you tested with a sufficiently new v4l2-compliance version. Prepackaged v4l2-compliance binaries tend to be too old, at least for the purpose of compliance checking a new driver. Regards, Hans > Compliance test for mtk-mdp3 device /dev/video0: > Driver Info: > Driver name : mtk-mdp3 > Card type : MediaTek MDP3 > Bus info : platform:14001000.mdp3-rdma0 > Driver version : 6.0.0 > Capabilities : 0x84204000 > Video Memory-to-Memory Multiplanar > Streaming > Extended Pix Format > Device Capabilities > Device Caps : 0x04204000 > Video Memory-to-Memory Multiplanar > Streaming > Extended Pix Format
WARNING: multiple messages have this Message-ID (diff)
From: Hans Verkuil <hverkuil-cisco@xs4all.nl> To: Moudy Ho <moudy.ho@mediatek.com>, Mauro Carvalho Chehab <mchehab@kernel.org>, Rob Herring <robh+dt@kernel.org>, Matthias Brugger <matthias.bgg@gmail.com>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org> Cc: Chun-Kuang Hu <chunkuang.hu@kernel.org>, Rob Landley <rob@landley.net>, Laurent Pinchart <laurent.pinchart@ideasonboard.com>, linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, Alexandre Courbot <acourbot@chromium.org>, tfiga@chromium.org, drinkcat@chromium.org, pihsun@chromium.org, hsinyi@google.com, Benjamin Gaignard <benjamin.gaignard@collabora.com>, AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>, Project_Global_Chrome_Upstream_Group@mediatek.com, cellopoint.kai@gmail.com, Ping-Hsun Wu <ping-hsun.wu@mediatek.com>, daoyuan huang <daoyuan.huang@mediatek.com> Subject: Re: [PATCH v25 4/4] media: platform: mtk-mdp3: add MediaTek MDP3 driver Date: Thu, 18 Aug 2022 13:02:23 +0200 [thread overview] Message-ID: <b0b4ff87-355e-1910-c6d2-a9690f3d7543@xs4all.nl> (raw) In-Reply-To: <20220817095629.29911-5-moudy.ho@mediatek.com> Hi Moudy, I noticed one more thing (and it is probably better to post a v26 after all): On 17/08/2022 11:56, Moudy Ho wrote: > This patch adds driver for MediaTek's Media Data Path ver.3 (MDP3). > It provides the following functions: > color transform, format conversion, resize, crop, rotate, flip > and additional image quality enhancement. > > The MDP3 driver is mainly used for Google Chromebook products to > import the new architecture to set the HW settings as shown below: > User -> V4L2 framework > -> MDP3 driver -> SCP (setting calculations) > -> MDP3 driver -> CMDQ (GCE driver) -> HW > > Each modules' related operation control is sited in mtk-mdp3-comp.c > Each modules' register table is defined in file with "mdp_reg_" prefix > GCE related API, operation control sited in mtk-mdp3-cmdq.c > V4L2 m2m device functions are implemented in mtk-mdp3-m2m.c > Probe, power, suspend/resume, system level functions are defined in > mtk-mdp3-core.c > > v4l2-compliance 1.22.1, 32 bits, 32-bit time_t First of all, the v4l2-compliance output belongs to the cover letter, not to a commit log for a patch. More importantly, I can tell that v4l2-compliance was a prepackaged version, but you need to compile it from the git repo yourself: git clone git://linuxtv.org/v4l-utils.git cd v4l-utils ./bootstrap.sh ./configure make sudo make install Running v4l2-compliance should start with this (or something close): v4l2-compliance 1.23.0-4941, 64 bits, 64-bit time_t v4l2-compliance SHA: 71112d214762 2022-07-28 15:31:13 I need to see the SHA to confirm that you tested with a sufficiently new v4l2-compliance version. Prepackaged v4l2-compliance binaries tend to be too old, at least for the purpose of compliance checking a new driver. Regards, Hans > Compliance test for mtk-mdp3 device /dev/video0: > Driver Info: > Driver name : mtk-mdp3 > Card type : MediaTek MDP3 > Bus info : platform:14001000.mdp3-rdma0 > Driver version : 6.0.0 > Capabilities : 0x84204000 > Video Memory-to-Memory Multiplanar > Streaming > Extended Pix Format > Device Capabilities > Device Caps : 0x04204000 > Video Memory-to-Memory Multiplanar > Streaming > Extended Pix Format _______________________________________________ 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:[~2022-08-18 11:02 UTC|newest] Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-08-17 9:56 [PATCH v25 0/4] media: mediatek: support mdp3 on mt8183 platform Moudy Ho 2022-08-17 9:56 ` Moudy Ho 2022-08-17 9:56 ` [PATCH v25 1/4] dt-binding: mediatek: add bindings for MediaTek MDP3 components Moudy Ho 2022-08-17 9:56 ` Moudy Ho 2022-08-22 14:31 ` Matthias Brugger 2022-08-22 14:31 ` Matthias Brugger 2022-08-22 15:10 ` moudy ho 2022-08-22 15:10 ` moudy ho 2022-08-17 9:56 ` [PATCH v25 2/4] dt-binding: mediatek: add bindings for MediaTek CCORR and WDMA Moudy Ho 2022-08-17 9:56 ` Moudy Ho 2022-08-22 14:32 ` Matthias Brugger 2022-08-22 14:32 ` Matthias Brugger 2022-08-23 2:19 ` moudy ho 2022-08-23 2:19 ` moudy ho 2022-08-17 9:56 ` [PATCH v25 3/4] arm64: dts: mt8183: add MediaTek MDP3 nodes Moudy Ho 2022-08-17 9:56 ` Moudy Ho 2022-08-17 9:56 ` [PATCH v25 4/4] media: platform: mtk-mdp3: add MediaTek MDP3 driver Moudy Ho 2022-08-18 10:55 ` Hans Verkuil 2022-08-18 10:55 ` Hans Verkuil 2022-08-18 11:02 ` Hans Verkuil [this message] 2022-08-18 11:02 ` Hans Verkuil 2022-08-19 7:42 ` Bo-Chen Chen 2022-08-19 7:42 ` Bo-Chen Chen
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=b0b4ff87-355e-1910-c6d2-a9690f3d7543@xs4all.nl \ --to=hverkuil-cisco@xs4all.nl \ --cc=Project_Global_Chrome_Upstream_Group@mediatek.com \ --cc=acourbot@chromium.org \ --cc=angelogioacchino.delregno@collabora.com \ --cc=benjamin.gaignard@collabora.com \ --cc=cellopoint.kai@gmail.com \ --cc=chunkuang.hu@kernel.org \ --cc=daoyuan.huang@mediatek.com \ --cc=devicetree@vger.kernel.org \ --cc=drinkcat@chromium.org \ --cc=hsinyi@google.com \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=laurent.pinchart@ideasonboard.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-media@vger.kernel.org \ --cc=linux-mediatek@lists.infradead.org \ --cc=matthias.bgg@gmail.com \ --cc=mchehab@kernel.org \ --cc=moudy.ho@mediatek.com \ --cc=pihsun@chromium.org \ --cc=ping-hsun.wu@mediatek.com \ --cc=rob@landley.net \ --cc=robh+dt@kernel.org \ --cc=tfiga@chromium.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: linkBe 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.