On Tue, 2009-11-17 at 16:16 -0800, pbath...@amcc.com wrote: > From: Pravin Bathija <pbath...@amcc.com> > > Powerpc 44x uses 36 bit real address while the real address defined > in MPT Fusion driver is of type 32 bit. This causes ioremap to fail and > driver > fails to initialize. This fix changes the data types representing the real > address from unsigned long 32-bit types to "phys_addr_t" which is 64-bit. > The > driver has been tested, the disks get discovered correctly and can do IO. > Removed > ioremap and used hose->io_base_virt for IO space to make it platform > independent. > Content-Type: text/plain; charset=utf-8
Except that this is all wrong :-) You basically made it powerpc specific since none of that pci controller stuff is generic. I don't understand what you are trying to do though. The -only- change you need to do is to change the longs into resource_size_t. IO Port numbers are "special" and handled as such already (and besides are never bigger than 32-bit neither, at least on x86 and powerpc). Just leave the PIO code alone, hopefully, if the driver isn't full of crack, the code should be fine already. If the driver does something wrong then you can attempt to fix it separately. The only problem that you need to address afaik, is purely that pci_resource_start() can return a resource_size_t which can be 64-bit, and as such it is broken for the driver to manipulate and store that value as an unsigned long or a u32. Cheers, Ben. _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
