[PATCH v7 0/3] Mediatek MT8189 JPEG support

[Jianhua Lin (林建华)]

Hi Nicolas

This series depends on commit 7560349ee0d9 ("media: mediatek: jpeg:
support 34bits"), which introduced the 'support_34bit' field in 'struct
mtk_jpeg_variant'. That commit has already been merged via the media
tree and is present in linux-next as of next-20260410.Therefore, this
series is based on linux-next/master rather than media-committers/next
to ensure the dependency is available.

The dt-bindings schema was validated against linux-next tag next-
20260410. JPEG encoder and decoder functionality was verified on MT8189
hardware.

Regards,
Jianhua Lin

On Fri, 2026-04-17 at 09:30 -0400, Nicolas Dufresne wrote:
> Hi,
> 
> Le vendredi 17 avril 2026 à 18:05 +0800, Jianhua Lin a écrit :
> > This series is based on tag: next-20260410, linux-next/master
> 
> What dependencies justify not submitting based on media-
> committers/next as usual
> ? Its fine to say you tested against linux-next of course, and if its
> only
> working there, its really nice to explain why.
> 
> Nicolas
> 
> > 
> > Changes compared with v6:
> > - Patches 1/3 (dt-bindings: decoder):
> >   update the existing `allOf` condition for mediatek,mt8189-jpgdec
> > to
> >   make the 'mediatek,larb' property strictly required for MT8189
> > SoC.
> > - Patches 2/3 (dt-bindings: encoder):
> >   Add an `allOf` condition to enforce that the `mediatek,larb`
> > property
> >   is strictly required when the compatible string contains
> >   mediatek,mt8189-jpgenc.
> > 
> > Changes compared with v5:
> > - Patches 1/3 (dt-bindings: decoder):
> >   - Drop top-level minItems/maxItems for clock-names per
> > Krzysztof's
> >     review.
> >   - Refine allOf block to strictly enforce clock constraints.
> > 
> > Changes compared with v4:
> > - Refines the device tree bindings for JPEG decoder and encoder.
> >   - Patches 1/3 (dt-bindings: decoder):
> >     Moved the standalone compatible string mediatek,mt8189-jpgdec
> >     into the first oneOf entry along with mt2701 and mt8173, as
> >     suggested by Rob Herring. This correctly groups all independent
> >     ICs and removes the redundant items wrapper.
> >   - Patches 2/3 (dt-bindings: encoder):
> >     Applied the same logic suggested by Rob Herring to the encoder
> >     binding. Restructured the compatible property to clearly
> >     distinguish between the standalone IC (mediatek,mt8189-jpgenc)
> >     and the ICs that must fallback to mediatek,mtk-jpgenc.
> > 
> > Changes compared with v3:
> > - The v4 is resending the cover-letter, because the v3 cover-letter 
> > was
> >   not sent successfully.
> > 
> > Changes compared with v2:
> > - Dropped the dts patch (arm64: dts: mt8188: update JPEG
> > encoder/decoder
> >   compatible) as it belongs to a different tree/series.
> > - Patches 1/3 (dt-bindings: decoder):
> >   - Changed the MT8189 compatible to be a standalone `const`
> > instead of
> >     an `enum`.
> >   - Added an `allOf` block with conditional checks to enforce the
> > single
> >     clock ("jpgdec") requirement for MT8189, while preserving the
> >     two-clock requirement for older SoCs.
> >   - Updated commit message to reflect the schema structure changes
> > and
> >     hardware differences.
> > - Patches 2/3 (dt-bindings: encoder):
> >   - Changed the MT8189 compatible to be a standalone `const`
> > instead of
> >     an `enum` inside the `items` list, as it does not fallback to
> >     "mediatek,mtk-jpgenc" due to 34-bit IOVA requirements.
> >   - Updated commit message to explain the standalone compatible
> > design.
> > - Patches 3/3 (media: mediatek: jpeg):
> >   - Refined commit message for better clarity regarding 34-bit IOVA
> > and
> >     single clock configuration.
> > 
> > Changes compared with v1:
> > - Patches 1/4:
> >   - Updating commit message
> > - Patches 2/4, 3/4: 
> >   - Updating commit message
> >   - Adjusted property descriptions acorrding to hardware
> > requirements
> >   - Improved formatting for better readability and consistency
> > - Patches 4/4:
> >   - Updating commit message
> > 
> > Jianhua Lin (3):
> >   dt-bindings: media: mediatek-jpeg-decoder: add MT8189 compatible
> >     string
> >   dt-bindings: media: mediatek-jpeg-encoder: add MT8189 compatible
> >     string
> >   media: mediatek: jpeg: add compatible for MT8189 SoC
> > 
> >  .../bindings/media/mediatek-jpeg-decoder.yaml | 48
> > +++++++++++++++----
> >  .../bindings/media/mediatek-jpeg-encoder.yaml | 29 ++++++++---
> >  .../platform/mediatek/jpeg/mtk_jpeg_core.c    | 44
> > +++++++++++++++++
> >  3 files changed, 107 insertions(+), 14 deletions(-)