From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Boyd Subject: Re: [PATCH] Revert "arm64: dts: qcom: Collapse usb support into one node" Date: Thu, 13 Jul 2017 15:02:31 -0700 Message-ID: <20170713220231.GD22780@codeaurora.org> References: <20170712125031.20245-1-robdclark@gmail.com> <20170713060933.GT22780@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]:60358 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751153AbdGMWCd (ORCPT ); Thu, 13 Jul 2017 18:02:33 -0400 Content-Disposition: inline In-Reply-To: Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Rob Clark Cc: linux-arm-msm , Andy Gross On 07/13, Rob Clark wrote: > On Thu, Jul 13, 2017 at 2:09 AM, Stephen Boyd wrote: > > Was device mode working on v4.12? I wonder if we could change > > dr_mode to host and configure the pmic gpio to mux out the right > > lines to the hub. Then we could retain host mode on db410c (which > > is probably all that people care about). It would be a smaller > > fix for this release while we work out the role switch patches. > > Like this patch? I still need to test it tomorrow, but this may > > work and allow us to delete the ehci-msm driver sooner. > > yup, was working on v4.12 so this patch that I reverted was a > regression. At least for host mode. (I don't really use device mode > much outside of android, so never tried that on v4.12, but it did work > in the past) Ok. > > I can try other patches.. although a bit wary of deleting ehci-msm or > a big dt change in general since some people seem to think taking dt > and kernel from two different places is a sane thing) I'm fine to leave ehci-msm driver in place for anyone wanting to use DT from somewhere outside the kernel, just hoping to push anybody I can towards the chipidea driver, which is a cleaner solution in the long run. It's scary that we have two usb drivers operating the same hardware with the DT in v4.12. I tested out my patch and it works well enough to get host mode working for me so I'll send it out as a proper regression fix. When we add role switching support we'll have to adjust the DT to support it and people who care to gain support for gadgets can use the updated DT. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project