| Message ID | 20191216133405.1001-3-afd@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Naming changes for DMA-BUF Heaps in drm-misc-next | expand |
On Mon, Dec 16, 2019 at 5:34 AM Andrew F. Davis <afd@ti.com> wrote: > > The heaps are already in a directory of heaps, adding _heap to a heap > name is redundant. This patch is only a name change, no logic is changed. > > Signed-off-by: Andrew F. Davis <afd@ti.com> Again, do wish we had caught/made this tweak earlier, but this seems sane to me. Acked-by: John Stultz <john.stultz@linaro.org> I've also reworked my userland changes to use this name, and will update them here shortly (assuming there's no objection to this). Sumit: Mind queuing for drm-misc-next? thanks -john
diff --git a/drivers/dma-buf/heaps/system_heap.c b/drivers/dma-buf/heaps/system_heap.c index 1aa01e98c595..0bf688e3c023 100644 --- a/drivers/dma-buf/heaps/system_heap.c +++ b/drivers/dma-buf/heaps/system_heap.c @@ -109,7 +109,7 @@ static int system_heap_create(void) struct dma_heap_export_info exp_info; int ret = 0; - exp_info.name = "system_heap"; + exp_info.name = "system"; exp_info.ops = &system_heap_ops; exp_info.priv = NULL;
The heaps are already in a directory of heaps, adding _heap to a heap name is redundant. This patch is only a name change, no logic is changed. Signed-off-by: Andrew F. Davis <afd@ti.com> --- drivers/dma-buf/heaps/system_heap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)