search for: pwm1_enabl

...ecified Component: Driver/nouveau Product: xorg This is a collection of 2 bugs, which I think are related (hence why I am reporting them both here) as they both happen as a result of hibernate/restore: 1. After a hibernate/restore cycle, the fan management option (the value of pwm1_enable) is not restored; and also 2. After a hibernate/restore cycle, when automatic fan management is applied ("echo 2 > .../pwm1_enable"), this doesn't work - the nouveau driver takes the fan speed to 2 and keeps it there *regardless* of the temperature of the video card, which may c...

...Unclassified OS: All Reporter: mr.dash.four at googlemail.com Hardware: Other Status: NEW Version: unspecified Component: Driver/nouveau Product: xorg When I have done the following: echo 1 > /sys/class/hwmon/hwmon0/pwm1_enable echo 30 > /sys/class/hwmon/hwmon0/pwm1 pwm1=30 pwm1_enable=1 pwm1_max=100 pwm1_min=0 temp1_auto_point1_pwm=100 temp1_auto_point1_temp=90000 temp1_auto_point1_temp_hyst=3000 temp1_crit=115000 temp1_crit_hyst=2000 temp1_emergency=135000 temp1_emergency_hyst=5000 temp1_input=41000 temp1_max=950...

...t I am trying to lower the fan speed on. I can change the fan RPM from high to not-as-high but I want more control over it. It is too loud and it was quieter with the blob. Maybe the temps are too high to make it quieter. # echo 1 >/sys/devices/pci0000:00/0000:00:02.0/0000:01:00.0/i2c-6/6-002e/pwm1_enable # sensors fam15h_power-pci-00c4 Adapter: PCI adapter power1: 89.28 W (crit = 95.01 W) k10temp-pci-00c3 Adapter: PCI adapter temp1: +32.4 C (high = +70.0 C) (crit = +70.0 C, hyst = +67.0 C) adt7473-i2c-6-2e Adapter: nouveau-0000:01:00.0-2 in1: +3.00...

https://bugs.freedesktop.org/show_bug.cgi?id=80901 Priority: medium Bug ID: 80901 Assignee: nouveau at lists.freedesktop.org Summary: [NVCF] PWM fan speed too high under kernel 3.15 Severity: normal Classification: Unclassified OS: Linux (All) Reporter: gianni at rootcube.net Hardware: x86-64 (AMD64)

