On 08/27/2014 10:54 PM, Mark Rutland wrote:
On Wed, Aug 27, 2014 at 01:38:29PM +0100, Zhangfei Gao wrote:Signed-off-by: Zhangfei Gao <zhangfei.gao@xxxxxxxxxx> --- arch/arm/boot/dts/hisi-x5hd2-dkb.dts | 26 ++++++++++++++++++++++++++ arch/arm/boot/dts/hisi-x5hd2.dtsi | 16 ++++++++++++++++ 2 files changed, 42 insertions(+) diff --git a/arch/arm/boot/dts/hisi-x5hd2-dkb.dts b/arch/arm/boot/dts/hisi-x5hd2-dkb.dts index 05b44c2..0344de0 100644 --- a/arch/arm/boot/dts/hisi-x5hd2-dkb.dts +++ b/arch/arm/boot/dts/hisi-x5hd2-dkb.dts @@ -51,3 +51,29 @@ &uart0 { status = "okay"; }; + +&gmac0 { + #address-cells = <1>; + #size-cells = <0>; + phy-handle = <&phy2>; + phy-mode = "mii"; + mac-address = [00 00 00 00 00 00];If this is just so space is pre-allocated for the bootloader to patch this, please place a comment to that effect. Otherwise that doesn't look right.
Yes, it is placeholder, bootloader will overwrite field. Also net driver will use random address if if not valid addr. if (!is_valid_ether_addr(ndev->dev_addr)) eth_hw_addr_random(ndev); Thanks -- 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
