From mboxrd@z Thu Jan 1 00:00:00 1970 From: Roland Dreier Subject: Re: [PATCH v2 12/51] IB/qib: Add qib_dma.c Date: Wed, 09 Dec 2009 16:09:33 -0800 Message-ID: References: <20091203190305.29507.58158.stgit@chromite.mv.qlogic.com> <20091203190408.29507.27708.stgit@chromite.mv.qlogic.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: In-Reply-To: <20091203190408.29507.27708.stgit-/vjeY7uYZjrPXfVEPVhPGq6RkeBMCJyt@public.gmane.org> (Ralph Campbell's message of "Thu, 03 Dec 2009 11:04:08 -0800") Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Ralph Campbell Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-rdma@vger.kernel.org > +struct ib_dma_mapping_ops qib_dma_mapping_ops = { > + qib_mapping_error, > + qib_dma_map_single, > + qib_dma_unmap_single, > + qib_dma_map_page, > + qib_dma_unmap_page, > + qib_map_sg, > + qib_unmap_sg, > + qib_sg_dma_address, > + qib_sg_dma_len, > + qib_sync_single_for_cpu, > + qib_sync_single_for_device, > + qib_dma_alloc_coherent, > + qib_dma_free_coherent > +}; I think it would be a good idea to use a designated initializer -- eg .mapping_error = qib_mapping_error, ... instead to avoid silent breakage if we extend the mapping ops structure. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html