Re: [Linaro-mm-sig] [RESEND][PATCH 0/5] Ion cleanup

From: Sumit Semwal
Date: Mon Aug 15 2016 - 11:09:03 EST


Hi Laura,

On 14 August 2016 at 14:17, Benjamin Gaignard
<benjamin.gaignard@xxxxxxxxxx> wrote:
> This series of patches look good for me, thanks for the clean up
>
> Reviewed-by: Benjamin Gaignard <benjamin.gaignard@xxxxxxxxxx>
>
> 2016-08-08 18:52 GMT+02:00 Laura Abbott <labbott@xxxxxxxxxx>:
>> I never saw any feedback or action on these patches so this is a resend.
>> Probably hit the window when everyone was busy/traveling.
>>
Looks good to me; please feel free to apply my
Reviewed-by: Sumit Semwal <sumit.semwal@xxxxxxxxxx>

>> ---
>>
>> Hi,
>>
>> This is some clean up of old Ion interfaces and APIs. These are interfaces that
>> mostly existed before dma_buf was well integrated into the kernel along with
>> reservations for board files.
>>
>> If there are objections to deletion I expect it to turn into a discussion about
>> what other APIs need to be extended.
>>
>> Thanks,
>> Laura
>>
Best,
Sumit.

>> Laura Abbott (5):
>> staging: android: ion: Get rid of ion_sg_table
>> staging: android: ion: Drop ion_phys interface
>> staging: android: ion: Get rid of map_dma/unmap_dma
>> staging: android: ion: Drop ion_carveout_allocate definitions
>> staging: android: ion: Get rid of ion_reserve
>>
>> drivers/staging/android/ion/ion.c | 103 ++----------------------
>> drivers/staging/android/ion/ion.h | 41 ----------
>> drivers/staging/android/ion/ion_carveout_heap.c | 33 +-------
>> drivers/staging/android/ion/ion_chunk_heap.c | 17 +---
>> drivers/staging/android/ion/ion_cma_heap.c | 34 +-------
>> drivers/staging/android/ion/ion_priv.h | 30 +------
>> drivers/staging/android/ion/ion_system_heap.c | 44 +---------
>> 7 files changed, 19 insertions(+), 283 deletions(-)
>>
>> --
>> 2.5.5
>>
>> _______________________________________________
>> Linaro-mm-sig mailing list
>> Linaro-mm-sig@xxxxxxxxxxxxxxxx
>> https://lists.linaro.org/mailman/listinfo/linaro-mm-sig
>
>
>
> --
> Benjamin Gaignard
>
> Graphic Study Group
>
> Linaro.org â Open source software for ARM SoCs
>
> Follow Linaro: Facebook | Twitter | Blog



--
Thanks and regards,

Sumit Semwal
Linaro Mobile Group - Kernel Team Lead
Linaro.org â Open source software for ARM SoCs