ata: Avoid comma separated statements
authorJoe Perches <joe@perches.com>
Tue, 25 Aug 2020 04:56:02 +0000 (21:56 -0700)
committerJens Axboe <axboe@kernel.dk>
Sun, 31 Jan 2021 15:05:46 +0000 (08:05 -0700)
Use semicolons and braces.

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
drivers/ata/pata_icside.c

index 08543ae..498383c 100644 (file)
@@ -202,14 +202,19 @@ static void pata_icside_set_dmamode(struct ata_port *ap, struct ata_device *adev
         * Choose the IOMD cycle timing which ensure that the interface
         * satisfies the measured active, recovery and cycle times.
         */
-       if (t.active <= 50 && t.recover <= 375 && t.cycle <= 425)
-               iomd_type = 'D', cycle = 187;
-       else if (t.active <= 125 && t.recover <= 375 && t.cycle <= 500)
-               iomd_type = 'C', cycle = 250;
-       else if (t.active <= 200 && t.recover <= 550 && t.cycle <= 750)
-               iomd_type = 'B', cycle = 437;
-       else
-               iomd_type = 'A', cycle = 562;
+       if (t.active <= 50 && t.recover <= 375 && t.cycle <= 425) {
+               iomd_type = 'D';
+               cycle = 187;
+       } else if (t.active <= 125 && t.recover <= 375 && t.cycle <= 500) {
+               iomd_type = 'C';
+               cycle = 250;
+       } else if (t.active <= 200 && t.recover <= 550 && t.cycle <= 750) {
+               iomd_type = 'B';
+               cycle = 437;
+       } else {
+               iomd_type = 'A';
+               cycle = 562;
+       }
 
        ata_dev_info(adev, "timings: act %dns rec %dns cyc %dns (%c)\n",
                     t.active, t.recover, t.cycle, iomd_type);