scsi: pm80xx: Clean up indentation of a code block
authorColin Ian King <colin.king@canonical.com>
Fri, 15 Jan 2021 09:58:24 +0000 (09:58 +0000)
committerMartin K. Petersen <martin.petersen@oracle.com>
Thu, 21 Jan 2021 02:38:56 +0000 (21:38 -0500)
A block of code is indented one level too deeply, clean this up.

Link: https://lore.kernel.org/r/20210115095824.9170-1-colin.king@canonical.com
Acked-by: Jack Wang <jinpu.wang@cloud.ionos.com>
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Addresses-Coverity: ("Indentation does not match nesting level")

drivers/scsi/pm8001/pm80xx_hwi.c

index e7fef42..6fd206a 100644 (file)
@@ -358,26 +358,22 @@ moreData:
                                        MEMBASE_II_SHIFT_REGISTER,
                                        pm8001_ha->fatal_forensic_shift_offset);
                }
-                       /* Read the next block of the debug data.*/
-                       length_to_read = pm8001_mr32(fatal_table_address,
-                       MPI_FATAL_EDUMP_TABLE_ACCUM_LEN) -
-                       pm8001_ha->forensic_preserved_accumulated_transfer;
-                       if (length_to_read != 0x0) {
-                               pm8001_ha->forensic_fatal_step = 0;
-                               goto moreData;
-                       } else {
-                               pm8001_ha->forensic_info.data_buf.direct_data +=
-                               sprintf(
-                               pm8001_ha->forensic_info.data_buf.direct_data,
+               /* Read the next block of the debug data.*/
+               length_to_read = pm8001_mr32(fatal_table_address,
+               MPI_FATAL_EDUMP_TABLE_ACCUM_LEN) -
+               pm8001_ha->forensic_preserved_accumulated_transfer;
+               if (length_to_read != 0x0) {
+                       pm8001_ha->forensic_fatal_step = 0;
+                       goto moreData;
+               } else {
+                       pm8001_ha->forensic_info.data_buf.direct_data +=
+                       sprintf(pm8001_ha->forensic_info.data_buf.direct_data,
                                "%08x ", 4);
-                               pm8001_ha->forensic_info.data_buf.read_len
-                                                               = 0xFFFFFFFF;
-                               pm8001_ha->forensic_info.data_buf.direct_len
-                                                               =  0;
-                               pm8001_ha->forensic_info.data_buf.direct_offset
-                                                               = 0;
-                               pm8001_ha->forensic_info.data_buf.read_len = 0;
-                       }
+                       pm8001_ha->forensic_info.data_buf.read_len = 0xFFFFFFFF;
+                       pm8001_ha->forensic_info.data_buf.direct_len =  0;
+                       pm8001_ha->forensic_info.data_buf.direct_offset = 0;
+                       pm8001_ha->forensic_info.data_buf.read_len = 0;
+               }
        }
        offset = (int)((char *)pm8001_ha->forensic_info.data_buf.direct_data
                        - (char *)buf);