Greg, Mark, Please find (attached because my mailer wraps lines.) an incremental patch that applies on top of the previous one (the one from the first message of the thread) and that adds parenthesis to the macro as pointed out by Mark. Please apply, Alex. Quoting "Mark M. Hoffman" <mhoffman at lightlink.com>: > Hello: > > > Quoting "Mark M. Hoffman" <mhoffman at lightlink.com>: > > > A recent driver core patch *necessitates* the trailing semicolon for > > > DEVICE_ATTR(). > > * Alexandre d'Alton <alex at alexdalton.org> [2004-06-30 09:33:01 +0200]: > > In fact, I already put the semicolons on the iner calls to DEVICE_ATTR(), > but as > > when I "instanciate" > > the macros I put a semicolon, I thought that it was unnecessary on the last > one. > > If you confirm that I have to add them, I'll be glad to do it. > > No, you're right. Sorry for the noise. > Mark : No problem :-) better to put only almost perfect stuff on Linux. > Regards, > > -- > Mark M. Hoffman > mhoffman at lightlink.com > > -------------- next part -------------- A non-text attachment was scrubbed... Name: adm1031-mmh-incremental.diff Type: text/x-patch Size: 577 bytes Desc: not available Url : http://lists.lm-sensors.org/pipermail/lm-sensors/attachments/20040701/0a09578a/attachment.bin
