Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Sat, 27 Apr 2019 13:11:03 +0100
From:   Jonathan Cameron <jic23@jic23.retrosnub.co.uk>
To:     Alexandre <amergnat@baylibre.com>
Cc:     Peter Meerwald-Stadler <pmeerw@pmeerw.net>,
        linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org,
        baylibre-upstreaming@groups.io
Subject: Re: [PATCH v2 3/3] iio: Add PAT9125 optical tracker sensor
Message-ID: <20190427131103.1ce00b1c@archlinux>
In-Reply-To: <8abd482d-ae1e-ba86-c050-317fc0208fe6@baylibre.com>
References: <1556016046-31231-1-git-send-email-amergnat@baylibre.com>
        <1556016046-31231-4-git-send-email-amergnat@baylibre.com>
        <alpine.DEB.2.21.1904231302210.24175@vps.pmeerw.net>
        <8abd482d-ae1e-ba86-c050-317fc0208fe6@baylibre.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Tue, 23 Apr 2019 16:06:56 +0200
Alexandre <amergnat@baylibre.com> wrote:

> Hi Peter,
> 
> On 4/23/19 13:33, Peter Meerwald-Stadler wrote:
> >
> > how is this thing different from a mouse device?  
> 
> I developed this driver to detect the board movement which can't be 
> detected by accelerometer (very slow motion). I admit this use case can 
> be handled by an input, and I'm agree with you, PAT9125 driver could be 
> an input. But this chip is able to track different kind of motion (flat 
> and rotation), and additionally have an interrupt GPIO, so using it like 
> input limit the driver potential. This chip is designed to work in 
> industrial measurement or embedded systems, and the IIO API match with 
> these environments, so IIO is the best way to exploit the entire 
> potential of this chip.
> 
> As I understand (from 
> https://www.kernel.org/doc/html/v4.12/input/event-codes.html#mice ), 
> mouse driver must report values when the device move. This feature 
> souldn't be mandatory for an optical tracker driver, specially for cases 
> where user prefers to use buffer or poll only when he need data.

Given the 'placement' of this driver isn't totally obvious, please
make sure to cc linux-input and Dmitry (maintainer) on V3.

Probably no one will mind, but the last thing we want is someone
else to put forward an input driver and we end up with two drivers
for the same thing.

Thanks,

Jonathan

> 
> All other comments will be fixed in V3.
> 
> Regards,
> 
> Alexandre
>