Bug 79204 - [NV50] No reported temperature/fan speed on GTX260
Summary: [NV50] No reported temperature/fan speed on GTX260
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Driver/nouveau (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Nouveau Project
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-05-25 10:15 UTC by Marcel Dopita
Modified: 2014-05-27 08:59 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
dmesg debug (59.95 KB, text/plain)
2014-05-25 10:15 UTC, Marcel Dopita
no flags Details
sensors (1.15 KB, text/plain)
2014-05-25 10:15 UTC, Marcel Dopita
no flags Details
gtx260 vbios (41.10 KB, application/octet-stream)
2014-05-25 10:16 UTC, Marcel Dopita
no flags Details
drm/nouveau/i2c: bump the i2c delay for the adt7473 (1.16 KB, patch)
2014-05-25 13:31 UTC, Martin Peres
no flags Details | Splinter Review
sensors (after patch) (1.79 KB, text/plain)
2014-05-25 17:42 UTC, Marcel Dopita
no flags Details

Description Marcel Dopita 2014-05-25 10:15:26 UTC
Created attachment 99756 [details]
dmesg debug

Nouveau doesn't report any temperature nor fan speed when using GTX260 (Gigabyte N26-896H-B). 
Reporting this because of discussion in bug 78116 as I didn't notice the fan management chip (the ADT7473) in logs.
Comment 1 Marcel Dopita 2014-05-25 10:15:44 UTC
Created attachment 99757 [details]
sensors
Comment 2 Marcel Dopita 2014-05-25 10:16:14 UTC
Created attachment 99758 [details]
gtx260 vbios
Comment 3 Martin Peres 2014-05-25 13:31:17 UTC
Created attachment 99767 [details] [review]
drm/nouveau/i2c: bump the i2c delay for the adt7473

Your card has an external sensor, but it doesn't answer to requests. This may be due to us driving the communication too fast. Please try this patch that halves the communication baudrate with your adt7473.
Comment 4 Marcel Dopita 2014-05-25 17:42:33 UTC
Created attachment 99785 [details]
sensors (after patch)

Seems that it's reporting accurate values. The temperature seems right and does change, it has single fan which is adjustable and correctly responds to pwm1 changes after switching to manual mode.
Comment 5 Martin Peres 2014-05-25 18:13:52 UTC
(In reply to comment #4)
> Created attachment 99785 [details]
> sensors (after patch)
> 
> Seems that it's reporting accurate values. The temperature seems right and
> does change, it has single fan which is adjustable and correctly responds to
> pwm1 changes after switching to manual mode.

Great, I'll send the patch to the ML! Thanks for testing!
Comment 6 Martin Peres 2014-05-27 08:59:33 UTC
Patch merged in Darktama's tree: http://cgit.freedesktop.org/~darktama/nouveau/commit/?id=74eec65d8f8041ead74cf301ed7baa115178aae6

Thanks again for reporting this bug!


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.