From mboxrd@z Thu Jan 1 00:00:00 1970 From: Karl Beldan Date: Sat, 20 Aug 2016 09:08:45 +0000 Subject: [U-Boot] [PATCH v2 0/5] Remove calls from net drivers to legacy miiphy_register() entry-point In-Reply-To: References: <1459760858-25071-1-git-send-email-joe.hershberger@ni.com> <1470673721-20590-1-git-send-email-joe.hershberger@ni.com> <20160819081456.GA30921@gobelin> Message-ID: <20160820090845.GA4865@gobelin> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Fri, Aug 19, 2016 at 01:48:00PM -0500, Joe Hershberger wrote: > Hi Karl, > > On Fri, Aug 19, 2016 at 3:14 AM, Karl Beldan wrote: > > On Mon, Aug 08, 2016 at 11:28:36AM -0500, Joe Hershberger wrote: > >> > >> This is an effort to migrate closer to the phylib approach so that a > >> new DM approach can be developed from a single starting point. The more > >> of this legacy code that can be removed, the easier it will be to move > >> forward. > >> > > > > Hi Joe, > > > > It was nice to see spatch put to work. However davinci_emac crashes > > U-Boot now. It is an invasive patch so it might take time to check > > everything is in order, so unless you already have a follow-up series I > > can send in a trivial patch just now to fix that for davinci_emac. > > That's a bummer. Is it a systemic problem that you expect to affect > more drivers or is it something that is likely to only affect davinci? > It is likely to only affect davinci, just a mix up of the internal MDIO accessors return code (not in the changes made by spatch proper). It would be better in the future in the davinci code to just use the same convention as the mdiodev accessors but I chose a non-invasive quick to check follow-up patch. Rgds, Karl