Re: [RFC][PATCH 1/2] WIP: Devicetree bindings for Ion

On Tue, Oct 13 2015 at 11:14:23 AM, Andrew <and...@ncrmnt.org> wrote: > On 2015-10-12 21:39, Mitchel Humpherys wrote: >> On Tue, Oct 06 2015 at 05:35:41 PM, Rob Herring <robherri...@gmail.com> >> wrote: >>> On Tue, Oct 6, 2015 at 3:47 PM, Laura Abbott &

Re: [RFC][PATCH 1/2] WIP: Devicetree bindings for Ion

On Tue, Oct 06 2015 at 05:35:41 PM, Rob Herring wrote: > On Tue, Oct 6, 2015 at 3:47 PM, Laura Abbott > wrote: [...] >> +Example: >> + >> + ion { >> + compatbile = "linux,ion"; >> + #address-cells = <1>; >> +

Re: [PATCH v2 2/2] staging: ion: chunk_heap: use %pad for printing dma_addr_t's

On Mon, May 04 2015 at 01:05:50 PM, Colin Cross ccr...@android.com wrote: On Mon, May 4, 2015 at 1:22 AM, Dan Carpenter dan.carpen...@oracle.com wrote: On Thu, Apr 09, 2015 at 06:10:04PM -0700, Mitchel Humpherys wrote: We're currently using %lu and %ld to print some variables of type

[PATCH] ion: improve ion_phys error message

Clients often get confused when ion_phys errors out due to some heap being used that they didn't expect. Add the heap name and heap type to the error message to make it more obvious. Signed-off-by: Mitchel Humpherys mitch...@codeaurora.org --- drivers/staging/android/ion/ion.c | 4 ++-- 1 file

[PATCH] ion: improve ion_phys error message

Clients often get confused when ion_phys errors out due to some heap being used that they didn't expect. Add the heap name and heap type to the error message to make it more obvious. Signed-off-by: Mitchel Humpherys mitch...@codeaurora.org --- drivers/staging/android/ion/ion.c | 4 ++-- 1 file

[PATCH] staging: ion: always initialize the free list parameters

in ion_device_add_heap, which is always called no matter which heap features are being used. Signed-off-by: Mitchel Humpherys mitch...@codeaurora.org --- drivers/staging/android/ion/ion.c | 3 +++ drivers/staging/android/ion/ion_heap.c | 2 -- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git

UAPI headers in staging drivers?

What's the protocol for UAPI headers in staging drivers? Specifically, I'm wondering how people usually install those headers. For example, there are a bunch of Android UAPI headers under drivers/staging/android/uapi which currently end up at INSTALL_HDR_PATH/drivers with a call to `make

Re: [PATCH v2 0/9] staging: ion: system heap and page pool fixes

trick with page.lru :). Reviewed-by: Mitchel Humpherys mitch...@codeaurora.org Heesub Shin (9): staging: ion: tidy up a bit staging: ion: simplify ion_page_pool_total() staging: ion: remove struct ion_page_pool_item staging: ion: use compound pages on high order pages for system heap

Re: [PATCH 6/9] staging: ion: remove struct page_info

On Mon, May 26 2014 at 03:04:58 AM, Heesub Shin heesub.s...@samsung.com wrote: ION system heap uses a temporary list holding meta data on pages allocated to build scatter/gather table. Now that the pages are compound pages, we do not need to introduce a new data type redundantly.

Re: [PATCH 7/9] staging: ion: remove order argument from free_buffer_page()

On Mon, May 26 2014 at 03:04:59 AM, Heesub Shin heesub.s...@samsung.com wrote: Not that the pages returned from the pool are compound pages, we do not need to pass the order information to free_buffer_page(). s/Not/Now/ -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora

Re: [PATCH] staging: ion: WARN when the handle kmap_cnt is going to wrap around

++greg-kh and de...@driverdev.osuosl.org (my bad for missing you the first time around) On Thu, May 22 2014 at 06:09:11 PM, Colin Cross ccr...@android.com wrote: On Thu, May 22, 2014 at 5:51 PM, Mitchel Humpherys mitch...@codeaurora.org wrote: There are certain client bugs (double unmap

[RESEND PATCH] staging: ion: WARN when the handle kmap_cnt is going to wrap around

a long time to complete when kmap_cnt starts at ~0 and can result in a watchdog timeout. WARN and bail when kmap_cnt is about to wrap around from zero. Signed-off-by: Mitchel Humpherys mitch...@codeaurora.org Acked-by: Colin Cross ccr...@android.com --- Resending since I missed a few folks