[PATCH] Add temperature-tracking mode to f71805f driver (v2)

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

 



Jean Delvare wrote:
>> --- linux-2.6-2.6.21/drivers/hwmon/f71805f.c.orig	2007-06-16 00:10:54.000000000 +0100
>> +++ linux-2.6-2.6.21/drivers/hwmon/f71805f.c	2007-06-19 21:07:26.000000000 +0100
[snip]
>> @@ -931,6 +1029,58 @@
>>  			  show_pwm_freq, set_pwm_freq, 2);
>>  static SENSOR_DEVICE_ATTR(pwm3_mode, S_IRUGO, show_pwm_mode, NULL, 2);
>>  
>> +static SENSOR_DEVICE_ATTR(pwm1_auto_channels_temp, S_IRUGO,
>> +					show_pwm_auto_channels, NULL, 0);
>> +static SENSOR_DEVICE_ATTR(pwm2_auto_channels_temp, S_IRUGO,
>> +					show_pwm_auto_channels, NULL, 1);
>> +static SENSOR_DEVICE_ATTR(pwm3_auto_channels_temp, S_IRUGO,
>> +					show_pwm_auto_channels, NULL, 2);
>> +static SENSOR_DEVICE_ATTR(pwm1_auto_channels_fan, S_IRUGO,
>> +					show_pwm_auto_channels, NULL, 0);
>> +static SENSOR_DEVICE_ATTR(pwm2_auto_channels_fan, S_IRUGO,
>> +					show_pwm_auto_channels, NULL, 1);
>> +static SENSOR_DEVICE_ATTR(pwm3_auto_channels_fan, S_IRUGO,
>> +					show_pwm_auto_channels, NULL, 2);
>
> Not sure what others think about this (Mark? Juerg?) but this (pwm1 to
> fan1 and to temp1, etc.) seems to be the usual mapping for chips where
> the mappings can't be changed, so I wonder what's the benefit of
> creating read-only sysfs files to express it. I think I'd be just as
> happy with no channel files at all (and it's cheaper.) No strong
> opinion though.

If anyone would like to express an opinion on this please do so as soon 
as possible.  The hardware on which I am able to test this will soon 
become unavailable.


Phil.








[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux