diff mbox series

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

Message ID 20231030100022.9262-17-moudy.ho@mediatek.com
State Superseded
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:43 a.m. UTC | #2
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 | #3
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