All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jonathan Cameron <jic23@cam.ac.uk>
To: "Pandruvada,
	Srinivas" <srinivas.pandruvada@intel.com>,Jiri Kosina
	<jkosina@suse.cz>
Cc: "linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>,Jonathan
	Cameron <jic23@kernel.org>
Subject: RE: [PATCH 0/8] HID-Sensor: v2
Date: Thu, 14 Jun 2012 17:04:51 +0100	[thread overview]
Message-ID: <fa30cbdd-0aba-400e-a367-1e14ac82e57f@email.android.com> (raw)
In-Reply-To: <4FA419E87744DF4DAECD5BCE1214B7A91932B625@ORSMSX104.amr.corp.intel.com>



"Pandruvada, Srinivas" <srinivas.pandruvada@intel.com> wrote:

>Hi Jonath=
an 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 t=
hink about this approach?
Sounds like a sensible plan.  I would put the ind=
ividual drivers in iio/accel and similar though. 
>
>Thanks,
>Srinivas
>
>
=
>-----Original Message-----
>From: Jonathan Cameron [mailto:jic23@cam.ac.uk=
] 
>Sent: Thursday, June 14, 2012 6:40 AM
>To: Jiri Kosina
>Cc: Pandruvada,=
 Srinivas; linux-iio@vger.kernel.org; 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 su=
ggest you resend the patch to staging 
>>>> maintainers so that it gets app=
lied and we can work on polishing
>the 
>>>> driver there.
>>>
>>>> Also, w=
hat are the plans regarding moving IIO out of staging,
>please?
>>> The cor=
e is out of staging as of the current cycle.
>>
>> Ah, you are right, I mis=
sed 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 end=
s.  A couple went with the last 
>>> merge window, lots more a queued up fo=
r the next one.
>>>
>>> Generally any new drivers shouldn't go into staging=
 but directly
>into 
>>> drivers/iio.
>>
>> For hid sensors I'd probably pr=
efer drivers/hid though.
>There's some pretty strong moves to clasify drive=
rs by function not by
>'bus' (which is kind of what hid is I guess?)
>
>I d=
o wonder if this driver would work better as an mfd type device with
>the s=
ensor specific bits each having their own module?
>
>Honestly I've never be=
en 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 vers=
ion, been a busy 
>>> week and it's a fairly big review to do.
>>
>> Thanks=
,
>>

-- 
Sent from my Android phone with K-9 Mail. Please excuse my brevit=
y.

  reply	other threads:[~2012-06-14 16:04 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-10  1:53 [PATCH 0/8] HID-Sensor: v2 srinivas pandruvada
2012-06-10  1:53 ` [PATCH 1/8] HID-Sensors: Add ST Micro Sensor hub srinivas pandruvada
2012-06-10  1:53 ` [PATCH 2/8] HID-Sensors: Add Intel HID sensor srinivas pandruvada
2012-06-10  1:53 ` [PATCH 3/8] HID-Sensors: Sensor framework srinivas pandruvada
2012-06-10  1:53 ` [PATCH 4/8] HID-Sensors: Common attributes and interfaces srinivas pandruvada
2012-06-10  1:53 ` [PATCH 5/8] HID-Sensors: Added accelerometer 3D srinivas pandruvada
2012-06-10  1:53 ` [PATCH 6/8] HID-Sensors: Added Gyro 3D srinivas pandruvada
2012-06-10  1:53 ` [PATCH 7/8] HID-Sensors: Added Compass 3D srinivas pandruvada
2012-06-10  1:53 ` [PATCH 8/8] HID-Sensors: Added ALS srinivas pandruvada
2012-06-14 13:08 ` [PATCH 0/8] HID-Sensor: v2 Jiri Kosina
2012-06-14 13:12   ` Jonathan Cameron
2012-06-14 13:25     ` Jiri Kosina
2012-06-14 13:39       ` Jonathan Cameron
2012-06-14 15:25         ` Pandruvada, Srinivas
2012-06-14 16:04           ` Jonathan Cameron [this message]
2012-06-15 12:50           ` Jonathan Cameron
2012-06-15 15:27             ` Pandruvada, Srinivas
2012-06-15 13:12 ` Jonathan Cameron
2012-06-15 15:31   ` Pandruvada, Srinivas
2012-06-18  7:43     ` Jonathan Cameron

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=fa30cbdd-0aba-400e-a367-1e14ac82e57f@email.android.com \
    --to=jic23@cam.ac.uk \
    --cc=jic23@kernel.org \
    --cc=jkosina@suse.cz \
    --cc=linux-iio@vger.kernel.org \
    --cc=srinivas.pandruvada@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.