Staging: rts5208: rtsx_transport.c: Align to open parenthesis
authorShaun Ren <shaun.ren@linux.com>
Mon, 15 Feb 2016 18:58:48 +0000 (10:58 -0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 20 Feb 2016 23:35:42 +0000 (15:35 -0800)
This patch fixes the alignment issue reported by checkpatch.pl:

CHECK: Alignment should match open parenthesis

Add a unsigned char *sgbuffer in rtsx_stor_access_xfer_buffer to make the
following memcpy logic easier to read.

Add a struct scatterlist *sg in the use_sg branch of
rtsx_transfer_data_partial to make the parameters of the
rtsx_transfer_sglist_adma_partial call fit in 80 character lines after
aligning them to the open parenthesis.

Refactor memcpy logic in rtsx_stor_access_xfer_buf to make it more legible.

Signed-off-by: Shaun Ren <shaun.ren@linux.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rts5208/rtsx_transport.c

index 91fdccf353a0c86f9cbb4e636b1d32a6e5726178..b6de93f76c710cf0a55d4686237bb9102905c4b6 100644 (file)
  */
 
 unsigned int rtsx_stor_access_xfer_buf(unsigned char *buffer,
-       unsigned int buflen, struct scsi_cmnd *srb, unsigned int *index,
-       unsigned int *offset, enum xfer_buf_dir dir)
+                                      unsigned int buflen,
+                                      struct scsi_cmnd *srb,
+                                      unsigned int *index,
+                                      unsigned int *offset,
+                                      enum xfer_buf_dir dir)
 {
        unsigned int cnt;
 
        /* If not using scatter-gather, just transfer the data directly. */
        if (scsi_sg_count(srb) == 0) {
+               unsigned char *sgbuffer;
+
                if (*offset >= scsi_bufflen(srb))
                        return 0;
                cnt = min(buflen, scsi_bufflen(srb) - *offset);
+
+               sgbuffer = (unsigned char *)scsi_sglist(srb) + *offset;
+
                if (dir == TO_XFER_BUF)
-                       memcpy((unsigned char *) scsi_sglist(srb) + *offset,
-                                       buffer, cnt);
+                       memcpy(sgbuffer, buffer, cnt);
                else
-                       memcpy(buffer, (unsigned char *) scsi_sglist(srb) +
-                                       *offset, cnt);
+                       memcpy(buffer, sgbuffer, cnt);
                *offset += cnt;
 
        /*
@@ -126,7 +132,7 @@ unsigned int rtsx_stor_access_xfer_buf(unsigned char *buffer,
  * SCSI residue.
  */
 void rtsx_stor_set_xfer_buf(unsigned char *buffer,
-       unsigned int buflen, struct scsi_cmnd *srb)
+                           unsigned int buflen, struct scsi_cmnd *srb)
 {
        unsigned int index = 0, offset = 0;
 
@@ -137,7 +143,7 @@ void rtsx_stor_set_xfer_buf(unsigned char *buffer,
 }
 
 void rtsx_stor_get_xfer_buf(unsigned char *buffer,
-       unsigned int buflen, struct scsi_cmnd *srb)
+                           unsigned int buflen, struct scsi_cmnd *srb)
 {
        unsigned int index = 0, offset = 0;
 
@@ -191,8 +197,8 @@ void rtsx_invoke_transport(struct scsi_cmnd *srb, struct rtsx_chip *chip)
                /* set the result so the higher layers expect this data */
                srb->result = SAM_STAT_CHECK_CONDITION;
                memcpy(srb->sense_buffer,
-                       (unsigned char *)&(chip->sense_buffer[SCSI_LUN(srb)]),
-                       sizeof(struct sense_data_t));
+                      (unsigned char *)&(chip->sense_buffer[SCSI_LUN(srb)]),
+                      sizeof(struct sense_data_t));
        }
 
        return;
@@ -202,7 +208,7 @@ Handle_Errors:
 }
 
 void rtsx_add_cmd(struct rtsx_chip *chip,
-               u8 cmd_type, u16 reg_addr, u8 mask, u8 data)
+                 u8 cmd_type, u16 reg_addr, u8 mask, u8 data)
 {
        u32 *cb = (u32 *)(chip->host_cmds_ptr);
        u32 val = 0;
@@ -321,9 +327,11 @@ static inline void rtsx_add_sg_tbl(
 }
 
 static int rtsx_transfer_sglist_adma_partial(struct rtsx_chip *chip, u8 card,
-               struct scatterlist *sg, int num_sg, unsigned int *index,
-               unsigned int *offset, int size,
-               enum dma_data_direction dma_dir, int timeout)
+                                            struct scatterlist *sg, int num_sg,
+                                            unsigned int *index,
+                                            unsigned int *offset, int size,
+                                            enum dma_data_direction dma_dir,
+                                            int timeout)
 {
        struct rtsx_dev *rtsx = chip->rtsx;
        struct completion trans_done;
@@ -486,8 +494,9 @@ out:
 }
 
 static int rtsx_transfer_sglist_adma(struct rtsx_chip *chip, u8 card,
-               struct scatterlist *sg, int num_sg,
-               enum dma_data_direction dma_dir, int timeout)
+                                    struct scatterlist *sg, int num_sg,
+                                    enum dma_data_direction dma_dir,
+                                    int timeout)
 {
        struct rtsx_dev *rtsx = chip->rtsx;
        struct completion trans_done;
@@ -633,7 +642,8 @@ out:
 }
 
 static int rtsx_transfer_buf(struct rtsx_chip *chip, u8 card, void *buf,
-               size_t len, enum dma_data_direction dma_dir, int timeout)
+                            size_t len, enum dma_data_direction dma_dir,
+                            int timeout)
 {
        struct rtsx_dev *rtsx = chip->rtsx;
        struct completion trans_done;
@@ -716,9 +726,9 @@ out:
 }
 
 int rtsx_transfer_data_partial(struct rtsx_chip *chip, u8 card,
-               void *buf, size_t len, int use_sg, unsigned int *index,
-               unsigned int *offset, enum dma_data_direction dma_dir,
-               int timeout)
+                              void *buf, size_t len, int use_sg,
+                              unsigned int *index, unsigned int *offset,
+                              enum dma_data_direction dma_dir, int timeout)
 {
        int err = 0;
 
@@ -726,13 +736,16 @@ int rtsx_transfer_data_partial(struct rtsx_chip *chip, u8 card,
        if (rtsx_chk_stat(chip, RTSX_STAT_ABORT))
                return -EIO;
 
-       if (use_sg)
-               err = rtsx_transfer_sglist_adma_partial(chip, card,
-                               (struct scatterlist *)buf, use_sg,
-                               index, offset, (int)len, dma_dir, timeout);
-       else
+       if (use_sg) {
+               struct scatterlist *sg = (struct scatterlist *)buf;
+
+               err = rtsx_transfer_sglist_adma_partial(chip, card, sg, use_sg,
+                                                       index, offset, (int)len,
+                                                       dma_dir, timeout);
+       } else {
                err = rtsx_transfer_buf(chip, card,
                                        buf, len, dma_dir, timeout);
+       }
        if (err < 0) {
                if (RTSX_TST_DELINK(chip)) {
                        RTSX_CLR_DELINK(chip);
@@ -745,7 +758,7 @@ int rtsx_transfer_data_partial(struct rtsx_chip *chip, u8 card,
 }
 
 int rtsx_transfer_data(struct rtsx_chip *chip, u8 card, void *buf, size_t len,
-               int use_sg, enum dma_data_direction dma_dir, int timeout)
+                      int use_sg, enum dma_data_direction dma_dir, int timeout)
 {
        int err = 0;
 
@@ -757,8 +770,8 @@ int rtsx_transfer_data(struct rtsx_chip *chip, u8 card, void *buf, size_t len,
 
        if (use_sg) {
                err = rtsx_transfer_sglist_adma(chip, card,
-                               (struct scatterlist *)buf,
-                               use_sg, dma_dir, timeout);
+                                               (struct scatterlist *)buf,
+                                               use_sg, dma_dir, timeout);
        } else {
                err = rtsx_transfer_buf(chip, card, buf, len, dma_dir, timeout);
        }