On 24/08/2023 17:30, Emmanuel Gil Peyrot wrote: > The BM92T36 is used in the Nintendo Switch as its USB-C Power Delivery > controller. > > Signed-off-by: Emmanuel Gil Peyrot <linkmauve@xxxxxxxxxxxx> > --- > .../bindings/misc/rohm,bm92txx.yaml | 67 +++++++++++++++++++ > MAINTAINERS | 5 ++ > 2 files changed, 72 insertions(+) > create mode 100644 Documentation/devicetree/bindings/misc/rohm,bm92txx.yaml That's not misc device but usb, so please put it in usb. > > diff --git a/Documentation/devicetree/bindings/misc/rohm,bm92txx.yaml b/Documentation/devicetree/bindings/misc/rohm,bm92txx.yaml > new file mode 100644 > index 000000000000..0322a7f096f0 > --- /dev/null > +++ b/Documentation/devicetree/bindings/misc/rohm,bm92txx.yaml > @@ -0,0 +1,67 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +# Copyright (C) 2023 Emmanuel Gil Peyrot > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/misc/rohm,bm92txx.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: BM92Txx USB-C Power Delivery Controller > + > +maintainers: > + - Emmanuel Gil Peyrot <linkmauve@xxxxxxxxxxxx> > + > +properties: > + compatible: > + enum: > + - rohm,bm92t10 > + - rohm,bm92t20 > + - rohm,bm92t30 > + - rohm,bm92t36 > + - rohm,bm92t50 Your driver suggests they are fully compatible. > + > + reg: > + maxItems: 1 > + > + rohm,dp-signal-toggle-on-resume: > + type: boolean > + description: | > + Do a toggle on resume instead of disable in suspend and enable in resume, > + because this also disables the LED effects. That's OS policy, not suitable for DT. Drop the property. > + > + rohm,led-static-on-suspend: > + type: boolean > + description: Dim or breathing dock LED. Ditto > + > + rohm,dock-power-limit-disable: > + type: boolean > + description: Disable the power limit in dock mode. Your description copies property name, so it is not much useful. Why this is board-configurable? > + > + rohm,dp-alerts-enable: > + type: boolean > + description: Enable DisplayPort alerts. Same questions. > + > + rohm,pd-5v-current-limit-ma: > + $ref: /schemas/types.yaml#/definitions/uint32 Use common property units. > + default: 2000 > + description: Current limit in mA when voltage is 5V. > + You miss connector, so binding is incomplete. All these properties look duplicating connector properties. > + rohm,pd-9v-current-limit-ma: > + $ref: /schemas/types.yaml#/definitions/uint32 > + default: 2000 > + description: Current limit in mA when voltage is 9V. > + > + rohm,pd-12v-current-limit-ma: > + $ref: /schemas/types.yaml#/definitions/uint32 > + default: 1500 > + description: Current limit in mA when voltage is 12V. > + > + rohm,pd-15v-current-limit-ma: > + $ref: /schemas/types.yaml#/definitions/uint32 > + default: 1200 > + description: Current limit in mA when voltage is 15V. > + > +required: > + - compatible > + - reg > + > +additionalProperties: false Missing example. You must have and it must be complete. Best regards, Krzysztof