diff mbox

[U-Boot] x86: galileo: Define mac addresses for the on-chip ethernet ports

Message ID BLU436-SMTP2519B5FAC7454535A883C44BFEF0@phx.gbl
State Rejected
Delegated to: Joe Hershberger
Headers show

Commit Message

Bin Meng April 21, 2015, 5:05 a.m. UTC
Not like other x86 chipset, there is no EEPROM for the ethernet
controller on the Intel Quark SoC to retreive the mac address
after power up. With pre-defined mac addresses, U-Boot boots up
and will not show "Error: dwmac.90006000 address not set" message.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
---

 include/configs/galileo.h | 2 ++
 1 file changed, 2 insertions(+)

Comments

Simon Glass April 21, 2015, 1:54 p.m. UTC | #1
Hi Bin,

On 20 April 2015 at 23:05, Bin Meng <bmeng.cn@gmail.com> wrote:
> Not like other x86 chipset, there is no EEPROM for the ethernet
> controller on the Intel Quark SoC to retreive the mac address
> after power up. With pre-defined mac addresses, U-Boot boots up
> and will not show "Error: dwmac.90006000 address not set" message.
>
> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
> ---
>
>  include/configs/galileo.h | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/include/configs/galileo.h b/include/configs/galileo.h
> index d4d0eb3..961d087 100644
> --- a/include/configs/galileo.h
> +++ b/include/configs/galileo.h
> @@ -62,6 +62,8 @@
>  #define CONFIG_DESIGNWARE_ETH
>  #define CONFIG_DW_ALTDESCRIPTOR
>  #define CONFIG_PHYLIB
> +#define CONFIG_ETHADDR         00:02:b3:00:00:00
> +#define CONFIG_ETH1ADDR                00:02:b3:00:00:01

I recall this coming up before with another board - we are not
supposed to set a default MAC address since it may create inexplicable
conflicts on the network if an org two boards on the same network one
day. Can you just set an environment variable?

>
>  /* Environment configuration */
>  #define CONFIG_ENV_SECT_SIZE           0x1000
> --
> 1.8.2.1
>

Regards,
Simon
Bin Meng April 22, 2015, 12:14 a.m. UTC | #2
Hi Simon,

On Tue, Apr 21, 2015 at 9:54 PM, Simon Glass <sjg@chromium.org> wrote:
> Hi Bin,
>
> On 20 April 2015 at 23:05, Bin Meng <bmeng.cn@gmail.com> wrote:
>> Not like other x86 chipset, there is no EEPROM for the ethernet
>> controller on the Intel Quark SoC to retreive the mac address
>> after power up. With pre-defined mac addresses, U-Boot boots up
>> and will not show "Error: dwmac.90006000 address not set" message.
>>
>> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
>> ---
>>
>>  include/configs/galileo.h | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/include/configs/galileo.h b/include/configs/galileo.h
>> index d4d0eb3..961d087 100644
>> --- a/include/configs/galileo.h
>> +++ b/include/configs/galileo.h
>> @@ -62,6 +62,8 @@
>>  #define CONFIG_DESIGNWARE_ETH
>>  #define CONFIG_DW_ALTDESCRIPTOR
>>  #define CONFIG_PHYLIB
>> +#define CONFIG_ETHADDR         00:02:b3:00:00:00
>> +#define CONFIG_ETH1ADDR                00:02:b3:00:00:01
>
> I recall this coming up before with another board - we are not
> supposed to set a default MAC address since it may create inexplicable
> conflicts on the network if an org two boards on the same network one
> day. Can you just set an environment variable?

Yes, I can. But I see lots of boards already use CONFIG_ETHADDR in
their board configuration files, so I assume this is allowed.

Regards,
Bin
diff mbox

Patch

diff --git a/include/configs/galileo.h b/include/configs/galileo.h
index d4d0eb3..961d087 100644
--- a/include/configs/galileo.h
+++ b/include/configs/galileo.h
@@ -62,6 +62,8 @@ 
 #define CONFIG_DESIGNWARE_ETH
 #define CONFIG_DW_ALTDESCRIPTOR
 #define CONFIG_PHYLIB
+#define CONFIG_ETHADDR		00:02:b3:00:00:00
+#define CONFIG_ETH1ADDR		00:02:b3:00:00:01
 
 /* Environment configuration */
 #define CONFIG_ENV_SECT_SIZE		0x1000