* [U-Boot] [PATCH] mvebu: turris_omnia: Only set eth?addr env if CONFIG_CMD_NET
@ 2017-09-03 13:13 Marek Behún
2017-09-04 6:42 ` Stefan Roese
0 siblings, 1 reply; 3+ messages in thread
From: Marek Behún @ 2017-09-03 13:13 UTC (permalink / raw)
To: u-boot
Otherwise the linking will fail since eth_env_set_enetaddr cannot
be found.
Signed-off-by: Marek Behun <marek.behun@nic.cz>
---
board/CZ.NIC/turris_omnia/turris_omnia.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/board/CZ.NIC/turris_omnia/turris_omnia.c b/board/CZ.NIC/turris_omnia/turris_omnia.c
index af66837909..7db3b8ff3b 100644
--- a/board/CZ.NIC/turris_omnia/turris_omnia.c
+++ b/board/CZ.NIC/turris_omnia/turris_omnia.c
@@ -470,6 +470,7 @@ out:
return 0;
}
+#if defined(CONFIG_ATSHA204A) && defined(CONFIG_CMD_NET)
static void increment_mac(u8 *mac)
{
int i;
@@ -480,10 +481,11 @@ static void increment_mac(u8 *mac)
break;
}
}
+#endif
int misc_init_r(void)
{
-#ifdef CONFIG_ATSHA204A
+#if defined(CONFIG_ATSHA204A) && defined(CONFIG_CMD_NET)
int err;
struct udevice *dev = get_atsha204a_dev();
u8 mac0[4], mac1[4], mac[6];
--
2.13.5
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH] mvebu: turris_omnia: Only set eth?addr env if CONFIG_CMD_NET
2017-09-03 13:13 [U-Boot] [PATCH] mvebu: turris_omnia: Only set eth?addr env if CONFIG_CMD_NET Marek Behún
@ 2017-09-04 6:42 ` Stefan Roese
2017-09-19 4:50 ` Stefan Roese
0 siblings, 1 reply; 3+ messages in thread
From: Stefan Roese @ 2017-09-04 6:42 UTC (permalink / raw)
To: u-boot
Hi Marek,
On 03.09.2017 15:13, Marek Behún wrote:
> Otherwise the linking will fail since eth_env_set_enetaddr cannot
> be found.
>
> Signed-off-by: Marek Behun <marek.behun@nic.cz>
> ---
> board/CZ.NIC/turris_omnia/turris_omnia.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/board/CZ.NIC/turris_omnia/turris_omnia.c b/board/CZ.NIC/turris_omnia/turris_omnia.c
> index af66837909..7db3b8ff3b 100644
> --- a/board/CZ.NIC/turris_omnia/turris_omnia.c
> +++ b/board/CZ.NIC/turris_omnia/turris_omnia.c
> @@ -470,6 +470,7 @@ out:
> return 0;
> }
>
> +#if defined(CONFIG_ATSHA204A) && defined(CONFIG_CMD_NET)
> static void increment_mac(u8 *mac)
> {
> int i;
> @@ -480,10 +481,11 @@ static void increment_mac(u8 *mac)
> break;
> }
> }
> +#endif
>
> int misc_init_r(void)
> {
> -#ifdef CONFIG_ATSHA204A
> +#if defined(CONFIG_ATSHA204A) && defined(CONFIG_CMD_NET)
> int err;
> struct udevice *dev = get_atsha204a_dev();
> u8 mac0[4], mac1[4], mac[6];
>
Perhaps its better to select or imply ATSHA204A and CMD_NET for
turris_omnia in Kconfig instead. You can then remove all the #ifdef's
from the code here.
Just curious:
Where did you see this compile / linking problem. Compiling current
mainline does not cause any issues.
Thanks,
Stefan
^ permalink raw reply [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH] mvebu: turris_omnia: Only set eth?addr env if CONFIG_CMD_NET
2017-09-04 6:42 ` Stefan Roese
@ 2017-09-19 4:50 ` Stefan Roese
0 siblings, 0 replies; 3+ messages in thread
From: Stefan Roese @ 2017-09-19 4:50 UTC (permalink / raw)
To: u-boot
Hi Marek,
On 04.09.2017 08:42, Stefan Roese wrote:
> On 03.09.2017 15:13, Marek Behún wrote:
>> Otherwise the linking will fail since eth_env_set_enetaddr cannot
>> be found.
>>
>> Signed-off-by: Marek Behun <marek.behun@nic.cz>
>> ---
>> board/CZ.NIC/turris_omnia/turris_omnia.c | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/board/CZ.NIC/turris_omnia/turris_omnia.c
>> b/board/CZ.NIC/turris_omnia/turris_omnia.c
>> index af66837909..7db3b8ff3b 100644
>> --- a/board/CZ.NIC/turris_omnia/turris_omnia.c
>> +++ b/board/CZ.NIC/turris_omnia/turris_omnia.c
>> @@ -470,6 +470,7 @@ out:
>> return 0;
>> }
>> +#if defined(CONFIG_ATSHA204A) && defined(CONFIG_CMD_NET)
>> static void increment_mac(u8 *mac)
>> {
>> int i;
>> @@ -480,10 +481,11 @@ static void increment_mac(u8 *mac)
>> break;
>> }
>> }
>> +#endif
>> int misc_init_r(void)
>> {
>> -#ifdef CONFIG_ATSHA204A
>> +#if defined(CONFIG_ATSHA204A) && defined(CONFIG_CMD_NET)
>> int err;
>> struct udevice *dev = get_atsha204a_dev();
>> u8 mac0[4], mac1[4], mac[6];
>>
>
> Perhaps its better to select or imply ATSHA204A and CMD_NET for
> turris_omnia in Kconfig instead. You can then remove all the #ifdef's
> from the code here.
>
> Just curious:
> Where did you see this compile / linking problem. Compiling current
> mainline does not cause any issues.
I didn't receive any answers on these questions. Could you please
check again?
Thanks,
Stefan
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-09-19 4:50 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-09-03 13:13 [U-Boot] [PATCH] mvebu: turris_omnia: Only set eth?addr env if CONFIG_CMD_NET Marek Behún
2017-09-04 6:42 ` Stefan Roese
2017-09-19 4:50 ` Stefan Roese
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.