Thread (9 messages) 9 messages, 4 authors, 2024-03-07

Re: [PATCH v14 1/3] dt-bindings: hwmon: fan: Add fan binding to schema

From: Guenter Roeck <linux@roeck-us.net>
Date: 2024-03-07 18:46:50
Also in: linux-aspeed, linux-devicetree, linux-doc, linux-hwmon, linux-pwm, lkml

On Wed, Feb 21, 2024 at 06:40:23PM +0800, Billy Tsai wrote:
From: Naresh Solanki <redacted>

Add common fan properties bindings to a schema.

Bindings for fan controllers can reference the common schema for the
fan

child nodes:

  patternProperties:
    "^fan@[0-2]":
      type: object
      $ref: fan-common.yaml#
      unevaluatedProperties: false

Reviewed-by: Rob Herring <robh@kernel.org>
Signed-off-by: Naresh Solanki <redacted>
Signed-off-by: Billy Tsai <redacted>
Applied to hwmon-next. There have been a couple of questions, specifically:

- min-rpm of 1000 seems low. Fans with higher minimum RPM
  are known to exist.
- A default value of '2' was suggested for pulses-per-revolution.

Those questions can be addressed with subsequent patches if needed.

Guenter

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help