From mboxrd@z Thu Jan 1 00:00:00 1970 From: bugzilla-daemon@freedesktop.org Subject: [Bug 73338] Fan speed in idle at 40% with radeonsi and at 18% with catalyst Date: Sun, 07 Sep 2014 09:38:37 +0000 Message-ID: References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1873723916==" Return-path: Received: from culpepper.freedesktop.org (unknown [131.252.210.165]) by gabe.freedesktop.org (Postfix) with ESMTP id 74EDD6E18B for ; Sun, 7 Sep 2014 02:38:37 -0700 (PDT) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============1873723916== Content-Type: multipart/alternative; boundary="1410082717.a7f8384.13335"; charset="us-ascii" --1410082717.a7f8384.13335 Date: Sun, 7 Sep 2014 09:38:37 +0000 MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" https://bugs.freedesktop.org/show_bug.cgi?id=73338 --- Comment #27 from Martin Peres --- (In reply to comment #26) > (In reply to comment #25) > > > I am the Nouveau developer who added fan management support. I would really > > advice you follow the hwmon recommendations and name this file pwm1. > > Sorry, I'm novice, didn't know about these. Don't feel bad, you cannot know possibly know everything ;) > Though it's not a hw interface > really, the exchange is done through io-mapped SMC registers... Not sure how more hw-oriented the interface could get :D That is still irrelevant to the discussion though. HWMON is software interface for the userspace. > > Ok, I'll make necessary changes. I assume it will allow lm-sensors to pick > it up automatically, is it correct? Yes, it will! Just to make sure we understood each others: - pwm1 (RW) exposes the current pwm value (what you called power) - pwm1_min(RW) min exposes the mininimum power that should be used by the fan - pwm1_max(RW) min exposes the maximum power that should be used by the fan - pwm1_enable (RW) exposes the current fan mode. 0 = DISABLED, 1 = MANUAL (the user writes the fan power he/she wants to pwm1), 2 = AUTOMATIC (linear fan management?). Here is Nouveau's documentation on fan management: http://cgit.freedesktop.org/nouveau/linux-2.6/tree/Documentation/thermal/nouveau_thermal I know it is a lot of changes I am asking you to do, but consistency across drivers is great feature! -- You are receiving this mail because: You are the assignee for the bug. --1410082717.a7f8384.13335 Date: Sun, 7 Sep 2014 09:38:37 +0000 MIME-Version: 1.0 Content-Type: text/html; charset="UTF-8"

Comment # 27 on bug 73338 from 
(In reply to comment #26)
> (In reply to comment #25)
> 
> > I am the Nouveau developer who added fan management support. I would really
> > advice you follow the hwmon recommendations and name this file pwm1.
> 
> Sorry, I'm novice, didn't know about these. 

Don't feel bad, you cannot know possibly know everything ;)

> Though it's not a hw interface
> really, the exchange is done through io-mapped SMC registers...

Not sure how more hw-oriented the interface could get :D That is still
irrelevant to the discussion though. HWMON is software interface for the
userspace.

> 
> Ok, I'll make necessary changes. I assume it will allow lm-sensors to pick
> it up automatically, is it correct?

Yes, it will!

Just to make sure we understood each others:
- pwm1 (RW) exposes the current pwm value (what you called power)
- pwm1_min(RW) min exposes the mininimum power that should be used by the fan
- pwm1_max(RW) min exposes the maximum power that should be used by the fan
- pwm1_enable (RW) exposes the current fan mode. 0 = DISABLED, 1 = MANUAL (the
user writes the fan power he/she wants to pwm1), 2 = AUTOMATIC (linear fan
management?).

Here is Nouveau's documentation on fan management:
http://cgit.freedesktop.org/nouveau/linux-2.6/tree/Documentation/thermal/nouveau_thermal

I know it is a lot of changes I am asking you to do, but consistency across
drivers is great feature!

You are receiving this mail because:
  • You are the assignee for the bug.
--1410082717.a7f8384.13335-- --===============1873723916== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel --===============1873723916==--