On Tue, Nov 11, 2025 at 05:49:24PM +0000, Conor Dooley wrote:

Tommaso, Rob,

On Mon, Nov 10, 2025 at 11:43:37PM +0100, Tommaso Merciai wrote:

quoted

Hi Conor,
Thanks for your comment!

On Mon, Nov 10, 2025 at 06:56:31PM +0000, Conor Dooley wrote:

quoted

On Mon, Nov 10, 2025 at 01:08:04PM +0100, Tommaso Merciai wrote:

quoted

Document the 'mux-controller' child node in the Renesas RZ/V2H(P)
USB2PHY reset binding to support describing the USB VBUS_SEL
multiplexer as a mux-controller.

This is required to properly configure the USB PHY VBUS source on
RZ/V2H(P), RZ/G3E SoCs.

Signed-off-by: Tommaso Merciai <tommaso.merciai.xr@bp.renesas.com>
---
v2->v3:
 - Manipulate mux-controller as an internal node.

Why is it a child node, rather than just putting the cell in the parent
reset node?

Getting "make dt_binding_check errors" [1] in v2
Adding #mux-state-cells = <1> into:

	usb20phyrst: reset-controller@15830000
	usb21phyrst: reset-controller@15840000

Nodes.

Please correct me if I'm wrong.

I think that that binding is not working as intended. Why require a
node-name pattern, when it matches on other things too:
select:
  anyOf:
    - properties:
        $nodename:
          pattern: '^mux-controller'
    - required:
        - '#mux-control-cells'
    - required:
        - '#mux-state-cells'
if the node name always contained mux-controller, the second two here
would not be needed. Looks to me like the intention was for it to allow
putting these control/state-cells properties into mfd type nodes.

I'd delete the node name property tbh. Rob, you converted this to
schema, what do you think?

Yeah, I'd probably just comment it out and drop the select.

I'm not sure what to do here in general. Matching on node names was 
useful early on, but that becomes less useful as we have schemas for 
everything. I'd still like to document standard node names in schemas 
somehow, but obviously we can't require a certain name when a node is 
multiple providers. Ideally, every node name in the list in the DT spec 
would be in schemas and we could generate that list.

Rob