Re: [PATCH v2 1/4] dt-bindings: reset: nuvoton,ma35d1-reset: add simple-mfd and child node support

[Joey Lu]

On 6/25/2026 3:51 PM, Krzysztof Kozlowski wrote:
> On Thu, Jun 25, 2026 at 10:39:55AM +0800, Joey Lu wrote:
> > The MA35D1 system-management syscon node hosts the USB PHY register
> > block at offset 0x60.  To model usb-phy@60 as a DT child of the syscon
> > node the binding must allow:
> Explain why do you need child node. If you have fixed device @0x60, you do
> not need DT child node at all. Compatible implies that child existence.
The commit message will be rewritten to state the motivation: the USB PHY
register block at offset 0x60 needs a referenceable DT node because consumer
drivers (dwc2, ehci0/ohci0, ehci1/ohci1) look it up via `phys = 
<&usb_phy N>`,
which requires an explicit node with a phandle. Optional per-board 
calibration
properties (`nuvoton,rcalcode`, `nuvoton,oc-active-high`) also need to be
expressed in DT.
> >    - simple-mfd as an optional third compatible so the MFD core can
> >      instantiate child platform devices.
> >
> >    - #address-cells and #size-cells (each const: 1) so child nodes can
> >      carry a reg property.
> >
> >    - An open child-node pattern (patternProperties "^.*@[0-9a-f]+$")
> >      to pass dt-schema validation.
> No. Do not explain what you did - we can read the diff. You must explain
> WHY you are doing that.
Addressed above.
> > Signed-off-by: Joey Lu <a0987203069@xxxxxxxxx>
> > ---
> >   .../bindings/reset/nuvoton,ma35d1-reset.yaml        | 13 ++++++++++++-
> >   1 file changed, 12 insertions(+), 1 deletion(-)
> >
> > diff --git a/Documentation/devicetree/bindings/reset/nuvoton,ma35d1-reset.yaml b/Documentation/devicetree/bindings/reset/nuvoton,ma35d1-reset.yaml
> > index 3ce7dcecd87a..1fda7e8f4b5d 100644
> > --- a/Documentation/devicetree/bindings/reset/nuvoton,ma35d1-reset.yaml
> > +++ b/Documentation/devicetree/bindings/reset/nuvoton,ma35d1-reset.yaml
> > @@ -19,6 +19,8 @@ properties:
> >       items:
> >         - const: nuvoton,ma35d1-reset
> >         - const: syscon
> > +      - const: simple-mfd
> > +    minItems: 2
> >   
> >     reg:
> >       maxItems: 1
> > @@ -26,6 +28,16 @@ properties:
> >     '#reset-cells':
> >       const: 1
> >   
> > +  '#address-cells':
> > +    const: 1
> > +
> > +  '#size-cells':
> > +    const: 1
> > +
> > +patternProperties:
> > +  "^.*@[0-9a-f]+$":
> This must be specific.
The `patternProperties` key will be changed from `"^.*@[0-9a-f]+$"` to
`"^usb-phy@[0-9a-f]+$"` so it matches only USB PHY child nodes, 
following the
naming convention used by existing vendor bindings.
> > +    type: object
> Missing ref and additionalProps. Please look at other simple-mfd.
`$ref: /schemas/phy/nuvoton,ma35d1-usb2-phy.yaml#` will be added under the
patternProperties entry, alongside `type: object`, so the child node will be
validated against the PHY binding schema. This follows the pattern used by
existing vendor bindings.

Thanks for review.
> > +
> >   required:
> >     - compatible
> >     - reg
> > @@ -43,4 +55,3 @@ examples:
> >           #reset-cells = <1>;
> >       };
> >   ...
> > -
> > --
> > 2.43.0