All of lore.kernel.org
 help / color / mirror / Atom feed
* [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.