On Wednesday 24 February 2010 09:41:00 am Ralf Baechle wrote: > On Wed, Feb 24, 2010 at 09:03:33AM +0900, Yoichi Yuasa wrote: > > > > approach Ben suggested long ago: > > > http://marc.info/?l=linux-kernel&m=119733290624544&w=2 > > > > It works fine with 2.6.34 queue tree. > > pci.c change is already committed by Ralf. > > Which I just dropped from queue. To keep the tree bisectable removal of > the old hack and adding the fixup should be done in the same patch so I'd > go for Bjorn's patch. Right, thanks. > There is another somewhat theoretical correctness issue. Because the > VIA SuperIO chip only decodes 24 bits of address space but port address > space currently being configured as 32MB there is the theoretical > possibility of I/O port addresses that alias with legacy addresses getting > allocated. Does this mean my comment: + * but the VT82C586 IDE controller does respond at 0x100001f0 because + * it only decodes the low 16 bits of the address. should say "24 bits" instead of "16 bits"? Bjorn