From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Warren Subject: Re: [PATCH 8/9] usb: host: ehci-tegra: fix PHY error handling Date: Mon, 18 Mar 2013 09:25:34 -0600 Message-ID: <514731EE.9010709@wwwdotorg.org> References: <1363338730-14581-1-git-send-email-balbi@ti.com> <1363338730-14581-9-git-send-email-balbi@ti.com> <51438EA8.1060301@wwwdotorg.org> <20130318080245.GI17135@arwen.pp.htv.fi> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20130318080245.GI17135-S8G//mZuvNWo5Im9Ml3/Zg@public.gmane.org> Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: balbi-l0cyMroinI0@public.gmane.org Cc: Linux USB Mailing List , Alan Stern , Greg KH , linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Venu Byravarasu List-Id: linux-tegra@vger.kernel.org On 03/18/2013 02:02 AM, Felipe Balbi wrote: > Hi, > > On Fri, Mar 15, 2013 at 03:12:08PM -0600, Stephen Warren wrote: >> On 03/15/2013 03:12 AM, Felipe Balbi wrote: >>> PHY layer no longer returns NULL, we must switch from >>> IS_ERR_OR_NULL() to IS_ERR(). >> >> This change will definitely conflict with some Tegra >> EHCI/USB-PHY changes that Venu plans to submit very soon, for >> 3.10. This is relevant > > but this is such a small change that, even if it conflicts, > resolution will be trivial. Well, Venu's changes re-organize exactly the code you're modifying. Wouldn't it be simpler to simply take all the Tegra USB patches through the PHY tree and avoid any conflicts at all? >> since we'd previously discussed you ack'ing Venu's patches, and >> my applying them to the Tegra tree, due to dependencies between >> the Tegra device tree files and his USB changes. >> >> To resolve this, we can do the following: >> >> 1) I will create a tiny topic branch containing just the Tegra >> DT changes that must happen before the USB driver changes. This >> can be based on v3.9-rc1 or similar, and be entirely >> self-contained. >> >> 2) You can merge that topic branch into your USB tree, so that >> the changes are present there. >> >> 3) I will merge that topic branch into the Tegra tree. >> >> (2) and (3) are both needed so that the exact same commit ID is >> present in each of our branches. It's needed in yours as a >> pre-cursor to Venu's changes. It's needed in Tegra's because I >> still hope to activate usage of the C pre-processor on the Tegra >> DT files, and that needs to build on top of the same DT change of >> Venu's that you also need. >> >> 4) Once you've done that, you can take Venu's USB patches through >> your USB tree rather than my applying them to the Tegra tree. >> This will allow you to resolve any conflicts between your changes >> and Venu's changes entirely within your branch simply by applying >> the patches one after the other. Nice and simple, and just like >> any other USB change. >> >> However, this goes against your statement that you "don't accept >> pull requests". Perhaps you can make an exception for this case? > > Greg won't like seeing merges from my pull request and I kinda > agree with him. We can sort out the conflicts later. That seems odd. There are plenty of cases of merges in other trees.