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


Groups > linux.kernel > #1577808

Re: [PATCH] [net-next] ARM: orion: fix PHYLIB dependency

From Andrew Lunn <andrew@lunn.ch>
Newsgroups linux.kernel
Subject Re: [PATCH] [net-next] ARM: orion: fix PHYLIB dependency
Date 2017-02-09 18:30 +0100
Message-ID <t90HD-yG-1@gated-at.bofh.it> (permalink)
References <t8YFR-7Mt-27@gated-at.bofh.it> <t904W-5k-25@gated-at.bofh.it> <t90xY-vh-17@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


> Adding a 'select' statement to something as broad as NETDEVICES sounds
> really bad, it has a significant risk of introducing dependency loops
> and may be confusing if you want to build a multiplatform config without
> networking support (note that NETDEVICES in turn depends on NET, which
> can also be disabled).

O.K, so overall it is not simple. So lets drop my idea.
 
> One possibility would be to have a special Kconfig symbol that controls
> mdiobus_register_board_info() being present and have that symbol
> force PHYLIB to never be "=m". Then we can either have no networking
> support and no phylib, turning mdiobus_register_board_info() into a
> stub, or we have the function built-in and reachable from the board
> code.

FYI: Florian is working on splitting MDIO out of PHYLIB. There will be
two separate symbols, so it will be possible to have MDIO without
PHYLIB. When this happens, i expect mdiobus_register_board_info() will
be in the MDIO part, not PHYLIB.

   Andrew

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


Thread

[PATCH] [net-next] ARM: orion: fix PHYLIB dependency Arnd Bergmann <arnd@arndb.de> - 2017-02-09 16:20 +0100
  Re: [PATCH] [net-next] ARM: orion: fix PHYLIB dependency Andrew Lunn <andrew@lunn.ch> - 2017-02-09 17:50 +0100
    Re: [PATCH] [net-next] ARM: orion: fix PHYLIB dependency Arnd Bergmann <arnd@arndb.de> - 2017-02-09 18:20 +0100
      Re: [PATCH] [net-next] ARM: orion: fix PHYLIB dependency Andrew Lunn <andrew@lunn.ch> - 2017-02-09 18:30 +0100
        Re: [PATCH] [net-next] ARM: orion: fix PHYLIB dependency Florian Fainelli <f.fainelli@gmail.com> - 2017-02-09 19:20 +0100
  Re: [PATCH] [net-next] ARM: orion: fix PHYLIB dependency Andrew Lunn <andrew@lunn.ch> - 2017-02-09 19:20 +0100
  Re: [PATCH] [net-next] ARM: orion: fix PHYLIB dependency Florian Fainelli <f.fainelli@gmail.com> - 2017-02-09 19:40 +0100
    Re: [PATCH] [net-next] ARM: orion: fix PHYLIB dependency Arnd Bergmann <arnd@arndb.de> - 2017-02-10 13:40 +0100
      Re: [PATCH] [net-next] ARM: orion: fix PHYLIB dependency Florian Fainelli <f.fainelli@gmail.com> - 2017-02-10 19:00 +0100
        Re: [PATCH] [net-next] ARM: orion: fix PHYLIB dependency Arnd Bergmann <arnd@arndb.de> - 2017-02-10 21:10 +0100
          Re: [PATCH] [net-next] ARM: orion: fix PHYLIB dependency Florian Fainelli <f.fainelli@gmail.com> - 2017-02-10 22:00 +0100
            Re: [PATCH] [net-next] ARM: orion: fix PHYLIB dependency Arnd Bergmann <arnd@arndb.de> - 2017-02-10 22:40 +0100
              Re: [PATCH] [net-next] ARM: orion: fix PHYLIB dependency Arnd Bergmann <arnd@arndb.de> - 2017-02-13 15:40 +0100

csiph-web