[PATCH v18 09/13] media: dt-bindings: mediatek,jpeg: Add mediatek, mt8196-jpgdec compatible

Krzysztof Kozlowski krzk at kernel.org
Mon Oct 12 02:33:18 PDT 2026


On Sat, Oct 10, 2026 at 04:07:13PM +0800, Kyrie Wu wrote:
> Compared to the previous generation IC, the MT8196 uses SMMU
> instead of IOMMU and supports features such as dynamic voltage
> and frequency scaling. Therefore, add "mediatek,mt8196-jpgdec"
> compatible to the binding document.
> 
> Signed-off-by: Kyrie Wu <kyrie.wu at mediatek.com>
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>
> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno at collabora.com>

So this has subject "media: dt-bindings:" but a new patch you added here
has completely different subject prefix. Including the re-introduced
nit, I am sure I already asked you.

A nit, subject: drop second/last, redundant "bindings". The
"dt-bindings" prefix is already stating that these are bindings.
See also:
https://elixir.bootlin.com/linux/v7.1-rc7/source/Documentation/devicetree/bindings/submitting-patches.rst#L23

This is random way of coding, not even checking your git history before
sending to lists.

Best regards,
Krzysztof




More information about the linux-arm-kernel mailing list