Re: [PATCH 0/8] HID-Sensor: v2
|[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]|
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