[PATCH v1 0/3] MediaTek MT8188 MDP3 Enablement

AngeloGioacchino Del Regno posted 3 patches 12 months ago
.../display/mediatek/mediatek,aal.yaml        |   4 +
.../display/mediatek/mediatek,color.yaml      |   4 +
.../display/mediatek/mediatek,merge.yaml      |   4 +
.../display/mediatek/mediatek,padding.yaml    |  10 +-
.../bindings/media/mediatek,mdp3-fg.yaml      |   8 +-
.../bindings/media/mediatek,mdp3-hdr.yaml     |   8 +-
.../bindings/media/mediatek,mdp3-rsz.yaml     |   1 +
.../bindings/media/mediatek,mdp3-stitch.yaml  |   8 +-
.../bindings/media/mediatek,mdp3-tcc.yaml     |   8 +-
.../bindings/media/mediatek,mdp3-tdshp.yaml   |   8 +-
.../bindings/media/mediatek,mdp3-wrot.yaml    |   1 +
arch/arm64/boot/dts/mediatek/mt8188.dtsi      | 313 ++++++++++++++++++
12 files changed, 364 insertions(+), 13 deletions(-)
[PATCH v1 0/3] MediaTek MT8188 MDP3 Enablement
Posted by AngeloGioacchino Del Regno 12 months ago
This series adds the necessary bindings and devicetree nodes to enable
the entire Multimedia Data Path 3 (MDP3) macro-block, found in MediaTek's
MT8188 SoC.

This was tested on a MediaTek Genio 700 EVK board.

AngeloGioacchino Del Regno (3):
  dt-bindings: display: mediatek: Add compatibles for MT8188 MDP3
  dt-bindings: media: mediatek: mdp3: Add compatibles for MT8188 MDP3
  arm64: dts: mediatek: mt8188: Add all Multimedia Data Path 3 nodes

 .../display/mediatek/mediatek,aal.yaml        |   4 +
 .../display/mediatek/mediatek,color.yaml      |   4 +
 .../display/mediatek/mediatek,merge.yaml      |   4 +
 .../display/mediatek/mediatek,padding.yaml    |  10 +-
 .../bindings/media/mediatek,mdp3-fg.yaml      |   8 +-
 .../bindings/media/mediatek,mdp3-hdr.yaml     |   8 +-
 .../bindings/media/mediatek,mdp3-rsz.yaml     |   1 +
 .../bindings/media/mediatek,mdp3-stitch.yaml  |   8 +-
 .../bindings/media/mediatek,mdp3-tcc.yaml     |   8 +-
 .../bindings/media/mediatek,mdp3-tdshp.yaml   |   8 +-
 .../bindings/media/mediatek,mdp3-wrot.yaml    |   1 +
 arch/arm64/boot/dts/mediatek/mt8188.dtsi      | 313 ++++++++++++++++++
 12 files changed, 364 insertions(+), 13 deletions(-)

-- 
2.46.1
Re: [PATCH v1 0/3] MediaTek MT8188 MDP3 Enablement
Posted by AngeloGioacchino Del Regno 8 months, 1 week ago
On Wed, 18 Dec 2024 11:53:17 +0100, AngeloGioacchino Del Regno wrote:
> This series adds the necessary bindings and devicetree nodes to enable
> the entire Multimedia Data Path 3 (MDP3) macro-block, found in MediaTek's
> MT8188 SoC.
> 
> This was tested on a MediaTek Genio 700 EVK board.
> 
> AngeloGioacchino Del Regno (3):
>   dt-bindings: display: mediatek: Add compatibles for MT8188 MDP3
>   dt-bindings: media: mediatek: mdp3: Add compatibles for MT8188 MDP3
>   arm64: dts: mediatek: mt8188: Add all Multimedia Data Path 3 nodes
> 
> [...]

Applied to v6.15-next/dts64, thanks!

[1/3] dt-bindings: display: mediatek: Add compatibles for MT8188 MDP3
      commit: 2971de063fa56c18b2720ab19bdebca23cd96471
[2/3] dt-bindings: media: mediatek: mdp3: Add compatibles for MT8188 MDP3
      commit: cfb00dfa1b778a8037faf6973cca226e5ad4f45a
[3/3] arm64: dts: mediatek: mt8188: Add all Multimedia Data Path 3 nodes
      commit: f0935480253ede5405045a4e733f4476343cbb91

