Hi Grant, Those are the sysfs names? If so something looks wrong with the SENSOR_ ones..maybe an unintended effect of the new sensor_device_attribute macro. I can't seem to find anything like that in "gcc -I ../../../include/ -E adm1026.c" though, would you mind sending me your script? Also make sure you are including the new i2c-sysfs.h header file. The group of attributes you've highlighted below don't use sensor_device_attribute on purpose because they don't benefit from the dynamic sysfs callbacks, mainly because they are singletons. Well its possible you could roll all the attribute callbacks into one if you wanted but that seems a bit extreme, and you'd have a large nasty if-else going on in it :-). BTW looks like a useful script :-), I'm always worried when doing these changes I might accidently change a sysfs attribute permission. Thanks, Yani On 5/17/05, Grant Coady <grant_lkml at dodo.com.au> wrote: > Hi Yani, > On Tue, 17 May 2005 07:18:51 -0400, Yani Ioannou <yani.ioannou at gmail.com> wrote: > > Following is derived from gcc -E adm1026.c with your patch (the > script strips all but base number from numbered names): > > adm1026.c SENSOR_fan1_div RW > adm1026.c SENSOR_fan1_input R > adm1026.c SENSOR_fan1_min RW > adm1026.c SENSOR_in0_input R > adm1026.c SENSOR_in0_max RW > adm1026.c SENSOR_in0_min RW > adm1026.c SENSOR_temp1_auto_point1_temp RW > adm1026.c SENSOR_temp1_auto_point1_temp_hyst R > adm1026.c SENSOR_temp1_crit RW > adm1026.c SENSOR_temp1_input R > adm1026.c SENSOR_temp1_max RW > adm1026.c SENSOR_temp1_min RW > adm1026.c SENSOR_temp1_offset RW > adm1026.c alarm_mask RW > adm1026.c alarms R > adm1026.c analog_out RW > adm1026.c gpio RW > adm1026.c gpio_mask RW > adm1026.c pwm1 RW << these also should also > adm1026.c pwm1_enable RW << have SENSOR_ in front > adm1026.c temp1_auto_point1_pwm RW << of them, seems you > adm1026.c temp1_crit_enable RW << missed a group? > adm1026.c vid R > adm1026.c vrm RW > > I'm assuming some magic elsewhere sees that 'SENSOR_' and removes > it before it gets to sysfs. I can test adm9240, w83627hf and it87 > drivers as well as watch the overall patterns as above. > > --Grant. > > _______________________________________________ lm-sensors mailing list lm-sensors at lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
