Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933865AbcJURic (ORCPT ); Fri, 21 Oct 2016 13:38:32 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:57011 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932507AbcJURi3 (ORCPT ); Fri, 21 Oct 2016 13:38:29 -0400 DMARC-Filter: OpenDMARC Filter v1.3.1 smtp.codeaurora.org 1F9086175E Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=pass smtp.mailfrom=sboyd@codeaurora.org Date: Fri, 21 Oct 2016 10:38:27 -0700 From: Stephen Boyd To: Bjorn Andersson Cc: Peter Chen , Greg Kroah-Hartman , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, Arnd Bergmann , Sriram Dash Subject: Re: [RESEND PATCH] usb: chipidea: Configure DMA properties and ops from DT Message-ID: <20161021173827.GJ26139@codeaurora.org> References: <1456119133-16114-1-git-send-email-bjorn.andersson@linaro.org> <1477069159-12399-1-git-send-email-bjorn.andersson@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1477069159-12399-1-git-send-email-bjorn.andersson@linaro.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1169 Lines: 29 On 10/21, Bjorn Andersson wrote: > hcd_alloc_coherent() and usb_alloc_coherent() ends up allocating coherent > memory on behalf of ci_hdrc driver. But as the ci_hdrc is instantiated manually > it will not have any dma_mem or dma_ops assigned, which makes the > dma_alloc_coherent() fail on some platforms (e.g. arm64). This patch solves > this by assigning the dma_mem and dma_ops based on the parent's DeviceTree > node. > > Cc: Stephen Boyd > Signed-off-by: Bjorn Andersson > --- > > Hi Peter, > > After (once more) debugging why USB doesn't work up on the 64-bit Qualcomm > systems I realized that we never concluded on this patch. Unfortunately I can't > find it in my mailbox either, so resending it to restart the discussion. > I thought we were going to go down the route that Arnd has been pushing[1]? That should work, but I haven't tried it yet and there are some more fixes on top from Sriram. I think Sriram is taking over the patch now? [1] https://patchwork.kernel.org/patch/9319527/ -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project