From: "Zhang, Yanmin" <yanmin_zhang@linux.intel.com>
To: linux-ia64@vger.kernel.org
Subject: Re: [Fedora-ia64-list] [PATCH] ide controller quirk to correct bad
Date: Wed, 14 Feb 2007 03:26:18 +0000 [thread overview]
Message-ID: <1171423578.2978.44.camel@ymzhang> (raw)
In-Reply-To: <45D1DE90.9080709@redhat.com>
On Tue, 2007-02-13 at 10:51 -0500, Prarit Bhargava wrote:
>
> Zhang, Yanmin wrote:
> > If ide controllers are at legacy mode, only the 4th BAR
> > is needed, so some BIOS initiate other BAR with incorrect
> > value. ata/ata_piix calls pci_enable_device on the ide
> > controller, which will check BAR resources. If the BAR
> > resource values are incorrect, pci_enable_device will fail,
> > and ata/ata_piix couldn't attach the ide controller.
> >
> > Below patch against 2.6.20 creates a quirk to correct the
> > bad BAR resources for a special ide controller which is
> > popular on tiger-4.
> >
> >
>
> If I understand the use of quirks, it is to fix hardware issues that
> cannot be resolved by bios fixes, etc.. ie) real HW problems. At least
> that's been my feeble understanding. If I'm wrong on that please
> correct me.
Correct.
Curent issue of ide controller also could be considered as ide hardware
issue. If the ide controller is at legacy mode, bar 4 is enough, but
my ide controller provides bar 5 as well as 4. If the controller
hardwires bar 5 as 0, there will be no such issue.
>
> Putting this sort of fix in opens up the kernel to resolving many
> vendors' bios issues within the kernel.
>
> I do understand that this is a special case -- it is unlikely a new bios
> will ship for this box. The way I see it, a future user of this
> platform will have to build kernels that use the old ide-cd/piix driver
> and/or patch the specific OS they are using with this patch.
From Alan's reply on linux-ide maillist, we could know another platform
PowerPC has the similiar issue. It assumes the BAR is not used if it is equal
to 0.
From the comments of function ide_pci_enable of the old ide driver, we could
see that such issue is not rare.
Is it better to put the patch into upstream kernel?
Yanmin
next prev parent reply other threads:[~2007-02-14 3:26 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-02-13 15:51 [Fedora-ia64-list] [PATCH] ide controller quirk to correct bad Prarit Bhargava
2007-02-14 3:15 ` [Fedora-ia64-list] [PATCH] ide controller quirk to correct Zhang, Yanmin
2007-02-14 3:26 ` Zhang, Yanmin [this message]
2007-02-15 13:46 ` [Fedora-ia64-list] [PATCH] ide controller quirk to correct badBAR Prarit Bhargava
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=1171423578.2978.44.camel@ymzhang \
--to=yanmin_zhang@linux.intel.com \
--cc=linux-ia64@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