On Fri, 2017-04-28 at 11:05 +0200, Maxime Ripard wrote:
On Fri, Apr 28, 2017 at 03:33:53PM +0800, Chen-Yu Tsai wrote:
The sunxi platform relies on the core boot sequence to load and process device tree blobs, including writing back any MAC addresses we generate by an implicit call to fdt_fixup_ethernet() within the image loading mechanism. This call was removed in commit 3f66149d9fb4 ("Remove extra fdt_fixup_ethernet() call"), resuling in Linux using random MAC addresses.
This patch adds an explicit call to fdt_fixup_ethernet() in our ft_board_setup() function.
Fixes: 3f66149d9fb4 ("Remove extra fdt_fixup_ethernet() call")
I would just revert that commit. The function mentionned in the commit log is platform specific, it just doesn't make any sense to remove some generic code just because it would work on your platform.
The Generic code was both added later and wrong so no need to restore that.
One could remove the platform specific code and add back a generic one but then make sure it is called BEFORE any board setup.
Jocke