Re: [PATCH v3] input: Add support for Hanwang tablet

From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
To: Xing Wei <weixing@hanwang.com.cn>
Cc: linux-input <linux-input@vger.kernel.org>,
	linux-kernel <linux-kernel@vger.kernel.org>,
	jkosina <jkosina@suse.cz>,
	Peter Hutterer <peter.hutterer@redhat.com>
Subject: Re: [PATCH v3] input: Add support for Hanwang tablet
Date: Wed, 1 Sep 2010 23:20:30 -0700	[thread overview]
Message-ID: <20100902062030.GA9416@core.coreip.homeip.net> (raw)
In-Reply-To: <201009021229501252232@hanwang.com.cn>

On Thu, Sep 02, 2010 at 12:29:50PM +0800, Xing Wei wrote:
> On Wed, Sep 01, 2010 mitry Torokhov wrote:
> > Hi Xing,
> > 
> > On Wed, Sep 01, 2010 at 02:55:44PM +0800, Xing Wei wrote:
> > > Hi Dmitry, it works well and I really appreciate your work. 
> > > BTN_3 - BTN_8 & ABS_DISTANCE events are not handled & used in our X system driver by now,
> > > sorry for my mistake.
> > >
> > 
> > Just to avoid confusion - are BTN_3 - BTN_8 not used by your X driver or
> > the hardware does not really report them? Because if it the former then
> > we should continue reporting them - some other user (as in
> > program/driver) might be interested in these.
> > 
> > Also, why do you develop your own X driver? Does not X's evdev driver
> > work properly for you?
> > 
> > Thanks.
> >  
> > 
> 
> Hi Dmitry, the hardware does really report BTN_3 - BTN_8.

OK, then we should keep reporting them then. I will add the patch to my
'next' branch and it should show up in 2.6.37.

> And I try to let the evdev driver(remove my X driver & .conf file ) take over the device,
> but seem like it has no reflection to the pressure and roll wheel pad,just run like a generic mouse.
> 

Then I suggest looking into modifying evdev so that it properly works
with your device. I think Peter Hutterer (CCed) is the man you need to
talk to.

Thanks.

-- 
Dmitry