2023-10-30 10:01:55

by Moudy Ho (何宗原)

[permalink] [raw]
Subject: [PATCH v8 16/16] dt-bindings: display: mediatek: padding: add compatible for MT8195

Add a compatible string for the PADDING block in MediaTek MT8195 that
is controlled by MDP3.

Signed-off-by: Moudy Ho <[email protected]>
---
.../bindings/display/mediatek/mediatek,padding.yaml | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,padding.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,padding.yaml
index db24801ebc48..19858d826808 100644
--- a/Documentation/devicetree/bindings/display/mediatek/mediatek,padding.yaml
+++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,padding.yaml
@@ -20,7 +20,9 @@ description:

properties:
compatible:
- const: mediatek,mt8188-padding
+ enum:
+ - mediatek,mt8188-padding
+ - mediatek,mt8195-mdp3-padding

reg:
maxItems: 1
--
2.18.0


2023-10-30 19:27:22

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH v8 16/16] dt-bindings: display: mediatek: padding: add compatible for MT8195


On Mon, 30 Oct 2023 18:00:22 +0800, Moudy Ho wrote:
> Add a compatible string for the PADDING block in MediaTek MT8195 that
> is controlled by MDP3.
>
> Signed-off-by: Moudy Ho <[email protected]>
> ---
> .../bindings/display/mediatek/mediatek,padding.yaml | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>

Acked-by: Rob Herring <[email protected]>

2023-10-31 07:38:58

by Moudy Ho (何宗原)

[permalink] [raw]
Subject: Re: [PATCH v8 16/16] dt-bindings: display: mediatek: padding: add compatible for MT8195

On Mon, 2023-10-30 at 18:00 +0800, Moudy Ho wrote:
> Add a compatible string for the PADDING block in MediaTek MT8195 that
> is controlled by MDP3.
>
> Signed-off-by: Moudy Ho <[email protected]>
> ---
> .../bindings/display/mediatek/mediatek,padding.yaml | 4
> +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git
> a/Documentation/devicetree/bindings/display/mediatek/mediatek,padding
> .yaml
> b/Documentation/devicetree/bindings/display/mediatek/mediatek,padding
> .yaml
> index db24801ebc48..19858d826808 100644
> ---
> a/Documentation/devicetree/bindings/display/mediatek/mediatek,padding
> .yaml
> +++
> b/Documentation/devicetree/bindings/display/mediatek/mediatek,padding
> .yaml
> @@ -20,7 +20,9 @@ description:
>
> properties:
> compatible:
> - const: mediatek,mt8188-padding
> + enum:
> + - mediatek,mt8188-padding

The compatible name of the mt8188 display padding, on which this patch
depends, has been changed in v11. Please disregard the typo here. it
will be updated in the next version as soon as possible.

Thanks,
Moudy

> + - mediatek,mt8195-mdp3-padding
>
> reg:
> maxItems: 1

2023-10-31 07:44:12

by Moudy Ho (何宗原)

[permalink] [raw]
Subject: Re: [PATCH v8 16/16] dt-bindings: display: mediatek: padding: add compatible for MT8195

On Mon, 2023-10-30 at 14:26 -0500, Rob Herring wrote:
>
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
>
> On Mon, 30 Oct 2023 18:00:22 +0800, Moudy Ho wrote:
> > Add a compatible string for the PADDING block in MediaTek MT8195
> that
> > is controlled by MDP3.
> >
> > Signed-off-by: Moudy Ho <[email protected]>
> > ---
> > .../bindings/display/mediatek/mediatek,padding.yaml | 4
> +++-
> > 1 file changed, 3 insertions(+), 1 deletion(-)
> >
>
> Acked-by: Rob Herring <[email protected]>
>

Hi Robe,

I apologize for not noticing the changes in the dependent patch and
failing to update one compatible name. I will remove the ACK tage and
would appreciate your help in reviewing it again in the next version.

Sincerely,
Moudy

2023-10-31 08:57:53

by Moudy Ho (何宗原)

[permalink] [raw]
Subject: Re: [PATCH v8 16/16] dt-bindings: display: mediatek: padding: add compatible for MT8195

On Tue, 2023-10-31 at 15:43 +0800, moudy ho wrote:
> On Mon, 2023-10-30 at 14:26 -0500, Rob Herring wrote:
> >
> > External email : Please do not click links or open attachments
> > until
> > you have verified the sender or the content.
> >
> > On Mon, 30 Oct 2023 18:00:22 +0800, Moudy Ho wrote:
> > > Add a compatible string for the PADDING block in MediaTek MT8195
> >
> > that
> > > is controlled by MDP3.
> > >
> > > Signed-off-by: Moudy Ho <[email protected]>
> > > ---
> > > .../bindings/display/mediatek/mediatek,padding.yaml |
> > > 4
> >
> > +++-
> > > 1 file changed, 3 insertions(+), 1 deletion(-)
> > >
> >
> > Acked-by: Rob Herring <[email protected]>
> >
>
> Hi Robe,
>
> I apologize for not noticing the changes in the dependent patch and
> failing to update one compatible name. I will remove the ACK tage and
> would appreciate your help in reviewing it again in the next version.
>
> Sincerely,
> Moudy

Hi Rob,

I am writing to apologize for mistakenly addressing you by the wrong
name in previous email. I kindly ask for your understanding regarding
the typo.

Sincerely,
Moudy