2023-03-09 09:07:20

by Alexandre Mergnat

[permalink] [raw]
Subject: [PATCH v5] dt-bindings: display: mediatek: clean unnecessary item

The item which have the mediatek,mt8192-disp-ccorr as const compatible
already exist above. Merge all compatibles which have the same fallback
under the same item.

Acked-by: Krzysztof Kozlowski <[email protected]>
Reviewed-by: AngeloGioacchino Del Regno <[email protected]>
Reviewed-by: Rob Herring <[email protected]>
Signed-off-by: Alexandre Mergnat <[email protected]>
---
Fix MTK color correction binding

The fallback compatible has been duplicated in the 137272ef1b0f commit.

To: Chun-Kuang Hu <[email protected]>
To: Philipp Zabel <[email protected]>
To: David Airlie <[email protected]>
To: Daniel Vetter <[email protected]>
To: Rob Herring <[email protected]>
To: Krzysztof Kozlowski <[email protected]>
To: Matthias Brugger <[email protected]>
To: AngeloGioacchino Del Regno <[email protected]>
To: Allen-KH Cheng <[email protected]>
Cc: Rob Herring <[email protected]>
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
---
Changes in v5:
- Add trailers.
- Link to v4: https://lore.kernel.org/r/[email protected]

Changes in v4:
- Reword commit title and message.
- Link to v3: https://lore.kernel.org/r/[email protected]

Changes in v3:
- Re-order compatible.
- Link to v2: https://lore.kernel.org/r/[email protected]

Changes in v2:
- Fix commit title.
- Link to v1: https://lore.kernel.org/r/[email protected]
---
.../devicetree/bindings/display/mediatek/mediatek,ccorr.yaml | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,ccorr.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,ccorr.yaml
index b04820c95b22..bda86e6857f5 100644
--- a/Documentation/devicetree/bindings/display/mediatek/mediatek,ccorr.yaml
+++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,ccorr.yaml
@@ -27,13 +27,10 @@ properties:
- const: mediatek,mt8192-disp-ccorr
- items:
- enum:
+ - mediatek,mt8186-disp-ccorr
- mediatek,mt8188-disp-ccorr
- mediatek,mt8195-disp-ccorr
- const: mediatek,mt8192-disp-ccorr
- - items:
- - enum:
- - mediatek,mt8186-disp-ccorr
- - const: mediatek,mt8192-disp-ccorr

reg:
maxItems: 1

---
base-commit: add072536971d7ce891fde3cdbf68c55e7cfa95a
change-id: 20230306-ccorr-binding-fix-718c6d725088

Best regards,
--
Alexandre Mergnat <[email protected]>


2023-03-13 14:54:05

by Chun-Kuang Hu

[permalink] [raw]
Subject: Re: [PATCH v5] dt-bindings: display: mediatek: clean unnecessary item

Hi, Alexandre:

Alexandre Mergnat <[email protected]> 於 2023年3月9日 週四 下午5:06寫道:
>
> The item which have the mediatek,mt8192-disp-ccorr as const compatible
> already exist above. Merge all compatibles which have the same fallback
> under the same item.

Applied to mediatek-drm-next [1], thanks.

[1] https://git.kernel.org/pub/scm/linux/kernel/git/chunkuang.hu/linux.git/log/?h=mediatek-drm-next

Regards,
Chun-Kuang.

>
> Acked-by: Krzysztof Kozlowski <[email protected]>
> Reviewed-by: AngeloGioacchino Del Regno <[email protected]>
> Reviewed-by: Rob Herring <[email protected]>
> Signed-off-by: Alexandre Mergnat <[email protected]>
> ---
> Fix MTK color correction binding
>
> The fallback compatible has been duplicated in the 137272ef1b0f commit.
>
> To: Chun-Kuang Hu <[email protected]>
> To: Philipp Zabel <[email protected]>
> To: David Airlie <[email protected]>
> To: Daniel Vetter <[email protected]>
> To: Rob Herring <[email protected]>
> To: Krzysztof Kozlowski <[email protected]>
> To: Matthias Brugger <[email protected]>
> To: AngeloGioacchino Del Regno <[email protected]>
> To: Allen-KH Cheng <[email protected]>
> Cc: Rob Herring <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> ---
> Changes in v5:
> - Add trailers.
> - Link to v4: https://lore.kernel.org/r/[email protected]
>
> Changes in v4:
> - Reword commit title and message.
> - Link to v3: https://lore.kernel.org/r/[email protected]
>
> Changes in v3:
> - Re-order compatible.
> - Link to v2: https://lore.kernel.org/r/[email protected]
>
> Changes in v2:
> - Fix commit title.
> - Link to v1: https://lore.kernel.org/r/[email protected]
> ---
> .../devicetree/bindings/display/mediatek/mediatek,ccorr.yaml | 5 +----
> 1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,ccorr.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,ccorr.yaml
> index b04820c95b22..bda86e6857f5 100644
> --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,ccorr.yaml
> +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,ccorr.yaml
> @@ -27,13 +27,10 @@ properties:
> - const: mediatek,mt8192-disp-ccorr
> - items:
> - enum:
> + - mediatek,mt8186-disp-ccorr
> - mediatek,mt8188-disp-ccorr
> - mediatek,mt8195-disp-ccorr
> - const: mediatek,mt8192-disp-ccorr
> - - items:
> - - enum:
> - - mediatek,mt8186-disp-ccorr
> - - const: mediatek,mt8192-disp-ccorr
>
> reg:
> maxItems: 1
>
> ---
> base-commit: add072536971d7ce891fde3cdbf68c55e7cfa95a
> change-id: 20230306-ccorr-binding-fix-718c6d725088
>
> Best regards,
> --
> Alexandre Mergnat <[email protected]>