Re: [PATCH 7/21] advansys: Convert to EISA driver model

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Matthew Wilcox wrote:
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?" ;-)

Your earlier statement was just confusing. You basically said "if PCIMEM doesn't work, we should fall back to PCIMEM" if I read it correctly.

I think you meant falling back to ioports, but wanted to clarify.

	Jeff



-
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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux