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 >