Bug 79204

Summary: [NV50] No reported temperature/fan speed on GTX260
Product: xorg Reporter: Marcel Dopita <mdop>
Component: Driver/nouveauAssignee: Nouveau Project <nouveau>
Status: RESOLVED FIXED QA Contact: Xorg Project Team <xorg-team>
Severity: normal    
Priority: medium    
Version: unspecified   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
dmesg debug
none
sensors
none
gtx260 vbios
none
drm/nouveau/i2c: bump the i2c delay for the adt7473
none
sensors (after patch) none

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.