- 03 Sep, 2008 40 commits
-
-
Ben Hutchings authored
When !port_enabled, defer the write to reconfigure_mac_wrapper. Whilst here, simplify the logic now that efx_start_port() always calls efx_reconfigure_port(). From: Steve Hodgson <shodgson@solarflare.com> Signed-off-by:
Ben Hutchings <bhutchings@solarflare.com> Signed-off-by:
Jeff Garzik <jgarzik@redhat.com>
-
Ben Hutchings authored
This was originally a kludge to fix broken locking, which has since been fixed properly. Signed-off-by:
-
Ben Hutchings authored
Include PMA/PMD in loopback self-tests as intended. Add NVRAM checksum validation and include it in self-tests. Add register self-tests. Run PHY self-tests where available. Signed-off-by:
-
Ben Hutchings authored
This will support register self-tests. From: Steve Hodgson <shodgson@solarflare.com> Signed-off-by:
-
Ben Hutchings authored
Move more code from efx_reset() into efx_reset_down() and efx_reset_up(). Stop propagating MAC/PHY setting failures from efx_reset_down() and efx_reset_up() as these should not be fatal. Signed-off-by:
-
Ben Hutchings authored
Some functions return an error code which is always 0. Change their return types to void and simplify their callers accordingly. Signed-off-by:
-
Ben Hutchings authored
Signed-off-by:
-
Ben Hutchings authored
Update comment on the dummy operation implementations. Line up the board operation initialisers. Signed-off-by:
-
Ben Hutchings authored
Signed-off-by:
-
Steve Hodgson authored
Signed-off-by:
-
Ben Hutchings authored
This allows updating PHY firmware for one interface without removing all other interfaces handled by the driver. Replace tx_disabled flags and 10Xpress status enumeration with flags in enum efx_phy_mode. Prevent an interface from being brought up while in PHY flash mode. Signed-off-by:
-
Ben Hutchings authored
Signed-off-by:
-
Ben Hutchings authored
Make efx_process_channel() and falcon_process_eventq() return the number of packets received rather than updating the quota, consistent with new NAPI. Since channels and RX queues are mapped one-to-one, remove return value from falcon_handle_rx_event() and add a warning for events with the wrong RX queue number. Signed-off-by:
-
Ben Hutchings authored
It is redundant with efx_channel::channel. Signed-off-by:
-
Ben Hutchings authored
Currently each channel can have at most one RX queue, so go straight to that one. Signed-off-by:
-
Ben Hutchings authored
We have long since given up doing RSS on Falcon A1 and therefore we would always write the default value of 0. Signed-off-by:
-
Ben Hutchings authored
Rename efx_nic::rss_queues to the more obvious n_rx_queues Remove efx_rx_queue::used and other stuff that's redundant with it. Signed-off-by:
-
Ben Hutchings authored
efx_channel::has_interrupt is redundant with efx_channel::used_flags. Remove efx_test_eventq() because it is now obviously unreachable. Signed-off-by:
-
Ben Hutchings authored
Signed-off-by:
-
Ben Hutchings authored
Extend the SPI device setup code to support this. Signed-off-by:
-
Ben Hutchings authored
gcc will automatically inline static functions with only one caller, and may inline other functions depending on the kernel configuration and size of the intermediate code. Signed-off-by:
-
Ben Hutchings authored
Squashed nested structures. Renamed remaining_len to out_len, ifc.len to in_len, header_length to header_len. Moved ipv4_id into the group of output variables where it belongs. Signed-off-by:
-
Ben Hutchings authored
Signed-off-by:
-
Ben Hutchings authored
Replace (cond ? 1 : 0) with cond or !!cond as appropriate, and (cond ? 0 : 1) with !cond. Remove some redundant boolean temporaries. Rename one field that looks like a flag but isn't. Signed-off-by:
-
Ben Hutchings authored
Remove unmap_addr since it can be calculated from dma_addr, len and unmap_len. This saves 4-16 bytes. Signed-off-by:
-
Ben Hutchings authored
Signed-off-by:
-
Ben Hutchings authored
Signed-off-by:
-
Ben Hutchings authored
Add efx_poll_loopback() function to test for successful completion of test. Change efx_test_loopback() to end the test after 1 ms if efx_poll_loopback() indicates success, and otherwise to wait for 100 ms as before. While we're here, rename efx_{rx,tx}_loopback() to efx_{begin,end}_loopback() which more accurately reflect their purpose. Signed-off-by: -
Ben Hutchings authored
Removed log messages that are redundant with calling functions. Fixed bitwise or-ing of return codes. Signed-off-by:
-
Ben Hutchings authored
This was making noise during 10Xpress self-test. Signed-off-by:
-
Ben Hutchings authored
These were clearly bogus. From: Steve Hodgson <shodgson@solarflare.com> Signed-off-by:
-
Ben Hutchings authored
efx.c does not use hardware workarounds. Signed-off-by:
-
Ben Hutchings authored
Exclude assumed size of RX control frames from rx_bad_bytes. Exclude assumed size of TX control frames from tx_good_bytes for consistency with rx_good_bytes. Signed-off-by:
-
Ben Hutchings authored
We believed that some valid SNAP frames were being marked as invalid. In fact this is not the case and no workaround is needed. Signed-off-by:
-
Ben Hutchings authored
Currently the board initialisation includes 2 delays of 1 second each. Usually it is unnecessary to wait that long, so check before doing so. Correct some of the comments and log messages while we're here. Signed-off-by:
-
Ben Hutchings authored
efx_test_loopback() used "|" to combine the results of the RX and TX phases. If both phases fail with different error codes, this results in a bogus error code. Signed-off-by:
-
Ben Hutchings authored
Checksum generation is an attribute of our hardware TX queues, not TX descriptors. We previously used a single queue and turned checksum generation on or off as requested through ethtool. However, this can result in regenerating checksums in raw packets that should not be modified. We now create 2 hardware TX queues with checksum generation on or off. They are presented to the net core as one queue since it does not know how to select between them. The self-test verifies that a bad checksum is unaltered on the queue with checksum generation off. Signed-off-by:
-
Steve Hodgson authored
Signed-off-by:
-
Ben Hutchings authored
This means the compiler doesn't need to use real division instructions. Signed-off-by:
-
Ben Hutchings authored
Use of the net_device::priv field is deprecated. Signed-off-by:
-
