RE: [PATCH 0/8] HID-Sensor: v2

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Jonathan and Jiri,

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


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux