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

Groups > linux.kernel > #1336588

Re: [PATCH] net: macb: Add support for phy-handle binding

Show all headers | View raw

From: Nathan Rossi <nathan@nathanrossi.com>
Date: Sun, 14 Feb 2016 02:02:48 +1000

> This patch adds support for the 'phy-handle' binding which allows for a
> system to specifically select a phy which can be attached via any MDIO
> bus available in the system.
> 
> Signed-off-by: Nathan Rossi <nathan@nathanrossi.com>
> Cc: Nicolas Ferre <nicolas.ferre@atmel.com>

I don't see how this can be backwards compatible at all.

In my opinion we are way too loose about handling things like this.

Existing chips that happened to have an OF node but lack a phy-handle
property are going to be broken by that change.

I know you are now going to bombard me with all kinds of reasons why
this won't happen.

Don't bother, I'm simply not interested.

All of these special cases we use (all the DT bindings are in text
files in the kernel sources, we control all of the bootloaders, etc.)
is the most shaky foundation I've ever seen upon which to erect a
stable device probing mechanism.

I'm not applying this patch until you add an error handling path from
the of_phy_connect() call that will do the existing PHY probing sequence
by hand using phy_find_first().

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

Thread

[PATCH] net: macb: Add support for phy-handle binding Nathan Rossi <nathan@nathanrossi.com> - 2016-02-13 17:10 +0100
  Re: [PATCH] net: macb: Add support for phy-handle binding David Miller <davem@davemloft.net> - 2016-02-17 18:20 +0100
    Re: [PATCH] net: macb: Add support for phy-handle binding Nathan Rossi <nathan@nathanrossi.com> - 2016-02-17 19:50 +0100
      Re: [PATCH] net: macb: Add support for phy-handle binding David Miller <davem@davemloft.net> - 2016-02-17 20:00 +0100

csiph-web