Comment # 27
on bug 73338
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.
_______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
