Re: [PATCH v2] DT: net: document Ethernet bindings in one place

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

 



Hello.

On 01/29/2014 07:16 PM, Rob Herring wrote:

This patch is an attempt to gather the Ethernet related bindings in one
file,
like it's done in the MMC and some other subsystems. It should save some
of
the trouble of documenting several properties over and over in each
binding
document, instead only making reference to the main file.

[snip]

   - compatible: should be "allwinner,sun4i-emac".
   - reg: address and length of the register set for the device.
   - interrupts: interrupt for the device
-- phy: A phandle to a phy node defining the PHY address (as the reg
-  property, a single integer).
+- phy: see ethernet.txt file in the same directory.
   - clocks: A phandle to the reference clock for this device

   Optional properties:
-- (local-)mac-address: mac address to be used by this driver
+- [local-]mac-address: see ethernet.txt file in the same directory

Sorry, but I meant you should just have a single statement like:

This binding uses standard ethernet properties defined in ethernet.txt.

    This way it won't be clear which of the properties are used and which are
not, no?

You're right. For the phy properties we need to specify which ones are
used. This could be done only for bindings using the not recommended
properties.

Sigh, the issue with e.g. "phy-mode" is I can't un-recommend it yet as the patch adding "phy-connection-type" parsing to of_get_phy_mode() seems to have been lost in transit. :-/

For optional properties like local-mac-address, I don't think these
need to be listed. They can be used for all devices and whether or not
a given Linux driver supports using the property is not really the
concern of the binding documentation. An exception could be a device

   Hm, this gets somewhat complicated... but let's try.

that requires a optional property (i.e. a NIC with no eeprom to store
a MAC address).

Can't remember an example of such device from the top of my head anyways (as it requires the boot loader to fill in this property which is still a problem).

Rob

WBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux