From 60ab99670ec43f10e4c73f027aa4a1dac3aac08c Mon Sep 17 00:00:00 2001 From: nbd Date: Tue, 10 Jun 2014 09:05:40 +0000 Subject: [PATCH] atheros: do not assignment in if condition Remove assignments from if conditions as suggested by checkpatch. Signed-off-by: Sergey Ryazanov git-svn-id: svn://svn.openwrt.org/openwrt/trunk@41084 3c298f89-4303-0410-b956-a3cf2f4a3e73 --- target/linux/atheros/patches-3.10/100-board.patch | 9 +++++---- target/linux/atheros/patches-3.10/110-ar2313_ethernet.patch | 5 +++-- target/linux/atheros/patches-3.10/130-watchdog.patch | 5 +++-- .../linux/atheros/patches-3.10/220-enet_micrel_workaround.patch | 2 +- target/linux/atheros/patches-3.10/230-3_10-updates.patch | 2 +- 5 files changed, 13 insertions(+), 10 deletions(-) diff --git a/target/linux/atheros/patches-3.10/100-board.patch b/target/linux/atheros/patches-3.10/100-board.patch index 0bd3410e38..df7dcf6020 100644 --- a/target/linux/atheros/patches-3.10/100-board.patch +++ b/target/linux/atheros/patches-3.10/100-board.patch @@ -1524,7 +1524,7 @@ + --- /dev/null +++ b/arch/mips/ar231x/ar5312.c -@@ -0,0 +1,579 @@ +@@ -0,0 +1,580 @@ +/* + * This file is subject to the terms and conditions of the GNU General Public + * License. See the file "COPYING" in the main directory of this archive @@ -1975,7 +1975,7 @@ +static int __init +ar5312_cpu_frequency(void) +{ -+ unsigned int result; ++ unsigned int scratch; + unsigned int predivide_mask, predivide_shift; + unsigned int multiplier_mask, multiplier_shift; + unsigned int clock_ctl1, predivide_select, predivisor, multiplier; @@ -1983,8 +1983,9 @@ + u16 devid; + + /* Trust the bootrom's idea of cpu frequency. */ -+ if ((result = ar231x_read_reg(AR5312_SCRATCH))) -+ return result; ++ scratch = ar231x_read_reg(AR5312_SCRATCH); ++ if (scratch) ++ return scratch; + + devid = ar231x_read_reg(AR531X_REV); + devid &= AR531X_REV_MAJ; diff --git a/target/linux/atheros/patches-3.10/110-ar2313_ethernet.patch b/target/linux/atheros/patches-3.10/110-ar2313_ethernet.patch index b9a56b7ac5..38b29691bf 100644 --- a/target/linux/atheros/patches-3.10/110-ar2313_ethernet.patch +++ b/target/linux/atheros/patches-3.10/110-ar2313_ethernet.patch @@ -32,7 +32,7 @@ +obj-$(CONFIG_NET_VENDOR_AR231X) += ar231x.o --- /dev/null +++ b/drivers/net/ethernet/ar231x/ar231x.c -@@ -0,0 +1,1265 @@ +@@ -0,0 +1,1266 @@ +/* + * ar231x.c: Linux driver for the Atheros AR231x Ethernet device. + * @@ -905,8 +905,9 @@ + + while (idx != sp->tx_prd) { + txdesc = &sp->tx_ring[idx]; ++ status = txdesc->status; + -+ if ((status = txdesc->status) & DMA_TX_OWN) { ++ if (status & DMA_TX_OWN) { + /* ar231x dma still owns descr */ + break; + } diff --git a/target/linux/atheros/patches-3.10/130-watchdog.patch b/target/linux/atheros/patches-3.10/130-watchdog.patch index 8c61c91a0a..f98b6b2168 100644 --- a/target/linux/atheros/patches-3.10/130-watchdog.patch +++ b/target/linux/atheros/patches-3.10/130-watchdog.patch @@ -1,6 +1,6 @@ --- /dev/null +++ b/drivers/watchdog/ar2315-wtd.c -@@ -0,0 +1,199 @@ +@@ -0,0 +1,200 @@ +/* + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by @@ -117,7 +117,8 @@ + break; + + case WDIOC_SETTIMEOUT: -+ if((ret = get_user(new_wdt_timeout, (int __user *)arg))) ++ ret = get_user(new_wdt_timeout, (int __user *)arg); ++ if (ret) + break; + wdt_timeout = HEARTBEAT(new_wdt_timeout); + ar2315_wdt_enable(); diff --git a/target/linux/atheros/patches-3.10/220-enet_micrel_workaround.patch b/target/linux/atheros/patches-3.10/220-enet_micrel_workaround.patch index 870b76c65b..35344ca73a 100644 --- a/target/linux/atheros/patches-3.10/220-enet_micrel_workaround.patch +++ b/target/linux/atheros/patches-3.10/220-enet_micrel_workaround.patch @@ -78,7 +78,7 @@ kfree(dev); return 0; } -@@ -1119,6 +1165,9 @@ static int ar231x_ioctl(struct net_devic +@@ -1120,6 +1166,9 @@ static int ar231x_ioctl(struct net_devic struct ar231x_private *sp = netdev_priv(dev); int ret; diff --git a/target/linux/atheros/patches-3.10/230-3_10-updates.patch b/target/linux/atheros/patches-3.10/230-3_10-updates.patch index 873296dbfc..80d7eaa395 100644 --- a/target/linux/atheros/patches-3.10/230-3_10-updates.patch +++ b/target/linux/atheros/patches-3.10/230-3_10-updates.patch @@ -1,6 +1,6 @@ --- a/drivers/net/ethernet/ar231x/ar231x.c +++ b/drivers/net/ethernet/ar231x/ar231x.c -@@ -1282,7 +1282,7 @@ static int ar231x_mdiobus_probe (struct +@@ -1283,7 +1283,7 @@ static int ar231x_mdiobus_probe (struct BUG_ON(!phydev); BUG_ON(phydev->attached_dev); -- 2.11.0