...intf(buf, PAGE_SIZE, "%d\n", nvkm_therm_fan_sense(therm)); } -static SENSOR_DEVICE_ATTR(fan1_input, S_IRUGO, nouveau_hwmon_show_fan1_input, +static SENSOR_DEVICE_ATTR(fan1_input, 0444, nouveau_hwmon_show_fan1_input, NULL, 0); static ssize_t @@ -392,7 +392,7 @@ nouveau_hwmon_set_pwm1_enable(struct device *d, struct device_attribute *a, else return count; } -static SENSOR_DEVICE_ATTR(pwm1_enable, S_IRUGO | S_IWUSR, +static SENSOR_DEVICE_ATTR(pwm1_enable, 0644, nouveau_hwmon_get_pwm1_enable, nouveau_hwmon_set_pwm1_enable, 0); @@ -431,7 +431,7 @@ nouveau_hwmon_set_pw...

...OR_DEVICE_ATTR(temp1_crit, S_IRUGO, amdgpu_hwmon_show_temp_thresh, NULL, 0); -static SENSOR_DEVICE_ATTR(temp1_crit_hyst, S_IRUGO, amdgpu_hwmon_show_temp_thresh, NULL, 1); -static SENSOR_DEVICE_ATTR(pwm1, S_IRUGO | S_IWUSR, amdgpu_hwmon_get_pwm1, amdgpu_hwmon_set_pwm1, 0); -static SENSOR_DEVICE_ATTR(pwm1_enable, S_IRUGO | S_IWUSR, amdgpu_hwmon_get_pwm1_enable, amdgpu_hwmon_set_pwm1_enable, 0); -static SENSOR_DEVICE_ATTR(pwm1_min, S_IRUGO, amdgpu_hwmon_get_pwm1_min, NULL, 0); -static SENSOR_DEVICE_ATTR(pwm1_max, S_IRUGO, amdgpu_hwmon_get_pwm1_max, NULL, 0); -static SENSOR_DEVICE_ATTR(fan1_input, S_IRUGO,...

...temp1_crit, S_IRUGO, amdgpu_hwmon_show_temp_thresh, NULL, 0); > -static SENSOR_DEVICE_ATTR(temp1_crit_hyst, S_IRUGO, amdgpu_hwmon_show_temp_thresh, NULL, 1); > -static SENSOR_DEVICE_ATTR(pwm1, S_IRUGO | S_IWUSR, amdgpu_hwmon_get_pwm1, amdgpu_hwmon_set_pwm1, 0); > -static SENSOR_DEVICE_ATTR(pwm1_enable, S_IRUGO | S_IWUSR, amdgpu_hwmon_get_pwm1_enable, amdgpu_hwmon_set_pwm1_enable, 0); > -static SENSOR_DEVICE_ATTR(pwm1_min, S_IRUGO, amdgpu_hwmon_get_pwm1_min, NULL, 0); > -static SENSOR_DEVICE_ATTR(pwm1_max, S_IRUGO, amdgpu_hwmon_get_pwm1_max, NULL, 0); > -static SENSOR_DEVICE_ATTR(fan1_i...

...-- Comment #10 from Krzysztof Katowicz-Kowalewski <vnd at vndh.net> --- As far as I've checked, the problem appeared in 3.15 kernel. I've tried hardened-sources-3.14.29 and there's no problem with the fan, at least in auto mode. But since you change the mode to manual (echo 1 > pwm1_enable), the fan gets mad as in 3.15 kernel branch. Related info from me: https://forums.gentoo.org/viewtopic-t-1009318.html -- You are receiving this mail because: You are the assignee for the bug. -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.freedes...

...- (In reply to Krzysztof Katowicz-Kowalewski from comment #10) > As far as I've checked, the problem appeared in 3.15 kernel. I've tried > hardened-sources-3.14.29 and there's no problem with the fan, at least in > auto mode. But since you change the mode to manual (echo 1 > pwm1_enable), > the fan gets mad as in 3.15 kernel branch. Related info from me: > https://forums.gentoo.org/viewtopic-t-1009318.html I didn't do anything to the kernel and I did change nothing in settings. the tests I have done are all in the live usb env and all from the 3.16 and above kernels....

https://bugs.freedesktop.org/show_bug.cgi?id=87885 Bug ID: 87885 Summary: Fan speed of Nvidia GeForce 9600GT (G94) constantly too high Product: xorg Version: unspecified Hardware: x86-64 (AMD64) OS: Linux (All) Status: NEW Severity: normal Priority: medium Component:

...at.com Hardware: Other Status: NEW Version: unspecified Component: Driver/nouveau Product: xorg It seems like the /sys/class/drm/card0/device/hwmon/hwmon0/pwm1 value for my GeForce 9600 GT (10de:0622) will never go above 80. If I do echo 1 > pwm1_enable (to set it to manual control), then try echoing values to pwm1, it basically acts as if any value above 80 is identical to 80. [root at adam hwmon0]# pwd /sys/class/drm/card0/device/hwmon/hwmon0 [root at adam hwmon0]# echo 79 > pwm1 [root at adam hwmon0]# cat pwm1 79 [root at adam hwmon0]# ech...

...Hardware: x86-64 (AMD64) Status: NEW Version: unspecified Component: Driver/nouveau Product: xorg I have Gigabyte GTX 560 Ti Ultra Durable card. (Logs for this issue are same as in Bug 78106) 1a) When I finish booting, nouveau selects AUTO mode (pwm1_enable: 2) and set's the fan power to 55% (pwm1: 55). 1b) Enable manual mode, set pwm1 to 100, enable auto mode. In both cases I would expect from the AUTO mode to lower PWM to go as low as pwm1_min if the temperature is low enough. My card temperature is 31-33 ?C but pwm1 is never lowered in AUTO...

...= nvxx_therm(&drm->client.device); - return snprintf(buf, PAGE_SIZE, "%d\n", nvkm_therm_fan_sense(therm)); + return nvkm_therm_fan_sense(therm); } -static SENSOR_DEVICE_ATTR(fan1_input, S_IRUGO, nouveau_hwmon_show_fan1_input, - NULL, 0); - static ssize_t -nouveau_hwmon_get_pwm1_enable(struct device *d, - struct device_attribute *a, char *buf) +static int +nouveau_hwmon_get_pwm1_enable(struct nouveau_drm *drm) { - struct drm_device *dev = dev_get_drvdata(d); - struct nouveau_drm *drm = nouveau_drm(dev); struct nvkm_therm *therm = nvxx_therm(&drm->client.device); -...

...= nvxx_therm(&drm->client.device); - return snprintf(buf, PAGE_SIZE, "%d\n", nvkm_therm_fan_sense(therm)); + return nvkm_therm_fan_sense(therm); } -static SENSOR_DEVICE_ATTR(fan1_input, S_IRUGO, nouveau_hwmon_show_fan1_input, - NULL, 0); - static ssize_t -nouveau_hwmon_get_pwm1_enable(struct device *d, - struct device_attribute *a, char *buf) +static int +nouveau_hwmon_get_pwm1_enable(struct nouveau_drm *drm) { - struct drm_device *dev = dev_get_drvdata(d); - struct nouveau_drm *drm = nouveau_drm(dev); struct nvkm_therm *therm = nvxx_therm(&drm->client.device); -...

...= nvxx_therm(&drm->client.device); - return snprintf(buf, PAGE_SIZE, "%d\n", nvkm_therm_fan_sense(therm)); + return nvkm_therm_fan_sense(therm); } -static SENSOR_DEVICE_ATTR(fan1_input, S_IRUGO, nouveau_hwmon_show_fan1_input, - NULL, 0); - static ssize_t -nouveau_hwmon_get_pwm1_enable(struct device *d, - struct device_attribute *a, char *buf) +static int +nouveau_hwmon_get_pwm1_enable(struct nouveau_drm *drm) { - struct drm_device *dev = dev_get_drvdata(d); - struct nouveau_drm *drm = nouveau_drm(dev); struct nvkm_therm *therm = nvxx_therm(&drm->client.device); -...

This v5 drops a check for attr_set. Versions: v1 -> v2: * Keep temp attrs as read only v2 -> v3: * Code fix-ups: struct and string as const and add return within switch due to fallthrough * Add Signed-off-by to all commits v3 -> v4: * Rever const to struct attribute. Kbuild complains. v4 -> v5: * Drops a check for attr_set in

...tf(buf, PAGE_SIZE, "%d\n", nvkm_therm_fan_sense(therm)); > + return nvkm_therm_fan_sense(therm); > } > -static SENSOR_DEVICE_ATTR(fan1_input, S_IRUGO, nouveau_hwmon_show_fan1_input, > - NULL, 0); > > - static ssize_t > -nouveau_hwmon_get_pwm1_enable(struct device *d, > - struct device_attribute *a, char *buf) > +static int > +nouveau_hwmon_get_pwm1_enable(struct nouveau_drm *drm) > { > - struct drm_device *dev = dev_get_drvdata(d); > - struct nouveau_drm *drm = nouveau_drm(dev); >...

This v6 fixes some comments pointed out by Martin Peres. Versions: v1 -> v2: * Keep temp attrs as read only v2 -> v3: * Code fix-ups: struct and string as const and add return within switch due to fallthrough * Add Signed-off-by to all commits v3 -> v4: * Rever const to struct attribute. Kbuild complains. v4 -> v5: * Drops a check for

Kbuild sent me an e-mail due to a fixup I introduced in v3. It complains due to an incompatible pointer type and it doesn't build. This version reverts it. Versions: v1 -> v2: * Keep temp attrs as read only v2 -> v3: * Code fix-ups: struct and string as const and add return within switch due to fallthrough * Add Signed-off-by to all commits v3 -> v4:

Hi, this is version v3 with some fix-ups: Versions: v1 -> v2: * Keep temp attrs as read only v2 -> v3: * Code fix-ups: struct and string as const and add return within switch due to fallthrough * Add Signed-off-by to all commits This patchseries replaces the deprecated hwmon_device_register function with the new one hwmon_device_register_with_info. It also does some cleanup. Here