From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Brown Subject: Re: [PATCH 1/6] Core driver for WM97xx touchscreens Date: Thu, 14 Feb 2008 15:55:43 +0000 Message-ID: <20080214155542.GA10009@rakim.wolfsonmicro.main> References: <20080213140648.GD31617@rakim.wolfsonmicro.main> <1202912042-2631-1-git-send-email-broonie@opensource.wolfsonmicro.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from opensource.wolfsonmicro.com ([80.75.67.52]:56310 "EHLO opensource2.wolfsonmicro.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753256AbYBNPzp (ORCPT ); Thu, 14 Feb 2008 10:55:45 -0500 Content-Disposition: inline In-Reply-To: Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Dmitry Cc: Dmitry Torokhov , linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, Liam Girdwood , Graeme Gregory , Mike Arthur , Stanley Cai , Rodolfo Giometti , Russell King , Pete MacKay , Marc Kleine-Budde , Ian Molton , Vincent Sanders , Andrew Zabolotny On Thu, Feb 14, 2008 at 06:10:19PM +0300, Dmitry wrote: > 2008/2/13, Mark Brown : > > +#define WM97XX_ADCSEL_X 0x1000 /* x coord measurement */ > > +#define WM97XX_ADCSEL_Y 0x2000 /* y coord measurement */ > > +#define WM97XX_ADCSEL_PRES 0x3000 /* pressure measurement */ > shouldn't this be 0x4000 ? No, these are an enumeration on the chip rather than a bitmask.