summaryrefslogtreecommitdiff
path: root/drivers/net/ethernet/freescale/dpa
AgeCommit message (Collapse)Author
2013-08-30Merge branch 'b4860rev2' into sdk-kernel-3.8J. German Rivera
2014-04-27dpaa_eth/mac: Sync with the unique MDIO bus name for "fixed-link" PHY(s)Shruti Kanetkar
'9e6c643 phy/fixed: use an unique MDIO bus name' changed the name of the fixed PHY bus to "fixed-0". This update has been already done for the non-dpaa SoC(s) in 'e5c7d1f of/mdio: fix fixed link bus name' Signed-off-by: Shruti Kanetkar <Shruti@Freescale.com> Change-Id: Ie2ead771bc74a2b62c26eca6b5618bdebbe1eb61 Reviewed-on: http://git.am.freescale.net:8181/4129 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Hamciuc Bogdan-BHAMCIU1 <bogdan.hamciuc@freescale.com> Reviewed-by: Rivera Jose-B46482 <German.Rivera@freescale.com>
2013-07-31dpaa_eth: replace pointer to default bpoolMadalin Bucur
Replace pointer to default bpool by bpid. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I10248ab7f01807c4e0b13f59c32ef9c4314b9635 Reviewed-on: http://git.am.freescale.net:8181/3661 Reviewed-by: Hamciuc Bogdan-BHAMCIU1 <bogdan.hamciuc@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-31dpaa_eth: reuse dpa_bp_alloc() for private interfacesMadalin Bucur
Make use of the common dpa_bp_alloc() function in the private interface code. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I1645bd36c37b807aa9c2da462579bd2907df7c8e Reviewed-on: http://git.am.freescale.net:8181/3660 Reviewed-by: Hamciuc Bogdan-BHAMCIU1 <bogdan.hamciuc@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-31dpaa_eth: private bpool seeding code refactoringMadalin Bucur
Moved globals used only by non-sg code to the apropriate file, renamed seeding functions. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: If9a4945be3e7a1d6ce3c752ba3c2f581651b5dec Reviewed-on: http://git.am.freescale.net:8181/3659 Reviewed-by: Hamciuc Bogdan-BHAMCIU1 <bogdan.hamciuc@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-31dpaa_eth: minor change in dpaa_eth_napi_schedule()Madalin Bucur
Moved counter increment in inline function. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: Iee1981c67f0b3daed65276cd3a3addf74c91e264 Reviewed-on: http://git.am.freescale.net:8181/3658 Reviewed-by: Hamciuc Bogdan-BHAMCIU1 <bogdan.hamciuc@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-31dpaa_eth: add drain and seed callbacks for dpa_bpMadalin Bucur
Add callbacks for seeding and draining the bpools. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: Ib6e2e0c6d6b8ea9187c0d90b5ab4af4c54f344a4 Reviewed-on: http://git.am.freescale.net:8181/3657 Reviewed-by: Hamciuc Bogdan-BHAMCIU1 <bogdan.hamciuc@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-31dpaa_eth: Move bpool seeding back to probe for SG driverIoana Radulescu
Only the non-SG ("optimized for forwarding") flavour of the driver needs to have the buffer pool seeding done after all interfaces are probed, as in this case the buffer size may vary between interfaces. For the termination optimized flavour, we have a fixed buffer size on Rx, so there's no need to wait for the pool seeding. Signed-off-by: Ioana Radulescu <ruxandra.radulescu@freescale.com> Change-Id: I101ffa38e9f83f90a432620898a2a24ede6c0895 Reviewed-on: http://git.am.freescale.net:8181/3656 Reviewed-by: Hamciuc Bogdan-BHAMCIU1 <bogdan.hamciuc@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-31dpaa_eth: Remove dpa_bp_count from percpu_priv structureIoana Radulescu
Instead just rely on the percpu field in dpa_bp structure. The performance lost due to the extra indirection is compensated by the smaller size of the percpu_priv structure. This way we begin to eliminate the assumption that private interfaces only have one default buffer pool, which may not remain true for long. Signed-off-by: Ioana Radulescu <ruxandra.radulescu@freescale.com> Change-Id: I0b2c199c3f8883fa7d5fa1a06d2940f8170fc959 Reviewed-on: http://git.am.freescale.net:8181/3655 Reviewed-by: Bucur Madalin-Cristian-B32716 <madalin.bucur@freescale.com> Reviewed-by: Hamciuc Bogdan-BHAMCIU1 <bogdan.hamciuc@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-31dpaa_eth: Fix size of DMA mappingsIoana Radulescu
Use the correct buffer size for DMA mappings on Tx path, otherwise we risk mapping too little if we have an MTU larger than the size of the default buffer pool. Signed-off-by: Ioana Radulescu <ruxandra.radulescu@freescale.com> Change-Id: Id197e3284a884fd417c62e13af8cb9688a2f1b8f Reviewed-on: http://git.am.freescale.net:8181/3654 Reviewed-by: Bucur Madalin-Cristian-B32716 <madalin.bucur@freescale.com> Reviewed-by: Hamciuc Bogdan-BHAMCIU1 <bogdan.hamciuc@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-30dpaa_eth: remove unused bpool depletion callbackMadalin Bucur
The stub bpool depletion callback is not used, removed. Change-Id: Ibf465d4c04bc867a7cda8a92c7cf07b3ae682de0 Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: Ie864646adbe77817b59fc9cf8223677a4048be2a Reviewed-on: http://git.am.freescale.net:8181/3600 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-30dpaa_eth: remove references to shared in the codeMadalin Bucur
There were several portions of code that were run conditionally based on the dpa_priv_s member variable shared. Removed the checks and the variable. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I80d29ca50cc0fa328e149ba43c1c5bcf0157d6c2 Reviewed-on: http://git.am.freescale.net:8181/3561 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-30dpaa_eth: bpool probing code separationMadalin Bucur
First changes in the series that will separate bpool initialization for private and other drivers. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I96ff087f37c0dd7117d9aabc3cf3554bf2c35684 Reviewed-on: http://git.am.freescale.net:8181/3560 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-30dpaa_eth: Fix freeing of bpool buffersIoana Radulescu
Make sure buffers get freed correctly even if they are part of compound pages. Signed-off-by: Ioana Radulescu <ruxandra.radulescu@freescale.com> Change-Id: I36aaa1d13b526fa34c7c6ac7d3a1d787091b50f2 Reviewed-on: http://git.am.freescale.net:8181/3481 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Hamciuc Bogdan-BHAMCIU1 <bogdan.hamciuc@freescale.com> Reviewed-by: Bucur Madalin-Cristian-B32716 <madalin.bucur@freescale.com> Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-30dpaa_eth: Fix computation of FD headroomIoana Radulescu
The Rx/Tx headrooms are aligned to a configurable value; if that value is zero however, FMD will interpret it as a "don't care" and use its own internal default, which is currently 64. If the user configures a zero headroom alignment, use our own default, otherwise we risk getting out of sync with the actual hardware configuration. Also, don't assume the Rx headroom is always equal to the Tx headroom plus the Kconfigurable Rx extra headroom, since alignment may change this. Signed-off-by: Ioana Radulescu <ruxandra.radulescu@freescale.com> Change-Id: Iaf12deb55f4330edb46c6d358114f8f757b64f08 Reviewed-on: http://git.am.freescale.net:8181/3479 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Hamciuc Bogdan-BHAMCIU1 <bogdan.hamciuc@freescale.com> Reviewed-by: Bucur Madalin-Cristian-B32716 <madalin.bucur@freescale.com> Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-30dpaa_eth: Recycle the buffers of linear skbsIoana Radulescu
Add recycling support to the "optimized for termination" flavour of the driver. For now, only the recycling of buffers pertaining to linear skbs is supported. The rules for recycling are similar to those used in the forwarding code; unlike that codebase however, only the page fragment is recycled, not the entire skb structure. Signed-off-by: Ioana Radulescu <ruxandra.radulescu@freescale.com> Change-Id: I6d3014e90119436d121f20d93a58a7d3054c13fa Reviewed-on: http://git.am.freescale.net:8181/3478 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Hamciuc Bogdan-BHAMCIU1 <bogdan.hamciuc@freescale.com> Reviewed-by: Bucur Madalin-Cristian-B32716 <madalin.bucur@freescale.com> Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-30dpaa_eth: Rename dpa_bp_add_8_pages()Bogdan Hamciuc
After we no longer allocate full page-size buffers, the main allocation function needs a rename. Ditto for some internal variable and one comment. Signed-off-by: Bogdan Hamciuc <bogdan.hamciuc@freescale.com> Change-Id: Id3e6888ad6554b696f3b59d39e26562588713d0b Reviewed-on: http://git.am.freescale.net:8181/3477 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Bucur Madalin-Cristian-B32716 <madalin.bucur@freescale.com> Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-30dpaa_eth: Half-page buffer managementBogdan Hamciuc
QMan automatically uses the FD to determine the frame size for congestion management purposes. But since we only allocated full-size pages, traffic consisting of small frames would pose a significant pressure on system memory, because the difference between the frame size and the buffer (page) size couldn't be accounted for. Consequently, the CGRs we set up for egress traffic would not detect congestion; in some cases, this quickly depletes system memory. Add half-page buffer support, for better memory accounting. Even if we now have to manually enforce buffer alignments which up until now were implicit, and even though the size overheads due to cacheline alignments are still non-negligible, the overall buffer accounting is significantly improved now. OOM situations, in particular, should be harder to produce. Signed-off-by: Bogdan Hamciuc <bogdan.hamciuc@freescale.com> Change-Id: I861ec2752124f92e58d278732ccd359703776032 Reviewed-on: http://git.am.freescale.net:8181/3476 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-30dpaa_eth: Simplify computation of skb headroomBogdan Hamciuc
Make it clear that on Rx the needed headroom is precisely fd.offset. Signed-off-by: Bogdan Hamciuc <bogdan.hamciuc@freescale.com> Change-Id: I62bf538c1dcb7a8f4a4feadc403e8bddd0707ef7 Reviewed-on: http://git.am.freescale.net:8181/3475 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-22dpaa_eth: Handle buffer pool depletionIoana Radulescu
If the system gets into an out-of-memory state, we are unable to refill the default buffer pool with new buffers. While this lasts, just release any incoming frame back to the pool without further processing. Signed-off-by: Ioana Radulescu <ruxandra.radulescu@freescale.com> Signed-off-by: Bogdan Hamciuc <bogdan.hamciuc@freescale.com> Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: Ia7cd0979adfa2d5c6f5be88017ed2bd0bfd202f8 Reviewed-on: http://git.am.freescale.net:8181/3335 Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-22dpaa_eth: Fix pool counters on error pathBogdan Hamciuc
When system memory is depleted, the Rx buffers are put back into their pool. Bad accounting on that error path made it so the software software counters remained behind the hardware counters, eventually allowing for negative totals. Fix this and keep the software pool counters in sync with the hardware counters. Signed-off-by: Ioana Radulescu <ruxandra.radulescu@freescale.com> Signed-off-by: Bogdan Hamciuc <bogdan.hamciuc@freescale.com> Change-Id: I1d1de60df0bc4fcec2dc1a4aadd9b2ffbc0ba89e Reviewed-on: http://git.am.freescale.net:8181/3334 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-22dpaa_eth: Switch compile-time default optimizationBogdan Hamciuc
Move back to the "optimize for IPFWD" compile-time choice. This effectively reverts commit id 692f5140. Signed-off-by: Bogdan Hamciuc <bogdan.hamciuc@freescale.com> Change-Id: I4235488c48141cfeebf2b7c108ed81ef73ab4afb Reviewed-on: http://git.am.freescale.net:8181/3333 Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-22fmd,dpaa_eth: Selectively reenable FMan allocation algorithmBogdan Hamciuc
On P3/P4/P5 platforms, as well as on P1023, FMan's default resource allocation values are not enough to enable jumbo frame forwarding in the absence of SG support. Enable the (previously deselected) FMan dynamic resource allocation algorithm on those platforms, if the legacy DPAA-Ethernet driver (missing SG support) is used. Signed-off-by: Bogdan Hamciuc <bogdan.hamciuc@freescale.com> Change-Id: Iec8224c331b95903d57291312791ac6bf0b7fa8a Reviewed-on: http://git.am.freescale.net:8181/3332 Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-22dpaa_eth: Fix endless loop in case of memory depletionBogdan Hamciuc
Check the return value from the buffer-allocating function, in order to prevent looping forever if memory depletes. Signed-off-by: Bogdan Hamciuc <bogdan.hamciuc@freescale.com> Change-Id: If817c367a39a47322dc3e55d6d133d500b324b3b Reviewed-on: http://git.am.freescale.net:8181/3331 Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-22dpaa_eth: Set a fixed upper limit for recycleable buffer sizeIoana Radulescu
Until now we considered a buffer recycleable if its size didn't exceed the buffer pool size by more than a certain amount. Recent changes in the stack however allow the allocation of skb buffers much larger than the requested size, which impacts our ability to recycle forwarded frames, especially when jumbo frame support is enabled. Make the recycling size upper limit a fixed and sufficiently large value, regardless of the default size of the buffer pool, in order to simplify recycling logic and guard against unfavourable cases. Signed-off-by: Ioana Radulescu <ruxandra.radulescu@freescale.com> Change-Id: I2fd9b6a057a4e54445233730805d5dc9c8188e5a Reviewed-on: http://git.am.freescale.net:8181/3330 Reviewed-by: Hamciuc Bogdan-BHAMCIU1 <bogdan.hamciuc@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-22dpaa_eth: Fix computation of cache paddingIoana Radulescu
When computing the padding between the beginning of a FD buffer and the head of the skb associated to it, we wrongly assume that this padding is always lower than PAGE_SIZE. In ip forward scenarios with jumbo frames support enabled and Rx extra headroom not zero, the above assumption is proven incorrect. Properly compute the padding size in order to avoid skb corruption. Signed-off-by: Ioana Radulescu <ruxandra.radulescu@freescale.com> Change-Id: I219dfc11d81639a94fc10cbf70d5fbbe365d3b07 Reviewed-on: http://git.am.freescale.net:8181/3329 Reviewed-by: Hamciuc Bogdan-BHAMCIU1 <bogdan.hamciuc@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: Fix unit testsMarian Rotariu
The unit tests will use the default pool as the buffer pool. Because at bootup the pool is empty, buffer seeding and draining must be provided around the tests. Signed-off-by: Marian Rotariu <marian.rotariu@freescale.com> Change-Id: I3cba65a13599e031dbfe14542699e7e74ac19c1c Reviewed-on: http://git.am.freescale.net:8181/3206 Reviewed-by: Bucur Madalin-Cristian-B32716 <madalin.bucur@freescale.com> Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: move common code to a separate fileMadalin Bucur
Created dpaa_eth_common.c to hold all common code. Moved legacy non-SG related code to its own file. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I5a79559714f515e6889655a1cc7256a556043c45 Reviewed-on: http://git.am.freescale.net:8181/3096 Reviewed-by: Rotariu Marian Cristian-B44860 <marian.rotariu@freescale.com> Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: remove module loading order dependencyMadalin Bucur
The dpa_rx_extra_headroom and dpa_max_frm are set by each module init functions so there is no chance they are used before being initialized. Also use the dpa_rx_extra_headroom for SG driver. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: If521e1f41e4857b5d03963aaa9fb2fa898a2183c Reviewed-on: http://git.am.freescale.net:8181/3095 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: removed empty ndo callbacksMadalin Bucur
Empty callbacks are not required. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: Ia41366d99c455b976bb0d4376a5fdc63332c58f2 Reviewed-on: http://git.am.freescale.net:8181/3094 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: free buf_layout on error pathsMadalin Bucur
The allocated buf_layout must be freed on all error paths. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: Icf2e68f90c28adcc99fdc56492942f3595033b21 Reviewed-on: http://git.am.freescale.net:8181/3093 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: print probed interfacesMadalin Bucur
Print a message for each probed interface. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: If108fc8828bc03c44b1e539eb340eb086cd4b2c7 Reviewed-on: http://git.am.freescale.net:8181/3092 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: check dpa_mac_probe() resultMadalin Bucur
Check for NULL returns from dpa_mac_probe() when probing private and shared interfaces. If a MACless interface is mistakenly identified with the old compatibility string "fsl,dpa-ethernet" instead of the required "fsl,dpa-ethernet-macless" compatibility string the interface is probed as private interface and dpa_mac_probe() fails. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I221d00aafb6ba25f32b7a451610f1e6185bdb7ca Reviewed-on: http://git.am.freescale.net:8181/3091 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: cleanup module load bannerMadalin Bucur
Remove function and line number from module load prints. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I9d34d6066c51263a04157f9f9d6b275fce706392 Reviewed-on: http://git.am.freescale.net:8181/3090 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: remove unnecessary function parameterMadalin Bucur
Remove unnecessary parameter dpa_bp to dpa_release_sgt(). Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: Icc489e62d294588b22b381e6f76d17307843b5a6 Reviewed-on: http://git.am.freescale.net:8181/3089 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: fix issue in dpa_fq_setup()Madalin Bucur
Only use the dpa_fq_list entries of type FQ_TYPE_TX as egress FQs. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I3310c2461e09b539a223d4c676b8b8ebe1b133d3 Reviewed-on: http://git.am.freescale.net:8181/3088 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: move MACless interface code to a new fileMadalin Bucur
Separated the code related to MACless interfaces. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I85ece5c7857c3044fa47592c33dc1e9e532fb33c Reviewed-on: http://git.am.freescale.net:8181/3087 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: move proxy initialization code to a new fileMadalin Bucur
Separated the code related to proxy interface initialization. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: Iee4f7c1ee359775901c7ac960df45537495baf47 Reviewed-on: http://git.am.freescale.net:8181/3086 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: move shared interface code to a new fileMadalin Bucur
Separated the code related to shared interfaces. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I082a351a7d73a128a9761b3ad245886e00355c53 Reviewed-on: http://git.am.freescale.net:8181/3085 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: remove interface type checks from dpa_fq_setup()Madalin Bucur
Instead of checking if the interface type is shared or MACless provide FQs and tx_port as function parameters. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I4b50dca52d8ece60b017fcf7582f42e779ce5101 Reviewed-on: http://git.am.freescale.net:8181/3084 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: remove some references to shared and macless port typesMadalin Bucur
Remove reference to shared and MACless port types from functions dpa_fq_probe_mac() and dpaa_eth_sysfs_init(). Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I3d2ab0f9d3ba01e0b39df43d9dc2cb036c8a7133 Reviewed-on: http://git.am.freescale.net:8181/3083 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: Moved DPA Ethernet unit test into separate file.Marian Rotariu
All unit tests should be created in this file from now on. Signed-off-by: Marian Rotariu <marian.rotariu@freescale.com> Change-Id: I53ef0032e3cc9b91c156f0e7776fd0dfb481fb52 Reviewed-on: http://git.am.freescale.net:8181/3082 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: separate dpa_eth probe() functionsMadalin Bucur
Separated the probe functions for the private, shared and MACless interfaces. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I86a05ba2dc80709b543583e5168694748e75fd05 Reviewed-on: http://git.am.freescale.net:8181/3081 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: add shared and MACless compatibility stringsMadalin Bucur
Add dedicated compatibility strings for shared and MACless interfaces: "fsl,dpa-ethernet-shared" and "fsl,dpa-ethernet-macless". Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I71a9f14e53e9ecbbba5f837589708cc847b41e4e Reviewed-on: http://git.am.freescale.net:8181/3080 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: use separated dpa_netdev_init pathsMadalin Bucur
Use separated dpa_netdev_init code for MACless, shared and private interfaces. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I74ef5f511a6b6ac8d76b5e9141629a284a53e31b Reviewed-on: http://git.am.freescale.net:8181/3079 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: free buf_layout on dpa_remove()Madalin Bucur
Memory allocated for buf_layout was not released. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I62d49b8b8c3726477c3631910a6c4227626f49f7 Reviewed-on: http://git.am.freescale.net:8181/3078 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: merge Rx and Tx calls for dpa_set_buffer_layout()Madalin Bucur
Use a single function call to set both Rx and Tx buffers layout. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I693ae89c058f418e4dc4f4c5a3a813badf9cab7e Reviewed-on: http://git.am.freescale.net:8181/3077 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: move common code to dpaa_eth_init_ports()Madalin Bucur
Create a function dpaa_eth_init_ports() to be used by all probing functions. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I2b713b2249bf1754a8789d7f269b0e6184a2183a Reviewed-on: http://git.am.freescale.net:8181/3076 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: separate probe function for proxy interfacesMadalin Bucur
Use a separate probe function for the proxy interfaces. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I365812e7c36383bd5c1952e16e60e8a5048e0d04 Reviewed-on: http://git.am.freescale.net:8181/3075 Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>
2013-07-15dpaa_eth: use explicit interface types in dpaa_eth_probe()Madalin Bucur
Changed interface type related conditional expressions from using inferred to using explicit interface types. Previous is_shared variable was true for both shared and MAC-less interfaces - disambiguated. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I87fef2a0a11472d61bbc8957fd1b92b7b0f040c2 Reviewed-on: http://git.am.freescale.net:8181/3074 Reviewed-by: Hamciuc Bogdan-BHAMCIU1 <bogdan.hamciuc@freescale.com> Reviewed-by: Sovaiala Cristian-Constantin-B39531 <Cristian.Sovaiala@freescale.com> Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com>