From: shyam_iyer@dell.com
To: linux-ide@vger.kernel.org
Subject: [PROBLEM + PATCH] Sata port disabled by BIOS gets initialized by ata_piix drive
Date: Fri, 8 Jun 2007 09:46:08 -0400 [thread overview]
Message-ID: <9493.1181310368198.JavaMail.root@wombat.diezmil.com> (raw)
Problem
Sata disks are connected to onboard sata ports of PowerEdge 1900 (ESB2 southbridge chipset). If one of the port is disabled in the bios then they get enabled again by the ata_piix driver because of a default port map being written to the Port control and status register(0x91-93).
Instead the driver should preserve the bios setting by way of a fix like this.
Fix:
The BIOS configured PCS value must be anded logically with the default port map for the chipset. This way the BIOS information will not be lost by the reinitialization of the config space by the ata_piix driver. The below patch is against 2.6.21 kernel.
--- ata_piix.c.orig 2007-06-08 02:38:03.000000000 +0530
+++ ata_piix.c 2007-06-08 02:39:45.000000000 +0530
@@ -990,7 +990,7 @@ static void __devinit piix_init_pcs(stru
pci_read_config_word(pdev, ICH5_PCS, &pcs);
- new_pcs = pcs | map_db->port_enable;
+ new_pcs = pcs & map_db->port_enable;
if (new_pcs != pcs) {
DPRINTK("updating PCS from 0x%x to 0x%x\n", pcs, new_pcs);
Signed-off-by: Shyam Iyer <shyam_iyer@dell.com>
--
This message was sent on behalf of shyam_iyer@dell.com at openSubscriber.com
http://www.opensubscriber.com/messages/linux-ide@vger.kernel.org/topic.html
next reply other threads:[~2007-06-08 14:17 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-06-08 13:46 shyam_iyer [this message]
2007-06-09 5:51 ` [PROBLEM + PATCH] Sata port disabled by BIOS gets initialized by ata_piix drive Tejun Heo
2007-06-11 13:32 ` Re: [PROBLEM + PATCH] Sata port disabled by BIOS gets initialized shyam_iyer
2007-06-11 17:54 ` Tejun Heo
2007-06-12 18:31 ` Matt Sealey
2007-06-14 9:27 ` Tejun Heo
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=9493.1181310368198.JavaMail.root@wombat.diezmil.com \
--to=shyam_iyer@dell.com \
--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).