All of lore.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <greg@kroah.com>
To: Andrew Morton <akpm@osdl.org>, kristen.c.accardi@intel.com
Cc: Bernhard Kaindl <bk@suse.de>,
	linux-kernel@vger.kernel.org, linux-pci@atrey.karlin.mff.cuni.cz,
	Alan Cox <alan@lxorguk.ukuu.org.uk>,
	Ivan Kokshaysky <ink@jurassic.park.msu.ru>
Subject: Re: [PATCH] PCI/Cardbus cards hidden, needs pci=assign-busses to fix
Date: Thu, 23 Feb 2006 17:47:55 -0800	[thread overview]
Message-ID: <20060224014755.GD25787@kroah.com> (raw)
In-Reply-To: <20060218014102.0647c0ce.akpm@osdl.org>

On Sat, Feb 18, 2006 at 01:41:02AM -0800, Andrew Morton wrote:
> Bernhard Kaindl <bk@suse.de> wrote:
> >
> > "In some cases, especially on modern laptops with a lot of PCI and cardbus
> >  bridges, we're unable to assign correct secondary/subordinate bus numbers
> >  to all cardbus bridges due to BIOS limitations unless we are using
> >  "pci=assign-busses" boot option." -- Ivan Kokshaysky (from a patch comment)
> > 
> >  Without it, Cardbus cards inserted are never seen by PCI because the
> >  parent PCI-PCI Bridge of the Cardbus bridge will not pass and translate
> >  Type 1 PCI configuration cycles correctly and the system will fail to
> >  find and initialise the PCI devices in the system.
> > 
> >  Reference: PCI-PCI Bridges: PCI Configuration Cycles and PCI Bus Numbering:
> >  http://www.science.unitn.it/~fiorella/guidelinux/tlk/node72.html
> > 
> >  The reason for this is that:
> >   ``All PCI busses located behind a PCI-PCI bridge must reside between the
> >  secondary bus number and the subordinate bus number (inclusive).''
> > 
> >  "pci=assign-busses" makes pcibios_assign_all_busses return 1 and this
> >  turns on PCI renumbering during PCI probing.
> > 
> >  Alan suggested to use DMI to make that function cause that on problem systems:
> >
> > [ snip patch which uses a DMI table to auto-set "pci=assign-busses" ]
> >
> 
> I guess if this is the only way in which we can do this, and nobody has any
> better solutions then sure, it'll get people's machines going.  We'll be
> forever patching that table though.
> 
> But _does_ anyone have any better solutions?

This patch might not be needed at all, as per Kristen's recent comments
on the linux-pci mailing list, and her small patch that is already in
the -mm tree.

I'll let the discussion continue on that list.

thanks,

greg k-h

  reply	other threads:[~2006-02-24  1:49 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-01-05 23:46 Problem with pci_fixups in drivers/pci/probe.c alan
2006-01-05 23:56 ` Dominik Brodowski
2006-01-06 13:48 ` Alan Cox
2006-02-16 22:06   ` [PATCH] PCI/Cardbus cards hidden, needs pci=assign-busses to fix Bernhard Kaindl
2006-02-18  9:41     ` Andrew Morton
2006-02-24  1:47       ` Greg KH [this message]
2006-02-24 13:16         ` Dominik Brodowski
2006-04-14 14:01       ` [PATCH] Cardbus cards hidden, fixup parent subordinate# carefully Bernhard Kaindl

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=20060224014755.GD25787@kroah.com \
    --to=greg@kroah.com \
    --cc=akpm@osdl.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=bk@suse.de \
    --cc=ink@jurassic.park.msu.ru \
    --cc=kristen.c.accardi@intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pci@atrey.karlin.mff.cuni.cz \
    /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.