On Thursday, August 19, 2010 02:39:47 pm Guennadi Liakhovetski wrote: > On Thu, 19 Aug 2010, Janusz Krzysztofik wrote: > > > Hi Marin, > > Since I've finaly managed to make use of your method without any previously > > observerd limitations (see below), I'm interested in it being implemented > > system-wide. Are you going to submit a patch? It is ready, I just wait for the invitation. Marin Mitov > > I'm about to submit a patch, which you'll be most welcome to test. Just > give me a couple more hours. > > > I would suggest creating one common function that allocates and fills the > > dev->dma_mem structure, and two wrappers that call it: a > > dma_declare_coherent_memory() replacement, that passes an ioremapped device > > memory address to the common fuction, and your proposed > > dma_reserve_coherent_memory(), that passes a pointer returned by the > > dma_alloc_coherent() instead. > No, I don't think you should go to the next power of 2 - that's too crude. > Try rounding your buffer size to the page size, that should suffice. Allocated coherent memory is always a power of 2. Thanks. Marin Mitov > > Thanks > Guennadi > --- > Guennadi Liakhovetski, Ph.D. > Freelance Open-Source Software Developer > http://www.open-technology.de/ > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html