[U-Boot] [PATCH 0/3] Update MAC address in "ethernet" node of Linux device tree

[York Sun]

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(-)
>