From: Jesper Juhl <jesper.juhl@gmail.com>
To: "Ju, Seokmann" <Seokmann.Ju@lsil.com>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
James Bottomley <James.Bottomley@steeleye.com>,
"Kolli, Neela" <Neela.Kolli@engenio.com>,
linux-scsi@vger.kernel.org
Subject: Re: [PATCH] Megaraid cleanup
Date: Tue, 10 Jan 2006 22:03:20 +0100 [thread overview]
Message-ID: <9a8748490601101303r2e44e39at5f0f10bd82180add@mail.gmail.com> (raw)
In-Reply-To: <9738BCBE884FDB42801FAD8A7769C265142001@NAMAIL1.ad.lsil.com>
On 1/10/06, Ju, Seokmann <Seokmann.Ju@lsil.com> wrote:
> Hi,
> > Ok, I went into drivers/scsi/megaraid.c just to fix a few
> > compiler warnings,
> > but ended up doing a big spring-cleaning.
>
> Thank you for the patch first of all.
You're welcome.
> I would like to step through the changes in the patch and
> Will get back to you.
> I would think you've already verified the patch by testing
> The updated driver. Please confirm this.
>
As far as I could without having access to actual hardware, yes.
That means I've read through my changes and tried to verify them
visually/logically.
I've also compile tested it - it builds fine.
And I've also attempted to load it, and it fails (due to no hardware)
as it's supposed to.
Beyond that I'm unable to test it.
--
Jesper Juhl <jesper.juhl@gmail.com>
Don't top-post http://www.catb.org/~esr/jargon/html/T/top-post.html
Plain text mails only, please http://www.expita.com/nomime.html
next parent reply other threads:[~2006-01-10 21:03 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <9738BCBE884FDB42801FAD8A7769C265142001@NAMAIL1.ad.lsil.com>
2006-01-10 21:03 ` Jesper Juhl [this message]
[not found] <9738BCBE884FDB42801FAD8A7769C265142031@NAMAIL1.ad.lsil.com>
2006-01-24 15:41 ` [PATCH] Megaraid cleanup Jesper Juhl
2006-01-24 17:05 ` James Bottomley
2006-01-25 21:22 ` Jesper Juhl
2006-02-09 21:06 ` Jesper Juhl
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=9a8748490601101303r2e44e39at5f0f10bd82180add@mail.gmail.com \
--to=jesper.juhl@gmail.com \
--cc=James.Bottomley@steeleye.com \
--cc=Neela.Kolli@engenio.com \
--cc=Seokmann.Ju@lsil.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
/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