Age | Commit message (Collapse) | Author |
|
|
|
'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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|