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

From: Macpaul Lin
Date: Fri Sep 13 2024 - 13:48:41 EST


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@xxxxxxxxxxxx>
Signed-off-by: Macpaul Lin <macpaul.lin@xxxxxxxxxxxx>

[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