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

[Prabhakar Kushwaha]

Hi Simon,

> -----Original Message-----
> From: York Sun
> Sent: Thursday, November 30, 2017 1:03 AM
> To: Simon Glass <sjg at chromium.org>
> Cc: Prabhakar Kushwaha <prabhakar.kushwaha at nxp.com>; u-
> boot at lists.denx.de
> Subject: Re: [PATCH 0/3][v3] Update MAC address in "ethernet" node of Linux
> device tree
> 
> On 11/23/2017 03:22 AM, Prabhakar Kushwaha wrote:
> > The MAC addresses get fixed in the device tree for "ethernet" nodes
> > is by using trailing number behind "ethernet" found in "/aliases".
> > It may not be necessary for the "ethernet" nodes to be sequential.
> > There can be gaps in between or any ethenet node disabled.
> >
> > Reason for gaps is because the available ethernet port are based on
> > SerDes protocol chosen for the SoC. Usually Linux device tree
> > contains "ethernet" node for all possible ports supported by SoC
> > for e.g. LS1046A. Where based on SerDes protocol some "ethernet"
> > ports are unavailable hence non-sequential.
> >
> > 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.
> >
> > This patch-set provide a support to fetch MAC addr sequentially
> > from env and apply them to "ethernet" nodes in the order they appear
> > in device tree only if "ethernet" is not "disabled"
> >
> > It also disables "ethernet" nodes in device tree based ethernet port
> > availability  in the selected SerDes protocol.
> > Currently the changes are done foe LS1046A. This has been added as a
> > compile time option for now, to keep backward compatibility.
> >
> > The associated documentation is also updated in the README.
> >
> > Other platforms should 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                           |  9 +++++++
> >  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, 98 insertions(+), 4 deletions(-)
> >
> 
> Simon,
> 
> What do you think about this set?
> 

There is no review comments on this patch-set. 

Can you please look into this and help with review/ack.

--pk