From mboxrd@z Thu Jan 1 00:00:00 1970 From: Grant Likely Subject: Re: [PATCH v3 1/1] davinci: spi: replace existing driver Date: Thu, 8 Jul 2010 13:34:02 -0600 Message-ID: References: <1278617003-10295-1-git-send-email-bniebuhr@efjohnson.com> <1278617003-10295-2-git-send-email-bniebuhr@efjohnson.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Cc: spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org, davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org To: Brian Niebuhr Return-path: In-Reply-To: <1278617003-10295-2-git-send-email-bniebuhr-JaPwekKOx1yaMJb+Lgu22Q@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: spi-devel-general-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: linux-spi.vger.kernel.org On Thu, Jul 8, 2010 at 1:23 PM, Brian Niebuhr wrote: > INTRODUCTION > > I have been working on a custom OMAP-L138 board that has multiple spi > devices (seven) on one controller. =A0These devices have a wide range of > transfer parameters (speed, phase, polarity, internal and gpio chip > selects). =A0During my testing I found multiple errors in the davinci spi > driver as a result of this complex setup. =A0The primary issues were: Nack. Sorry for being abrupt, but the patch still is full of unrelated whitespace changes (tabs converted to spaces) which makes the patch very difficult to review. This needs to be fixed. Let me know if you need help with how to do this. g. ---------------------------------------------------------------------------= --- This SF.net email is sponsored by Sprint What will you do first with EVO, the first 4G phone? Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first