Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1509940

Re: [PATCH 0/3] add ION driver for STIh4xx SoC

Show all headers | View raw

On 10/26/2016 08:05 AM, Benjamin Gaignard wrote:
> 2016-10-26 16:44 GMT+02:00 Sumit Semwal <sumit.semwal@linaro.org>:
>> On 26 October 2016 at 20:11, Benjamin Gaignard
>> <benjamin.gaignard@linaro.org> wrote:
>>> 2016-10-26 15:51 GMT+02:00 Sumit Semwal <sumit.semwal@linaro.org>:
>>>> Hello Benjamin,
>>>>
>>>> On 26 October 2016 at 19:02, Benjamin Gaignard
>>>> <benjamin.gaignard@linaro.org> wrote:
>>>>> It is more or less a copy of Hisilicon driver but with a heap definition
>>>>> fitting with STIH4xx SoC needs.
>>>>> I have just chnage the some function prefix from "hi6220" to "sti".
>>>>>
>>>> Thanks for your patches!
>>>>
>>>> I was just wondering if you couldn't convert the HiSilicon driver into
>>>> something like a 'simple-ion' driver, and have just the DT definitions
>>>> as specifics? This would save a lot of code duplication, and keep it
>>>> as a simple interface for common heaps like cma.
>>>
>>> Create a simple-ion driver is a good idea but it means that heaps
>>> (configuration, name, etc..)
>>> will have to be describe into devicetree. I'm not sure if that will is
>>> acceptable.
>>>
>>>>
>>>> If there are any ST-specific requirements that are incompatible with
>>>> the existing driver, it should be clearly documented out here I think.
>>>
>>> heaps names and Ids aren't the same so I can't reuse hisilicon driver.
>>>
>> But I'd suspect both these are solvable with using something like
>> 'generic,cma' instead of 'hisi,cma' or 'st,cma'?
>
> yes, but it requires to describe the heaps in devicetree.
> Hisilicon driver was doing like that until last month but Laura
> convert it to common platform:
> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/drivers/staging/android/ion/hisilicon?id=b6e336dbeda85585c3ba6d935753d8240e18baf1
>
> I bet she got good reasons do to that so I have implemented sti ION
> driver in this mindset.
>

I agree that having common code would be useful. There are some
generic bindings listed in drivers/staging/android/ion/devicetree.txt
so we could go with linux,ion-heap-dma.

The changes got merged and there were never mailing list objections
but that's because the devicetree maintainers got busy and never
actually looked at them and Arnd at least still didn't like the
idea of Ion in devicetree. I'd like to wait until after plumbers
next week to decide what to do.



>>>
>> Best,
>> Sumit.
>
>
>

Thanks,
Laura

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 0/3] add ION driver for STIh4xx SoC Benjamin Gaignard <benjamin.gaignard@linaro.org> - 2016-10-26 15:40 +0200
  [PATCH 1/3] add binding for STIh4xx ION driver Benjamin Gaignard <benjamin.gaignard@linaro.org> - 2016-10-26 15:40 +0200
  [PATCH 2/3] add STIH4xx ION driver Benjamin Gaignard <benjamin.gaignard@linaro.org> - 2016-10-26 15:40 +0200
  Re: [PATCH 0/3] add ION driver for STIh4xx SoC Sumit Semwal <sumit.semwal@linaro.org> - 2016-10-26 16:00 +0200
    Re: [PATCH 0/3] add ION driver for STIh4xx SoC Sumit Semwal <sumit.semwal@linaro.org> - 2016-10-26 16:50 +0200
      Re: [PATCH 0/3] add ION driver for STIh4xx SoC Benjamin Gaignard <benjamin.gaignard@linaro.org> - 2016-10-26 17:10 +0200
        Re: [PATCH 0/3] add ION driver for STIh4xx SoC Laura Abbott <labbott@redhat.com> - 2016-10-27 01:30 +0200
          Re: [PATCH 0/3] add ION driver for STIh4xx SoC Benjamin Gaignard <benjamin.gaignard@linaro.org> - 2016-11-07 11:20 +0100
            Re: [PATCH 0/3] add ION driver for STIh4xx SoC Laura Abbott <labbott@redhat.com> - 2016-11-07 23:20 +0100
              Re: [PATCH 0/3] add ION driver for STIh4xx SoC Benjamin Gaignard <benjamin.gaignard@linaro.org> - 2016-11-08 10:20 +0100
                Re: [PATCH 0/3] add ION driver for STIh4xx SoC Laura Abbott <labbott@redhat.com> - 2016-11-09 06:00 +0100
    Re: [PATCH 0/3] add ION driver for STIh4xx SoC Benjamin Gaignard <benjamin.gaignard@linaro.org> - 2016-10-26 16:50 +0200

csiph-web