if (err)
                return err;
 
-       switch(value) {
+       switch (value) {
        case 1: /* manual, percent-based */
                rdev->asic->dpm.fan_ctrl_set_mode(rdev, FDO_PWM_MODE_STATIC);
                break;
                                         struct device_attribute *attr,
                                         char *buf)
 {
-       return sprintf(buf, "%i\n", 100); /* pwm uses percent-based fan-control */
+       return sprintf(buf, "%i\n", 255);
 }
 
 static ssize_t radeon_hwmon_set_pwm1(struct device *dev,
        if (err)
                return err;
 
+       value = (value * 100) / 255;
+
        err = rdev->asic->dpm.set_fan_speed_percent(rdev, value);
        if (err)
                return err;
        if (err)
                return err;
 
+       speed = (speed * 255) / 100;
+
        return sprintf(buf, "%i\n", speed);
 }