On Thu, Feb 10, 2005 at 09:43:18AM -0700, Bjorn Helgaas wrote:
> On Wed, 2005-02-09 at 21:16 -0500, Adam Belay wrote:
> > On Wed, Feb 09, 2005 at 01:12:47PM -0700, Bjorn Helgaas wrote:
> > > PNP ports tend to be built-in, and discovering them after
> > > PCI ports means the names of the built-in ports can change
> > > if you add or remove PCI ports.
> > > 
> > > (And yes, we should look at getting rid of 8250_acpi.c
> > > now that we have PNPACPI, but that's for another patch.)
> > > 
> > > Signed-off-by: Bjorn Helgaas <[EMAIL PROTECTED]>>
> > 
> > I think this used to be the old behavior.  It was changed because some
> > of the resources reserved by pnp would also be reserved by pci, causing
> > pci to fail.  We should see if this is still the case.
> 
> That must be before the 2.5 series ...  Everything in the BK
> history (which admittedly only goes back to 7/21/2002) has
> PCI before PNP.

Yeah, I think it was quite a while ago.

> 
> Wouldn't it be a bug if PNP described a resource that can be
> discovered by standard PCI discovery?

Maybe.  It's usually one of these:

pnp: 00:09: ioport range 0x4d0-0x4d1 has been reserved
pnp: 00:09: ioport range 0xcf8-0xcff could not be reserved
pnp: 00:0b: ioport range 0x800-0x87f has been reserved

The "could not be reserved" was touched by pci.  From memory, I think
it's the pci configuration range.

#cat /proc/ioports
0000-001f : dma1
0020-0021 : pic1
0040-005f : timer
0060-006f : keyboard
0070-0077 : rtc
0080-008f : dma page reg
00a0-00a1 : pic2
00c0-00df : dma2
00f0-00ff : fpu
0170-0177 : ide1
01f0-01f7 : ide0
0376-0376 : ide1
0378-037a : parport0
03c0-03df : vga+
03f6-03f6 : ide0
03f8-03ff : serial
0400-047f : 0000:00:1f.0
04d0-04d1 : pnp 00:09
0500-053f : 0000:00:1f.0
0778-077a : parport0
0800-087f : pnp 00:0b
0cf8-0cff : PCI conf1
d800-d8ff : 0000:01:08.0
  d800-d8ff : tulip
df80-df9f : 0000:01:09.0
  df80-df9f : EMU10K1
dfe0-dfe7 : 0000:01:0b.0
  dfe0-dfe7 : serial
dff0-dff7 : 0000:01:09.1
  dff0-dff7 : emu10k1-gp
ef80-ef9f : 0000:00:1f.2
  ef80-ef9f : uhci_hcd
efa0-efaf : 0000:00:1f.3
ffa0-ffaf : 0000:00:1f.1
  ffa0-ffa7 : ide0
  ffa8-ffaf : ide1

Also here's the resources reported by pnpbios:

/sys/devices/pnp0/00:09# cat resources
state = active
io 0x4d0-0x4d1
io 0xcf8-0xcff
io 0x10-0x1f
io 0x22-0x2d
io 0x30-0x3f
io 0x50-0x52
io 0x72-0x77
io 0x91-0x93
mem disabled
mem disabled
mem disabled
mem disabled

> 
> I think we should move PNP before PCI and fix whatever resource
> problems crop up.

We could always add this change to -mm and see what effect it has.  Let me
know if you still think this makes sense.

Thanks,
Adam
-
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to