[PATCH v8 5/7] dt-binding: mt8183: add Mediatek MDP3 dt-bindings

[moudy ho]

On Sat, 2021-10-16 at 09:18 -0500, Rob Herring wrote:
> On Fri, Oct 15, 2021 at 08:38:30PM +0800, Moudy Ho wrote:
> > This patch adds DT binding document for Media Data Path 3 (MDP3)
> > a unit in multimedia system used for scaling and color format
> > convert.
> > 
> > Signed-off-by: Moudy Ho <moudy.ho at mediatek.com>
> > ---
> >  .../bindings/arm/mediatek/mediatek,ccorr.yaml |  59 +++++
> >  .../bindings/arm/mediatek/mediatek,rdma.yaml  | 211
> > ++++++++++++++++++
> >  .../bindings/arm/mediatek/mediatek,wdma.yaml  |  70 ++++++
> 
> These belong in a functional directory, not 'arm'.
> 

Hi Matthias,

We tried to integrate the common components of DRM & MDP3 into the same
binding files and move them to the "bindings/arm/mediatek" directory,
but it did not meet the requirements.
Is it appropriate for us to change the path to "bindings/soc/mediatek",
or do you have other suggestions?

Thanks & Regards,
Moudy Ho

> >  .../bindings/media/mediatek,mdp3-rsz.yaml     |  68 ++++++
> >  .../bindings/media/mediatek,mdp3-wrot.yaml    |  70 ++++++
> >  5 files changed, 478 insertions(+)
> >  create mode 100644
> > Documentation/devicetree/bindings/arm/mediatek/mediatek,ccorr.yaml
> >  create mode 100644
> > Documentation/devicetree/bindings/arm/mediatek/mediatek,rdma.yaml
> >  create mode 100644
> > Documentation/devicetree/bindings/arm/mediatek/mediatek,wdma.yaml
> >  create mode 100644
> > Documentation/devicetree/bindings/media/mediatek,mdp3-rsz.yaml
> >  create mode 100644
> > Documentation/devicetree/bindings/media/mediatek,mdp3-wrot.yaml

(snip)

> 
> Fix this.
> 
> Similar comments on the rest...
> 
> Rob
>