Re: [PATCH v2 2/2] dt-bindings: hwmon: update TI TPS23861 bindings with per-port schema

[Gregory Fuchedgi <gfuchedgi@gmail.com>]

On Tue, Aug 12, 2025 at 12:20 AM Krzysztof Kozlowski <krzk@kernel.org> wrote:
> > +  shutdown-gpios:
> powerdown-gpios, see gpio-consumer-common.yaml
It is called shutdown in the datasheet, but seems like neither powerdown nor
shutdown truly reflects its purpose. This pin doesn't power down the controller
itself. It shuts down the ports while keeping the controller available for
configuration over i2c. Should I call it ti,ports-shutdown-gpios or maybe
ti,shutdown-gpios? Any other suggestions?

> > +patternProperties:
> > +  "^port@[0-3]$":
> This goes to ports property.
Do you mean I should add another DT node that groups all ports? such as:
compatible = "ti,tps23861"; ports { port@0 {...} port@1 {...} }

If that's the case would it make sense to use "^.*$" pattern to allow any name
and drop the port label? Is patternProperties even needed in this case?

> > +        tps23861@28 {
> Node names should be generic. See also an explanation and list of
Ack. Should I also fix the existing example in this patch?

> > +            label = "my_poe_controller";
> Use useful names or just drop it.
I thought this is good as an example? A useful name would be board specific.