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

Message ID 20231030100022.9262-17-moudy.ho@mediatek.com
State New
Headers
Series introduce more MDP3 components in MT8195 |

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 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
  

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