ata: pata_parport: Fix aten module code indentation and style
authorDamien Le Moal <dlemoal@kernel.org>
Mon, 15 May 2023 07:32:51 +0000 (16:32 +0900)
committerDamien Le Moal <dlemoal@kernel.org>
Tue, 16 May 2023 11:17:04 +0000 (20:17 +0900)
Fix the header, indentation and coding style in the aten pata parport
protocol module to suppress warnings from smatch and other static code
analyzers.

No functional changes.

Signed-off-by: Damien Le Moal <dlemoal@kernel.org>
drivers/ata/pata_parport/aten.c

index 1bd248c..8328a49 100644 (file)
@@ -1,13 +1,12 @@
-/* 
-        aten.c  (c) 1997-8  Grant R. Guenther <grant@torque.net>
-                            Under the terms of the GNU General Public License.
-
-       aten.c is a low-level protocol driver for the ATEN EH-100
-       parallel port adapter.  The EH-100 supports 4-bit and 8-bit
-        modes only.  There is also an EH-132 which supports EPP mode
-        transfers.  The EH-132 is not yet supported.
-
-*/
+// SPDX-License-Identifier: GPL-2.0-or-later
+/*
+ * (c) 1997-8  Grant R. Guenther <grant@torque.net>
+ *
+ * aten.c is a low-level protocol driver for the ATEN EH-100
+ * parallel port adapter.  The EH-100 supports 4-bit and 8-bit
+ * modes only.  There is also an EH-132 which supports EPP mode
+ * transfers.  The EH-132 is not yet supported.
+ */
 
 #include <linux/module.h>
 #include <linux/init.h>
 
 #define j44(a,b)                ((((a>>4)&0x0f)|(b&0xf0))^0x88)
 
-/* cont = 0 - access the IDE register file 
-   cont = 1 - access the IDE command set 
-*/
-
+/*
+ * cont = 0 - access the IDE register file
+ * cont = 1 - access the IDE command set
+ */
 static int  cont_map[2] = { 0x08, 0x20 };
 
 static void aten_write_regr(struct pi_adapter *pi, int cont, int regr, int val)
-
-{      int r;
-
-       r = regr + cont_map[cont] + 0x80;
+{
+       int r = regr + cont_map[cont] + 0x80;
 
        w0(r); w2(0xe); w2(6); w0(val); w2(7); w2(6); w2(0xc);
 }
 
 static int aten_read_regr(struct pi_adapter *pi, int cont, int regr)
+{
+       int  a, b, r;
 
-{      int  a, b, r;
-
-        r = regr + cont_map[cont] + 0x40;
+       r = regr + cont_map[cont] + 0x40;
 
        switch (pi->mode) {
 
-        case 0: w0(r); w2(0xe); w2(6); 
+       case 0:
+               w0(r); w2(0xe); w2(6);
                w2(7); w2(6); w2(0);
                a = r1(); w0(0x10); b = r1(); w2(0xc);
                return j44(a,b);
 
-        case 1: r |= 0x10;
-               w0(r); w2(0xe); w2(6); w0(0xff); 
+       case 1:
+               r |= 0x10;
+               w0(r); w2(0xe); w2(6); w0(0xff);
                w2(0x27); w2(0x26); w2(0x20);
                a = r0();
                w2(0x26); w2(0xc);
@@ -59,27 +58,30 @@ static int aten_read_regr(struct pi_adapter *pi, int cont, int regr)
 }
 
 static void aten_read_block(struct pi_adapter *pi, char *buf, int count)
-
-{      int  k, a, b, c, d;
+{
+       int  k, a, b, c, d;
 
        switch (pi->mode) {
 
-       case 0: w0(0x48); w2(0xe); w2(6);
-               for (k=0;k<count/2;k++) {
+       case 0:
+               w0(0x48); w2(0xe); w2(6);
+               for (k = 0; k < count / 2; k++) {
                        w2(7); w2(6); w2(2);
                        a = r1(); w0(0x58); b = r1();
                        w2(0); d = r1(); w0(0x48); c = r1();
-                       buf[2*k] = j44(c,d);
-                       buf[2*k+1] = j44(a,b);
+                       buf[2 * k] = j44(c, d);
+                       buf[2 * k + 1] = j44(a, b);
                }
                w2(0xc);
                break;
 
-       case 1: w0(0x58); w2(0xe); w2(6);
-               for (k=0;k<count/2;k++) {
+       case 1:
+               w0(0x58); w2(0xe); w2(6);
+               for (k = 0; k < count / 2; k++) {
                        w2(0x27); w2(0x26); w2(0x22);
                        a = r0(); w2(0x20); b = r0();
-                       buf[2*k] = b; buf[2*k+1] = a;
+                       buf[2 * k] = b;
+                       buf[2 * k + 1] = a;
                }
                w2(0x26); w2(0xc);
                break;
@@ -87,36 +89,37 @@ static void aten_read_block(struct pi_adapter *pi, char *buf, int count)
 }
 
 static void aten_write_block(struct pi_adapter *pi, char *buf, int count)
-
-{      int k;
+{
+       int k;
 
        w0(0x88); w2(0xe); w2(6);
-       for (k=0;k<count/2;k++) {
-               w0(buf[2*k+1]); w2(0xe); w2(6);
-               w0(buf[2*k]); w2(7); w2(6);
+       for (k = 0; k < count / 2; k++) {
+               w0(buf[2 * k + 1]); w2(0xe); w2(6);
+               w0(buf[2 * k]); w2(7); w2(6);
        }
        w2(0xc);
 }
 
 static void aten_connect(struct pi_adapter *pi)
-
-{       pi->saved_r0 = r0();
-        pi->saved_r2 = r2();
-       w2(0xc);        
+{
+       pi->saved_r0 = r0();
+       pi->saved_r2 = r2();
+       w2(0xc);
 }
 
 static void aten_disconnect(struct pi_adapter *pi)
-
-{       w0(pi->saved_r0);
-        w2(pi->saved_r2);
-} 
+{
+       w0(pi->saved_r0);
+       w2(pi->saved_r2);
+}
 
 static void aten_log_adapter(struct pi_adapter *pi)
+{
+       char *mode_string[2] = { "4-bit", "8-bit" };
 
-{       char    *mode_string[2] = {"4-bit","8-bit"};
-
-       dev_info(&pi->dev, "ATEN EH-100 at 0x%x, mode %d (%s), delay %d\n",
-               pi->port, pi->mode, mode_string[pi->mode], pi->delay);
+       dev_info(&pi->dev,
+                "ATEN EH-100 at 0x%x, mode %d (%s), delay %d\n",
+                pi->port, pi->mode, mode_string[pi->mode], pi->delay);
 }
 
 static struct pi_protocol aten = {