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

[Krzysztof Kozlowski]

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.

Best regards,
Krzysztof