linux.git/drivers/net/sky2.c, branch v2.6.32

sky2: set carrier off in probe

2009-10-30T19:25:16+00:00

Before bringing up a sky2 interface up ethtool reports
"Link detected: yes". Do as ixgbe does and netif_carrier_off() on
probe().

Signed-off-by: Brandon Philips 
Signed-off-by: David S. Miller

sky2: irqname based on pci address

2009-10-01T22:14:55+00:00

This is based on Michal Schmidt fix for skge.

Most network drivers request their IRQ when the interface is activated.
sky2 does it in ->probe() instead, because it can work with two-port
cards where the two net_devices use the same IRQ. This works fine most
of the time, except in some situations when the interface gets renamed.
Consider this example:

1. modprobe sky2
   The card is detected as eth0 and requests IRQ 17. Directory
   /proc/irq/17/eth0 is created.
2. There is an udev rule which says this interface should be called
   eth1, so udev renames eth0 -> eth1.
3. modprobe 8139too
   The Realtek card is detected as eth0. It will be using IRQ 17 too.
4. ip link set eth0 up
   Now 8139too requests IRQ 17.

The result is:
WARNING: at fs/proc/generic.c:590 proc_register ...
proc_dir_entry '17/eth0' already registered

The fix is for sky2 to name the irq based on the pci device, as is done
by some other devices DRM, infiniband, ...  ie. sky2@pci:0000:00:00

Signed-off-by: Stephen Hemminger 
Reviewed-by: Michal Schmidt 
Signed-off-by: David S. Miller

Merge branch 'master' of /home/davem/src/GIT/linux-2.6/

2009-09-24T22:13:11+00:00

Conflicts:
	drivers/staging/Kconfig
	drivers/staging/Makefile
	drivers/staging/cpc-usb/TODO
	drivers/staging/cpc-usb/cpc-usb_drv.c
	drivers/staging/cpc-usb/cpc.h
	drivers/staging/cpc-usb/cpc_int.h
	drivers/staging/cpc-usb/cpcusb.h

sky2: Set SKY2_HW_RAM_BUFFER in sky2_init

2009-09-22T21:00:11+00:00

The SKY2_HW_RAM_BUFFER bit in hw->flags was checked in sky2_mac_init(),
 before being set later in sky2_up().

Setting SKY2_HW_RAM_BUFFER in sky2_init() where other hw->flags are set
 should avoid this problem recurring.

Signed-off-by: Mike McCormack 
Acked-by: Stephen Hemminger 
Signed-off-by: David S. Miller

trivial: remove unnecessary semicolons

2009-09-21T13:14:58+00:00

Signed-off-by: Joe Perches 
Signed-off-by: Jiri Kosina

sky2: Make sure both ports initialize correctly

2009-09-15T09:50:04+00:00

Sorry Mike, I sent you off the wrong way. The following is simpler and the
second port is diffrent enough in setup (because of NAPI), that the
following is simpler.

Signed-off-by: Stephen Hemminger 
Signed-off-by: David S. Miller

sky2: transmit ring accounting

2009-09-15T09:49:57+00:00

Be more accurate about number of transmit list elements required.

Signed-off-by: Stephen Hemminger 
Signed-off-by: David S. Miller

netdev: Remove redundant checks for CAP_NET_ADMIN in MDIO implementations

2009-09-04T03:02:11+00:00

dev_ioctl() already checks capable(CAP_NET_ADMIN) before calling the
driver's implementation of MDIO ioctls.

Signed-off-by: Ben Hutchings 
Signed-off-by: David S. Miller

sky2: only enable Vaux if capable of wakeup

2009-09-04T03:02:05+00:00

While perusing vendor driver, I saw that it did not enable the Vaux
power unless device was able to wake from lan for D3cold.
This might help for Rene's power issue.

Signed-off-by: Stephen Hemminger 
Signed-off-by: David S. Miller

sky2: Use 32bit read to read Y2_VAUX_AVAIL

2009-09-02T00:40:44+00:00

B0_CTST is a 24bit register according to the vendor driver (sk98lin).
A 16bit read on B0_CTST will always return 0 for Y2_VAUX_AVAIL (1<<16),
 so use a 32bit read when testing Y2_VAUX_AVAIL

Signed-off-by: Mike McCormack 
Acked-by: Stephen Hemminger 
Signed-off-by: David S. Miller