Re: [U-Boot] [PATCH v2] fdt: fix setting MAC addresses for multiple interfaces

2016-05-04 Thread Lev Iserovich
Hi Joe, Thanks for fixing that up, I'll try to stick to patman in the future. --Lev On 05/03/2016 03:52 PM, Joe Hershberger wrote: Hi Lev, I had already applied your previous patch with a few minor adjustments. Please look at what's in my tree and send a patch on top of that if you care to

Re: [U-Boot] [PATCH v2] fdt: fix setting MAC addresses for multiple interfaces

2016-05-03 Thread Joe Hershberger
Hi Lev, On Mon, May 2, 2016 at 11:01 AM, Lev Iserovich wrote: > Hi, > > Updated with C-style comments: > > For multiple ethernet interfaces the FDT offset of '/aliases' will change as > we > are adding MAC addresses to the FDT. > Therefore only the first

Re: [U-Boot] [PATCH v2] fdt: fix setting MAC addresses for multiple interfaces

2016-05-02 Thread Lev Iserovich
Hi, Updated with C-style comments: For multiple ethernet interfaces the FDT offset of '/aliases' will change as we are adding MAC addresses to the FDT. Therefore only the first interface ('ethernet0') will get properly updated in the FDT, with the rest getting FDT errors when we try to set

Re: [U-Boot] [PATCH v2] fdt: fix setting MAC addresses for multiple interfaces

2016-05-02 Thread Joe Hershberger
On Tue, Apr 26, 2016 at 8:57 PM, Bin Meng wrote: > On Wed, Apr 27, 2016 at 8:06 AM, Bin Meng wrote: >> Hi Lev, >> >> On Fri, Jan 8, 2016 at 7:04 AM, Lev Iserovich >> wrote: >>> For multiple ethernet interfaces the FDT

Re: [U-Boot] [PATCH v2] fdt: fix setting MAC addresses for multiple interfaces

2016-04-26 Thread Bin Meng
On Wed, Apr 27, 2016 at 8:06 AM, Bin Meng wrote: > Hi Lev, > > On Fri, Jan 8, 2016 at 7:04 AM, Lev Iserovich > wrote: >> For multiple ethernet interfaces the FDT offset of '/aliases' will change as >> we >> are adding MAC addresses to the

Re: [U-Boot] [PATCH v2] fdt: fix setting MAC addresses for multiple interfaces

2016-04-26 Thread Bin Meng
Hi Lev, On Fri, Jan 8, 2016 at 7:04 AM, Lev Iserovich wrote: > For multiple ethernet interfaces the FDT offset of '/aliases' will change as > we > are adding MAC addresses to the FDT. > Therefore only the first interface ('ethernet0') will get properly updated >

Re: [U-Boot] [PATCH v2] fdt: fix setting MAC addresses for multiple interfaces

2016-04-26 Thread Joe Hershberger
On Thu, Jan 7, 2016 at 5:04 PM, Lev Iserovich wrote: > For multiple ethernet interfaces the FDT offset of '/aliases' will change as > we > are adding MAC addresses to the FDT. > Therefore only the first interface ('ethernet0') will get properly updated > in > the

[U-Boot] [PATCH v2] fdt: fix setting MAC addresses for multiple interfaces

2016-01-07 Thread Lev Iserovich
For multiple ethernet interfaces the FDT offset of '/aliases' will change as we are adding MAC addresses to the FDT. Therefore only the first interface ('ethernet0') will get properly updated in the FDT, with the rest getting FDT errors when we try to set their MAC address. Signed-off-by: Lev