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

Moudy Ho posted 16 patches 1 year ago
There is a newer version of this series
[PATCH v8 16/16] dt-bindings: display: mediatek: padding: add compatible for MT8195
Posted by Moudy Ho 1 year ago
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
+      - mediatek,mt8195-mdp3-padding
 
   reg:
     maxItems: 1
-- 
2.18.0
Re: [PATCH v8 16/16] dt-bindings: display: mediatek: padding: add compatible for MT8195
Posted by Moudy Ho (何宗原) 1 year ago
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
Re: [PATCH v8 16/16] dt-bindings: display: mediatek: padding: add compatible for MT8195
Posted by Rob Herring 1 year ago
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>
Re: [PATCH v8 16/16] dt-bindings: display: mediatek: padding: add compatible for MT8195
Posted by Moudy Ho (何宗原) 1 year ago
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
Re: [PATCH v8 16/16] dt-bindings: display: mediatek: padding: add compatible for MT8195
Posted by Moudy Ho (何宗原) 1 year ago
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