From mboxrd@z Thu Jan 1 00:00:00 1970 From: clabbe.montjoie@gmail.com (Corentin Labbe) Date: Tue, 24 Oct 2017 21:02:26 +0200 Subject: [PATCH v7 10/10] of: mdio: Prevent of_mdiobus_register from scanning mdio-mux nodes In-Reply-To: <20171024184654.ggfrsrjd3nobku5n@rob-hp-laptop> References: <20171018114458.17891-1-clabbe.montjoie@gmail.com> <20171018114458.17891-11-clabbe.montjoie@gmail.com> <20171024184654.ggfrsrjd3nobku5n@rob-hp-laptop> Message-ID: <20171024190226.GA23866@Red> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, Oct 24, 2017 at 01:46:54PM -0500, Rob Herring wrote: > On Wed, Oct 18, 2017 at 01:44:58PM +0200, Corentin Labbe wrote: > > Each child node of an MDIO node is scanned as a PHY when calling > > of_mdiobus_register() givint the following result: > > [ 18.175379] mdio_bus stmmac-0: /soc/ethernet at 1c30000/mdio/mdio-mux has invalid PHY address > > [ 18.175408] mdio_bus stmmac-0: scan phy mdio-mux at address 0 > > [ 18.175450] mdio_bus stmmac-0: scan phy mdio-mux at address 1 > > [...] > > [ 18.176420] mdio_bus stmmac-0: scan phy mdio-mux at address 30 > > [ 18.176452] mdio_bus stmmac-0: scan phy mdio-mux at address 31 > > > > Since mdio-mux nodes are not PHY, this patch a way to to not scan > > them. > > This can be dropped now, right? Yes and it is dropped in my two last serie. Forgot to said it in changelog From mboxrd@z Thu Jan 1 00:00:00 1970 From: Corentin Labbe Subject: Re: [PATCH v7 10/10] of: mdio: Prevent of_mdiobus_register from scanning mdio-mux nodes Date: Tue, 24 Oct 2017 21:02:26 +0200 Message-ID: <20171024190226.GA23866@Red> References: <20171018114458.17891-1-clabbe.montjoie@gmail.com> <20171018114458.17891-11-clabbe.montjoie@gmail.com> <20171024184654.ggfrsrjd3nobku5n@rob-hp-laptop> Reply-To: clabbe.montjoie-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: Sender: linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org Content-Disposition: inline In-Reply-To: <20171024184654.ggfrsrjd3nobku5n@rob-hp-laptop> List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , To: Rob Herring Cc: mark.rutland-5wv7dgnIgG8@public.gmane.org, maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org, wens-jdAy2FN1RRM@public.gmane.org, linux-I+IVW8TIWO2tmTQ+vhA3Yw@public.gmane.org, catalin.marinas-5wv7dgnIgG8@public.gmane.org, will.deacon-5wv7dgnIgG8@public.gmane.org, peppe.cavallaro-qxv4g6HH51o@public.gmane.org, alexandre.torgue-qxv4g6HH51o@public.gmane.org, andrew-g2DYL2Zd6BY@public.gmane.org, f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, frowand.list-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org List-Id: devicetree@vger.kernel.org On Tue, Oct 24, 2017 at 01:46:54PM -0500, Rob Herring wrote: > On Wed, Oct 18, 2017 at 01:44:58PM +0200, Corentin Labbe wrote: > > Each child node of an MDIO node is scanned as a PHY when calling > > of_mdiobus_register() givint the following result: > > [ 18.175379] mdio_bus stmmac-0: /soc/ethernet@1c30000/mdio/mdio-mux has invalid PHY address > > [ 18.175408] mdio_bus stmmac-0: scan phy mdio-mux at address 0 > > [ 18.175450] mdio_bus stmmac-0: scan phy mdio-mux at address 1 > > [...] > > [ 18.176420] mdio_bus stmmac-0: scan phy mdio-mux at address 30 > > [ 18.176452] mdio_bus stmmac-0: scan phy mdio-mux at address 31 > > > > Since mdio-mux nodes are not PHY, this patch a way to to not scan > > them. > > This can be dropped now, right? Yes and it is dropped in my two last serie. Forgot to said it in changelog From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932185AbdJXTCe (ORCPT ); Tue, 24 Oct 2017 15:02:34 -0400 Received: from mail-wr0-f169.google.com ([209.85.128.169]:47939 "EHLO mail-wr0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751354AbdJXTCb (ORCPT ); Tue, 24 Oct 2017 15:02:31 -0400 X-Google-Smtp-Source: ABhQp+RxfxQvO/fOn8e5+72Hr83pdSKxwBHQWa/e6Rj9c90wzIVMIELFta4n5idbk30/29Cf36hpVQ== Date: Tue, 24 Oct 2017 21:02:26 +0200 From: Corentin Labbe To: Rob Herring Cc: mark.rutland@arm.com, maxime.ripard@free-electrons.com, wens@csie.org, linux@armlinux.org.uk, catalin.marinas@arm.com, will.deacon@arm.com, peppe.cavallaro@st.com, alexandre.torgue@st.com, andrew@lunn.ch, f.fainelli@gmail.com, frowand.list@gmail.com, netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-sunxi@googlegroups.com Subject: Re: [PATCH v7 10/10] of: mdio: Prevent of_mdiobus_register from scanning mdio-mux nodes Message-ID: <20171024190226.GA23866@Red> References: <20171018114458.17891-1-clabbe.montjoie@gmail.com> <20171018114458.17891-11-clabbe.montjoie@gmail.com> <20171024184654.ggfrsrjd3nobku5n@rob-hp-laptop> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171024184654.ggfrsrjd3nobku5n@rob-hp-laptop> User-Agent: Mutt/1.7.2 (2016-11-26) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Oct 24, 2017 at 01:46:54PM -0500, Rob Herring wrote: > On Wed, Oct 18, 2017 at 01:44:58PM +0200, Corentin Labbe wrote: > > Each child node of an MDIO node is scanned as a PHY when calling > > of_mdiobus_register() givint the following result: > > [ 18.175379] mdio_bus stmmac-0: /soc/ethernet@1c30000/mdio/mdio-mux has invalid PHY address > > [ 18.175408] mdio_bus stmmac-0: scan phy mdio-mux at address 0 > > [ 18.175450] mdio_bus stmmac-0: scan phy mdio-mux at address 1 > > [...] > > [ 18.176420] mdio_bus stmmac-0: scan phy mdio-mux at address 30 > > [ 18.176452] mdio_bus stmmac-0: scan phy mdio-mux at address 31 > > > > Since mdio-mux nodes are not PHY, this patch a way to to not scan > > them. > > This can be dropped now, right? Yes and it is dropped in my two last serie. Forgot to said it in changelog