* [PATCH 4/7] staging: cxt1e1: fix checkpatch error 'assignment in if condition'
@ 2014-03-04 2:09 Daeseok Youn
2014-03-04 9:04 ` Dan Carpenter
0 siblings, 1 reply; 3+ messages in thread
From: Daeseok Youn @ 2014-03-04 2:09 UTC (permalink / raw)
To: gregkh; +Cc: sachin.kamat, davem, shaun, ying.xue, devel, linux-kernel
checkpatch.pl error in linux.c:
ERROR: do not use assignment in if condition
Signed-off-by: Daeseok Youn <daeseok.youn@gmail.com>
---
drivers/staging/cxt1e1/linux.c | 28 +++++++++++++++++-----------
1 files changed, 17 insertions(+), 11 deletions(-)
diff --git a/drivers/staging/cxt1e1/linux.c b/drivers/staging/cxt1e1/linux.c
index 579e68e..cade004 100644
--- a/drivers/staging/cxt1e1/linux.c
+++ b/drivers/staging/cxt1e1/linux.c
@@ -229,7 +229,8 @@ c4_wq_port_init(mpi_t *pi)
pr_info(">> %s: creating workqueue <%s> for Port %d.\n",
__func__, name, pi->portnum); /* RLD DEBUG */
#endif
- if (!(pi->wq_port = create_singlethread_workqueue(name)))
+ pi->wq_port = create_singlethread_workqueue(name);
+ if (!pi->wq_port)
return -ENOMEM;
return 0; /* success */
}
@@ -286,12 +287,14 @@ chan_open(struct net_device *ndev)
const struct c4_priv *priv = hdlc->priv;
int ret;
- if ((ret = hdlc_open(ndev)))
- {
+ ret = hdlc_open(ndev);
+ if (ret) {
pr_info("hdlc_open failure, err %d.\n", ret);
return ret;
}
- if ((ret = c4_chan_up(priv->ci, priv->channum)))
+
+ ret = c4_chan_up(priv->ci, priv->channum);
+ if (ret)
return -ret;
try_module_get(THIS_MODULE);
netif_start_queue(ndev);
@@ -648,7 +651,8 @@ do_get_chan(struct net_device *ndev, void *data)
sizeof(struct sbecom_chan_param)))
return -EFAULT;
- if ((ret = mkret(c4_get_chan(cp.channum, &cp))))
+ ret = mkret(c4_get_chan(cp.channum, &cp));
+ if (ret)
return ret;
if (copy_to_user(data, &cp, sizeof(struct sbecom_chan_param)))
@@ -864,7 +868,8 @@ c4_ioctl(struct net_device *ndev, struct ifreq *ifr, int cmd)
return -EPERM;
if (cmd != SIOCDEVPRIVATE + 15)
return -EINVAL;
- if (!(ci = get_ci_by_dev(ndev)))
+ ci = get_ci_by_dev(ndev);
+ if (!ci)
return -EINVAL;
if (ci->state != C_RUNNING)
return -ENODEV;
@@ -1100,9 +1105,8 @@ c4_add_dev(hdw_info_t *hi, int brdno, unsigned long f0, unsigned long f1,
tasklet_enable(&ci->ci_musycc_isr_tasklet);
#endif
-
- if ((error_flag = c4_init2(ci)) != SBE_DRVR_SUCCESS)
- {
+ error_flag = c4_init2(ci);
+ if (error_flag != SBE_DRVR_SUCCESS) {
#ifdef CONFIG_PROC_FS
sbecom_proc_brd_cleanup(ci);
#endif
@@ -1123,7 +1127,8 @@ c4_mod_init(void)
int rtn;
pr_warning("%s\n", pmcc4_OSSI_release);
- if ((rtn = c4hw_attach_all()))
+ rtn = c4hw_attach_all();
+ if (rtn)
return -rtn; /* installation failure - see system log */
/* housekeeping notifications */
@@ -1174,7 +1179,8 @@ cleanup_hdlc(void)
ci = (ci_t *)(netdev_priv(hi->ndev));
for (j = 0; j < ci->max_port; j++)
for (k = 0; k < MUSYCC_NCHANS; k++)
- if ((ndev = ci->port[j].chan[k]->user))
+ ndev = ci->port[j].chan[k]->user;
+ if (ndev)
{
do_deluser(ndev, 0);
}
--
1.7.4.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 4/7] staging: cxt1e1: fix checkpatch error 'assignment in if condition'
2014-03-04 2:09 [PATCH 4/7] staging: cxt1e1: fix checkpatch error 'assignment in if condition' Daeseok Youn
@ 2014-03-04 9:04 ` Dan Carpenter
2014-03-04 10:18 ` DaeSeok Youn
0 siblings, 1 reply; 3+ messages in thread
From: Dan Carpenter @ 2014-03-04 9:04 UTC (permalink / raw)
To: Daeseok Youn
Cc: gregkh, devel, shaun, sachin.kamat, linux-kernel, ying.xue, davem
On Tue, Mar 04, 2014 at 11:09:39AM +0900, Daeseok Youn wrote:
> @@ -1174,7 +1179,8 @@ cleanup_hdlc(void)
> ci = (ci_t *)(netdev_priv(hi->ndev));
> for (j = 0; j < ci->max_port; j++)
> for (k = 0; k < MUSYCC_NCHANS; k++)
> - if ((ndev = ci->port[j].chan[k]->user))
> + ndev = ci->port[j].chan[k]->user;
> + if (ndev)
> {
> do_deluser(ndev, 0);
> }
This patch introduces a bug here because the inner for loop now has two
statement but no curly braces.
regards,
dan carpenter
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 4/7] staging: cxt1e1: fix checkpatch error 'assignment in if condition'
2014-03-04 9:04 ` Dan Carpenter
@ 2014-03-04 10:18 ` DaeSeok Youn
0 siblings, 0 replies; 3+ messages in thread
From: DaeSeok Youn @ 2014-03-04 10:18 UTC (permalink / raw)
To: Dan Carpenter
Cc: gregkh, devel, shaun, sachin.kamat, linux-kernel, ying.xue, davem
Yes, it need to add brace in inner loop.
I will send again.
Thanks for review.
Daeseok Youn
2014-03-04 18:04 GMT+09:00, Dan Carpenter <dan.carpenter@oracle.com>:
> On Tue, Mar 04, 2014 at 11:09:39AM +0900, Daeseok Youn wrote:
>> @@ -1174,7 +1179,8 @@ cleanup_hdlc(void)
>> ci = (ci_t *)(netdev_priv(hi->ndev));
>> for (j = 0; j < ci->max_port; j++)
>> for (k = 0; k < MUSYCC_NCHANS; k++)
>> - if ((ndev = ci->port[j].chan[k]->user))
>> + ndev = ci->port[j].chan[k]->user;
>> + if (ndev)
>> {
>> do_deluser(ndev, 0);
>> }
>
>
> This patch introduces a bug here because the inner for loop now has two
> statement but no curly braces.
>
> regards,
> dan carpenter
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-03-04 10:18 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-03-04 2:09 [PATCH 4/7] staging: cxt1e1: fix checkpatch error 'assignment in if condition' Daeseok Youn
2014-03-04 9:04 ` Dan Carpenter
2014-03-04 10:18 ` DaeSeok Youn
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).