Re: [PATCH] dt: net: enhance DWC EQoS binding to support Tegra186

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 08/24/2016 02:10 AM, Lars Persson wrote:
On 08/23/2016 10:47 PM, Stephen Warren wrote:
The Synopsys DWC EQoS is a configurable IP block which supports multiple
options for bus type, clocking and reset structure, and feature list.
Extend the DT binding to define a "compatible value" for the
configuration
contained in NVIDIA's Tegra186 SoC, and define some new properties and
list property entries required by that configuration.

diff --git
a/Documentation/devicetree/bindings/net/snps,dwc-qos-ethernet.txt
b/Documentation/devicetree/bindings/net/snps,dwc-qos-ethernet.txt

 Optional properties:

+- phy-reset-gpios: Phandle and specifier for any GPIO used to reset
the PHY.
+  See ../gpio/gpio.txt.

IMHO the phy reset gpio belongs in the binding for the PHY. I notice
some other ethernet drivers have this, but the PHY should be managed
entirely through the phylib and any special handling for reset can be
hidden in phy specific drivers.

I can see that argument; this GPIO certainly does control the PHY so seems part of it. However, presumably this GPIO must be manipulated before being able to communicate with the PHY at all, and hence instantiate any driver that might control the PHY. As such, this seems more like a property of the MDIO bus than the PHY itself, even if it electrically is part of the PHY. Also, Documentation/devicetree/bindings/net/phy.txt doesn't contain any phy-reset-gpios property or similar, so we'd have to add that if we wanted to rely upon it.

For now I'll post V2 without changing this, but I can always post V3 if needed.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux