Re: [PATCH 3/4] dt-binding: remoteproc: venus rproc dt binding document

From	Marek Szyprowski <m.szyprowski@samsung.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH 3/4] dt-binding: remoteproc: venus rproc dt binding document
Date	2016-09-02 14:00 +0200
Message-ID	<scViy-7sc-27@gated-at.bofh.it> (permalink)
References	(5 earlier) <sa0Rs-fC-23@gated-at.bofh.it> <saxNn-4Bu-11@gated-at.bofh.it> <sbteF-6LX-1@gated-at.bofh.it> <sbURz-7My-9@gated-at.bofh.it> <scBDb-3cq-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hi,

On 2016-09-01 16:58, Stanimir Varbanov wrote:
> Hi,
>
> Cc: Marek
>

...

>>>> But I presume we have the implementation issue of dma_alloc_coherent()
>>>> failing in either case with the 5MB size. I think we need to look into
>>> I'd be good to include Marek Szyprowski? At least he will know what
>>> design restrictions there are.
>>>
>> Please do. The more I look at this the more I think we must use the
>> existing infrastructure for allocating "dma memory". Getting
>> dma_alloc_coherent() supporting non-power-of-2 memory regions would
> Just to be precise it should be dma_alloc_from_coherent().
>
> Marek, what is your opinion on that, can we make dma_alloc_from_coherent
> able to allocate memory for sizes with bigger granularity.
>
> For your convenience here [1] is the mail thread.

There should be no technical restrictions to add support for bigger 
granularity
than power-of-2. dma_alloc_from_coherent uses standard bitmap based 
allocator,
so it already support tracking allocations of arbitrary size. However 
for the
small allocations (smaller than 64KiB?, 512KiB?) it would make sense to keep
nearest-power-of-2 round up to prevent memory fragmentation.

Best regards
-- 
Marek Szyprowski, PhD
Samsung R&D Institute Poland

Thread

Re: [PATCH 3/4] dt-binding: remoteproc: venus rproc dt binding  document Stanimir Varbanov <stanimir.varbanov@linaro.org> - 2016-08-25 13:20 +0200
  Re: [PATCH 3/4] dt-binding: remoteproc: venus rproc dt binding  document Bjorn Andersson <bjorn.andersson@linaro.org> - 2016-08-27 00:30 +0200
    Re: [PATCH 3/4] dt-binding: remoteproc: venus rproc dt binding  document Stanimir Varbanov <stanimir.varbanov@linaro.org> - 2016-08-29 13:50 +0200
      Re: [PATCH 3/4] dt-binding: remoteproc: venus rproc dt binding  document Bjorn Andersson <bjorn.andersson@linaro.org> - 2016-08-30 19:20 +0200
        Re: [PATCH 3/4] dt-binding: remoteproc: venus rproc dt binding  document Stanimir Varbanov <stanimir.varbanov@linaro.org> - 2016-09-01 17:00 +0200
          Re: [PATCH 3/4] dt-binding: remoteproc: venus rproc dt binding  document Andy Gross <andy.gross@linaro.org> - 2016-09-01 23:20 +0200
          Re: [PATCH 3/4] dt-binding: remoteproc: venus rproc dt binding document Marek Szyprowski <m.szyprowski@samsung.com> - 2016-09-02 14:00 +0200
            Re: [PATCH 3/4] dt-binding: remoteproc: venus rproc dt binding  document Bjorn Andersson <bjorn.andersson@linaro.org> - 2016-09-02 22:20 +0200
              Re: [PATCH 3/4] dt-binding: remoteproc: venus rproc dt binding  document Stanimir Varbanov <stanimir.varbanov@linaro.org> - 2016-09-07 14:00 +0200

csiph-web