diff options
author | Cristian Bercaru <cristian.bercaru@freescale.com> | 2013-08-14 13:34:56 (GMT) |
---|---|---|
committer | Rivera Jose-B46482 <German.Rivera@freescale.com> | 2013-09-05 19:18:26 (GMT) |
commit | 188c08c4064ffc2fa303b8512a673016d1d4071c (patch) | |
tree | 017eed27da3009eb7b032be6622e059c4f9e1737 /drivers/net/ethernet | |
parent | d83e6ec78bf67be5f2470dfe80c201b4063b9e8d (diff) | |
download | linux-fsl-qoriq-188c08c4064ffc2fa303b8512a673016d1d4071c.tar.xz |
dpaa_eth: fix coccinelle code analysis issues
Assign boolean constants to boolean variables instead of integer
constants.
Signed-off-by: Cristian Bercaru <cristian.bercaru@freescale.com>
Change-Id: Ib7ecb86273355d0dd6c4f06fe410d572e613c570
Reviewed-on: http://git.am.freescale.net:8181/4021
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Bucur Madalin-Cristian-B32716 <madalin.bucur@freescale.com>
Reviewed-by: Radulescu Ruxandra Ioana-B05472 <ruxandra.radulescu@freescale.com>
Reviewed-by: Rivera Jose-B46482 <Jose.G.Rivera@freescale.com>
Reviewed-on: http://git.am.freescale.net:8181/4459
Reviewed-by: Rivera Jose-B46482 <German.Rivera@freescale.com>
Diffstat (limited to 'drivers/net/ethernet')
-rw-r--r-- | drivers/net/ethernet/freescale/dpa/dpaa_eth_common.c | 8 | ||||
-rw-r--r-- | drivers/net/ethernet/freescale/dpa/dpaa_eth_unit_test.c | 2 | ||||
-rw-r--r-- | drivers/net/ethernet/freescale/dpa/offline_port.c | 4 |
3 files changed, 7 insertions, 7 deletions
diff --git a/drivers/net/ethernet/freescale/dpa/dpaa_eth_common.c b/drivers/net/ethernet/freescale/dpa/dpaa_eth_common.c index 013ff0f..3a74cbb 100644 --- a/drivers/net/ethernet/freescale/dpa/dpaa_eth_common.c +++ b/drivers/net/ethernet/freescale/dpa/dpaa_eth_common.c @@ -335,7 +335,7 @@ static void dpa_ts_tx_enable(struct net_device *dev) if (mac_dev->ptp_enable) mac_dev->ptp_enable(mac_dev->get_mac_handle(mac_dev)); - priv->ts_tx_en = TRUE; + priv->ts_tx_en = true; } static void dpa_ts_tx_disable(struct net_device *dev) @@ -355,7 +355,7 @@ static void dpa_ts_tx_disable(struct net_device *dev) mac_dev->ptp_disable(mac_dev->get_mac_handle(mac_dev)); #endif - priv->ts_tx_en = FALSE; + priv->ts_tx_en = false; } static void dpa_ts_rx_enable(struct net_device *dev) @@ -368,7 +368,7 @@ static void dpa_ts_rx_enable(struct net_device *dev) if (mac_dev->ptp_enable) mac_dev->ptp_enable(mac_dev->get_mac_handle(mac_dev)); - priv->ts_rx_en = TRUE; + priv->ts_rx_en = true; } static void dpa_ts_rx_disable(struct net_device *dev) @@ -388,7 +388,7 @@ static void dpa_ts_rx_disable(struct net_device *dev) mac_dev->ptp_disable(mac_dev->get_mac_handle(mac_dev)); #endif - priv->ts_rx_en = FALSE; + priv->ts_rx_en = false; } static int dpa_ts_ioctl(struct net_device *dev, struct ifreq *rq, int cmd) diff --git a/drivers/net/ethernet/freescale/dpa/dpaa_eth_unit_test.c b/drivers/net/ethernet/freescale/dpa/dpaa_eth_unit_test.c index 62095a7..4856e15 100644 --- a/drivers/net/ethernet/freescale/dpa/dpaa_eth_unit_test.c +++ b/drivers/net/ethernet/freescale/dpa/dpaa_eth_unit_test.c @@ -283,7 +283,7 @@ static int dpa_tx_unit_test(struct net_device *net_dev) } /* Was it good? */ - if (tx_unit_test_passed == false) { + if (!tx_unit_test_passed) { pr_err("Test failed:\n"); pr_err("size: %d pad: %d head: %p end: %p\n", size, headroom, tx_unit_skb_head, diff --git a/drivers/net/ethernet/freescale/dpa/offline_port.c b/drivers/net/ethernet/freescale/dpa/offline_port.c index c002dc9..95dd28a 100644 --- a/drivers/net/ethernet/freescale/dpa/offline_port.c +++ b/drivers/net/ethernet/freescale/dpa/offline_port.c @@ -134,7 +134,7 @@ oh_port_probe(struct platform_device *_of_dev) uint32_t queues_count; uint32_t crt_fqid_base; uint32_t crt_fq_count; - bool frag_enabled = FALSE; + bool frag_enabled = false; struct fm_port_params oh_port_tx_params; struct fm_port_pcd_param oh_port_pcd_params; struct dpa_buffer_layout_s buf_layout; @@ -369,7 +369,7 @@ oh_port_probe(struct platform_device *_of_dev) buf_layout.manip_extra_space != FRAG_MANIP_SPACE) goto init_port; - frag_enabled = TRUE; + frag_enabled = true; dev_info(dpa_oh_dev, "IP Fragmentation enabled for OH port %d", *port_id); |