diff mbox series

[v8,16/16] dt-bindings: display: mediatek: padding: add compatible for MT8195

Message ID 20231030100022.9262-17-moudy.ho@mediatek.com (mailing list archive)
State New, archived
Headers show
Series introduce more MDP3 components in MT8195 | expand

Commit Message

Moudy Ho Oct. 30, 2023, 10 a.m. UTC
Add a compatible string for the PADDING block in MediaTek MT8195 that
is controlled by MDP3.

Signed-off-by: Moudy Ho <moudy.ho@mediatek.com>
---
 .../bindings/display/mediatek/mediatek,padding.yaml           | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Comments

Rob Herring (Arm) Oct. 30, 2023, 7:26 p.m. UTC | #1
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 <moudy.ho@mediatek.com>
> ---
>  .../bindings/display/mediatek/mediatek,padding.yaml           | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 

Acked-by: Rob Herring <robh@kernel.org>
Moudy Ho Oct. 31, 2023, 7:37 a.m. UTC | #2
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 <moudy.ho@mediatek.com>
> ---
>  .../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
Moudy Ho Oct. 31, 2023, 7:43 a.m. UTC | #3
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 <moudy.ho@mediatek.com>
> > ---
> >  .../bindings/display/mediatek/mediatek,padding.yaml           | 4
> +++-
> >  1 file changed, 3 insertions(+), 1 deletion(-)
> > 
> 
> Acked-by: Rob Herring <robh@kernel.org>
> 

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
Moudy Ho Oct. 31, 2023, 8:57 a.m. UTC | #4
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 <moudy.ho@mediatek.com>
> > > ---
> > >  .../bindings/display/mediatek/mediatek,padding.yaml           |
> > > 4
> > 
> > +++-
> > >  1 file changed, 3 insertions(+), 1 deletion(-)
> > > 
> > 
> > Acked-by: Rob Herring <robh@kernel.org>
> > 
> 
> 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
diff mbox series

Patch

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