Re: [PATCH v1] dma: imx-sdma: add virt-dma support

2018-05-23 Thread Robin Gong
Okay, I'll try to split it. On 三, 2018-05-23 at 19:04 +0530, Vinod wrote: > On 23-05-18, 12:56, s.ha...@pengutronix.de wrote: > > > > > Well, it's somewhat related to virtual dma support, but that's not > > my > > point. My point is that this patch is quite big and thus hard to > > review. > > If

Re: [PATCH v1] dma: imx-sdma: add virt-dma support

2018-05-23 Thread Vinod
On 23-05-18, 12:56, s.ha...@pengutronix.de wrote: > Well, it's somewhat related to virtual dma support, but that's not my > point. My point is that this patch is quite big and thus hard to review. > If we find ways to make it smaller and to split it up in multiple > patches then we should do so, b

Re: [PATCH v1] dma: imx-sdma: add virt-dma support

2018-05-23 Thread s.ha...@pengutronix.de
On Wed, May 23, 2018 at 10:26:23AM +, Robin Gong wrote: > > > > > > > > + u32 bd_size_sum; > > This variable is never used for anything. > Yes, it's not for significative use but debug to see how many current > bds used. I am not convinced this is useful. The vari

Re: [PATCH v1] dma: imx-sdma: add virt-dma support

2018-05-23 Thread Robin Gong
On 二, 2018-05-22 at 12:09 +0200, Sascha Hauer wrote: > Hi Robin, > > Several comments inside. > > Sascha > > On Fri, Mar 23, 2018 at 12:18:19AM +0800, Robin Gong wrote: > > > > The legacy sdma driver has below limitations or drawbacks: > >   1. Hardcode the max BDs number as "PAGE_SIZE / sizeof

Re: [PATCH v1] dma: imx-sdma: add virt-dma support

2018-05-22 Thread Sascha Hauer
Hi Robin, Several comments inside. Sascha On Fri, Mar 23, 2018 at 12:18:19AM +0800, Robin Gong wrote: > The legacy sdma driver has below limitations or drawbacks: > 1. Hardcode the max BDs number as "PAGE_SIZE / sizeof(*)", and alloc > one page size for one channel regardless of only few

Re: [PATCH v1] dma: imx-sdma: add virt-dma support

2018-05-22 Thread Robin Gong
Ok, I'll resend it after rebase and test. On 二, 2018-05-22 at 12:25 +0530, Vinod wrote: > On 22-05-18, 06:16, Robin Gong wrote: > > > > Ping. > Looks like I missed this one, can you please rebase and resend. > While at it, modify the subject to dmaengine, as thats the subsystem > name >

Re: [PATCH v1] dma: imx-sdma: add virt-dma support

2018-05-21 Thread Vinod
On 22-05-18, 06:16, Robin Gong wrote: > Ping. Looks like I missed this one, can you please rebase and resend. While at it, modify the subject to dmaengine, as thats the subsystem name -- ~Vinod

RE: [PATCH v1] dma: imx-sdma: add virt-dma support

2018-05-21 Thread Robin Gong
Ping. -Original Message- From: Robin Gong Sent: 2018年3月23日 0:18 To: dan.j.willi...@intel.com; vinod.k...@intel.com Cc: dmaeng...@vger.kernel.org; linux-arm-ker...@lists.infradead.org; linux-kernel@vger.kernel.org; dl-linux-imx Subject: [PATCH v1] dma: imx-sdma: add virt-dma support

[PATCH v1] dma: imx-sdma: add virt-dma support

2018-03-22 Thread Robin Gong
The legacy sdma driver has below limitations or drawbacks: 1. Hardcode the max BDs number as "PAGE_SIZE / sizeof(*)", and alloc one page size for one channel regardless of only few BDs needed most time. But in few cases, the max PAGE_SIZE maybe not enough. 2. One SDMA channel can't st