Re: [PATCH v1 1/8] dt-bindings: media: mediatek: decoder: Add MT8189 mediatek,vcodec-decoder

["Kyrie Wu (吴晗)" <Kyrie.Wu@mediatek.com>]

On Tue, 2025-07-22 at 13:09 +0200, Krzysztof Kozlowski wrote:
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
> 
> 
> On 22/07/2025 10:11, Kyrie Wu (吴晗) wrote:
> > On Tue, 2025-07-22 at 09:25 +0200, Krzysztof Kozlowski wrote:
> > > External email : Please do not click links or open attachments
> > > until
> > > you have verified the sender or the content.
> > > 
> > > 
> > > On Mon, Jul 21, 2025 at 06:55:13PM +0800, Kyrie Wu wrote:
> > > > Add compatible for video decoder on MT8189 platform, which is a
> > > > pure single core architecture.
> > > 
> > > Looks compatible with existing ones and nothing in commit msg
> > > explains
> > > why it was not made compatible. Describe the hardware, say
> > > something
> > > useful to avoid such questions.
> > > 
> > > Best regards,
> > > Krzysztof
> > > 
> > 
> > Dear Krzysztof,
> > 
> > There are two reasons for adding a new compatible string to MT8189:
> > On the one hand, MT8189 is a pure single-core hardware chip, so the
> > of_device_id data is different;
> 
> Not relevant. Your of_device_id is wrong and based on that you claim
> the
> hardware is different?
> 
> Describe the hardware.
> 
> Best regards,
> Krzysztof

Dear Krzysztof,

Sorry for the misunderstanding in your reply. I will update the commit
messages in the next version of patchset to introduce the hardware
differences of MT8189.

In summary, compared with other ICs, the MT8189 encoder and decoder use
iommu instead of smmu, and use scp architecture, the frequency is only
406MHZ, and cannot reach more than 700MHZ. At the same time, the
decoder supports the vp9 decoding protocol for the first time.

Thanks.

Regards,
Kyrie.