On 11/21/2017 08:26 AM, Prabhakar Kushwaha wrote: > Current implementation of MAC address fix-up of device tree uses > tailing number behind "ethernet" found in "/aliases". It is not > necessary for trailing number of “ethernet” to be sequential. There > can be hole in between or any node disabled.
I would use "gap" instead of "hole". > > hole can be created based on SerDes protocol supported by SoCs. > Usually Linux device tree contains "ethernet" node for all possible > interface supported by SoC i.e. LS1046A. Where based on SerDes > protocol some "ethernet" interface are unavailable. I think it would be more clear to explain the "ethernet" port numbers are not consecutive in this case. > > This patch-set provide way to fetch MAC addr sequentially from env > variables and also create hole in "ethernet" node (disable) based > on SerDes protocol for LS1046ARDB. I think you meant to fetch MAC addresses sequentially from environmental variables and apply them to "ethernet" nodes in the order they appear in device tree. > > Other platforms can also follow same approach. > > Prabhakar Kushwaha (3): > common: Fix-up MAC addr in dts by fetching env variable serially > arm: Add support of updating dts before fix-up > boards: ls1046ardb: disable unavailable "ethernet" node in dts > > README | 10 ++++++++ > arch/arm/lib/bootm-fdt.c | 12 ++++++++++ > board/freescale/ls1046ardb/eth.c | 51 > ++++++++++++++++++++++++++++++++++++++++ > common/fdt_support.c | 25 ++++++++++++++++---- > include/configs/ls1046ardb.h | 2 ++ > include/fdt_support.h | 3 +++ > 6 files changed, 99 insertions(+), 4 deletions(-) > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
