From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from wf-out-1314.google.com (wf-out-1314.google.com [209.85.200.171]) by ozlabs.org (Postfix) with ESMTP id E9D10DDE1F for ; Wed, 13 Aug 2008 16:06:28 +1000 (EST) Received: by wf-out-1314.google.com with SMTP id 24so5555995wfg.15 for ; Tue, 12 Aug 2008 23:06:27 -0700 (PDT) Message-ID: <792f5f410808122306u478b3ere8167016ef892323@mail.gmail.com> Date: Wed, 13 Aug 2008 07:06:27 +0100 From: "Tim Yamin" Sender: plasmaroo@gmail.com To: "Grant Likely" Subject: Re: [PATCH]: [MPC5200] Add ATA DMA support In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 References: <792f5f410808120643md72e679o39692c4bfa3f285e@mail.gmail.com> <792f5f410808120730j7b4be4f2n7a40de880178ccca@mail.gmail.com> <792f5f410808120830v311e0446kc8fddfb97d0b9ea6@mail.gmail.com> <792f5f410808122257t3711eb2lfc1e02bacade16a3@mail.gmail.com> Cc: linuxppc-dev@ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Wed, Aug 13, 2008 at 7:02 AM, Grant Likely wrote: > While on this topic; have you had a chance to address the comments you > received on v2 of your patch? I'm keen to get your change merged in, > but there are a few more things that need to be sorted out. I'm still trying to determine whether the locking stuff is needed or not. I think ultimately it might be a bug caused by hardware problems at this end as opposed to silicon problems affecting everybody, but I won't have an answer regarding this for a while because new hardware needs to be prototyped to test out this theory. I think the patch can probably be resubmitted with the locking stuff removed, and the driver can be marked as experimental for everybody to try out: if people experience any corruption problems I'm sure they can speak up :-) Any objections? Tim