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

[Sumit Semwal]

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