From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Brownell Subject: Re: [PATCH 1/4] SPI: tsc2xxx core Date: Tue, 14 Aug 2007 13:12:47 -0700 Message-ID: <200708141312.48347.david-b@pacbell.net> References: <20070814191229.27333.62004.stgit@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20070814191229.27333.62004.stgit@localhost.localdomain> Content-Disposition: inline List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-omap-open-source-bounces@linux.omap.com Errors-To: linux-omap-open-source-bounces@linux.omap.com To: Ragner Magalhaes Cc: linux-omap-open-source@linux.omap.com List-Id: linux-omap@vger.kernel.org On Tuesday 14 August 2007, Ragner Magalhaes wrote: > This is the start to build tsc2xxx core with all sharable routines > between tsc2101, tsc2102, tsc2301 ... and more later tsc2100, tsc2111, etc. I'm getting confused here. What's the plan? Is this an entirely different approach from the tsc210x stuff ... and if so, why? I took a look at the tsc2101 bits, and I'm puzzled by why the original "int tsc2101_read_sync()" became "u16 tsc2101_read_sync()". First, why return "int" instead of "negative erno or value"? Not that errors will be common, but returning "u16" means they can't ever be reported or handled... Second, since that just wraps tsc2xxx_read_sync(), why even bother wrapping it? Better to call that directly ... more efficient and easier to understand/follow. ... and for that matter, the single-register read/write commands seem like they'd best use spi_write_then_read(), since DMA to stacks is unsafe and nonportable. I certainly like the idea of having *one* core for all these TSC chips ... with reusable bits on top. And I know that audio will be one of the nastier bits, so being able to reuse e.g. the H2 (and H3?) audio support right away seems like a win. Do your patches also give us shared tsc2101/2102 touchscreen and hwmon support? I just don't see how these parts are expected to fit together yet. The patch from Andrzej seemed a bit more clear, although it did not cover as much ground at the beginning. I suspect you have a plan, but it's just not yet apparent to me... - Dave