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

Groups > linux.kernel > #1265872

Re: [PATCH] mvneta: add FIXED_PHY dependency

Show all headers | View raw

On Monday 09 November 2015 18:08:49 Andrew Lunn wrote:
> > > I suppose it comes down to, are we allowed to optionally implement
> > > part of the DT binding?
> > 
> > I'm not sure what you are asking. A lot of DT bindings have both
> > optional and mandatory properties. For mvneta, the "phy" and "phy-mode"
> > properties are listed as mandatory, so the driver can safely assume
> > that they are always present. If there are reasons to leave them out,
> > and for the driver to handle that case correctly, the binding
> > should be updated to mark them as optional.
> 
> Hi Arnd
> 
> You are looking at it from the perspective of the driver. I was
> meaning from the perspective of the DT blob. Can be blob assume the
> driver implements all of the binding, all of the time?

That question is not really relevant: the DT describes the hardware,
it doesn't matter whether there are drivers for all the bits or
whether all properties are read.

> You use fixed-phy when the MAC is connected to a switch, not a phy. Or
> when the MAC is connected to an SFP module. The driver can currently
> be built to not implement the fixed-phy party of the binding. Is that
> O.K. from the perspective of the DT blob? Or should the driver always
> implement all of the binding, in which these NOP stubs should be
> removed and fixed phy always be enabled for the drivers that use it.

Sure, that is ok.

	Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

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

Thread

[PATCH] mvneta: add FIXED_PHY dependency Arnd Bergmann <arnd@arndb.de> - 2015-11-09 15:10 +0100
  Re: [PATCH] mvneta: add FIXED_PHY dependency David Miller <davem@davemloft.net> - 2015-11-09 17:40 +0100
  Re: [PATCH] mvneta: add FIXED_PHY dependency Andrew Lunn <andrew@lunn.ch> - 2015-11-09 17:50 +0100
    Re: [PATCH] mvneta: add FIXED_PHY dependency Arnd Bergmann <arnd@arndb.de> - 2015-11-09 18:00 +0100
      Re: [PATCH] mvneta: add FIXED_PHY dependency Andrew Lunn <andrew@lunn.ch> - 2015-11-09 18:10 +0100
        Re: [PATCH] mvneta: add FIXED_PHY dependency Arnd Bergmann <arnd@arndb.de> - 2015-11-09 18:20 +0100
        Re: [PATCH] mvneta: add FIXED_PHY dependency Russell King - ARM Linux <linux@arm.linux.org.uk> - 2015-11-09 18:40 +0100
      Re: [PATCH] mvneta: add FIXED_PHY dependency Russell King - ARM Linux <linux@arm.linux.org.uk> - 2015-11-09 18:10 +0100
        Re: [PATCH] mvneta: add FIXED_PHY dependency Arnd Bergmann <arnd@arndb.de> - 2015-11-09 18:20 +0100
          Re: [PATCH] mvneta: add FIXED_PHY dependency Russell King - ARM Linux <linux@arm.linux.org.uk> - 2015-11-09 18:40 +0100

csiph-web