On Tue, Mar 3, 2026 at 9:55 PM John Stultz <[email protected]> wrote:
>
> On Tue, Mar 3, 2026 at 4:34 AM Albert Esteve <[email protected]> wrote:
> >
> > This patch introduces a new heap driver to expose DT non‑reusable
> > "shared-dma-pool" coherent regions as dma-buf heaps, so userspace can
> > allocate buffers from each reserved, named region.
>
> Just a nit here: Might be good to provide some higher level context as
> to why this is wanted, and what it enables.
>
> Also, "shared-dma-pool" is also used for CMA regions, so it might be
> unclear initially how this is different from the CMA heap (you do
> mention non-reusable, but that's a prettty subtle detail).

Sure, I will expand this for the next revision and try to clarify the
points you mentioned here (and add these points to the relevant
patch).

BR,
Albert

>
> Might be good to add some of the rationale to the patch adding the
> heap implementation as well so it makes it into the git history.
>
> thanks
> -john
>

Reply via email to