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
