On Fri, Jul 27, 2018 at 11:05:48PM +0200, Sam Ravnborg wrote: > Hi Christoph. > > Some observations below - nitpick and bikeshedding only. > > The parameter of phys_addr_t is sometimes renamed > to use the same name as in the original prototype (good), > and sometimes uses the old name (bad). > This makes it inconsistent as the local name changes in the > different functions, but they represents the same.
I'll change it. > You can add my: > Acked-by: Sam Ravnborg <s...@ravnborg.org> I will resend it with your nitpicks addressed and your ack added. > > +void *arch_dma_alloc(struct device *dev, size_t len, dma_addr_t *pba, > > gfp_t gfp, > > + unsigned long attrs) > > This function was renamed in ee664a9252d24 and is now renamed again. > The printk statements should be updated to use arch_dma_alloc. I've switched the printk statements to use __func__ instead, not that I plan for another rename any time soon. I've also update the comments referring to this function to use the right name. _______________________________________________ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu
