mbox series

[RFC,0/3] net: phy: at803x device tree binding

Message ID 20191030224251.21578-1-michael@walle.cc
Headers show
Series net: phy: at803x device tree binding | expand

Message

Michael Walle Oct. 30, 2019, 10:42 p.m. UTC
Adds a device tree binding to configure the clock and the RGMII voltage.

I do have the following questions:
 - Who should be the maintainer of the atheros,at803x.yaml file?
 - Is the "atheros,rgmii-io-1v8" boolean property ok or should it be a
   "atheros,rgmii-io-microvolt = <1800000>"?
 - There is actually a typo throughout the whole at803x file. The actual
   name of the PHY is "Atheros AR803x". What should be the name of the yaml
   file and the dt-bindings header file? atheros,at803x.yaml or
   atheros,ar803x.yaml. Likewise for the header file.

Michael Walle (3):
  net: phy: at803x: fix Kconfig description
  dt-bindings: net: phy: Add support for AT803X
  net: phy: at803x: add device tree binding

 .../bindings/net/atheros,at803x.yaml          |  58 +++++++
 drivers/net/phy/Kconfig                       |   4 +-
 drivers/net/phy/at803x.c                      | 156 +++++++++++++++++-
 include/dt-bindings/net/atheros-at803x.h      |  13 ++
 4 files changed, 227 insertions(+), 4 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/net/atheros,at803x.yaml
 create mode 100644 include/dt-bindings/net/atheros-at803x.h

Cc: "David S. Miller" <davem@davemloft.net>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Andrew Lunn <andrew@lunn.ch>
Cc: Florian Fainelli <f.fainelli@gmail.com>
Cc: Heiner Kallweit <hkallweit1@gmail.com>