This patch solves the following checkpatch.pl's message in 
drivers/staging/rts5208/rtsx_transport.c:397.

WARNING: line over 80 characters
+                               option = RTSX_SG_VALID | RTSX_SG_END | 
RTSX_SG_TRANS_DATA;

Signed-off-by: Prakhar Sinha <prakharsinha2...@gmail.com>
---
Changes in v2:
  - Re-structured assignment to solve checkpath.pl's warning.

 drivers/staging/rts5208/rtsx_transport.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/rts5208/rtsx_transport.c 
b/drivers/staging/rts5208/rtsx_transport.c
index 8277d7895608..e61bc7c6ac33 100644
--- a/drivers/staging/rts5208/rtsx_transport.c
+++ b/drivers/staging/rts5208/rtsx_transport.c
@@ -393,10 +393,9 @@ static int rtsx_transfer_sglist_adma_partial(struct 
rtsx_chip *chip, u8 card,
                        *offset = 0;
                        *index = *index + 1;
                }
-               if ((i == (sg_cnt - 1)) || !resid)
-                       option = RTSX_SG_VALID | RTSX_SG_END | 
RTSX_SG_TRANS_DATA;
-               else
-                       option = RTSX_SG_VALID | RTSX_SG_TRANS_DATA;
+               option = RTSX_SG_VALID | RTSX_SG_TRANS_DATA;
+               if ((i == sg_cnt - 1) || !resid)
+                       option |= RTSX_SG_END;
 
                rtsx_add_sg_tbl(chip, (u32)addr, (u32)len, option);
 
-- 
2.20.1

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to