Re: [PATCH v2 1/7] regulator: dt-bindings: mt6323: Convert to DT schema

[Macpaul Lin <macpaul.lin@mediatek.com>]

On 8/30/24 22:50, Rob Herring wrote:
> 	
> 
> External email : Please do not click links or open attachments until you 
> have verified the sender or the content.
> 
> On Fri, Aug 30, 2024 at 07:07:26PM +0800, Macpaul Lin wrote:
>> Convert this from the old style text based binding to the new DT schema
>> style.
>> 
>> The examples have been trimmed down and move to parent schema
>> mfd/mediatek,mt6397.yaml.
>> 
>> Add new maintainers and submitter from MediaTek.
>> 
>> Signed-off-by: Sen Chu <sen.chu@mediatek.com>
>> Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>

[snip]

>> +
>> +patternProperties:
>> +  "^(buck_)?v(pa|proc|sys)$":
>> +    description: Buck regulators
>> +    type: object
>> +    $ref: regulator.yaml#
>> +    properties:
>> +      regulator-allowed-modes: false
>> +    unevaluatedProperties: false
>> +
>> +  "^(ldo_)?v(camio|cn18)$":
> 
> Why are buck_ and ldo_ prefixes optional? The old binding didn't reflect
> actual (upstream) users? If so, that's fine, but mention that in the
> commit message.
> 
> Rob
> 

Will use "^buck_v" and "^ldo_v" as the prefixes of each items in 
patternProperties. Thanks!

Regards,
Macpaul Lin