On Mon, Oct 25, 2010 at 11:27:42PM -0400, Zhang, Sonic wrote: > > > >-----Original Message----- > >From: Jonathan Cameron [mailto:jic23@xxxxxxxxx] > >Sent: Monday, October 25, 2010 6:54 AM > >To: Mike Frysinger > >Cc: linux-iio@xxxxxxxxxxxxxxx; > >device-drivers-devel@xxxxxxxxxxxxxxxxxxxx; Zhang, Sonic; Guenter Roeck > >Subject: Re: [PATCH 12/14] staging: iio: adc: new driver for > >ADT7408 temperature sensors > > > >On 10/23/10 21:29, Mike Frysinger wrote: > >> From: Sonic Zhang <sonic.zhang@xxxxxxxxxx> > >Here we enter new territory. This device is already supported > >in hwmon. Do we have a usecase that is not covered by that driver? > > > > I don't find a way to get event notification other than poll in hwmon framework. So, I move all temperature devic with interrupt available to IIO framework. > A much better approach, if such notifications are required and make sense, would be to add it to the hwmon framework. Rewriting drivers just to get notifications is not a solution. Besides, did you consider using kobject_uevent() ? The generic gpio driver uses that mechanism to create udev notifications on alarms. Guenter -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html