Hi Grant, > In my cross-reference checking of sysfs names, the via686a needs > special case treatment as it the only driver expands S_IWUSR to > 00200 with gcc -E. (00200 is the correct value for S_IWUSR). > > This is caused by the driver including <linux/delay.h>, it compiles > fine without that header but I am unable to test drive the change. Although your reason for removing that include is bad, it is actually correct to remove it, because we don't need it. I've removed it from the lm_sensors CVS driver as well. Thanks, -- Jean Delvare