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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.