Re: [PATCH v11 07/16] dt-bindings: arm: mediatek: move common module from display folder

[Chun-Kuang Hu]

Hi, Jason:

jason-jh.lin <jason-jh.lin@xxxxxxxxxxxx> 於 2021年9月21日 週二 下午11:52寫道：
>
> AAL, COLOR, CCORR, MUTEX, WDMA could be used by other modules,
> such as MDP, so move their binding document into the common folder.

Reviewed-by: Chun-Kuang Hu <chunkuang.hu@xxxxxxxxxx>

>
> Signed-off-by: jason-jh.lin <jason-jh.lin@xxxxxxxxxxxx>
> ---
>  .../{display => arm}/mediatek/mediatek,aal.yaml      |  9 ++++-----
>  .../{display => arm}/mediatek/mediatek,ccorr.yaml    |  9 ++++-----
>  .../{display => arm}/mediatek/mediatek,color.yaml    | 11 +++++------
>  .../{display => arm}/mediatek/mediatek,mutex.yaml    | 12 +++++-------
>  .../{display => arm}/mediatek/mediatek,wdma.yaml     |  9 ++++-----
>  5 files changed, 22 insertions(+), 28 deletions(-)
>  rename Documentation/devicetree/bindings/{display => arm}/mediatek/mediatek,aal.yaml (88%)
>  rename Documentation/devicetree/bindings/{display => arm}/mediatek/mediatek,ccorr.yaml (87%)
>  rename Documentation/devicetree/bindings/{display => arm}/mediatek/mediatek,color.yaml (86%)
>  rename Documentation/devicetree/bindings/{display => arm}/mediatek/mediatek,mutex.yaml (85%)
>  rename Documentation/devicetree/bindings/{display => arm}/mediatek/mediatek,wdma.yaml (90%)
>
> diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,aal.yaml b/Documentation/devicetree/bindings/arm/mediatek/mediatek,aal.yaml
> similarity index 88%
> rename from Documentation/devicetree/bindings/display/mediatek/mediatek,aal.yaml
> rename to Documentation/devicetree/bindings/arm/mediatek/mediatek,aal.yaml
> index 311bbf05a967..ab6eb9b550a4 100644
> --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,aal.yaml
> +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,aal.yaml
> @@ -1,17 +1,16 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: http://devicetree.org/schemas/display/mediatek/mediatek,aal.yaml#
> +$id: http://devicetree.org/schemas/arm/mediatek/mediatek,aal.yaml#
>  $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> -title: Mediatek display adaptive ambient light processor
> +title: MediaTek adaptive ambient light processor
>
>  maintainers:
> -  - Chun-Kuang Hu <chunkuang.hu@xxxxxxxxxx>
> -  - Philipp Zabel <p.zabel@xxxxxxxxxxxxxx>
> +  - Matthias Brugger <matthias.bgg@xxxxxxxxx>
>
>  description: |
> -  Mediatek display adaptive ambient light processor, namely AAL,
> +  MediaTek adaptive ambient light processor, namely AAL,
>    is responsible for backlight power saving and sunlight visibility improving.
>    AAL device node must be siblings to the central MMSYS_CONFIG node.
>    For a description of the MMSYS_CONFIG binding, see
> diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,ccorr.yaml b/Documentation/devicetree/bindings/arm/mediatek/mediatek,ccorr.yaml
> similarity index 87%
> rename from Documentation/devicetree/bindings/display/mediatek/mediatek,ccorr.yaml
> rename to Documentation/devicetree/bindings/arm/mediatek/mediatek,ccorr.yaml
> index 60752ce45d49..de86e9ae35f3 100644
> --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,ccorr.yaml
> +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,ccorr.yaml
> @@ -1,17 +1,16 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: http://devicetree.org/schemas/display/mediatek/mediatek,ccorr.yaml#
> +$id: http://devicetree.org/schemas/arm/mediatek/mediatek,ccorr.yaml#
>  $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> -title: Mediatek display color correction
> +title: MediaTek color correction
>
>  maintainers:
> -  - Chun-Kuang Hu <chunkuang.hu@xxxxxxxxxx>
> -  - Philipp Zabel <p.zabel@xxxxxxxxxxxxxx>
> +  - Matthias Brugger <matthias.bgg@xxxxxxxxx>
>
>  description: |
> -  Mediatek display color correction, namely CCORR, reproduces correct color
> +  MediaTek color correction, namely CCORR, reproduces correct color
>    on panels with different color gamut.
>    CCORR device node must be siblings to the central MMSYS_CONFIG node.
>    For a description of the MMSYS_CONFIG binding, see
> diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,color.yaml b/Documentation/devicetree/bindings/arm/mediatek/mediatek,color.yaml
> similarity index 86%
> rename from Documentation/devicetree/bindings/display/mediatek/mediatek,color.yaml
> rename to Documentation/devicetree/bindings/arm/mediatek/mediatek,color.yaml
> index f6636869909c..73be301b50d2 100644
> --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,color.yaml
> +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,color.yaml
> @@ -1,18 +1,17 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: http://devicetree.org/schemas/display/mediatek/mediatek,color.yaml#
> +$id: http://devicetree.org/schemas/arm/mediatek/mediatek,color.yaml#
>  $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> -title: Mediatek display color processor
> +title: MediaTek color processor
>
>  maintainers:
> -  - Chun-Kuang Hu <chunkuang.hu@xxxxxxxxxx>
> -  - Philipp Zabel <p.zabel@xxxxxxxxxxxxxx>
> +  - Matthias Brugger <matthias.bgg@xxxxxxxxx>
>
>  description: |
> -  Mediatek display color processor, namely COLOR, provides hue, luma and
> -  saturation adjustments to get better picture quality and to have one panel
> +  MediaTek color processor, namely COLOR, provides hue, luma and saturation
> +  adjustments to get better picture quality and to have one panel
>    resemble the other in their output characteristics.
>    COLOR device node must be siblings to the central MMSYS_CONFIG node.
>    For a description of the MMSYS_CONFIG binding, see
> diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,mutex.yaml b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mutex.yaml
> similarity index 85%
> rename from Documentation/devicetree/bindings/display/mediatek/mediatek,mutex.yaml
> rename to Documentation/devicetree/bindings/arm/mediatek/mediatek,mutex.yaml
> index 6eca525eced0..713c7485e11a 100644
> --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,mutex.yaml
> +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mutex.yaml
> @@ -1,19 +1,17 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: http://devicetree.org/schemas/display/mediatek/mediatek,mutex.yaml#
> +$id: http://devicetree.org/schemas/arm/mediatek/mediatek,mutex.yaml#
>  $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> -title: Mediatek mutex
> +title: MediaTek mutex
>
>  maintainers:
> -  - Chun-Kuang Hu <chunkuang.hu@xxxxxxxxxx>
> -  - Philipp Zabel <p.zabel@xxxxxxxxxxxxxx>
> +  - Matthias Brugger <matthias.bgg@xxxxxxxxx>
>
>  description: |
> -  Mediatek mutex, namely MUTEX, is used to send the triggers signals called
> -  Start Of Frame (SOF) / End Of Frame (EOF) to each sub-modules on the display
> -  data path or MDP data path.
> +  MediaTek mutex, namely MUTEX, is used to send the triggers signals called
> +  Start Of Frame(SOF) / End Of Frame(EOF) to each sub-modules on the data path.
>    In some SoC, such as mt2701, MUTEX could be a hardware mutex which protects
>    the shadow register.
>    MUTEX device node must be siblings to the central MMSYS_CONFIG node.
> diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,wdma.yaml b/Documentation/devicetree/bindings/arm/mediatek/mediatek,wdma.yaml
> similarity index 90%
> rename from Documentation/devicetree/bindings/display/mediatek/mediatek,wdma.yaml
> rename to Documentation/devicetree/bindings/arm/mediatek/mediatek,wdma.yaml
> index 25f9a63fe7af..5222535d98c6 100644
> --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,wdma.yaml
> +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,wdma.yaml
> @@ -1,17 +1,16 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: http://devicetree.org/schemas/display/mediatek/mediatek,wdma.yaml#
> +$id: http://devicetree.org/schemas/arm/mediatek/mediatek,wdma.yaml#
>  $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> -title: Mediatek display WDMA
> +title: MediaTek WDMA
>
>  maintainers:
> -  - Chun-Kuang Hu <chunkuang.hu@xxxxxxxxxx>
> -  - Philipp Zabel <p.zabel@xxxxxxxxxxxxxx>
> +  - Matthias Brugger <matthias.bgg@xxxxxxxxx>
>
>  description: |
> -  Mediatek display WDMA stands for Write Direct Memory Access.
> +  MediaTek WDMA stands for Write Direct Memory Access.
>    It can write the data in display pipeline into DRAM.
>    WDMA device node must be siblings to the central MMSYS_CONFIG node.
>    For a description of the MMSYS_CONFIG binding, see
> --
> 2.18.0
>