On Fri, Nov 17, 2006 at 04:36:37PM +0800, Peer Chen wrote: > I didn't get any comment from you guys for new patch, does someone take > care this patch, do we still need some modification upon it? Or do we > need re-submit it in other thread? One small change suggestion: > +static inline bool nv_sgpio_capable(const struct pci_device_id *ent) > +{ > + if (ent->device == PCI_DEVICE_ID_NVIDIA_NFORCE_MCP55_SATA2) > + return 1; > + else > + return 0; > +} Make it shorter -> return ent->device == PCI_DEVICE_ID_NVIDIA_NFORCE_MCP55_SATA2; - Heikki - To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html