linux-kernel - Re: [PATCH v2 3/9] dt-bindings: mfd: document ASUS Transformer EC

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20260210-sexy-grumpy-sambar-44edd2@quoll>
Date: Tue, 10 Feb 2026 10:22:35 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Svyatoslav Ryhel <clamor95@...il.com>
Cc: Lee Jones <lee@...nel.org>, Rob Herring <robh@...nel.org>, 
	Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, 
	Dmitry Torokhov <dmitry.torokhov@...il.com>, Pavel Machek <pavel@...nel.org>, Arnd Bergmann <arnd@...db.de>, 
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>, Sebastian Reichel <sre@...nel.org>, 
	Michał Mirosław <mirq-linux@...e.qmqm.pl>, Ion Agorria <ion@...rria.com>, devicetree@...r.kernel.org, 
	linux-kernel@...r.kernel.org, linux-input@...r.kernel.org, linux-leds@...r.kernel.org, 
	linux-pm@...r.kernel.org
Subject: Re: [PATCH v2 3/9] dt-bindings: mfd: document ASUS Transformer EC

On Mon, Feb 09, 2026 at 12:44:01PM +0200, Svyatoslav Ryhel wrote:
> Document embedded controller used in ASUS Transformer device series.
> 
> Signed-off-by: Svyatoslav Ryhel <clamor95@...il.com>
> ---
>  .../devicetree/bindings/mfd/asus,ec.yaml      | 152 ++++++++++++++++++
>  1 file changed, 152 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/mfd/asus,ec.yaml
> 
> diff --git a/Documentation/devicetree/bindings/mfd/asus,ec.yaml b/Documentation/devicetree/bindings/mfd/asus,ec.yaml
> new file mode 100644
> index 000000000000..1d1a62761b71
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/mfd/asus,ec.yaml

EC do not go to MFD, but to dedicated directory (there is such).

> @@ -0,0 +1,152 @@
> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/mfd/asus,ec.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: ASUS Transformer's Embedded Controller
> +
> +description:
> +  Several Nuvoton based Embedded Controller attached to an I2C bus,
> +  running a custom ASUS firmware, specific to the Asus Transformer
> +  device series.
> +
> +maintainers:
> +  - Svyatoslav Ryhel <clamor95@...il.com>
> +
> +properties:
> +  compatible:
> +    oneOf:

Drop oneOf

> +      - enum:
> +          - asus,ec-pad  # Pad part of Asus Transformer
> +          - asus,ec-dock # Dock part of Asus Transformer


Compatibles are way too generic. "ec" basically means you document all
Asus EC, which is for sure not true. You need specific compatible for
this specific model.

Missing blank line.

> +  reg:
> +    maxItems: 1
> +
> +  interrupts:
> +    maxItems: 1
> +
> +  request-gpio:

gpio is deprecated. gpios, look at any other binding.

> +    maxItems: 1
> +
> +  asus,dockram:
> +    $ref: /schemas/types.yaml#/definitions/phandle
> +    description: I2C device used to access power related functions.
> +
> +  asus,clear-factory-mode:
> +    type: boolean
> +    description: clear Factory Mode bit in EC control register

Why would this be a static/fixed property over lifecycle of all devices?

> +
> +  battery:
> +    type: object
> +    $ref: /schemas/power/supply/power-supply.yaml
> +    unevaluatedProperties: false
> +
> +    properties:
> +      compatible:
> +        const: asus,ec-battery
> +
> +    required:
> +      - compatible
> +
> +  charger:
> +    type: object
> +    $ref: /schemas/power/supply/power-supply.yaml
> +    additionalProperties: false
> +
> +    properties:
> +      compatible:
> +        const: asus,ec-charger
> +
> +      monitored-battery: true
> +
> +    required:
> +      - compatible
> +
> +  keyboard-ext:
> +    type: object
> +    description: top row of multimedia keys
> +    additionalProperties: false
> +
> +    properties:
> +      compatible:
> +        const: asus,ec-keys
> +
> +    required:
> +      - compatible
> +
> +  led:
> +    type: object
> +    additionalProperties: false
> +
> +    properties:
> +      compatible:
> +        const: asus,ec-led
> +
> +    required:
> +      - compatible
> +
> +  serio:

All of these children are pointless - no resources. Drop all of them,
it's btw explicitly documented rule in writing bindings.

Best regards,
Krzysztof