Cheers,
Angelo
Re: [PATCH v1 0/3] MediaTek MT8188 MDP3 Enablement
Posted by Rob Herring 8 months ago
On Tue, Apr 15, 2025 at 2:38 AM AngeloGioacchino Del Regno
<angelogioacchino.delregno@collabora.com> wrote:
>
> On Wed, 18 Dec 2024 11:53:17 +0100, AngeloGioacchino Del Regno wrote:
> > This series adds the necessary bindings and devicetree nodes to enable
> > the entire Multimedia Data Path 3 (MDP3) macro-block, found in MediaTek's
> > MT8188 SoC.
> >
> > This was tested on a MediaTek Genio 700 EVK board.
> >
> > AngeloGioacchino Del Regno (3):
> >   dt-bindings: display: mediatek: Add compatibles for MT8188 MDP3
> >   dt-bindings: media: mediatek: mdp3: Add compatibles for MT8188 MDP3
> >   arm64: dts: mediatek: mt8188: Add all Multimedia Data Path 3 nodes
> >
> > [...]
>
> Applied to v6.15-next/dts64, thanks!
>
> [1/3] dt-bindings: display: mediatek: Add compatibles for MT8188 MDP3
>       commit: 2971de063fa56c18b2720ab19bdebca23cd96471
> [2/3] dt-bindings: media: mediatek: mdp3: Add compatibles for MT8188 MDP3
>       commit: cfb00dfa1b778a8037faf6973cca226e5ad4f45a
> [3/3] arm64: dts: mediatek: mt8188: Add all Multimedia Data Path 3 nodes
>       commit: f0935480253ede5405045a4e733f4476343cbb91

A couple of warnings added with this:

     33 (mediatek,mt8188-mdp3-wrot): '#dma-cells' is a required property
     33 (mediatek,mt8188-mdp3-tdshp): 'power-domains' does not match
any of the regexes: 'pinctrl-[0-9]+'
     33 (mediatek,mt8188-mdp3-rsz): 'power-domains' does not match any
of the regexes: 'pinctrl-[0-9]+'
     33 (mediatek,mt8188-mdp3-rsz): 'mediatek,gce-events' is a required property
     33 (mediatek,mt8188-mdp3-rdma): compatible: 'oneOf' conditional
failed, one must be fixed:
     33 (mediatek,mt8188-mdp3-hdr): 'power-domains' does not match any
of the regexes: 'pinctrl-[0-9]+'
     33 (mediatek,mt8188-mdp3-fg): 'power-domains' does not match any
of the regexes: 'pinctrl-[0-9]+'
     33 (mediatek,mt8188-mdp3-color): 'oneOf' conditional failed, one
must be fixed:
     33 (mediatek,mt8188-mdp3-aal): 'oneOf' conditional failed, one
must be fixed:
     11 (mediatek,mt8188-mdp3-tcc): 'power-domains' does not match any
of the regexes: 'pinctrl-[0-9]+'
     11 (mediatek,mt8188-mdp3-ovl): 'oneOf' conditional failed, one
must be fixed:
     11 (mediatek,mt8188-mdp3-ovl): 'iommus' is a required property
      8 (mediatek,mt8188-mdp3-rdma): clocks: [[51, 12], [46, 186],
[46, 191]] is too long
      8 (mediatek,mt8188-mdp3-rdma): clocks: [[51, 10], [46, 186],
[46, 191]] is too long
      8 (mediatek,mt8188-mdp3-rdma): clocks: [[50, 12], [45, 186],
[45, 191]] is too long
      8 (mediatek,mt8188-mdp3-rdma): clocks: [[50, 10], [45, 186],
[45, 191]] is too long
      8 (mediatek,mt8188-mdp3-rdma): clocks: [[49, 24], [46, 185],
[46, 190], [49, 4], [49, 41], [49, 42], [49, 7], [51, 41], [51, 42],
[49, 8]] is too long
      8 (mediatek,mt8188-mdp3-rdma): clocks: [[48, 24], [45, 185],
[45, 190], [48, 4], [48, 41], [48, 42], [48, 7], [50, 41], [50, 42],
[48, 8]] is too long
      4 (mediatek,mt8188-mdp3-rsz): clocks: [[51, 24], [51, 25]] is too long
      4 (mediatek,mt8188-mdp3-rsz): clocks: [[51, 20], [51, 21]] is too long
      4 (mediatek,mt8188-mdp3-rsz): clocks: [[50, 24], [50, 25]] is too long
      4 (mediatek,mt8188-mdp3-rsz): clocks: [[50, 20], [50, 21]] is too long
      4 (mediatek,mt8188-mdp3-rdma): power-domains: [[67, 15], [67,
20]] is too long
      4 (mediatek,mt8188-mdp3-rdma): power-domains: [[66, 15], [66,
20]] is too long
      2 (mediatek,mt8188-mdp3-rdma): mboxes: [[121, 13, 1], [121, 14,
1], [121, 16, 1], [121, 21, 1]] is too short
      2 (mediatek,mt8188-mdp3-rdma): mboxes: [[120, 13, 1], [120, 14,
1], [120, 16, 1], [120, 21, 1]] is too short
      2 (mediatek,mt8188-mdp3-rdma): mboxes: [[119, 13, 1], [119, 14,
1], [119, 16, 1], [119, 21, 1]] is too short
      2 (mediatek,mt8188-mdp3-rdma): mboxes: [[118, 13, 1], [118, 14,
1], [118, 16, 1], [118, 21, 1]] is too short
      2 (mediatek,mt8188-mdp3-rdma): iommus: [[132, 164], [132, 166]]
