summaryrefslogtreecommitdiffstats
path: root/drivers/staging/rts5208
diff options
context:
space:
mode:
authorAya Mahfouz <mahfouz.saif.elyazal@gmail.com>2015-03-04 08:19:24 +0200
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2015-03-06 15:24:23 -0800
commit5a80ee6f8871a81afb4fd919133ae5be3b656d44 (patch)
tree956ef9a5016f4d541e5a489c941bc3112a324e31 /drivers/staging/rts5208
parentdbc9eb5af20578a931efb6fac721e9a977c5171b (diff)
downloadop-kernel-dev-5a80ee6f8871a81afb4fd919133ae5be3b656d44.zip
op-kernel-dev-5a80ee6f8871a81afb4fd919133ae5be3b656d44.tar.gz
staging: rts5208: remove extra parentheses around left bit shift operation
Removes extra parentheses around bitwise left shift operations. The case handled is when resultant value is assigned to a variable. The issue was detected and resolved using the following coccinelle script: @@ expression e, e1; constant c; @@ e = -(e1 +e1 << -c); +c; @@ identifier i; constant c; type t; expression e; @@ t i = -(e +e << -c); +c; @@ expression e, e1; identifier f; constant c; @@ e1 = f(..., -(e +e << -c) +c ,...); Signed-off-by: Aya Mahfouz <mahfouz.saif.elyazal@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/rts5208')
-rw-r--r--drivers/staging/rts5208/rtsx_transport.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/rts5208/rtsx_transport.c b/drivers/staging/rts5208/rtsx_transport.c
index dab1995..03caa9b 100644
--- a/drivers/staging/rts5208/rtsx_transport.c
+++ b/drivers/staging/rts5208/rtsx_transport.c
@@ -642,7 +642,7 @@ static int rtsx_transfer_buf(struct rtsx_chip *chip, u8 card, void *buf,
dma_addr_t addr;
u8 dir;
int err = 0;
- u32 val = (1 << 31);
+ u32 val = 1 << 31;
long timeleft;
if ((buf == NULL) || (len <= 0))
OpenPOWER on IntegriCloud