Age | Commit message (Collapse) | Author |
|
This patch removes the per-packet lookup of a
completed request by adding an opaque after the SG
entries in the FD. While here, also the software
congestion control is removed and replaced with
a proper CGR with a sensible threshold. While here,
some very likely branches are decorated.
Change-Id: I48f2c71b6ac0d537843a44f8c0627c9b70c77592
Signed-off-by: Alex Porosanu <alexandru.porosanu@freescale.com>
Reviewed-on: http://git.am.freescale.net:8181/31368
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Alexandru Marginean <Alexandru.Marginean@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
This reverts commit fa008b7d96cc30a7d711641d1513e226badd3de9.
Change-Id: If2569f9e57d697ef1cceb865326c52eeaa5b03ad
Reviewed-on: http://git.am.freescale.net:8181/33914
Tested-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Added the B4860QDS platform DTS file example for NF DPAA offloading
demo application.
Signed-off-by: Marian Chereji <marian.chereji@freescale.com>
Change-Id: Ib546a6b2984735f933df90cce1be376bd60245d5
Reviewed-on: http://git.am.freescale.net:8181/33632
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Signed-off-by: Cristian Sovaiala <cristian.sovaiala@freescale.com>
Change-Id: Ic80a3f28c92aba18af32416d9216759f819c3e8c
Reviewed-on: http://git.am.freescale.net:8181/33687
Reviewed-by: Madalin-Cristian Bucur <madalin.bucur@freescale.com>
Tested-by: Cristian-Constantin Sovaiala <Cristian.Sovaiala@freescale.com>
|
|
The hooks in the DPAA Ethernet driver are guarded by a new config option
FSL_DPAA_HOOKS therefore select this option when enabling FSL_FMAN_TEST.
Signed-off-by: Cristian Sovaiala <cristian.sovaiala@freescale.com>
Change-Id: I52a22ae0357f9169741b8ccb17fc669a087cde36
Reviewed-on: http://git.am.freescale.net:8181/33686
Reviewed-by: Madalin-Cristian Bucur <madalin.bucur@freescale.com>
Tested-by: Cristian-Constantin Sovaiala <Cristian.Sovaiala@freescale.com>
|
|
Replace existing CONFIG_AS_FASTPATH and CONFIG_FSL_FMAN_TEST options
guarding the hooks with a new option CONFIG_FSL_DPAA_HOOKS.
Signed-off-by: Cristian Sovaiala <cristian.sovaiala@freescale.com>
Change-Id: Ie571a761c7971bd558ec711e4c1a811f9eee1c89
Reviewed-on: http://git.am.freescale.net:8181/33685
Reviewed-by: Madalin-Cristian Bucur <madalin.bucur@freescale.com>
Tested-by: Cristian-Constantin Sovaiala <Cristian.Sovaiala@freescale.com>
|
|
The check must be added in the RX hook instead of conditioning the
call of the hook based on the frame size.
The hooks might be utilized by multiple drivers therefore each must add
its specific checks inside the hooks.
Signed-off-by: Cristian Sovaiala <cristian.sovaiala@freescale.com>
Change-Id: I9f1ee9321bad4b6c9c38fdc332a0a8785153f81b
Reviewed-on: http://git.am.freescale.net:8181/33684
Reviewed-by: Madalin-Cristian Bucur <madalin.bucur@freescale.com>
Tested-by: Cristian-Constantin Sovaiala <Cristian.Sovaiala@freescale.com>
|
|
Make sure that saving and restoration of usb phy
registers only happen in case of deep-sleep, and
not for any other feature like sleep, etc. Also
export pm_suspend_state() and set_pm_suspend_state()
Signed-off-by: Ramneek Mehresh <ramneek.mehresh@freescale.com>
Change-Id: I2e08208c975ec7aee7ed42e7d424aaa8d14899e3
Reviewed-on: http://git.am.freescale.net:8181/33384
Reviewed-by: Nikhil Badola <nikhil.badola@freescale.com>
Tested-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Require 64 bytes align.
Remove compilation warnings.
Signed-off-by: Pan Jiafei <Jiafei.Pan@freescale.com>
Change-Id: Ic97c690adfca42c8c9ac83de9d89b05ea32b3926
Reviewed-on: http://git.am.freescale.net:8181/33327
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Write fladj register adjusts (micro)frame length to appropriate
value thus avoiding USB 2.0 devices to time-out over a longer run
Signed-off-by: Nikhil Badola <nikhil.badola@freescale.com>
Change-Id: I26df8b7d134d08171a096dba7871f7334be02315
Reviewed-on: http://git.am.freescale.net:8181/33664
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Ramneek Mehresh <ramneek.mehresh@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
include delay.h to fix compile issue on arm platform
Signed-off-by: Shengzhou Liu <Shengzhou.Liu@freescale.com>
Change-Id: I1da95676894bbdb10d6173288a04cdcdb8f26642
Reviewed-on: http://git.am.freescale.net:8181/33659
Reviewed-by: Huan Wang <alison.wang@freescale.com>
Tested-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Read RTL8211F PHY registers(e.g. PHY_ID, MII_STAT1000) will fail
with xgmac_mdio_read on T1023RDB. so add the MDIO timing delay in
xgmac_mdio_read to make it work stably with RTL8211F on T1023RDB.
Signed-off-by: Shengzhou Liu <Shengzhou.Liu@freescale.com>
Change-Id: Ib85b1858d1cca310992b892dd1c86a129c36e7d2
Reviewed-on: http://git.am.freescale.net:8181/33272
Tested-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Signed-off-by: Pan Jiafei <Jiafei.Pan@freescale.com>
Change-Id: I52411e0d3cef00e5d8a3c7a45ec4fd9ac5907686
Reviewed-on: http://git.am.freescale.net:8181/33326
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Horia Ioan Geanta Neag <horia.geanta@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Catch exceptions and return from functions safely in qbman. Exception handling
holes were found in static analysis review
Signed-off-by: Ahmed Mansour <Ahmed.Mansour@freescale.com>
Change-Id: Ie1e2feadffd2c5c968164d0c569af147ab7929fb
Reviewed-on: http://git.am.freescale.net:8181/25568
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Technical Details : Add support for the following BMI counters
and make them available to the DPA stats interface in the
User Space:
e_FM_PORT_COUNTERS_DISCARD_FRAME,
/* BMI stat counter */
e_FM_PORT_COUNTERS_RX_BAD_FRAME,
/* BMI Rx stat counter */
e_FM_PORT_COUNTERS_RX_LARGE_FRAME,
/* BMI Rx stat counter */
e_FM_PORT_COUNTERS_RX_LIST_DMA_ERR,
/* BMI Rx OP stat counter */
e_FM_PORT_COUNTERS_RX_OUT_OF_BUFFERS_DISCARD,
/* BMI Rx OP stat counter */
e_FM_PORT_COUNTERS_WRED_DISCARD,
/* BMI OP stat counter */
@Function FM_PORT_GetBmiCounters
@Description Read port's BMI stat counters and place them into
a designated structure of counters.
@Param[in] h_FmPort A handle to a FM Port module.
@Param[out] p_BmiStats counters structure
Change-Id: I464b5defc29e149252002c911b22e69343e61adf
Signed-off-by: Mandy Lavi <mandy.lavi@freescale.com>
Reviewed-on: http://git.am.freescale.net:8181/32755
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Igal Liberman <Igal.Liberman@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
This reverts commit 1aa49383a4e16ce0c98c73cf81e1a9b2938e68fc.
Change-Id: I67ab8f420b1d0777b8229dd91cba56b1bdc88d27
Reviewed-on: http://git.am.freescale.net:8181/33431
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
This patch updates Realtek PHY driver to add RTL8211F support.
RTL8211F has different register definitions from RTL8211E.
Tested with RTL8211F(RGMII, SGMII) on Freescale T1023RDB board.
Signed-off-by: Shengzhou Liu <Shengzhou.Liu@freescale.com>
Change-Id: I5251bf757888e8cbef5cf3c6f851f68824e2cd8e
Reviewed-on: http://git.am.freescale.net:8181/33250
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Xiaobo Xie <X.Xie@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
CONFIG_FMAN_T4240 was eliminated.
Instead, there are now two flags representing FmanV3:
CONFIG_FMAN_V3H for FmanV3H arch and
CONFIG_FMAN_V3L for FmanV3L arch
Signed-off-by: Pan Jiafei <Jiafei.Pan@freescale.com>
Change-Id: Ia0de7b70a0c110e379edb38ee61779e7560979a7
Reviewed-on: http://git.am.freescale.net:8181/33329
Reviewed-by: Mandy Lavi <Mandy.Lavi@freescale.com>
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Signed-off-by: Pan Jiafei <Jiafei.Pan@freescale.com>
Change-Id: I4110d5a606dd4d1e266fdb28d3ead1ba84382c22
Reviewed-on: http://git.am.freescale.net:8181/33328
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Mandy Lavi <Mandy.Lavi@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Technical Details : Add support for the following BMI counters
and make them available to the DPA stats interface in the
User Space:
e_FM_PORT_COUNTERS_DISCARD_FRAME,
/* BMI stat counter */
e_FM_PORT_COUNTERS_RX_BAD_FRAME,
/* BMI Rx stat counter */
e_FM_PORT_COUNTERS_RX_LARGE_FRAME,
/* BMI Rx stat counter */
e_FM_PORT_COUNTERS_RX_LIST_DMA_ERR,
/* BMI Rx OP stat counter */
e_FM_PORT_COUNTERS_RX_OUT_OF_BUFFERS_DISCARD,
/* BMI Rx OP stat counter */
e_FM_PORT_COUNTERS_WRED_DISCARD,
/* BMI OP stat counter */
@Function FM_PORT_GetBmiCounters
@Description Read port's BMI stat counters and place them into
a designated structure of counters.
@Param[in] h_FmPort A handle to a FM Port module.
@Param[out] p_BmiStats counters structure
Change-Id: I464b5defc29e149252002c911b22e69343e61adf
Signed-off-by: Mandy Lavi <mandy.lavi@freescale.com>
Reviewed-on: http://git.am.freescale.net:8181/32755
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Igal Liberman <Igal.Liberman@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Change-Id: I1498478e96ac52523283e41fd047b1162dad11ba
Signed-off-by: Mandy Lavi <mandy.lavi@freescale.com>
Reviewed-on: http://git.am.freescale.net:8181/33210
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Camelia Alexandra Groza <camelia.groza@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
fm_manip.c - Base cascading decision on current reparsing
option rather than the next one
Change-Id: Ic24a9c4c441337548d8f5a1c6a09d5409276e37a
Signed-off-by: Mandy Lavi <mandy.lavi@freescale.com>
Reviewed-on: http://git.am.freescale.net:8181/30768
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Eyal Harari <Eyal.Harari@freescale.com>
Reviewed-by: Nir Erez <nir.erez@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
Reviewed-on: http://git.am.freescale.net:8181/33215
|
|
|
|
This patch checks fd->length20 therefore not allowing jumbo packet
to ASF when JUMBO flag is not enabled.
Signed-off-by: Alok Makhariya <B46187@freescale.com>
Change-Id: Id6a111b823acd3a4b3152f5044262abfb3ca1f06
Reviewed-on: http://git.am.freescale.net:8181/27894
Reviewed-by: Madalin-Cristian Bucur <madalin.bucur@freescale.com>
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Sandeep Malik <Sandeep.Malik@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Signed-off-by: Cristian Sovaiala <cristian.sovaiala@freescale.com>
Change-Id: Icdfb39e75fd1c7268b26209f1bcd9b0755e6192f
Reviewed-on: http://git.am.freescale.net:8181/33299
Reviewed-by: Madalin-Cristian Bucur <madalin.bucur@freescale.com>
Tested-by: Cristian-Constantin Sovaiala <Cristian.Sovaiala@freescale.com>
|
|
When disabled auto-neg for 2.5 sgmii, it needs to set
MDIO_SGMII_IF_MODE[SGMII_SPEED] to '10'(for gigabit) for
2.5G operation, otherwise large packet loss issue occured.
Signed-off-by: Shengzhou Liu <Shengzhou.Liu@freescale.com>
Change-Id: I21ddecd01aaf1eced8d62a40b737218cf47ca6b4
Reviewed-on: http://git.am.freescale.net:8181/32697
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
- correct the location of disabling auto-neg for 2.5G in SetupSgmiiInternalPhy.
- add the missing condition e_ENET_MODE_SGMII_2500 in memac_init_phy.
- fix sgmii_2500 to sgmii-2500 to match u-boot.
Signed-off-by: Shengzhou Liu <Shengzhou.Liu@freescale.com>
Change-Id: I3e3481f1a22e0931acaaaf739dccce944e91e76c
Reviewed-on: http://git.am.freescale.net:8181/25752
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Mandy Lavi <Mandy.Lavi@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
t1024 requires for its 10g "best effort" port a different
value for TX_FIFO_SECTIONS[TX_AVAIL] than other 10g ports
in other devices:
0x60 instead of 0x19
Change-Id: I06fa4166fdff7ef39c02ebe4d97807230ae01f3d
Signed-off-by: Mandy Lavi <mandy.lavi@freescale.com>
Reviewed-on: http://git.am.freescale.net:8181/31540
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Igal Liberman <Igal.Liberman@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
CONFIG_FMAN_T4240 was eliminated
Instead, there are now two flags representing FmanV3:
CONFIG_FMAN_V3H for FmanV3H arch and
CONFIG_FMAN_V3L for FmanV3L arch.
FmanV3H h/w block is integrated in B4860, B4420, T4240, T4160, T2080 etc
FmanV3L h/w block is integrated in T1040, T1024, T1020, T1022 etc
defconfig files changes:
corenet64_fmanv3_smp_defconfig -> corenet64_fmanv3l_smp_defconfig
corenet32_fmanv3_smp_defconfig -> corenet32_fmanv3l_smp_defconfig
85xx/e6500rev2_defconfig -> corenet64_fmanv3h_smp_defconfig
Change-Id: I9fcfb454bc3bd2d72c5d55c616400a808e181413
Signed-off-by: Mandy Lavi <mandy.lavi@freescale.com>
Reviewed-on: http://git.am.freescale.net:8181/30539
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Shengzhou Liu <Shengzhou.Liu@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Signed-off-by: Yangbo Lu <yangbo.lu@freescale.com>
Change-Id: I56995d4297e8b1001dc8089ae650604dea311428
Reviewed-on: http://git.am.freescale.net:8181/32977
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
The dpa_ipsec driver was only providing statistics for the first policy
of the outbound SAs. It is now correctly suming up the statistics for all
the outbound SA's policies.
Signed-off-by: Marian Chereji <marian.chereji@freescale.com>
Reviewed-on: http://git.am.freescale.net:8181/28731
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Bogdan Constantin Popescu <bogdan.c.popescu@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
Change-Id: Ide98e16c6a6c684b6d51fa190388c7a95b1a8490
Reviewed-on: http://git.am.freescale.net:8181/33188
|
|
This allows users to pass in additional compiler flags through
the environment variable EXTRA_CFLAGS, e.g.
make EXTRA_CFLAGS=-Wno-error
Signed-off-by: Wang Dongsheng <dongsheng.wang@freescale.com>
CC: Mandy Lavi <mandy.lavi@freescale.com>
Change-Id: I73b952f3c3862e6bd96f6b90df110f8f195721d9
Reviewed-on: http://git.am.freescale.net:8181/33154
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
|
|
This allows users to pass in additional compiler flags through the
environment variable EXTRA_CFLAGS, e.g.
make EXTRA_CFLAGS=-Wno-error
Signed-off-by: Wang Dongsheng <dongsheng.wang@freescale.com>
Change-Id: I36513506ee1671c31bb076e9b73c2e8e2442cf4f
Reviewed-on: http://git.am.freescale.net:8181/33146
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Change-Id: I8b2b5e146358ab5aa9ce995c458471b4d037310b
Signed-off-by: Mandy Lavi <mandy.lavi@freescale.com>
Reviewed-on: http://git.am.freescale.net:8181/29110
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Jianhua Xie <jianhua.xie@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Change-Id: Id4166dad03d6e51202caa367de188ff8486fa9dc
Signed-off-by: Mandy Lavi <mandy.lavi@freescale.com>
Reviewed-on: http://git.am.freescale.net:8181/28750
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Igal Liberman <Igal.Liberman@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Removed unnecessary argument to dpa_bp_free().
Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com>
Change-Id: Ib16b37b0057857875805eb5692e1fd5a273883cb
Reviewed-on: http://git.am.freescale.net:8181/32981
Reviewed-by: Cristian-Constantin Sovaiala <Cristian.Sovaiala@freescale.com>
Reviewed-by: Marian Cristian Rotariu <marian.rotariu@freescale.com>
|
|
|
|
The condition that needs to be evaluated to determine if a certain
fd is to be recycled (buffers released in the buffer pool by HW)
is (fd.bpid != 0xff). Several places in the code were using the
previous condition (fd.cmd & FM_FD_CMD_FCO) that is always true
after the unification of the recycling and confirmation paths.
Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com>
Change-Id: I23789acc6879086f356ca9accfe400122fd4f0f3
Reviewed-on: http://git.am.freescale.net:8181/32879
Reviewed-by: Marian Cristian Rotariu <marian.rotariu@freescale.com>
|
|
This is the MACSEC lower API integration.
Change-Id: I3b6f172b323f1e297d6202856524ac059b7745c1
Signed-off-by: Mandy Lavi <mandy.lavi@freescale.com>
Reviewed-on: http://git.am.freescale.net:8181/19442
Reviewed-by: Mandy Lavi <Mandy.Lavi@freescale.com>
Tested-by: Mandy Lavi <Mandy.Lavi@freescale.com>
Reviewed-on: http://git.am.freescale.net:8181/28484
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Cristian-Constantin Sovaiala <Cristian.Sovaiala@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
In order to do so it was also required to introduce
a new if type for SGMII2.5G
Change-Id: Iddbe223d8b716c3ed348c7a8a53bee0c037f04f4
Signed-off-by: Mandy Lavi <mandy.lavi@freescale.com>
Reviewed-on: http://git.am.freescale.net:8181/23474
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Shengzhou Liu <Shengzhou.Liu@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
The key statistics for match tables are not accessible from user space
using fmlib. This update implements the support for the function
FM_PCD_MatchTableGetKeyStatistics to be accessible from user space.
Signed-off-by: Marian Chereji <marian.chereji@freescale.com>
Change-Id: Ibcf40fdcf7a60afc65b2f926c2a1474513ae8950
Reviewed-on: http://git.am.freescale.net:8181/25376
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Mandy Lavi <Mandy.Lavi@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Commit 6ce29b0e2a04 ("gianfar: Avoid unnecessary reg accesses in adjust_link()")
eliminates unnecessary calls to adjust_link for phy devices which don't support
interrupts and need polling. As part of that work, the 'new_state' local flag,
which was used to reduce logging noise on the console, was eliminated.
Unfortunately, that means that a 'Link is Down' log message will now be
issued continuously if a link is configured as UP, the link state is down,
and the associated phy requires polling. This occurs because priv->oldduplex
is -1 in this case, which always differs from phydev->duplex. In addition,
phydev->speed may also differ from priv->oldspeed. gfar_update_link_state()
is therefore called each time a phy is polled, even if the link state did not
change.
Cc: Claudiu Manoil <claudiu.manoil@freescale.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Reviewed-by: Claudiu Manoil <claudiu.manoil@freescale.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Change-Id: Iab3e64c4c5828133ff0e1441a087e55c5287408f
Reviewed-on: http://git.am.freescale.net:8181/32703
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
This patch correct the bad expression while writing the
bit-pattern from software's buffer to hardware registers.
Signed-off-by: Sanjeev Sharma <Sanjeev_Sharma@mentor.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Change-Id: I3cdc480f17d1c18f47c5dbea47b843c76c9669a2
Reviewed-on: http://git.am.freescale.net:8181/32702
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Since commit cd1e65044d44 ("of/device: Don't register disabled
devices"), the disabled device will not be registered at all. So we
don't need to do the check again in the platform device driver.
Signed-off-by: Kevin Hao <haokexin@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Change-Id: I3a851a9b6eb649a1fa9694970c6aa34d11f76fda
Reviewed-on: http://git.am.freescale.net:8181/32701
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Fix the following spare warning:
drivers/net/ethernet/freescale/gianfar.c:3521:60: warning: incorrect type in argument 1 (different address spaces)
drivers/net/ethernet/freescale/gianfar.c:3521:60: expected unsigned int [noderef] <asn:2>*addr
drivers/net/ethernet/freescale/gianfar.c:3521:60: got unsigned int [usertype] *rfbptr
drivers/net/ethernet/freescale/gianfar.c:205:16: warning: incorrect type in assignment (different address spaces)
drivers/net/ethernet/freescale/gianfar.c:205:16: expected unsigned int [usertype] *rfbptr
drivers/net/ethernet/freescale/gianfar.c:205:16: got unsigned int [noderef] <asn:2>*<noident>
drivers/net/ethernet/freescale/gianfar.c:2918:44: warning: incorrect type in argument 1 (different address spaces)
drivers/net/ethernet/freescale/gianfar.c:2918:44: expected unsigned int [noderef] <asn:2>*addr
drivers/net/ethernet/freescale/gianfar.c:2918:44: got unsigned int [usertype] *rfbptr
Signed-off-by: Kevin Hao <haokexin@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Change-Id: I85e5f6de656028a5ff96a0083fd36aa07869e6dc
Reviewed-on: http://git.am.freescale.net:8181/32700
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Signed-off-by: Kevin Hao <haokexin@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
The results of backporting this upstream fix to SDK are:
* removal of unused lock|unlock_tx_qs();
* gfar_new_skb() cannot be made static beacause it is
exported to ASF;
Signed-off-by: Claudiu Manoil <claudiu.manoil@freescale.com>
Change-Id: I42b348fbfa33858b0db84c1055dd6b92000e3b3e
Reviewed-on: http://git.am.freescale.net:8181/32699
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
For each Rx frame the eTSEC writes its FCS (Frame Check Sequence)
to the Rx buffer.
The eTSEC h/w manual states in the "Receive Buffer Descriptor Field
Descriptions" table:
"Data length is the number of octets written by the eTSEC into this BD's
data buffer if L is cleared (the value is equal to MRBLR), or, if L is
set, the length of the frame including *CRC*, FCB (if RCTRL[PRSDEP > 00),
preamble (if MACCFG2[PreAmRxEn]=1), time stamp (if RCTRL[TS] = 1) and
any padding (RCTRL[PAL])."
Though the FCS bytes are removed by the driver before passing the skb
to the net stack, the Rx buffer size computation does not currently
take into account the FCS bytes (4 bytes).
Because the Rx buffer size is multiple of 512 bytes, leaving out the
FCS is not a problem for the default MTU of 1500, as the Rx buffer size
is 1536 in this case. However, for custom MTUs, where the difference
between the MTU size and the Rx buffer size is less, this can be a
problem as the computed Rx buffer size won't be enough to accomodate
the FCS for a received frame that is big enough (close to MTU size).
In such case the received frame is considered to be incomplete (L flag
not set in the RxBD status) and silently dropped.
Note that the driver does not currently support S/G on Rx, so it has to
compute its Rx buffer size based on the MTU of the device.
Reported-by: Kristian Otnes <kotnes@cisco.com>
Signed-off-by: Claudiu Manoil <claudiu.manoil@freescale.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Change-Id: I75eab6badeb8cd57afffee4e88a2e47fc1685a2f
Reviewed-on: http://git.am.freescale.net:8181/32698
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
Change-Id: Ifb357bd9b78c06042aa6e9fbb7c50b821f955064
Signed-off-by: Mandy Lavi <mandy.lavi@freescale.com>
Reviewed-on: http://git.am.freescale.net:8181/32794
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Liron Himi <Liron.Himi@freescale.com>
Reviewed-by: Pinghua An <pinghua.an@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|
|
generic_handle_irq() should always be called in interrupt disabled context,
this patch ensures that interrupts are disabled while calling generic_handle_irq().
It call local_irq_disable() before calling generic_handle_irq() and local_irq_enable() on exit.
In case of non-rt kernel, ls_pcie_msi_irq_handler is called in interrupt disabled context,
so no need to explicitly disabling interrupt while calling generic_handle_irq().
Signed-off-by: Meenakshi Aggarwal <meenakshi.aggarwal@freescale.com>
Change-Id: I13d83423ea45c1ce5021474c04640ac7118664af
Reviewed-on: http://git.am.freescale.net:8181/31478
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Priyanka Jain <Priyanka.Jain@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
|