is too long
      2 (mediatek,mt8188-mdp3-rdma): iommus: [[131, 164], [131, 166]]
is too long
      2 (mediatek,mt8188-mdp3-rdma): iommus: [[130, 164], [130, 166]]
is too long
      2 (mediatek,mt8188-mdp3-rdma): iommus: [[129, 164], [129, 166]]
is too long
      2 (mediatek,mt8188-mdp3-rdma): iommus: [[122, 192], [122, 194]]
is too long
      2 (mediatek,mt8188-mdp3-rdma): iommus: [[122, 128], [122, 131]]
is too long
      2 (mediatek,mt8188-mdp3-rdma): iommus: [[121, 192], [121, 194]]
is too long
      2 (mediatek,mt8188-mdp3-rdma): iommus: [[121, 128], [121, 131]]
is too long
      2 (mediatek,mt8188-mdp3-rdma): iommus: [[120, 192], [120, 194]]
is too long
      2 (mediatek,mt8188-mdp3-rdma): iommus: [[120, 128], [120, 131]]
is too long
      2 (mediatek,mt8188-mdp3-rdma): iommus: [[119, 192], [119, 194]]
is too long
      2 (mediatek,mt8188-mdp3-rdma): iommus: [[119, 128], [119, 131]]
is too long
      2 (mediatek,mt8188-mdp3-rdma): clocks: [[44, 12], [39, 186],
[39, 191]] is too long
      2 (mediatek,mt8188-mdp3-rdma): clocks: [[44, 10], [39, 186],
[39, 191]] is too long
      2 (mediatek,mt8188-mdp3-rdma): clocks: [[42, 24], [39, 185],
[39, 190], [42, 4], [42, 41], [42, 42], [42, 7], [44, 41], [44, 42],
[42, 8]] is too long
      2 (mediatek,mt8188-mdp3-rdma): clocks: [[42, 12], [37, 186],
[37, 191]] is too long
      2 (mediatek,mt8188-mdp3-rdma): clocks: [[42, 10], [37, 186],
[37, 191]] is too long
      2 (mediatek,mt8188-mdp3-rdma): clocks: [[40, 24], [37, 185],
[37, 190], [40, 4], [40, 41], [40, 42], [40, 7], [42, 41], [42, 42],
[40, 8]] is too long
      2 (mediatek,mt8188-mdp3-rdma): clocks: [[38, 12], [34, 186],
[34, 191]] is too long
      2 (mediatek,mt8188-mdp3-rdma): clocks: [[38, 10], [34, 186],
[34, 191]] is too long
      2 (mediatek,mt8188-mdp3-rdma): clocks: [[42, 24], [39, 185],
[39, 190], [42, 4], [42, 41], [42, 42], [42, 7], [44, 41], [44, 42],
[42, 8]] is too long
      2 (mediatek,mt8188-mdp3-rdma): clocks: [[42, 12], [37, 186],
[37, 191]] is too long
      2 (mediatek,mt8188-mdp3-rdma): clocks: [[42, 10], [37, 186],
[37, 191]] is too long
      2 (mediatek,mt8188-mdp3-rdma): clocks: [[40, 24], [37, 185],
[37, 190], [40, 4], [40, 41], [40, 42], [40, 7], [42, 41], [42, 42],
[40, 8]] is too long
      2 (mediatek,mt8188-mdp3-rdma): clocks: [[38, 12], [34, 186],
[34, 191]] is too long
      2 (mediatek,mt8188-mdp3-rdma): clocks: [[38, 10], [34, 186],
[34, 191]] is too long
      2 (mediatek,mt8188-mdp3-rdma): clocks: [[36, 24], [34, 185],
[34, 190], [36, 4], [36, 41], [36, 42], [36, 7], [38, 41], [38, 42],
[36, 8]] is too long
      1 (mediatek,mt8188-mdp3-rsz): clocks: [[44, 24], [44, 25]] is too long
      1 (mediatek,mt8188-mdp3-rsz): clocks: [[44, 20], [44, 21]] is too long
      1 (mediatek,mt8188-mdp3-rsz): clocks: [[42, 24], [42, 25]] is too long
      1 (mediatek,mt8188-mdp3-rsz): clocks: [[42, 20], [42, 21]] is too long
      1 (mediatek,mt8188-mdp3-rsz): clocks: [[38, 24], [38, 25]] is too long
      1 (mediatek,mt8188-mdp3-rsz): clocks: [[38, 20], [38, 21]] is too long
      1 (mediatek,mt8188-mdp3-rdma): power-domains: [[60, 15], [60,
20]] is too long
      1 (mediatek,mt8188-mdp3-rdma): power-domains: [[58, 15], [58,
20]] is too long
      1 (mediatek,mt8188-mdp3-rdma): power-domains: [[54, 15], [54,
20]] is too long
      1 (mediatek,mt8188-mdp3-rdma): mboxes: [[92, 13, 1], [92, 14,
1], [92, 16, 1], [92, 21, 1]] is too short
      1 (mediatek,mt8188-mdp3-rdma): mboxes: [[130, 13, 1], [130, 14,
1], [130, 16, 1], [130, 21, 1]] is too short
      1 (mediatek,mt8188-mdp3-rdma): mboxes: [[128, 13, 1], [128, 14,
1], [128, 16, 1], [128, 21, 1]] is too short
      1 (mediatek,mt8188-mdp3-rdma): iommus: [[93, 192], [93, 194]] is too long
      1 (mediatek,mt8188-mdp3-rdma): iommus: [[93, 128], [93, 131]] is too long
      1 (mediatek,mt8188-mdp3-rdma): iommus: [[141, 164], [141, 166]]
