[<prev] [next>] [day] [month] [year] [list]
Message-ID: <CALAqxLUXKYibt6e4ji=kpP4ROFkU_4YQZoAE-ciq2bnFJxM_PA@mail.gmail.com>
Date: Wed, 2 Oct 2019 09:14:42 -0700
From: John Stultz <john.stultz@...aro.org>
To: Hillf Danton <hdanton@...a.com>
Cc: lkml <linux-kernel@...r.kernel.org>,
"Andrew F . Davis" <afd@...com>, Laura Abbott <labbott@...hat.com>,
Benjamin Gaignard <benjamin.gaignard@...aro.org>,
Sumit Semwal <sumit.semwal@...aro.org>,
Liam Mark <lmark@...eaurora.org>,
Pratik Patel <pratikp@...eaurora.org>,
Brian Starkey <Brian.Starkey@....com>,
Vincent Donnefort <Vincent.Donnefort@....com>,
Sudipto Paul <Sudipto.Paul@....com>,
Christoph Hellwig <hch@...radead.org>,
Chenbo Feng <fengc@...gle.com>,
Alistair Strachan <astrachan@...gle.com>,
Hridya Valsaraju <hridya@...gle.com>,
dri-devel <dri-devel@...ts.freedesktop.org>
Subject: Re: [RESEND][PATCH v8 1/5] dma-buf: Add dma-buf heaps framework
On Sun, Sep 29, 2019 at 8:27 PM Hillf Danton <hdanton@...a.com> wrote:
> On Fri, 6 Sep 2019 18:47:08 +0000 John Stultz wrote:
> > +/**
> > + * dma_heap_get_data() - get per-heap driver data
> > + * @heap: DMA-Heap to retrieve private data for
> > + *
> > + * Returns:
> > + * The per-heap data for the heap.
> > + */
> > +void *dma_heap_get_data(struct dma_heap *heap);
> > +
>
> It will help readers more than thought understand this framework
> if s/get_data/get_drvdata/
Sounds good!
Thanks for the review and suggestion!
-john
Powered by blists - more mailing lists