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:

quoted

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;
On the other hand, the correct chip name can be set through the
compatible strings;
The above two reasons are reflected in the following patch:

https://patchwork.linuxtv.org/project/linux-media/patch/20250721105520.5625-3-kyrie.wu@mediatek.com/

Thanks.

Regards,
Kyrie.