Messages in this thread | | | From | Arnd Bergmann <> | Subject | Re: [PATCH 00/21] dmaengine/ARM: Merge the edma drivers into one | Date | Thu, 10 Sep 2015 10:47:28 +0200 |
| |
On Thursday 10 September 2015 11:37:29 Peter Ujfalusi wrote: > > with this series the edma two driver setup will be changed to have only one > driver to support eDMA3. The legacy edma interface will be removed and eDMA can > only be used via dmaengine API from this point on. > In order to do the merge the following improvements has been done: > - One driver instance per eDMA: > - Any number of eDMA instances are supported (both legacy and DT boot) > - Not relying on global variables, arrays, etc > - Code simplification and optimizations in several places > > This change will also help us to do bigger changes in the eDMA driver since, > since now we have only one driver to work with. >
Very nice series, thanks for addressing this long-standing issue!
I've briefly read over the series and did not see anything major sticking out, but I did not attempt to do a thorough review.
I found two minor things that I commented on that could be improved.
Arnd
| |