powerpc/windfarm: Improve display of fan speeds in sysfs
Controls registered as RPM and PWM fans are now displayed with the "RPM" or "%" suffix respectively to make it clearer to the user what the value actually means since the fan type isn't otherwise obvious. Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
This commit is contained in:
parent
a94d72b136
commit
ea5c64b067
|
@ -164,13 +164,24 @@ static ssize_t wf_show_control(struct device *dev,
|
||||||
struct device_attribute *attr, char *buf)
|
struct device_attribute *attr, char *buf)
|
||||||
{
|
{
|
||||||
struct wf_control *ctrl = container_of(attr, struct wf_control, attr);
|
struct wf_control *ctrl = container_of(attr, struct wf_control, attr);
|
||||||
|
const char *typestr;
|
||||||
s32 val = 0;
|
s32 val = 0;
|
||||||
int err;
|
int err;
|
||||||
|
|
||||||
err = ctrl->ops->get_value(ctrl, &val);
|
err = ctrl->ops->get_value(ctrl, &val);
|
||||||
if (err < 0)
|
if (err < 0)
|
||||||
return err;
|
return err;
|
||||||
return sprintf(buf, "%d\n", val);
|
switch(ctrl->type) {
|
||||||
|
case WF_CONTROL_RPM_FAN:
|
||||||
|
typestr = " RPM";
|
||||||
|
break;
|
||||||
|
case WF_CONTROL_PWM_FAN:
|
||||||
|
typestr = " %";
|
||||||
|
break;
|
||||||
|
default:
|
||||||
|
typestr = "";
|
||||||
|
}
|
||||||
|
return sprintf(buf, "%d%s\n", val, typestr);
|
||||||
}
|
}
|
||||||
|
|
||||||
/* This is really only for debugging... */
|
/* This is really only for debugging... */
|
||||||
|
|
Loading…
Reference in New Issue