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, 21 Sep 2014 21:34:36 +0000 Message-ID: References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1836986256==" Return-path: Received: from culpepper.freedesktop.org (unknown [131.252.210.165]) by gabe.freedesktop.org (Postfix) with ESMTP id 984CB6E054 for ; Sun, 21 Sep 2014 14:34: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 --===============1836986256== Content-Type: multipart/alternative; boundary="1411335276.efdBe0eA4.20396"; charset="us-ascii" --1411335276.efdBe0eA4.20396 Date: Sun, 21 Sep 2014 21:34:36 +0000 MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" https://bugs.freedesktop.org/show_bug.cgi?id=73338 --- Comment #38 from Martin Peres --- (In reply to comment #37) > Created attachment 106588 [details] [review] > experimental patch for CI (BONAIRE tested) for enabling fan control with > hwmon interface Good, I left some comments on the patch :) > > (In reply to comment #35) > > Have fun ;) > > So here's the final version of what I've done. I found existing radeon hwmon > interface and added pwm1* sysfs files to it, so it exposes needed values > (some like pwm1_max are RO though). > > I need someone aware of radeon driver workflow to review this patch (maybe > Alex) and to point in a right direction. Posting the patch on radeon's ML (if there is none, then use dri-devel) and CC Alex or any other radeon dev working on PM. > > When all is good, I'll request some help from public (reddit maybe, or local > LUG) to test it extensively. When the radeon devs will think the patch is in good-enough shape, they will likely commit it. Make sure your patch does not change the default behavior and that should be fine. Users willing to have manual fan management will be the testers ;) -- You are receiving this mail because: You are the assignee for the bug. --1411335276.efdBe0eA4.20396 Date: Sun, 21 Sep 2014 21:34:36 +0000 MIME-Version: 1.0 Content-Type: text/html; charset="UTF-8"

Comment # 38 on bug 73338 from 
(In reply to comment #37)
> Created attachment 106588 [details] [review] [review]
> experimental patch for CI (BONAIRE tested) for enabling fan control with
> hwmon interface

Good, I left some comments on the patch :)

> 
> (In reply to comment #35)
> > Have fun ;)
> 
> So here's the final version of what I've done. I found existing radeon hwmon
> interface and added pwm1* sysfs files to it, so it exposes needed values
> (some like pwm1_max are RO though).
> 
> I need someone aware of radeon driver workflow to review this patch (maybe
> Alex) and to point in a right direction.

Posting the patch on radeon's ML (if there is none, then use dri-devel) and CC
Alex or any other radeon dev working on PM.

> 
> When all is good, I'll request some help from public (reddit maybe, or local
> LUG) to test it extensively.

When the radeon devs will think the patch is in good-enough shape, they will
likely commit it. Make sure your patch does not change the default behavior and
that should be fine.

Users willing to have manual fan management will be the testers ;)

You are receiving this mail because:
  • You are the assignee for the bug.
--1411335276.efdBe0eA4.20396-- --===============1836986256== 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 --===============1836986256==--