From: Andi Shyti <andi@etezian.org>
To: Greg KH <gregkh@linuxfoundation.org>
Cc: Andi Shyti <andi@etezian.org>,
oatilla@gmail.com, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/2] etzkx accelerometer support
Date: Fri, 9 Aug 2013 00:21:08 +0200 [thread overview]
Message-ID: <20130808222108.GC11387@jack.whiskey> (raw)
In-Reply-To: <20130617025854.GB12522@kroah.com>
Hi Greg,
> > > > these two patches provide the etzkx device driver which supports
> > > > the following accelerometers:
> > >
> > > Why isn't this a drivers/iio/accel/ driver instead of a misc driver?
> > > Please make this an iio driver, then you will be using the correct
> > > user/kernel api to talk to your device.
> >
> > I was thinking it may be an issue, but it's not iio because the
> > time I developed the driver, iio was in the first staging period
> > and mainly because the on the board I'm using I cannot update the
> > Kernel to use the latest iio.
> >
> > I guess you don't like the promise "I can do it later" once the
> > supplier sends me a development board (which is planned) :)
>
> Sure, I'll gladly accept "I can do it later" from anyone, as long as you
> don't mind my, "I will merge it later" as well :)
>
> Sorry, I can't accept a driver now, that is known to going to be
> changing the kernel/user api in the future. So please, rewrite it to
> use IIO, that's what that interface / subsystem is there for.
What about submitting this accelerometer driver under staging
even if it's still not iio and slowly put it in shape?
Andi
next prev parent reply other threads:[~2013-08-08 22:21 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-16 21:40 [PATCH 0/2] etzkx accelerometer support Andi Shyti
2013-06-16 21:40 ` [PATCH 1/2] drivers/misc: etzkx driver for kxcnl/kxtnk accelerometer Andi Shyti
2013-06-16 21:40 ` [PATCH 2/2] Documentation: added etzkx driver documentation Andi Shyti
2013-06-16 21:51 ` [PATCH 0/2] etzkx accelerometer support Greg KH
2013-06-16 22:26 ` Andi Shyti
2013-06-17 2:58 ` Greg KH
2013-06-17 8:27 ` Andi Shyti
2013-08-08 22:21 ` Andi Shyti [this message]
2013-08-08 22:25 ` Greg KH
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=20130808222108.GC11387@jack.whiskey \
--to=andi@etezian.org \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=oatilla@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox