From: Thorsten Leemhuis <fedora@leemhuis.info>
To: Christian Lamparter <chunkeey@web.de>
Cc: Chuck Ebbert <cebbert@redhat.com>, Jeff Garzik <jeff@garzik.org>,
IDE/ATA development list <linux-ide@vger.kernel.org>,
Jason Gaston <jason.d.gaston@intel.com>
Subject: Re: [patch 2.6.22-rc6] ATA: add a PCI ID for Intel Santa Rosa PATA controller
Date: Mon, 02 Jul 2007 11:07:32 +0200 [thread overview]
Message-ID: <4688C054.2030208@leemhuis.info> (raw)
In-Reply-To: <468886E1.7080302@leemhuis.info>
On 02.07.2007 07:02, Thorsten Leemhuis wrote:
> On 28.06.2007 21:47, Chuck Ebbert wrote:
>> On 06/28/2007 03:16 PM, Chuck Ebbert wrote:
>>
>> [Forgot to mark subj: as a patch; attached]
>>
>>> From: Christian Lamparter <chunkeey@web.de>
>>>
>>> ATA: add a PCI ID for Intel Santa Rosa PATA controller.
>>>
>>> Signed-off-by: Christian Lamparter <chunkeey@web.de>
>>> ---
>> [...]
>>> + /* ICH8 Mobile PATA Controller */
>>> + { 0x8086, 0x2850, PCI_ANY_ID, PCI_ANY_ID, 0, 0, ich_pata_133 },
>
> FYI: Patch works fine for me on my brand new Dell D630 (Santa Rosa) --
> the DVD-Drive gets recognized with it applied.
Hmmm. From looking at the ICH8-Datasheet (
http://www.intel.com/design/chipsets/datashts/313056.htm
or http://download.intel.com/design/chipsets/datashts/31305603.pdf )
page 189 and later it seems to me that the ICH8M (just as all the other
Intel-Chipsets I know) officially only supports Ultra ATA 100 (and not
133!).
So is the ich_pata_133 correct in the patch (see quoted line above)?
CU
thl
next prev parent reply other threads:[~2007-07-02 9:07 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-06-28 19:16 ATA: add a PCI ID for Intel Santa Rosa PATA controller Chuck Ebbert
2007-06-28 19:47 ` [patch 2.6.22-rc6] " Chuck Ebbert
2007-06-29 14:31 ` Jeff Garzik
2007-07-02 5:02 ` Thorsten Leemhuis
2007-07-02 9:07 ` Thorsten Leemhuis [this message]
2007-07-02 11:24 ` Chr
2007-07-02 11:44 ` Thorsten Leemhuis
2007-07-02 14:36 ` Chr
2007-07-04 14:03 ` Thorsten Leemhuis
2007-07-05 20:30 ` Chr
2007-07-06 19:06 ` Gaston, Jason D
2007-07-06 19:45 ` Chr
2007-07-06 20:01 ` Thorsten Leemhuis
2007-07-06 20:43 ` Jeff Garzik
2007-07-05 21:17 ` Alan Cox
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=4688C054.2030208@leemhuis.info \
--to=fedora@leemhuis.info \
--cc=cebbert@redhat.com \
--cc=chunkeey@web.de \
--cc=jason.d.gaston@intel.com \
--cc=jeff@garzik.org \
--cc=linux-ide@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;
as well as URLs for NNTP newsgroup(s).