Re: [lm-sensors] [PATCH 4/5] hwmon: DNS323 rev C1 fan support

[Simon Guinot <simon@sequanux.org>]

[-- Attachment #1.1: Type: text/plain, Size: 1270 bytes --]

Hi Benjamin,

What is the status for the DNS323 fan support ?
Have this patch been merged yet ?

I am currently looking to add hwmon support for the GPIO fan found
on Network Space Max v2 boards. Obviously, some attributes are shared
with the DNS323 fan. Maybe there is some room for a generic GPIO fan
driver ?

Platform data could provide the board specific GPIO pinout and a speed
conversion array (rpm from/to GPIO value). Here is a proposal for this
platform data interface:

struct gpio_fan {
        const char      *name;
        unsigned        gpio;
        unsigned        active_low;
};

struct gpio_fan_speed {
        int value;
        int rpm;
};

struct gpio_fan_platform_data {
        struct gpio_fan         *alarm; /* fan alarm GPIO. */
        struct gpio_fan         *ctrl;  /* fan control GPIOs. */
        int                     num_ctrl;
        /*
         * Speed conversion array: rpm from/to GPIO bit field.
         * This array _must_ be sorted in ascending rpm order.
         */
        struct gpio_fan_speed   *speed;
        int                     num_speed;
};

Based on this informations the GPIO fan driver could perform the
speed conversions (pwm, rpm, GPIO value) and then provide a hwmon
interface.

Thanks for advice.

Simon

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

[-- Attachment #2: Type: text/plain, Size: 153 bytes --]

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors