ACK. Inspected only. Looks ok. Sincerely -- Mark Salyzyn
> -----Original Message----- > From: [EMAIL PROTECTED] > [mailto:[EMAIL PROTECTED] On Behalf Of Jeff Garzik > Sent: Wednesday, October 24, 2007 7:49 PM > To: LKML; [EMAIL PROTECTED] > Cc: [EMAIL PROTECTED] > Subject: [PATCH 3/4] [SCSI] ips: PCI API cleanups > > * pass Scsi_Host to ips_remove_device() via pci_set_drvdata(), > allowing us to eliminate the ips_ha[] search loop and call > ips_release() directly. > > * call pci_{request,release}_regions() and eliminate individual > request/release_[mem_]region() calls > > * call pci_disable_device(), paired with pci_enable_device() > > * s/0/NULL/ in a few places > > * check ioremap() return value > > Signed-off-by: Jeff Garzik <[EMAIL PROTECTED]> > --- > drivers/scsi/ips.c | 72 > ++++++++++++++++++++++----------------------------- > 1 files changed, 31 insertions(+), 41 deletions(-) - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/