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

dm: net: Fix DM for targets which use MANUAL_RELOC

2015-12-18T12:15:58Z

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:52Z

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:51Z

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:10Z

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:15Z

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:45Z

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:03Z

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

dm: eth: Do not print misleading "Net Initialization Skipped"

2015-09-09T13:48:03Z

With driver model, board_eth_init() or cpu_eth_init() is not a must. Thus we don't need print a misleading "Net Initialization Skipped". Signed-off-by: Bin Meng Acked-by: Joe Hershberger

net: Allow drivers to return -ENOSYS with the write_hwaddr() method

2015-07-21T23:39:41Z

Some drivers may want to implement this method for some of their devices but not for others. So it is not possible to just leave the operation out of the table. Drivers could get around this by masquerading as two separate drivers but that seems unpleasant. Allow the driver to return an error when it does not want to process the write_hwaddr() method. Signed-off-by: Simon Glass

dm: eth: Avoid blocking on packet reception

2015-07-21T23:39:40Z

Some devices can take a long time to work out whether they have a new packet or now. For example the ASIX USB Ethernet dongle can take 5 seconds to do this, since it waits until it gets a new packet on the wire before allowing the USB bulk read packet to be submitted. At present with driver mode the Ethernet receive code reads 32 packets. This can take a very long time if we must wait for all 32 packets. The old code (before driver model) worked by reading a single set of packets from the USB device, then processing all the packets with in. It would be nice to use the same behaviour with driver model. Add a flag to the receive method which indicates that the driver should try to find a packet if available, by consulting the hardware. When the flag is not set, it should just return any packet data it has already received. If there is none, it should return -EAGAIN so that the loop will terminate. Signed-off-by: Simon Glass