[Bug 29572] Radeon card reports wrong temperature when switched off

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.kernel.org/show_bug.cgi?id=29572





--- Comment #5 from Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>  2011-02-24 05:14:11 ---
(In reply to comment #4)
> The old behavior was wrong.

Don't care really. We shouldn't change interfaces.

>  The temperature value in the register was
> interpreted incorrectly prior to my recent patch (improper handling of signed
> values).

That seems unrelated.

>  Also, if the card is disabled, the value of the mmio registers is
> undefined.

So reads should have returned -EINVAL from day one.  Too late to fix that.  The
best thing to do now would be to detect this situation and to return zero,
preserving the API.

-- 
Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching the assignee of the bug.

------------------------------------------------------------------------------
Free Software Download: Index, Search & Analyze Logs and other IT data in 
Real-Time with Splunk. Collect, index and harness all the fast moving IT data 
generated by your applications, servers and devices whether physical, virtual
or in the cloud. Deliver compliance at lower cost and gain new business 
insights. http://p.sf.net/sfu/splunk-dev2dev 
--
_______________________________________________
Dri-devel mailing list
Dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux