Re: [PATCH v6 08/14] dt-bindings: media: mediatek: vcodec: add decoder dt-bindings for mt8196

[Kyrie Wu (吴晗)]

On Mon, 2026-07-20 at 07:49 +0200, Krzysztof Kozlowski wrote:
> On 20/07/2026 07:46, Krzysztof Kozlowski wrote:
> > On Mon, Jul 20, 2026 at 09:20:50AM +0800, Kyrie Wu wrote:
> > > Add the MT8196 compatible string to the MediaTek vcodec subdev
> > > decoder binding.
> > > 
> > > Compared to previous ICs, the MT8196 supports a 10-bit decoder
> > > and has a decoding capability of 4K@120fps. It also supports
> > > 36-bit DRAM IOVA address and Video Power Control to optimize
> > > bandwidth and voltage usage.
> > 
> > Please wrap commit message according to Linux coding style /
> > submission
> > process (neither too early nor over the limit):
> > https://urldefense.com/v3/__https://elixir.bootlin.com/linux/v6.4-rc1/source/Documentation/process/submitting-patches.rst*L597__;Iw!!CTRNKA9wMg0ARbw!gDjbq0EZNkhLTzuJM_w5PopnDFej1mG4qnkRFgyoiU7Dvhm7tj3MeG5eZYNGA0L0G_61j-cmdIxORA$
> >  
> > 
> > > 
> > > Signed-off-by: Kyrie Wu <kyrie.wu@xxxxxxxxxxxx>
> > > Acked-by: Nicolas Dufresne <nicolas.dufresne@xxxxxxxxxxxxx>
> > > ---
> > >  .../bindings/media/mediatek,vcodec-subdev-decoder.yaml          
> > > | 1 +
> > >  1 file changed, 1 insertion(+)
> > 
> > Acked-by: Krzysztof Kozlowski
> > <krzysztof.kozlowski@xxxxxxxxxxxxxxxx>
> 
> Actually no. I checked previous comments and you received feedback
> which
> you basically ignored, so explain why this model has flexible number
> of
> clocks
> 
> Best regards,
> Krzysztof

Dear Krzysztof,

As mentioned in the previous version of the communication, the
information regarding clk was incorrect, and I have corrected it in
this version. 

Thank you for your attention and reminder.

Regards,
Kyrie.