From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joe Perches Subject: Re: PATCH 1/3 ] SCSI: Support 1024 scatter-gather list entries and improve AP while FW trapped and behaviors of EHs Date: Tue, 25 May 2010 22:11:29 -0700 Message-ID: <1274850689.1845.11.camel@Joe-Laptop.home> References: <2D7BE052D37A48D59461AB6B775FE070@areca4e4f859d4> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <2D7BE052D37A48D59461AB6B775FE070@areca4e4f859d4> Sender: linux-kernel-owner@vger.kernel.org To: nick.cheng@areca.com.tw Cc: linux-scsi@vger.kernel.org, James.Bottomley@HansenPartnership.com, linux-kernel@vger.kernel.org List-Id: linux-scsi@vger.kernel.org On Wed, 2010-05-26 at 12:55 +0800, Nick Cheng wrote: > 1. To support 4M/1024 scatter-gather list entry, reorganize struct > ARCMSR_CDB and struct CommandControlBlock > 2. To modify arcmsr_probe > 3. In order to help fix F/W issue, add the driver mode for type B card > 4. To improve AP's behavior while F/W resets > 5. To unify struct MessageUnit_B's members' naming in all OS drivers' > 6. To improve error handlers, arcmsr_bus_reset(), arcmsr_abort() > 7. To fix the arcmsr_queue_command() in bus reset stage, just let the > commands pass down to FW, don't block > Signed-off-by: Nick Cheng< nick.cheng@areca.com.tw > > --- Hi Nick. These patches don't use normal kernel coding style. Can you run scripts/checkpatch.pl against them please. You've also got continuation lines like this: + printk(KERN_NOTICE "arcmsr%d: wait 'flush adapter cache' \ + timeout, retry count down = %d \n", acb->host->host_no, which are defective because of the additional white space introduced into the format string after the line continuations. These should written as: printk(KERN_NOTICE "arcmsr%d: wait 'flush adapter cache' " "timeout, retry count down = %d\n", I did submit patches to Erich Chen to fix the some of the existing problems. http://patchwork.kernel.org/patch/75997/