Re: [V1,01/12] dt-bindings: mediatek: Add mediatek, mt8196-jpgdec compatible

Krzysztof Kozlowski
2025-04-07T07:45:36Z

On 07/04/2025 09:13, Kyrie Wu (吴晗) wrote:
> On Thu, 2025-01-09 at 14:54 +0100, Krzysztof Kozlowski wrote:

...

>>>
>>> properties:
>>> compatible:
>>> - const: mediatek,mt8195-jpgdec
>>> + enum:
>>> + - mediatek,mt8195-jpgdec
>>> + - mediatek,mt8196-jpgdec
>>
>> Devices look compatible, so express it with fallback.
>>
>> Best regards,
>> Krzysztof
>
> Dear Krzysztof,
>
> The MT8195 and MT8196 are both multi-hardware chips, I renamed the file
> to treat it as a common file, if I don't do that it will need to
> recreate a file with similar contents. Is this acceptable?

No, do not create own rules. I do not see how this is related to my
comment at all.

You responded three months after my review, so entire context including
actual mails in my inbox are gone.

If you disagree with feedback, respond in timely manner. I feel
discussing something after three months like wasting my time.

> If the above description is acceptable, I will apply your request in
> the next version.

Best regards,
Krzysztof