Message ID | MMXP123MB137666A8C158380B6CE1EF65AC1D0@MMXP123MB1376.GBRP123.PROD.OUTLOOK.COM |
---|---|
State | Accepted |
Commit | ecc3bd73b35398d8337096b19493028a29ed038e |
Delegated to: | Simon Glass |
Headers | show |
On 22 April 2017 at 02:57, Jonas Karlman <jonas@kwiboo.se> wrote: > Set ethernet mac address in late init for Tinker Board, > prevents getting a random mac address each boot. > > Read mac address from eeprom, first 6 bytes from m24c08@50. > Same as /etc/init.d/rockchip.sh on Tinker OS. > > Signed-off-by: Jonas Karlman <jonas@kwiboo.se> > --- > > Changes in v3: > - Use uclass_first_device_err() to find eeprom > - Use i2c_eeprom_read() to read eeprom > > Changes in v2: > - Change to use i2c_eeprom device driver > > arch/arm/dts/rk3288-tinker.dts | 7 +++++++ > board/rockchip/tinker_rk3288/tinker-rk3288.c | 28 ++++++++++++++++++++++++++++ > configs/tinker-rk3288_defconfig | 3 +++ > 3 files changed, 38 insertions(+) Reviewed-by: Simon Glass <sjg@chromium.org>
On 22 April 2017 at 02:57, Jonas Karlman <jonas@kwiboo.se> wrote: > Set ethernet mac address in late init for Tinker Board, > prevents getting a random mac address each boot. > > Read mac address from eeprom, first 6 bytes from m24c08@50. > Same as /etc/init.d/rockchip.sh on Tinker OS. > > Signed-off-by: Jonas Karlman <jonas@kwiboo.se> > --- > > Changes in v3: > - Use uclass_first_device_err() to find eeprom > - Use i2c_eeprom_read() to read eeprom > > Changes in v2: > - Change to use i2c_eeprom device driver > > arch/arm/dts/rk3288-tinker.dts | 7 +++++++ > board/rockchip/tinker_rk3288/tinker-rk3288.c | 28 ++++++++++++++++++++++++++++ > configs/tinker-rk3288_defconfig | 3 +++ > 3 files changed, 38 insertions(+) Reviewed-by: Simon Glass <sjg@chromium.org> Applied to u-boot-rockchip/next, thanks!
diff --git a/arch/arm/dts/rk3288-tinker.dts b/arch/arm/dts/rk3288-tinker.dts index 22881cb785..ea2f715922 100644 --- a/arch/arm/dts/rk3288-tinker.dts +++ b/arch/arm/dts/rk3288-tinker.dts @@ -67,3 +67,10 @@ &gpio8 { u-boot,dm-pre-reloc; }; + +&i2c2 { + m24c08@50 { + compatible = "at,24c08", "i2c-eeprom"; + reg = <0x50>; + }; +}; diff --git a/board/rockchip/tinker_rk3288/tinker-rk3288.c b/board/rockchip/tinker_rk3288/tinker-rk3288.c index 79541a3939..c2872e7330 100644 --- a/board/rockchip/tinker_rk3288/tinker-rk3288.c +++ b/board/rockchip/tinker_rk3288/tinker-rk3288.c @@ -5,3 +5,31 @@ */ #include <common.h> +#include <dm.h> +#include <i2c_eeprom.h> +#include <netdev.h> + +static int get_ethaddr_from_eeprom(u8 *addr) +{ + int ret; + struct udevice *dev; + + ret = uclass_first_device_err(UCLASS_I2C_EEPROM, &dev); + if (ret) + return ret; + + return i2c_eeprom_read(dev, 0, addr, 6); +} + +int rk_board_late_init(void) +{ + u8 ethaddr[6]; + + if (get_ethaddr_from_eeprom(ethaddr)) + return 0; + + if (is_valid_ethaddr(ethaddr)) + eth_setenv_enetaddr("ethaddr", ethaddr); + + return 0; +} diff --git a/configs/tinker-rk3288_defconfig b/configs/tinker-rk3288_defconfig index cec39384b3..dc3699d5c4 100644 --- a/configs/tinker-rk3288_defconfig +++ b/configs/tinker-rk3288_defconfig @@ -11,6 +11,7 @@ CONFIG_CONSOLE_MUX=y # CONFIG_DISPLAY_CPUINFO is not set CONFIG_SPL_STACK_R=y CONFIG_SPL_STACK_R_MALLOC_SIMPLE_LEN=0x2000 +CONFIG_SPL_I2C_SUPPORT=y # CONFIG_CMD_IMLS is not set CONFIG_CMD_GPT=y CONFIG_CMD_MMC=y @@ -39,6 +40,8 @@ CONFIG_CLK=y CONFIG_SPL_CLK=y CONFIG_ROCKCHIP_GPIO=y CONFIG_SYS_I2C_ROCKCHIP=y +CONFIG_MISC=y +CONFIG_I2C_EEPROM=y CONFIG_MMC_DW=y CONFIG_MMC_DW_ROCKCHIP=y CONFIG_DM_ETH=y
Set ethernet mac address in late init for Tinker Board, prevents getting a random mac address each boot. Read mac address from eeprom, first 6 bytes from m24c08@50. Same as /etc/init.d/rockchip.sh on Tinker OS. Signed-off-by: Jonas Karlman <jonas@kwiboo.se> --- Changes in v3: - Use uclass_first_device_err() to find eeprom - Use i2c_eeprom_read() to read eeprom Changes in v2: - Change to use i2c_eeprom device driver arch/arm/dts/rk3288-tinker.dts | 7 +++++++ board/rockchip/tinker_rk3288/tinker-rk3288.c | 28 ++++++++++++++++++++++++++++ configs/tinker-rk3288_defconfig | 3 +++ 3 files changed, 38 insertions(+)