Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1621681

Re: [RFC net-next] of: mdio: Honor hints from MDIO bus drivers

From Andrew Lunn <andrew@lunn.ch>
Newsgroups linux.kernel
Subject Re: [RFC net-next] of: mdio: Honor hints from MDIO bus drivers
Date 2017-04-12 01:20 +0200
Message-ID <tvdeN-58v-11@gated-at.bofh.it> (permalink)
References <tuPma-6ni-19@gated-at.bofh.it> <tvciJ-4wo-3@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


> To give some more background and rational for this change.
> 
> On a platform where we have a parent MDIO bus, backed by the
> mdio-bcm-unimac.c driver, we also register a slave MII bus (through
> net/dsa/dsa2.c) which is parented to this UniMAC MDIO bus through an
> assignment of of_node. This slave MII bus is created in order to
> intercept reads/writes to problematic addresses (e.g: that clashes with
> another piece of hardware).
> 
> This means that the slave DSA MII bus inherits all child nodes from the
> originating master MII bus. This also means that when the slave MII bus
> is probed via of_mdiobus_register(), we probe the same devices twice:
> once through the master, another time through the slave.

Ah, O.K. This makes more sense. On the hardware i have, we get three
deep in MDIO busses. We have the FEC mdio bus. On top of that we have
a gpio-mux-mdio, and on top of that we have the mv88e6xxx mdio
bus. And i've never seen issues.

So your real problem here is you have two mdio busses using the same
device tree properties. I would actually say that is just plain
broken.

	Andrew

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[RFC net-next] of: mdio: Honor hints from MDIO bus drivers Florian Fainelli <f.fainelli@gmail.com> - 2017-04-10 23:50 +0200
  Re: [RFC net-next] of: mdio: Honor hints from MDIO bus drivers Florian Fainelli <f.fainelli@gmail.com> - 2017-04-12 00:20 +0200
    Re: [RFC net-next] of: mdio: Honor hints from MDIO bus drivers Andrew Lunn <andrew@lunn.ch> - 2017-04-12 01:20 +0200
      Re: [RFC net-next] of: mdio: Honor hints from MDIO bus drivers Florian Fainelli <f.fainelli@gmail.com> - 2017-04-12 01:30 +0200
        Re: [RFC net-next] of: mdio: Honor hints from MDIO bus drivers Florian Fainelli <f.fainelli@gmail.com> - 2017-04-12 23:50 +0200
          Re: [RFC net-next] of: mdio: Honor hints from MDIO bus drivers Andrew Lunn <andrew@lunn.ch> - 2017-04-13 00:20 +0200
            Re: [RFC net-next] of: mdio: Honor hints from MDIO bus drivers Florian Fainelli <f.fainelli@gmail.com> - 2017-04-13 02:00 +0200
              Re: [RFC net-next] of: mdio: Honor hints from MDIO bus drivers Andrew Lunn <andrew@lunn.ch> - 2017-04-14 00:00 +0200
                Re: [RFC net-next] of: mdio: Honor hints from MDIO bus drivers Florian Fainelli <f.fainelli@gmail.com> - 2017-04-14 01:30 +0200
  Re: [RFC net-next] of: mdio: Honor hints from MDIO bus drivers David Miller <davem@davemloft.net> - 2017-04-13 18:50 +0200

csiph-web