Re: [lm-sensors] [PATCH 2.6.12-rc4 15/15] drivers/i2c/chips/adm1026.c: use dynamic sysfs callbacks

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

 



On 5/17/05, Grant Coady <[email protected]> wrote:
> Hi Yani,
> 
> On Tue, 17 May 2005 16:56:04 -0400, Yani Ioannou <[email protected]> wrote:
> >Those are the sysfs names? If so something looks wrong with the
> 
> Not the final ones, just from first macro expansion of driver
> source, that's why I'd like to see changes on w83627hf driver
> as I can test it right through.
>
> No, I'm not doing a proper compile, I'm intentionally doing partial
> compile of driver.c and _not_ including headers, ignoring errors due
> to missing headers.

Ah..OK, that is probably why, I've put the macros which would be
expanded in the first level in a separate header because it will
probably be shared amongst many drivers. Although I still don't see
where SENSOR_blah is coming from at all at the moment, if you can
track that down I'd be interested to know if its just something to do
with the script or a problem with the patch.


> Script is work in progress, updated to current version up at:
> 
>   http://scatter.mine.nu/hwmon/sysfs-names/

Ok, I'll have a look at it later.
> 
> >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
> 
> Not singletons, 3 of each (from an intermediate file):
> 
> adm1026.c       temp1_crit_enable       S_IRUGO S_IWUSR
> adm1026.c       temp2_crit_enable       S_IRUGO S_IWUSR
> adm1026.c       temp3_crit_enable       S_IRUGO S_IWUSR
> adm1026.c       pwm1    S_IRUGO S_IWUSR
> adm1026.c       pwm2    S_IRUGO S_IWUSR
> adm1026.c       pwm3    S_IRUGO S_IWUSR
> adm1026.c       temp1_auto_point1_pwm   S_IRUGO S_IWUSR
> adm1026.c       temp2_auto_point1_pwm   S_IRUGO S_IWUSR
> adm1026.c       temp3_auto_point1_pwm   S_IRUGO S_IWUSR
> adm1026.c       temp1_auto_point2_pwm   S_IRUGO
> adm1026.c       temp2_auto_point2_pwm   S_IRUGO
> adm1026.c       temp3_auto_point2_pwm   S_IRUGO
> 
> Yet, in related groups of three you have:
> 
> adm1026.c       SENSOR_temp1_crit       S_IRUGO S_IWUSR
> adm1026.c       SENSOR_temp2_crit       S_IRUGO S_IWUSR
> adm1026.c       SENSOR_temp3_crit       S_IRUGO S_IWUSR

Well I said mainly singletons :-), some of the attributes don't
benefit from the dynamic sysfs callbacks simply because they already
only use one callback for a few different attributes, I believe that's
the case with the non-singletons in this case.

Thanks,
Yani
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux