From: Willy Tarreau <w@1wt.eu>
To: Jeff Garzik <jeff@garzik.org>, Filippo Carletti <carletti@mobilia.it>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH] 2.4.34 - Add some ahci pci ids
Date: Wed, 6 Jun 2007 07:43:04 +0200 [thread overview]
Message-ID: <20070606054304.GA4241@1wt.eu> (raw)
In-Reply-To: <466005C6.4050303@garzik.org>
On Fri, Jun 01, 2007 at 07:40:54AM -0400, Jeff Garzik wrote:
> Filippo Carletti wrote:
> >This patch adds support for some chipsets in ahci driver.
> >The list comes from a patch for redhat kernel 2.6.9-34.
> >I only tested ICH8.
> >
> >The original patch contained also this lines (that I omitted):
> >+ /* JMicron-specific fixup: make sure we're in AHCI mode */
> >+ if (pdev->vendor == 0x197b)
> >+ pci_write_config_byte(pdev, 0x41, 0xa1);
>
> NAK. Don't omit obviously needed lines, if you are going to add JMicron
> PCI IDs.
Filippo,
would you please resend your patch including the lines you have omitted ?
Jeff is right, those lines are explicitly labelled as a fixup, so either
we merge the complete patch, or nothing at all.
I expect to release 2.4.35 around the end of this month, so if you can
provide me with an acceptable patch in time so that I can run at least
one -rc with it, I'm OK to merge it (provided that Jeff has no objection
of course). I will merge your other patch (VIA) at the same time.
Thanks in advance,
Willy
next prev parent reply other threads:[~2007-06-06 5:43 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-06-01 9:21 [PATCH] 2.4.34 - Add some ahci pci ids Filippo Carletti
2007-06-01 11:40 ` Jeff Garzik
2007-06-06 5:43 ` Willy Tarreau [this message]
2007-06-06 20:16 ` Filippo Carletti
2007-06-06 20:57 ` Willy Tarreau
2007-06-06 21:02 ` Filippo Carletti
2007-06-06 21:09 ` Willy Tarreau
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=20070606054304.GA4241@1wt.eu \
--to=w@1wt.eu \
--cc=carletti@mobilia.it \
--cc=jeff@garzik.org \
--cc=linux-kernel@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 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.