Re: [PATCH] dt-bindings: soc: mediatek: Allow pwrap regs

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] dt-bindings: missing referenced mediatek,mt6380-regulator.yaml schema
--

commit 73b0295b13e9c91b5f296071d21498bc710f649e
Author: Rob Herring (Arm) <robh@kernel.org>

dt-bindings: soc: mediatek: Allow pwrap regs

This commit updates the MT7622 PMIC wrapper binding to add a child schema
reference for regulators. This change routes regulator subnodes to the
MT6380 regulator binding for evaluation.

> diff --git a/Documentation/devicetree/bindings/soc/mediatek/mediatek,pwrap.yaml b/Documentation/devicetree/bindings/soc/mediatek/mediatek,pwrap.yaml
> --- a/Documentation/devicetree/bindings/soc/mediatek/mediatek,pwrap.yaml
> +++ b/Documentation/devicetree/bindings/soc/mediatek/mediatek,pwrap.yaml
> @@ -105,6 +105,9 @@ properties:
>    pmic:
>      type: object
>  
> +  regulators:
> +    $ref: /schemas/regulator/mediatek,mt6380-regulator.yaml#

[Severity: Low]
Does this introduce an invalid schema reference for the regulators property?

It appears the referenced mediatek,mt6380-regulator.yaml file does not exist
in the repository, as the MT6380 regulator binding is currently a legacy text
file (mt6380-regulator.txt).

This can cause make dt_binding_check to fail when resolving the
cross-reference or silently skip validating the regulators node. Subsequent
patches in the series also do not appear to add the missing YAML file.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260921232527.3127894-1-robh@kernel.org?part=1