From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Mon, 17 Apr 2017 12:31:36 -0400 Subject: [U-Boot] [PATCH] drivers/usb/ehci: Use platform-specific accessors In-Reply-To: <1492445956.7458.16.camel@synopsys.com> References: <1488372733.4659.19.camel@synopsys.com> <2004fcc3-7ae2-209e-957d-48eadc588b12@denx.de> <1488547367.2940.1.camel@synopsys.com> <1490353692.5689.2.camel@synopsys.com> <1c621928-7149-59a6-9a6b-c08109c3dd3a@denx.de> <1492190170.3223.25.camel@synopsys.com> <20170414200605.GU19897@bill-the-cat> <1492445956.7458.16.camel@synopsys.com> Message-ID: <20170417163136.GF19897@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: u-boot@lists.denx.de On Mon, Apr 17, 2017 at 04:19:17PM +0000, Alexey Brodkin wrote: > Hi Tom, Marek, > > On Fri, 2017-04-14 at 16:06 -0400, Tom Rini wrote: > > On Fri, Apr 14, 2017 at 05:16:11PM +0000, Alexey Brodkin wrote: > > > > > > Hi Marek, > > > > > > On Fri, 2017-04-14 at 16:44 +0200, Marek Vasut wrote: > > > > > > > > On 03/24/2017 01:56 PM, Marek Vasut wrote: > > > > > > > > > >   > > > > Even better, this patch breaks powerpc board lwmon5, so dropped. > > > > > > Are you sure the problem is in my patch? > > > > Yes, it is.  Dropping this from the USB PR gets everything going again. > > Ok the problem was obvious missing inclusion f "asm/io.h". > Fix is submitted here http://patchwork.ozlabs.org/patch/751397/ > > Please consider for applying. Thanks. > > > Maybe this PPC board just needs to UNdefine > > > CONFIG_EHCI_DESC_BIG_ENDIAN or there's a problem in its either > > > cpu_to_XXX() or readl()/writel()? > > > > > > Stefan, could you please look at what's wrong here > > > if you have the board handy? > > > > Well, since this is your set of changes, you should have some idea > > what's right/wrong here given what things are before/after.  Also, for > > wide changes like this, please give things a build either in travis-ci > > (so that most of the world is built) or do most of the world locally > > before posting, thanks! > > Well I tried to run a buildman locally (thanks to machine resources I have) > but out of 1243 boards 154 have fatal problems if I use toolchains obtained > via buildbot. Which is barely useful for capturing issues introduced by > incremental patches :( Did you mean buildman? If you look at .travis.yml that builds all of those boards (well, with the exception of some we need gcc-6 in order to have small enough binaries for) and that's pretty close to 1200 I think. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: