Re: [PATCH v2 3/4] dt-bindings: soc: mediatek: pwrap: Add compatible for MT8188

[Jason-ch Chen (陳建豪)]

Hi Krzysztof,

On Tue, 2023-07-11 at 08:07 +0200, Krzysztof Kozlowski wrote:
>  	 
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
>  On 11/07/2023 04:39, Jason-ch Chen wrote:
> > From: jason-ch chen <Jason-ch.Chen@xxxxxxxxxxxx>
> > 
> > Add MT8188 PMIC Wrapper compatible to binding document.
> > 
> > Signed-off-by: jason-ch chen <Jason-ch.Chen@xxxxxxxxxxxx>
> > ---
> >  .../devicetree/bindings/soc/mediatek/mediatek,pwrap.yaml     | 5
> +++++
> >  1 file changed, 5 insertions(+)
> > 
> > diff --git
> a/Documentation/devicetree/bindings/soc/mediatek/mediatek,pwrap.yaml
> b/Documentation/devicetree/bindings/soc/mediatek/mediatek,pwrap.yaml
> > index a06ac2177444..c2f22e7dbcfb 100644
> > ---
> a/Documentation/devicetree/bindings/soc/mediatek/mediatek,pwrap.yaml
> > +++
> b/Documentation/devicetree/bindings/soc/mediatek/mediatek,pwrap.yaml
> > @@ -50,6 +50,11 @@ properties:
> >                - mediatek,mt8186-pwrap
> >                - mediatek,mt8195-pwrap
> >            - const: syscon
> > +      - items:
> > +          - enum:
> > +              - mediatek,mt8188-pwrap
> 
> Why do you add duplicated entries? Test your DTS before sending
> patches.
In the next patch, I will remove the duplicated part. Thank you.

> 
> 
> Best regards,
> Krzysztof
>