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

[Jason-ch Chen (陳建豪)]

Hi Krzysztof,

On Tue, 2023-07-18 at 08:36 +0200, Krzysztof Kozlowski wrote:
>  	 
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
>  On 18/07/2023 08:34, Jason-ch Chen (陳建豪) wrote:
> > Hi Krzysztof,
> > 
> > On Mon, 2023-07-17 at 21:55 +0200, Krzysztof Kozlowski wrote:
> >>   
> >> External email : Please do not click links or open attachments
> until
> >> you have verified the sender or the content.
> >>  On 17/07/2023 14:02, Jason-ch Chen wrote:
> >>> From: jason-ch chen <Jason-ch.Chen@xxxxxxxxxxxx>
> >>>
> >>> Add MT8188 PMIC Wrapper compatible to binding document.
> >>
> >> No improvements here - the compatible is already there. Subject
> has
> >> the
> >> same issue. Explain what you are doing and why.
> > 
> > The reason for changing the patch was that while MT8188 uses the
> same
> > pwrap as MT8195, the original code was applicable to 'compatible =
> > "mediatek,mt8188-pwrap"'.
> > 
> > To resolve the DTBS check warning that '['mediatek,mt8188-pwrap',
> > 'mediatek,mt8195-pwrap', 'syscon'] is too long', I modified the
> code as
> > per the current patch.
> 
> Explain it in commit msg. Fix the subject in new patch.

Thank you for your kind response. In the next version, I will explain
it and correct the subject accordingly.

> 
> Best regards,
> Krzysztof
> 
Regards,
Jason