From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Lunn Subject: Re: [PATCH] net/phy: micrel: Follow attached_dev to get to the MAC device Date: Sat, 17 Oct 2015 01:50:42 +0200 Message-ID: <20151016235042.GA28795@lunn.ch> References: <1445028545-22599-1-git-send-email-andrew@lunn.ch> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Florian Fainelli , netdev To: Dinh Nguyen Return-path: Received: from vps0.lunn.ch ([178.209.37.122]:42075 "EHLO vps0.lunn.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751201AbbJPXuo (ORCPT ); Fri, 16 Oct 2015 19:50:44 -0400 Content-Disposition: inline In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: On Fri, Oct 16, 2015 at 05:58:41PM -0500, Dinh Nguyen wrote: > On Fri, 16 Oct 2015, Andrew Lunn wrote: > > > Commit "8b63ec1837fa phylib: Make PHYs children of their MDIO bus, not > > the bus' parent." broke finding PHY properties in the MAC device tree > > node. The parent device is now the MDIO bus, not the MAC. Use > > attached_dev towards the MAC device tree node. > > > > Fixes: 8b63ec1837fa ("phylib: Make PHYs children of their MDIO bus, not the bus' parent.") > > Signed-off-by: Andrew Lunn > > --- > > > > Compile tested only. > > > > Dinh, please could you test it and report back if it works or not. > > > > This patch did not seem to fix the problem. The following code did seem to > fix the problem: > > if (!of_node && dev->parent->of_node) > - of_node = dev->parent->of_node; > + do { > + of_node = dev->of_node; > + dev = dev->parent; > + i++; > + } while (!of_node && dev); This might fix the issue, but it has disadvantages. As i said before, it allows people to place phy properties into the mdio device node. We want to be reducing placing you can add phy properties, not adding more. Please could you try to debug why my patch did not work. Is attached_dev null? Andrew