* [lm-sensors] New abituguru version (Re:
@ 2005-12-29 11:24 Hans de Goede
0 siblings, 0 replies; only message in thread
From: Hans de Goede @ 2005-12-29 11:24 UTC (permalink / raw)
To: lm-sensors
Jim Cromie wrote:
> Hi Jean, Hans,
>
> This patch refactors SENSOR_DEVICE_ATTR_2 macro, following pattern set by
> SENSOR_ATTR. First it creates a new macro SENSOR_ATTR_2() which expands
> to an initialization expression, then it uses that in
> SENSOR_DEVICE_ATTR_2, which declares and initializes a struct
> sensor_device_attribute_2.
>
>
> This version differs from that in Hans' abituguru driver,
> in that it preserves the argument order used in the original
> SENSOR_DEVICE_ATTR_2 macro, Hans' version invites confusion by
> transposing _nr and _index between 2 closely related macros.
>
> Happily though, Hans coded with __SENSOR_DEVICE_ATTR2,
> so his driver could coexist with the macro defined here,
> and he could switch at his discretion.
>
> Signed-off-by: Jim Cromie <jim.cromie at gmail.com>
I've attached a new version which uses the new SENSOR_ATTR_2() macro as
defined in Jim's patch. Currently a copy of the macro surrounded by an
#ifndef SENSOR_ATTR_2 is included in the driver for compiling against a
clean 2.6.14, once this macro gets merged upstream the copy will be removed.
Regards,
Hans
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: abituguru.c.txt
Url: http://lists.lm-sensors.org/pipermail/lm-sensors/attachments/20051229/b0196dc1/abituguru.c-0001.txt
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2005-12-29 11:24 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-12-29 11:24 [lm-sensors] New abituguru version (Re: Hans de Goede
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.