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

[Sumit Semwal]

Hi Laura,

On 14 August 2016 at 14:17, Benjamin Gaignard
 wrote:
> This series of patches look good for me, thanks for the clean up
>
> Reviewed-by: Benjamin Gaignard 
>
> 2016-08-08 18:52 GMT+02:00 Laura Abbott :
>> 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 

>> ---
>>
>> 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-...@lists.linaro.org
>> 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
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

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

[Benjamin Gaignard]

This series of patches look good for me, thanks for the clean up

Reviewed-by: Benjamin Gaignard 

2016-08-08 18:52 GMT+02:00 Laura Abbott :
> I never saw any feedback or action on these patches so this is a resend.
> Probably hit the window when everyone was busy/traveling.
>
> ---
>
> 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
>
> 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-...@lists.linaro.org
> 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
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel