[PATCH v2 2/5] dt-bindings: mediatek: display: Change format to yaml

Chun-Kuang Hu chunkuang.hu at kernel.org
Sat Jul 31 18:04:15 PDT 2021


Hi, Jason:

jason-jh.lin <jason-jh.lin at mediatek.com> 於 2021年7月29日 週四 下午11:49寫道:
>
> Change mediatek,dislpay.txt to mediatek,display.yaml
>
> Signed-off-by: jason-jh.lin <jason-jh.lin at mediatek.com>
> ---
>  .../display/mediatek/mediatek,disp.txt        | 219 ---------
>  .../display/mediatek/mediatek,disp.yaml       | 432 ++++++++++++++++++
>  2 files changed, 432 insertions(+), 219 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt
>  create mode 100644 Documentation/devicetree/bindings/display/mediatek/mediatek,disp.yaml
>

[snip]

> +
> +  DISP function blocks
> +  ====================
> +  A display stream starts at a source function block that reads pixel data from
> +  memory and ends with a sink function block that drives pixels on a display
> +  interface, or writes pixels back to memory. All DISP function blocks have
> +  their own register space, interrupt, and clock gate. The blocks that can
> +  access memory additionally have to list the IOMMU and local arbiter they are
> +  connected to.
> +
> +properties:
> +  compatible:
> +    description: |
> +      If the display function block of different soc have the same function,
> +      you can use the same compatible name after it.
> +      For example, if mt8183 COLOR function is the same as mt8173, then the
> +      compatible of mt8183 cholud be set as:
> +      compatible = "mediatek,mt8183-disp-color", "mediatek,mt8173-disp-color";
> +    oneOf:
> +      # OVL: overlay (4 layers, blending, csc)
> +      - items:
> +          - const: mediatek,mt2701-disp-ovl
> +      - items:
> +          - const: mediatek,mt8173-disp-ovl
> +      - items:
> +          - const: mediatek,mt8183-disp-ovl
> +      - items:
> +          - enum:
> +              - mediatek,mt7623-disp-ovl
> +              - mediatek,mt2712-disp-ovl
> +          - enum:
> +              - mediatek,mt2701-disp-ovl
> +      - items:
> +          - enum:
> +              - mediatek,mt8192-disp-ovl
> +          - enum:
> +              - mediatek,mt8183-disp-ovl
> +
> +      # OVL2L: overlay (2 layers, blending, csc)
> +      - items:
> +          - const: mediatek,mt8183-disp-ovl-2l
> +      - items:
> +          - enum:
> +              - mediatek,mt8192-disp-ovl-2l
> +          - enum:
> +              - mediatek,mt8183-disp-ovl-2l
> +
> +      # RDMA: read DMA / line buffer
> +      - items:
> +          - const: mediatek,mt2701-disp-rdma
> +      - items:
> +          - const: mediatek,mt8173-disp-rdma
> +      - items:
> +          - const: mediatek,mt8183-disp-rdma
> +      - items:
> +          - enum:
> +              - mediatek,mt7623-disp-rdma
> +              - mediatek,mt2712-disp-rdma
> +          - enum:
> +              - mediatek,mt2701-disp-rdma
> +      - items:
> +          - enum:
> +              - mediatek,mt8192-disp-rdma
> +          - enum:
> +              - mediatek,mt8183-disp-rdma
> +
> +      # WDMA: write DMA
> +      - items:
> +          - const: mediatek,mt8173-disp-wdma
> +
> +      # CCORR: color correction
> +      - items:
> +          - const: mediatek,mt8183-disp-ccorr
> +      - items:
> +          - enum:
> +              - mediatek,mt8192-disp-ccorr
> +          - enum:
> +              - mediatek,mt8183-disp-ccorr
> +
> +      # COLOR: color processor
> +      - items:
> +          - const: mediatek,mt2701-disp-color
> +      - items:
> +          - const: mediatek,mt8167-disp-color
> +      - items:
> +          - const: mediatek,mt8173-disp-color
> +      - items:
> +          - enum:
> +              - mediatek,mt7623-disp-color
> +              - mediatek,mt2712-disp-color
> +          - enum:
> +              - mediatek,mt2701-disp-color
> +      - items:
> +          - enum:
> +              - mediatek,mt8183-disp-color
> +              - mediatek,mt8192-disp-color
> +          - enum:
> +              - mediatek,mt8173-disp-color
> +
> +      # DITHER
> +      - items:
> +          - const: mediatek,mt8183-disp-dither
> +      - items:
> +          - enum:
> +              - mediatek,mt8192-disp-dither
> +          - enum:
> +              - mediatek,mt8183-disp-dither
> +
> +      # AAL: adaptive ambient light controller
> +      - items:
> +          - const: mediatek,mt8173-disp-aal
> +      - items:
> +          - enum:
> +              - mediatek,mt2712-disp-aal
> +              - mediatek,mt8183-disp-aal
> +              - mediatek,mt8192-disp-aal
> +          - enum:
> +              - mediatek,mt8173-disp-aal
> +
> +      # GAMMA: gamma correction
> +      - items:
> +          - const: mediatek,mt8173-disp-gamma
> +      - items:
> +          - const: mediatek,mt8183-disp-gamma
> +      - items:
> +          - enum:
> +              - mediatek,mt8192-disp-gamma
> +          - enum:
> +              - mediatek,mt8183-disp-gamma
> +
> +      # MERGE: merge streams from two RDMA sources
> +
> +      # POSTMASK: control round corner for display frame
> +      - items:
> +          - const: mediatek,mt8192-disp-postmask
> +
> +      # SPLIT: split stream to two encoders
> +
> +      # UFOE: data compression engine
> +      - items:
> +          - const: mediatek,mt8173-disp-ufoe
> +
> +      # DSI: see Documentation/devicetree/bindings/display/mediatek/mediatek,dsi.txt for details.
> +      - items:
> +          - const: mediatek,mt2701-disp-dsi
> +      - items:
> +          - const: mediatek,mt8173-disp-dsi
> +      - items:
> +          - const: mediatek,mt8183-disp-dsi
> +      - items:
> +          - enum:
> +              - mediatek,mt7623-disp-dsi
> +              - mediatek,mt2712-disp-dsi
> +          - enum:
> +              - mediatek,mt2701-disp-dsi
> +      - items:
> +          - enum:
> +              - mediatek,mt8192-disp-dsi
> +          - enum:
> +              - mediatek,mt8183-disp-dsi
> +
> +      # DPI: see Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml for details.
> +      - items:
> +          - const: mediatek,mt2701-disp-dpi
> +      - items:
> +          - const: mediatek,mt8173-disp-dpi
> +      - items:
> +          - const: mediatek,mt8183-disp-dpi
> +      - items:
> +          - const: mediatek,mt8192-disp-dpi
> +      - items:
> +          - enum:
> +              - mediatek,mt7623-disp-dpi
> +              - mediatek,mt2712-disp-dpi
> +          - enum:
> +              - mediatek,mt2701-disp-dpi
> +
> +      # MUTEX: display mutex
> +      - items:
> +          - const: mediatek,mt2701-disp-mutex
> +      - items:
> +          - const: mediatek,mt2712-disp-mutex
> +      - items:
> +          - const: mediatek,mt8167-disp-mutex
> +      - items:
> +          - const: mediatek,mt8173-disp-mutex
> +      - items:
> +          - const: mediatek,mt8183-disp-mutex
> +      - items:
> +          - const: mediatek,mt8192-disp-mutex
> +
> +      # OD: overdrive
> +      - items:
> +          - const: mediatek,mt2712-disp-od
> +      - items:
> +          - const: mediatek,mt8173-disp-od
> +

I think you should follow Rob's suggestion [1] to split each function
block into one schema.

[1] https://patchwork.kernel.org/project/linux-mediatek/patch/20210715173750.10852-2-jason-jh.lin@mediatek.com/

Regards,
Chun-Kuang.



More information about the Linux-mediatek mailing list