From: James Bottomley <James.Bottomley@suse.de>
To: Mike Christie <michaelc@cs.wisc.edu>
Cc: Vikas Chaudhary <vikas.chaudhary@qlogic.com>,
"linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
Ravi Anand <ravi.anand@qlogic.com>,
Lalit Chandivade <lalit.chandivade@qlogic.com>,
Karen Higgins <karen.higgins@qlogic.com>,
Prasanna Mumbai <prasanna.mumbai@qlogic.com>,
Sarang Radke <sarang.radke@qlogic.com>
Subject: Re: [PATCH 00/13] qla4xxx: Updates for scsi-misc-2.6
Date: Sat, 19 Mar 2011 10:49:43 -0400 [thread overview]
Message-ID: <1300546183.12679.9.camel@mulgrave.site> (raw)
In-Reply-To: <4D82D647.7050002@cs.wisc.edu>
On Thu, 2011-03-17 at 22:49 -0500, Mike Christie wrote:
> On 03/14/2011 12:23 AM, Vikas Chaudhary wrote:
> > James,
> >
> > Please apply these patches on scsi-misc-2.6 tree.
> >
> > Karen Higgins (2):
> > qla4xxx: cleanup DDB relogin logic during initialization
> > qla4xxx: Prevent other port reinitialization during remove_adapter
> >
> > Prasanna Mumbai (3):
> > qla4xxx: Do not send mbox command if FW is in failed state
> > qla4xxx: Do not retry ISP82XX initialization if H/W state is failed
> > qla4xxx: masking required bits of add_fw_options during initialization
> >
> > Sarang Radke (1):
> > qla4xxx: Use polling mode for disable interrupt mailbox completion
> >
> > Vikas Chaudhary (7):
> > qla4xxx: cleanup qla4xxx_initialize_ddb_list()
> > qla4xxx: remove unused ddb flag DF_NO_RELOGIN
> > qla4xxx: cleanup function qla4xxx_process_ddb_changed
> > qla4xxx: Add support for ql4xmaxqdepth command line parameter
> > qla4xxx: add support for ql4xsess_recovery_tmo cmd line param
> > qla4xxx: added new function qla4xxx_relogin_all_devices
> > qla4xxx: Update driver version to 5.02.00-k6
> >
>
> Patches look ok with updated 06/13 patch. James, Vikas actually sent a
> couple revisions of 06/13. The final one is here:
> http://marc.info/?l=linux-scsi&m=130035150917342&w=2
>
> I could not compile/checkpatch test the patches, because my
> mailer/browser keeps replacing = with =D. Other people seem to be able
> to see it ok, so I think it is problem on my side. Code was reviewed.
I don't think that's entirely your fault. This entire patch series is
whitespace damaged. Vikas, please resubmit with the correct mailer
settings so it can be applied.
> Reviewed-by: Mike Christie <michaelc@cs.wisc.edu>
I'll add this to the resubmit, thanks!
James
next prev parent reply other threads:[~2011-03-19 14:49 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-03-14 5:23 [PATCH 00/13] qla4xxx: Updates for scsi-misc-2.6 Vikas Chaudhary
2011-03-18 3:49 ` Mike Christie
2011-03-19 14:49 ` James Bottomley [this message]
2011-03-21 10:35 ` Vikas Chaudhary
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1300546183.12679.9.camel@mulgrave.site \
--to=james.bottomley@suse.de \
--cc=karen.higgins@qlogic.com \
--cc=lalit.chandivade@qlogic.com \
--cc=linux-scsi@vger.kernel.org \
--cc=michaelc@cs.wisc.edu \
--cc=prasanna.mumbai@qlogic.com \
--cc=ravi.anand@qlogic.com \
--cc=sarang.radke@qlogic.com \
--cc=vikas.chaudhary@qlogic.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).