On Thu, 24 May 2012, Ralf Baechle wrote: > > diff --git a/arch/mips/mti-malta/malta-pci.c b/arch/mips/mti-malta/malta-pci.c > > index bf80921..afeb619 100644 > > --- a/arch/mips/mti-malta/malta-pci.c > > +++ b/arch/mips/mti-malta/malta-pci.c > > @@ -241,8 +241,9 @@ void __init mips_pcibios_init(void) > > return; > > } > > > > - if (controller->io_resource->start < 0x00001000UL) /* FIXME */ > > - controller->io_resource->start = 0x00001000UL; > > + /* Change start address to avoid conflicts with ACPI and SMB devices */ > > + if (controller->io_resource->start < 0x00002000UL) /* FIXME */ > > + controller->io_resource->start = 0x00002000UL; > > I think raising this value to 0x2000 solves the FIXME which is there since > Maciej's 66d9ad704b25287bfee7e86a5af50b92642b9c72 commit in 2005. Maciej, > do you recall you added the FIXME? Vaguely. I reckon the bump was required because PIIX4 ACPI/SMB didn't reserve their resources that are not properly set/announced with standard PCI BARs in the PCI configuration space. Chances therefore were something else would take this range and I reckon this was of course exactly what happened, ruining everything in a weird way. And I think it was the other way round -- I think bumping it up yet more is the wrong way of "fixing this up" (why is it needed anyway, did the assignment change? -- I don't remember if power-on defaults are used on Malta or if that is YAMON that initialises these). I am fairly sure if I added that FIXME it was meant as a reminder to fix that properly and not do something as trivial as this. And a proper fix is IMO either of these: 1. If there is a proper PIIX4 ACPI/SMB driver available then it should reserve these resources to avoid the conflict. 2. Otherwise a PCI quirk should do that based on the PCI ID of the device and the resource ranges obtained directly from the device (I reckon they still use BARs, but in the vendor-specific space, they're not fixed assignments), pretty much like some PC/AT legacy resources are reserved (82xx series PIC, PIT, etc). I don't remember why I didn't do either of these, sorry. Maciej
