u-boot.git/net/eth.c, branch v2016.01

net: remove scary warning about EEPROM provided MAC address

2016-01-08T15:15:44+00:00

In many parts of the computing world having a unique MAC address
sitting in some on-NIC storage is considered the normal case.
Remove the warning to not scare the user unnecessarily.
This applies to Highbank/Midway and ARM's Juno, for instance.
Besides that this fixes the formatting on Midway, for instance,
which currently looks like:
...
Net:   xgmac0
Warning: xgmac0 using MAC address from net device
, xgmac1
Warning: xgmac1 using MAC address from net device
...

Signed-off-by: Andre Przywara

dm: eth: Stick to 'ethact' when 'ethrotate' is 'no' in eth_init()

2016-01-07T17:27:07+00:00

When 'ethrotate' variable is set to 'no' and 'ethact' variable
is already set to an ethernet device, we should stick to 'ethact'.

Signed-off-by: Bin Meng 
Acked-by: Joe Hershberger 
Acked-by: Simon Glass 
Tested-by: Simon Glass

dm: eth: Test 'ethrotate' before changing current ethernet device

2016-01-07T17:27:07+00:00

In eth_current_changed(), the call to eth_get_dev() below has a side
effect of rotating ethernet device if uc_priv->current == NULL. This
is not what we want when 'ethrotate' variable is 'no'.

Signed-off-by: Bin Meng 
Acked-by: Joe Hershberger 
Acked-by: Simon Glass 
Tested-by: Simon Glass

dm: net: Fix DM for targets which use MANUAL_RELOC

2015-12-18T12:15:58+00:00

All ethernet operation needs to be updated for architectures which
requires MANUAL_RELOC.

Signed-off-by: Michal Simek 
Acked-by: Joe Hershberger

net: eth: Check return value in various places

2015-10-29T19:05:52+00:00

eth_get_dev() can return NULL which means device_probe() fails for
that ethernet device. Add return value check in various places or
U-Boot will crash due to NULL pointer access.

With this commit, 'dm_test_eth_act' test case passes.

Signed-off-by: Bin Meng 
Acked-by: Joe Hershberger

net: eth: Clear MAC address in eth_pre_remove()

2015-10-29T19:05:51+00:00

platdata->enetaddr was assigned to a value in dev_probe() last time.
If we don't clear it, for dev_probe() at the second time, dm eth
will end up treating it as a MAC address from ROM no matter where it
came from originally (maybe env, ROM, or even random). Fix this by
clearing platdata->enetaddr when removing an Ethernet device.

Signed-off-by: Bin Meng 
Acked-by: Joe Hershberger

net: change the env name to use const

2015-10-28T20:01:10+00:00

As we don't modify the 'name' parameter, so change it to const.

Signed-off-by: Josh Wu 
Reviewed-by: Simon Glass 
Acked-by: Joe Hershberger

net/eth: fix a bug in on_ethaddr()

2015-10-28T20:00:15+00:00

The loop should check all ethenet devices, not only the first device,
to set each specified ethaddr, or it'll cause failure when we use other
devices.

Signed-off-by: Gong Qianyu 
Acked-by: Joe Hershberger

net: expose eth_is_active() function to test network device state

2015-09-30T02:54:45+00:00

The previous eth_device struct returned by eth_get_dev() allowed
code to directly query the state member field. However, with
CONFIG_DM_ETH this data gets encapsulated (i.e. private), and
eth_get_dev() returns a udevice struct 'abstraction' instead.

This breaks legacy code relying on the former behaviour - e.g.
netconsole.
(see http://lists.denx.de/pipermail/u-boot/2015-June/216528.html)

The patch introduces a method to retrieve the ethernet device
state in a 'clean' and uniform way, supporting both legacy code
and driver model. The new function eth_is_active() accepts a
device struct pointer and tests it for ETH_STATE_ACTIVE.

Signed-off-by: Bernhard Nortmann 
Reviewed-by: Simon Glass 
Acked-by: Joe Hershberger

dm: eth: Correctly detect alias in eth_get_dev_by_name()

2015-09-09T13:48:03+00:00

When given a device name string, we should test to see if it is
really an alias like "eth#".

Signed-off-by: Bin Meng 
Acked-by: Joe Hershberger