summaryrefslogtreecommitdiffstats
path: root/drivers/staging/et131x/et1310_rx.c
Commit message (Collapse)AuthorAgeFilesLines
* staging: et131x: Put all .c files into one big fileMark Einon2011-10-231-1160/+0
| | | | | | | | Created one big .c file for the driver, moving the contents of all driver .c files into it. Signed-off-by: Mark Einon <mark.einon@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: et131x: Remove file et131x_version.hMark Einon2011-10-111-1/+0
| | | | | | | | Removed defines in et131x_version.h and replaced them by actual strings where convinient, or moved them to et131x.h Signed-off-by: Mark Einon <mark.einon@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: et131x: Updating copyright statements and module authorsMark Einon2011-10-041-0/+2
| | | | | | | Adding copyright notices and adding myself as a module author. Signed-off-by: Mark Einon <mark.einon@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: et131x: Remove private adapter->linkspeed and use phydev->speed insteadMark Einon2011-09-061-2/+7
| | | | | | | | | | | The phy device keeps a note of the link speed, so use that value instead of the driver private one. Also use the phydev defines for link speeds, and remove the driver private ones. adapter->hw_errs was never used, even in the vendor driver, so remove that too. Signed-off-by: Mark Einon <mark.einon@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: et131x: Remove redundant replica loopback codeMark Einon2011-08-291-11/+0
| | | | | | | A mechanism used to set the phy in loopback mode is not present in the driver, making associated checks and variables redundant. Removing them. Signed-off-by: Mark Einon <mark.einon@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: et131x: Rename var name 'etdev' to 'adapter' throughout moduleMark Einon2011-08-231-87/+87
| | | | | | | | | | | | Used regex (from vim) :%s/\<etdev\>/adapter/g Changed because: * 'etdev' is easily confused as a misspelling of 'netdev' * 'adapter' is more widely used for this struct type in net drivers. This change made comparing code with other drivers much easier. Signed-off-by: Mark Einon <mark.einon@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: et131x: Converting et1310_adapter.h variable names from CamelCaseMark Einon2011-08-231-22/+32
| | | | | | | Tested on an ET-131x device. Signed-off-by: Mark Einon <mark.einon@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: et131x: Converting et1310_rx.h variable names from CamelCaseMark Einon2011-08-231-165/+173
| | | | | | | Tested on an ET-131x device. Signed-off-by: Mark Einon <mark.einon@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: et131x: Make static some local functions in et1310_rx.cMark Einon2011-08-231-2/+2
| | | | | | | Tested on an ET-131x device. Signed-off-by: Mark Einon <mark.einon@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: et131x: Converting et1310_rx.c function and local names from CamelCaseMark Einon2011-08-231-98/+93
| | | | | | | | Also renamed some items to improve readability, and other minor tidy-ups. Tested on an ET-131x device. Signed-off-by: Mark Einon <mark.einon@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: et131x: rename adapter->Flags to adapter->flagsMark Einon2011-07-081-3/+3
| | | | | | | | Trivial rename of the adapter flags struct member to remove camel case. Tested on a ET-131x device. Signed-off-by: Mark Einon <mark.einon@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: et131x: Remove some unecessary packet accounting codeMark Einon2011-07-051-1/+1
| | | | | | | | The ce_stats ipackets and opackets members are only used to update the net_device_stats->[r,t]x_packets counters, so get rid of them. Signed-off-by: Mark Einon <mark.einon@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: et131x: Fix stats->rx_packets accountingMark Einon2011-07-051-3/+2
| | | | | | | | | | | | | The net_device_stats->rx_packets counter is not getting updated. This is due to checking a fMP_ADAPTER_LINK_DETECTION flag prior to updating which is never set. Solved by using netif_carrier_ok() to test for a link, and removing the fMP_ADAPTER_LINK_DETECTION flag, which looks to be a broken mechanism. Signed-off-by: Mark Einon <mark.einon@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: et131x: Removing '_t' from ce_stats_t structMark Einon2011-06-281-5/+5
| | | | | | | | Removing '_t' from ce_stats_t struct name and renaming et131x_adapter ce_stats member from 'Stats' to 'stats' Signed-off-by: Mark Einon <mark.einon@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: et131x: et1310_rx.c checkpatch fixesMark Einon2011-06-281-97/+96
| | | | | Signed-off-by: Mark Einon <mark.einon@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Fix common misspellingsLucas De Marchi2011-03-311-1/+1
| | | | | | Fixes generated by 'codespell' and manually reviewed. Signed-off-by: Lucas De Marchi <lucas.demarchi@profusion.mobi>
* staging/et131x: fix et131x_rx_dma_disable halt_status usageMarcin Slusarz2011-02-231-2/+2
| | | | | | | | | | | | | | | | | | Commit 1bd751c1abc1029e8a0ae63ef4f19357c735a2a3 ("Staging: et131x: Clean up rxdma_csr") changed csr from bitfield to u32, but failed to convert 2 uses of halt_status bit. It did: - if (csr.bits.halt_status != 1) + if ((csr & 0x00020000) != 1) which is wrong, because second version is always true. Fix it. This bug was found by coccinelle (http://coccinelle.lip6.fr/). Signed-off-by: Marcin Slusarz <marcin.slusarz@gmail.com> Acked-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: et131x: Clean up the RFD struct/typesAlan Cox2011-02-041-22/+22
| | | | | Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* staging: et131x: Turn a few more LongCapitalisedThings into Linuxish namesAlan Cox2011-02-041-9/+8
| | | | | Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: Drop memory allocation castJulia Lawall2010-05-111-1/+1
| | | | | | | | | | | | | | | | | | | | Drop cast on the result of kmalloc and similar functions. The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ type T; @@ - (T *) (\(kmalloc\|kzalloc\|kcalloc\|kmem_cache_alloc\|kmem_cache_zalloc\| kmem_cache_alloc_node\|kmalloc_node\|kzalloc_node\)(...)) // </smpl> Signed-off-by: Julia Lawall <julia@diku.dk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: fix most coding style issues in et131xMichael Sprecher2010-05-111-27/+26
| | | | | | | | This is a patch to the et131x driver that fixes up almost all coding style issues Signed-off-by: Michael Sprecher <sprecher.m@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: Collapse all the function definitions into one placeAlan Cox2010-03-031-6/+1
| | | | | | | | We have lots of tiny files right now that could be one Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: Kill off RX_RING_tAlan Cox2010-03-031-27/+27
| | | | | | | | | This completes the typedef clean up of the rx specific structures, although there is plenty do on field names and the like Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: clean up the pkt_desc_stat_t typesAlan Cox2010-03-031-25/+24
| | | | | | Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: Clean up the fbr lookup tableAlan Cox2010-03-031-33/+32
| | | | | | | | | All the subtypes are sane so just turn it into something struct and linux like Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: Fix rx_status typingAlan Cox2010-03-031-11/+10
| | | | | | | | | Use the proper pointer types for the higher level pointers to the rx_status object and kill casts Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: rx_status_block_t is now cleanAlan Cox2010-03-031-6/+6
| | | | | | | | So kill off the top level type and turn it into a struct Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: Clean up the rx status word typesAlan Cox2010-03-031-6/+4
| | | | | | Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: Kill off the rxdma typeAlan Cox2010-03-031-2/+2
| | | | | | | | | We have a clean struct of this now so turn the top level typedefs into a struct Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: Clean up rxdma_csrAlan Cox2010-03-031-24/+22
| | | | | | | | This is another set of flags as typedef that can be cleaned up. Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: fbr_desc is now only sane typesAlan Cox2010-03-031-18/+17
| | | | | | | | Clean up the typdef for fbr_desc itself so we know it is done Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: kill of FBR_WORD2Alan Cox2010-03-031-4/+4
| | | | | | | | | This is only used meaningfully as a definition, we never mask and fetch the bits apart Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: Fix 2.6.33rc1 regression in et131xAlan Cox2010-01-201-3/+3
| | | | | | | | | | | | | | et131x: Fix 12bit wrapping From: Alan Cox <alan@linux.intel.com> The 12bit wrap logic conversion is wrong and this shows up for some memory sizes and layouts of card. Patch it up for now, once the kernel view of status is cleaned up it'll become two variables and a lot saner. Signed-off-by: Alan Cox <alan@linux.intel.com> Cc: stable <stable@kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: clean up the avail fields in the rx registersAlan Cox2009-12-111-13/+15
| | | | | | | | | These have a wrap bit but again need little work to clean out. There are a couple of uglies left that want addressing in later clean up. Notably we should probably keep the local psr copy and wrap as two values. Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: Clean up number fieldsAlan Cox2009-12-111-11/+11
| | | | | | | | Lots of RX typedefs are just low bits of a u32, so clean them all up in one go and just work them directly. Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: kill RX_DMA_MAX_PKT_TIMEAlan Cox2009-12-111-2/+2
| | | | | | | Another one bits the dust ... Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: Kill the RX pending listAlan Cox2009-12-111-22/+0
| | | | | | | | As with tx there was a pending list Linux doesn't use Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: fold up simple wrapper functionsAlan Cox2009-12-111-34/+5
| | | | | | Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: Clean up the receive arraysAlan Cox2009-12-111-18/+1
| | | | | | | | | We don't use them for anything having stripped out the debug gunge in the original driver. Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: first pass RX cleanupAlan Cox2009-12-111-209/+205
| | | | | | | | Sort out the variable naming and clean up types and obvious trivia Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: pci_alloc_consistent DMA alignment is guaranteedAlan Cox2009-12-111-38/+12
| | | | | | | | | So we can remove this alignment work. Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: PHY loopback cannot be set (and isn't useful for us anyway)Alan Cox2009-12-111-28/+23
| | | | | | | | | Remove the stuff that falls out from this always being zero. Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: Correct WRAP bit handlingAlan Cox2009-10-141-6/+14
| | | | | | | | add_10bit loses the existing wrap value Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: prune all the debug codeAlan Cox2009-09-151-116/+22
| | | | | | | | We don't need it, we have a perfectly good set of debug tools. For this pass keep a few debug printks around which are "should not happen" items Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: clean up MP_FLAG macrosAlan Cox2009-09-151-6/+6
| | | | | Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: clean up DMA10/DMA4 typesAlan Cox2009-09-151-38/+22
| | | | | Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: de-hungarianise a bitAlan Cox2009-09-151-68/+68
| | | | | | | bOverrideAddress is write only so kill it rather than fix it Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: clean up constant rx/tx registry fieldsAlan Cox2009-09-151-3/+3
| | | | | Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging: et131x: CSRAddress to regsAlan Cox2009-09-151-13/+13
| | | | | | | Switch this to a Linux like naming as it occurs all over. Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
* Staging et131x: kill refcountAlan Cox2009-09-151-7/+0
| | | | | | | | The RefCount field is accessed only by a macro and the only use of it in the tree is to read it, so it can go Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
OpenPOWER on IntegriCloud