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

["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:16, Kyrie Wu (吴晗) wrote:
> > On Tue, 2025-07-22 at 09:26 +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:19PM +0800, Kyrie Wu wrote:
> > > > Add MT8189 encoder compatible string, which will reference SCP
> > > > device.
> > > 
> > > And the device is not compatible or is different because of what?
> > > 
> > > Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> > > 
> > > Best regards,
> > > Krzysztof
> > > 
> > 
> > Dear Krzysztof,
> > 
> > The reason, adding a new compatible strings for the MT8189 encoder,
> > is
> > same as for decoder, to use different of_device_id data and to
> > distinguish the chip name.
> 
> You speak about drivers, this is not relevant. Explain the hardware -
> WHY it is not compatible, what is the difference
> 
> Best regards,
> Krzysztof

Dear Krzysztof,

For encoder, the hardware differences are shown as follow:
Compared with MT8196, the maximum resolution of MT8189 encoder is only
4K, and the fps is only 30, which cannot reach the highest parameter of
MT8196: level6.2, 8K@60fps.
Compared with MT8188, the level can only support 5.1, which is less
than 5.2 of MT8188. But the maximum bitrate is 100Mbps, which is twice
that of MT8188.

The above informations will be updated to the commit messages in the
coming version.

Thanks.

Regards,
Kyire.