is too long
      1 (mediatek,mt8188-mdp3-rdma): iommus: [[139, 164], [139, 166]]
is too long
      1 (mediatek,mt8188-mdp3-rdma): iommus: [[131, 192], [131, 194]]
is too long
      1 (mediatek,mt8188-mdp3-rdma): iommus: [[131, 128], [131, 131]]
is too long
      1 (mediatek,mt8188-mdp3-rdma): iommus: [[129, 192], [129, 194]]
is too long
      1 (mediatek,mt8188-mdp3-rdma): iommus: [[129, 128], [129, 131]]
is too long
      1 (mediatek,mt8188-mdp3-rdma): iommus: [[103, 164], [103, 166]]
is too long
Re: [PATCH v1 0/3] MediaTek MT8188 MDP3 Enablement
Posted by AngeloGioacchino Del Regno 10 months, 1 week ago
Il 18/12/24 11:53, AngeloGioacchino Del Regno ha scritto:
> This series adds the necessary bindings and devicetree nodes to enable
> the entire Multimedia Data Path 3 (MDP3) macro-block, found in MediaTek's
> MT8188 SoC.
> 
> This was tested on a MediaTek Genio 700 EVK board.
> 
> AngeloGioacchino Del Regno (3):
>    dt-bindings: display: mediatek: Add compatibles for MT8188 MDP3
>    dt-bindings: media: mediatek: mdp3: Add compatibles for MT8188 MDP3

CK, Mauro,

If it's okay for you, I'd like to take the display and media bindings
patches in the MediaTek trees, so that I can also safely take the DT patch
without immutable branching, etc.

It's a very old series, yes, but it's still applicable.

Can you please ack the display and media bindings so that I can safely pick?

For display: 20241218105320.38980-2-angelogioacchino.delregno@collabora.com
For media:   20241218105320.38980-3-angelogioacchino.delregno@collabora.com

Thanks!
Angelo

>    arm64: dts: mediatek: mt8188: Add all Multimedia Data Path 3 nodes
> 
>   .../display/mediatek/mediatek,aal.yaml        |   4 +
>   .../display/mediatek/mediatek,color.yaml      |   4 +
>   .../display/mediatek/mediatek,merge.yaml      |   4 +
>   .../display/mediatek/mediatek,padding.yaml    |  10 +-
>   .../bindings/media/mediatek,mdp3-fg.yaml      |   8 +-
>   .../bindings/media/mediatek,mdp3-hdr.yaml     |   8 +-
>   .../bindings/media/mediatek,mdp3-rsz.yaml     |   1 +
>   .../bindings/media/mediatek,mdp3-stitch.yaml  |   8 +-
>   .../bindings/media/mediatek,mdp3-tcc.yaml     |   8 +-
>   .../bindings/media/mediatek,mdp3-tdshp.yaml   |   8 +-
>   .../bindings/media/mediatek,mdp3-wrot.yaml    |   1 +
>   arch/arm64/boot/dts/mediatek/mt8188.dtsi      | 313 ++++++++++++++++++
>   12 files changed, 364 insertions(+), 13 deletions(-)
>