From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vinod Koul Date: Fri, 05 Jul 2013 05:05:50 +0000 Subject: Re: [PATCH] dmaengine: shdma: fix a build failure on platforms with no DMA support Message-Id: <20130705045350.GA10947@intel.com> List-Id: References: <51A77CF1.7040105@ti.com> <20130531024438.GB32564@verge.net.au> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-arm-kernel@lists.infradead.org On Wed, Jun 19, 2013 at 09:32:18PM +0200, Guennadi Liakhovetski wrote: > Hi Vinod > > On Wed, 19 Jun 2013, Kevin Hilman wrote: > > > On Thu, May 30, 2013 at 7:44 PM, Simon Horman wrote: > > > > [...] > > > > > thanks for this. I will wait for a refresh (as we discussed earlier today). > > > Can I confirm that this is a fix for v3.10? If so, could ou note > > > that when you post your revised patch? > > > > Any progress on this patch? > > > > The SH-mobile defconfigs are still all failing in linux-next. > > In > > https://patchwork.kernel.org/patch/2640061/ And you havent CC maintainers on this patch, so I dont have it! > > I proposed a simple immediate fix for this problem. Arnd at the same time > developed an alternative solution: > > https://patchwork.kernel.org/patch/2644121/ > https://patchwork.kernel.org/patch/2644111/ Reading these patches I agree with Arnd that client drivers should not depend on dma slave drivers. Existing issue need to be fixed Arnd, Have you merged these changes? -- ~Vinod > > Which, I think, is a much larger change and isn't suitable as a bug-fix > this late in the release cycle. Instead those patches address other issues > with the shdma drivers, which might need to be carefully studied and > solutions have to be found, which, however, would require more work and > need a longer time. What is your opinion on this? Would you agree to apply > v2 of this patch, as referenced above or would you prefer a different > solution? > > Thanks > Guennadi > --- > Guennadi Liakhovetski, Ph.D. > Freelance Open-Source Software Developer > http://www.open-technology.de/ --