On Fri, Jul 27, 2007 at 09:50:58AM -0400, Jeff Garzik wrote: > >Would it make sense to put in a comment that says "No idea why we do this"? > > IMO, yes. It should be noticeable at the very least. Added to todo for next spin of these patches. > >>5) I would suggest putting a "remove inpw/outpw pointless wrappers" > >>cleanup patch before patches #2 .. #N. > > > >Next thing on my todo list is to convert the driver over to ioread8 and > >friends. Currently, only the wide boards get accessed through PCIMEM, > >but the narrow boards also have PCIMEM BARs. It would also allow the wide > >boards to be used on machines where PCIMEM is broken, like the VisWS. > > PCIMEM versus PCIMEM... huh? This driver is really two drivers in one -- the wide boards and the narrow boards. Christoph is working on a new driver for the wide boards only, and once he's finished that, I'll delete the support for the wide boards in this driver. Right now, only the wide boards get to take advantage of PCIMEM. I want to give the narrow boards (like the one I have) the opportunity to take advantage of PCIMEM, if they have it. There's also a pending todo for wide boards to fall back to ioports if PCIMEM doesn't work. If that doesn't answer your question, I need something more detailed than "huh?" ;-) -- "Bill, look, we understand that you're interested in selling us this operating system, but compare it to ours. We can't possibly take such a retrograde step." - To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
