Srinivas,
shall I hold off reviewing until your next version?
Might be a little while longer before I get to it either way!
I am in process of implementing as mfd device. In this case I can keep the core HID stuff in driver/hid and move sensor implantation using IIO to drivers/iio/hid-sensors.
In this way if some driver if just want to use HID sensor but want to use some other mechanism to communicate with user mode, they can do in their respective drivers.
What do you think about this approach?
Thanks,
Srinivas
-----Original Message-----
From: Jonathan Cameron [mailto:jic23@xxxxxxxxx]
Sent: Thursday, June 14, 2012 6:40 AM
To: Jiri Kosina
Cc: Pandruvada, Srinivas; linux-iio@xxxxxxxxxxxxxxx; Jonathan Cameron
Subject: Re: [PATCH 0/8] HID-Sensor: v2
On 6/14/2012 2:25 PM, Jiri Kosina wrote:
On Thu, 14 Jun 2012, Jonathan Cameron wrote:
As this is however a staging driver (and depends on IIO, which is a
staging infrastructure), I suggest you resend the patch to staging
maintainers so that it gets applied and we can work on polishing the
driver there.
Also, what are the plans regarding moving IIO out of staging, please?
The core is out of staging as of the current cycle.
Ah, you are right, I missed that this has already happened.
Drivers are moving out whenever someone has time to take a look at
each one and clean up any loose ends. A couple went with the last
merge window, lots more a queued up for the next one.
Generally any new drivers shouldn't go into staging but directly into
drivers/iio.
For hid sensors I'd probably prefer drivers/hid though.
There's some pretty strong moves to clasify drivers by function not by 'bus' (which is kind of what hid is I guess?)
I do wonder if this driver would work better as an mfd type device with the sensor specific bits each having their own module?
Honestly I've never been much of a stickler for where things are as long as someone is happy to look after them.
Sorry for my lack of responses on this revised version, been a busy
week and it's a fairly big review to do.
Thanks,
--
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
