ata: pata_parport: use dev_* instead of printk
authorOndrej Zary <linux@zary.sk>
Sat, 18 Feb 2023 22:01:31 +0000 (23:01 +0100)
committerDamien Le Moal <damien.lemoal@opensource.wdc.com>
Thu, 23 Mar 2023 01:29:25 +0000 (10:29 +0900)
Use dev_info/dev_err/dev_dbg instead of printk.

Reviewed-by: Sergey Shtylyov <s.shtylyov@omp.ru>
Signed-off-by: Ondrej Zary <linux@zary.sk>
Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
15 files changed:
drivers/ata/pata_parport/aten.c
drivers/ata/pata_parport/bpck.c
drivers/ata/pata_parport/bpck6.c
drivers/ata/pata_parport/comm.c
drivers/ata/pata_parport/dstr.c
drivers/ata/pata_parport/epat.c
drivers/ata/pata_parport/epia.c
drivers/ata/pata_parport/fit2.c
drivers/ata/pata_parport/fit3.c
drivers/ata/pata_parport/friq.c
drivers/ata/pata_parport/frpw.c
drivers/ata/pata_parport/kbic.c
drivers/ata/pata_parport/ktti.c
drivers/ata/pata_parport/on20.c
drivers/ata/pata_parport/on26.c

index 3f5c50c2c43ab76a367856a29b6274b7b22b744d..ccbba14ccab107a2ecc2eb36d6fe082cf684d750 100644 (file)
@@ -124,8 +124,8 @@ static void aten_log_adapter(struct pi_adapter *pi)
 
 {       char    *mode_string[2] = {"4-bit","8-bit"};
 
-       printk("aten %s, ATEN EH-100 at 0x%x, ", ATEN_VERSION, pi->port);
-        printk("mode %d (%s), delay %d\n",pi->mode,
+       dev_info(&pi->dev, "aten %s, ATEN EH-100 at 0x%x, ", ATEN_VERSION, pi->port);
+       dev_info(&pi->dev, "mode %d (%s), delay %d\n", pi->mode,
                mode_string[pi->mode],pi->delay);
 
 }
index f475a25769f0d758dc3a83b0963f53b31eec62c8..1cba098b4b82869657948e3e3725660c68499f39 100644 (file)
@@ -435,9 +435,9 @@ static void bpck_log_adapter(struct pi_adapter *pi)
           printk("             %64.64s\n", &scratch[64]);
 #endif
 
-       printk("bpck %s, backpack %8.8s unit %d",
+       dev_info(&pi->dev, "bpck %s, backpack %8.8s unit %d",
                BPCK_VERSION, &scratch[110], pi->unit);
-       printk(" at 0x%x, mode %d (%s), delay %d\n",pi->port,
+       dev_info(&pi->dev, " at 0x%x, mode %d (%s), delay %d\n", pi->port,
                pi->mode,mode_string[pi->mode],pi->delay);
 }
 
index 41395a97d77c147d09bde5e67cb31510c9eab654..4770e29bfdeb430725698541281b644655bf336f 100644 (file)
@@ -90,10 +90,7 @@ static void bpck6_read_block(struct pi_adapter *pi, char *buf, int len)
 
 static void bpck6_connect(struct pi_adapter *pi)
 {
-       if(verbose)
-       {
-               printk(KERN_DEBUG "connect\n");
-       }
+       dev_dbg(&pi->dev, "connect\n");
 
        if(pi->mode >=2)
        {
@@ -114,22 +111,16 @@ static void bpck6_connect(struct pi_adapter *pi)
 
 static void bpck6_disconnect(struct pi_adapter *pi)
 {
-       if(verbose)
-       {
-               printk("disconnect\n");
-       }
+       dev_dbg(&pi->dev, "disconnect\n");
        ppc6_wr_extout(PPCSTRUCT(pi),0x0);
        ppc6_close(PPCSTRUCT(pi));
 }
 
 static int bpck6_test_port(struct pi_adapter *pi)   /* check for 8-bit port */
 {
-       if(verbose)
-       {
-               printk(KERN_DEBUG "PARPORT indicates modes=%x for lp=0x%lx\n",
-                               ((struct pardevice*)(pi->pardev))->port->modes,
-                       ((struct pardevice *)(pi->pardev))->port->base); 
-       }
+       dev_dbg(&pi->dev, "PARPORT indicates modes=%x for lp=0x%lx\n",
+               ((struct pardevice *)(pi->pardev))->port->modes,
+               ((struct pardevice *)(pi->pardev))->port->base);
 
        /*copy over duplicate stuff.. initialize state info*/
        PPCSTRUCT(pi)->ppc_id=pi->unit;
@@ -158,10 +149,7 @@ static int bpck6_probe_unit(struct pi_adapter *pi)
 {
        int out;
 
-       if(verbose)
-       {
-               printk(KERN_DEBUG "PROBE UNIT %x on port:%x\n",pi->unit,pi->port);
-       }
+       dev_dbg(&pi->dev, "PROBE UNIT %x on port:%x\n", pi->unit, pi->port);
 
        /*SET PPC UNIT NUMBER*/
        PPCSTRUCT(pi)->ppc_id=pi->unit;
@@ -171,26 +159,17 @@ static int bpck6_probe_unit(struct pi_adapter *pi)
 
        out=ppc6_open(PPCSTRUCT(pi));
 
-       if(verbose)
-       {
-               printk(KERN_DEBUG "ppc_open returned %2x\n",out);
-       }
+       dev_dbg(&pi->dev, "ppc_open returned %2x\n", out);
 
        if(out)
        {
                ppc6_close(PPCSTRUCT(pi));
-               if(verbose)
-               {
-                       printk(KERN_DEBUG "leaving probe\n");
-               }
+               dev_dbg(&pi->dev, "leaving probe\n");
                return(1);
        }
        else
        {
-               if(verbose)
-               {
-                       printk(KERN_DEBUG "Failed open\n");
-               }
+               dev_dbg(&pi->dev, "Failed open\n");
                return(0);
        }
 }
@@ -200,9 +179,9 @@ static void bpck6_log_adapter(struct pi_adapter *pi)
        char *mode_string[5]=
                {"4-bit","8-bit","EPP-8","EPP-16","EPP-32"};
 
-       printk("BACKPACK %s, Micro Solutions BACKPACK Drive at 0x%x\n",
+       dev_info(&pi->dev, "BACKPACK %s, Micro Solutions BACKPACK Drive at 0x%x\n",
                BACKPACK_VERSION, pi->port);
-       printk("Unit: %d Mode:%d (%s) Delay %d\n",
+       dev_info(&pi->dev, "Unit: %d Mode:%d (%s) Delay %d\n",
                pi->unit,pi->mode,mode_string[pi->mode],pi->delay);
 }
 
@@ -215,7 +194,7 @@ static int bpck6_init_proto(struct pi_adapter *pi)
                return 0;
        }
 
-       printk(KERN_ERR "ERROR COULDN'T ALLOCATE MEMORY\n");
+       dev_err(&pi->dev, "ERROR COULDN'T ALLOCATE MEMORY\n");
        return -1;
 }
 
index 88476072b7085817904c8d2e9ff3a4be5d74df3a..7b59eca52185c76f2de414fe59a8c27209942a8a 100644 (file)
@@ -183,8 +183,8 @@ static void comm_log_adapter(struct pi_adapter *pi)
 
 {       char    *mode_string[5] = {"4-bit","8-bit","EPP-8","EPP-16","EPP-32"};
 
-       printk("comm %s, DataStor Commuter at 0x%x, ", COMM_VERSION, pi->port);
-        printk("mode %d (%s), delay %d\n",pi->mode,
+       dev_info(&pi->dev, "comm %s, DataStor Commuter at 0x%x, ", COMM_VERSION, pi->port);
+       dev_info(&pi->dev, "mode %d (%s), delay %d\n", pi->mode,
                mode_string[pi->mode],pi->delay);
 
 }
index a8b238828061c666520910de2322e5689c86053f..eb6ddceaeb839cc5a7ff19dbd6d0074cd06795fd 100644 (file)
@@ -195,8 +195,8 @@ static void dstr_log_adapter(struct pi_adapter *pi)
 {       char    *mode_string[5] = {"4-bit","8-bit","EPP-8",
                                   "EPP-16","EPP-32"};
 
-       printk("dstr %s, DataStor EP2000 at 0x%x, ", DSTR_VERSION, pi->port);
-        printk("mode %d (%s), delay %d\n",pi->mode,
+       dev_info(&pi->dev, "dstr %s, DataStor EP2000 at 0x%x, ", DSTR_VERSION, pi->port);
+       dev_info(&pi->dev, "mode %d (%s), delay %d\n", pi->mode,
                mode_string[pi->mode],pi->delay);
 
 }
index a30313a2c3d782fa9086adf319e1ea1afc73fc9f..389faa6ae49d0a4e6ab9957f2a7752693f2cc78c 100644 (file)
@@ -279,10 +279,8 @@ static int epat_test_proto(struct pi_adapter *pi, char *scratch, int verbose)
         }
         epat_disconnect(pi);
 
-        if (verbose)  {
-               printk("epat: port 0x%x, mode %d, ccr %x, test=(%d,%d,%d)\n",
-                      pi->port, pi->mode, cc, e[0], e[1], f);
-       }
+       dev_dbg(&pi->dev, "epat: port 0x%x, mode %d, ccr %x, test=(%d,%d,%d)\n",
+              pi->port, pi->mode, cc, e[0], e[1], f);
        
         return (e[0] && e[1]) || f;
 }
@@ -298,9 +296,9 @@ static void epat_log_adapter(struct pi_adapter *pi)
         ver = RR(0xb);
         epat_disconnect(pi);
 
-       printk("epat %s, Shuttle EPAT chip %x at 0x%x, ",
+       dev_info(&pi->dev, "epat %s, Shuttle EPAT chip %x at 0x%x, ",
                EPAT_VERSION, ver, pi->port);
-       printk("mode %d (%s), delay %d\n",pi->mode,
+       dev_info(&pi->dev, "mode %d (%s), delay %d\n", pi->mode,
                mode_string[pi->mode],pi->delay);
 
 }
index ece7862dc05824aeb746aaaf49bb17b098c8f3dc..48d12f56c5017fed64eca263e71ed2f66a650e4c 100644 (file)
@@ -262,10 +262,8 @@ static int epia_test_proto(struct pi_adapter *pi, char *scratch, int verbose)
         WR(0x84,0);
         epia_disconnect(pi);
 
-        if (verbose)  {
-               printk("epia: port 0x%x, mode %d, test=(%d,%d,%d)\n",
-                      pi->port, pi->mode, e[0], e[1], f);
-        }
+       dev_dbg(&pi->dev, "epia: port 0x%x, mode %d, test=(%d,%d,%d)\n",
+              pi->port, pi->mode, e[0], e[1], f);
         
         return (e[0] && e[1]) || f;
 
@@ -277,8 +275,8 @@ static void epia_log_adapter(struct pi_adapter *pi)
 {       char    *mode_string[6] = {"4-bit","5/3","8-bit",
                                   "EPP-8","EPP-16","EPP-32"};
 
-       printk("epia %s, Shuttle EPIA at 0x%x, ", EPIA_VERSION, pi->port);
-        printk("mode %d (%s), delay %d\n",pi->mode,
+       dev_info(&pi->dev, "epia %s, Shuttle EPIA at 0x%x, ", EPIA_VERSION, pi->port);
+       dev_info(&pi->dev, "mode %d (%s), delay %d\n", pi->mode,
                mode_string[pi->mode],pi->delay);
 
 }
index 056e92d8e015f994f665b0b2f7124c215a03846e..d0a781f04411a39292596e30d0e66f71fcff1e05 100644 (file)
@@ -116,7 +116,7 @@ static void fit2_disconnect(struct pi_adapter *pi)
 static void fit2_log_adapter(struct pi_adapter *pi)
 
 {
-       printk("fit2 %s, FIT 2000 adapter at 0x%x, delay %d\n",
+       dev_info(&pi->dev, "fit2 %s, FIT 2000 adapter at 0x%x, delay %d\n",
                FIT2_VERSION, pi->port, pi->delay);
 
 }
index fa37f7f17fb27412dc88dc6532dc950e70dbd63d..f86da2d478cbbb1d4fe699f952a7dffb03899c9c 100644 (file)
@@ -173,7 +173,7 @@ static void fit3_log_adapter(struct pi_adapter *pi)
 
 {       char    *mode_string[3] = {"4-bit","8-bit","EPP"};
 
-       printk("fit3 %s, FIT 3000 adapter at 0x%x, "
+       dev_info(&pi->dev, "fit3 %s, FIT 3000 adapter at 0x%x, "
               "mode %d (%s), delay %d\n",
                FIT3_VERSION, pi->port,
                pi->mode,mode_string[pi->mode],pi->delay);
index bf597ee520b7d6d33a77b04b718b4cb259c01602..ec564e7cb9308d20b1261f93e321165bd2d4d4a4 100644 (file)
@@ -207,10 +207,8 @@ static int friq_test_proto(struct pi_adapter *pi, char *scratch, int verbose)
         for (k=0;k<128;k++) if (scratch[k] != k) r++;
        friq_disconnect(pi);
 
-        if (verbose)  {
-               printk("friq: port 0x%x, mode %d, test=(%d,%d,%d)\n",
-                      pi->port, pi->mode, e[0], e[1], r);
-        }
+       dev_dbg(&pi->dev, "friq: port 0x%x, mode %d, test=(%d,%d,%d)\n",
+              pi->port, pi->mode, e[0], e[1], r);
 
         return (r || (e[0] && e[1]));
 }
@@ -221,9 +219,9 @@ static void friq_log_adapter(struct pi_adapter *pi)
 {       char    *mode_string[6] = {"4-bit","8-bit",
                                   "EPP-8","EPP-16","EPP-32"};
 
-       printk("friq %s, Freecom IQ ASIC-2 adapter at 0x%x, ",
+       dev_info(&pi->dev, "friq %s, Freecom IQ ASIC-2 adapter at 0x%x, ",
                FRIQ_VERSION,pi->port);
-        printk("mode %d (%s), delay %d\n",pi->mode,
+       dev_info(&pi->dev, "mode %d (%s), delay %d\n", pi->mode,
                mode_string[pi->mode],pi->delay);
 
        pi->private = 1;
index 9b8db1122154fe8e938cc535b12cc812a1ebabc8..5ea16a247257897e6f5fcbd82f63b9b5f2f7cf58 100644 (file)
@@ -230,14 +230,12 @@ static int frpw_test_proto(struct pi_adapter *pi, char *scratch, int verbose)
           pi->private = frpw_test_pnp(pi) + 2*pi->port;
 
        if (((pi->private%2) == 0) && (pi->mode > 2)) {
-          if (verbose) 
-               printk("frpw: Xilinx does not support mode %d\n", pi->mode);
+               dev_dbg(&pi->dev, "frpw: Xilinx does not support mode %d\n", pi->mode);
           return 1;
        }
 
        if (((pi->private%2) == 1) && (pi->mode == 2)) {
-          if (verbose)
-               printk("frpw: ASIC does not support mode 2\n");
+               dev_dbg(&pi->dev, "frpw: ASIC does not support mode 2\n");
           return 1;
        }
 
@@ -258,10 +256,8 @@ static int frpw_test_proto(struct pi_adapter *pi, char *scratch, int verbose)
         for (k=0;k<128;k++) if (scratch[k] != k) r++;
        frpw_disconnect(pi);
 
-        if (verbose)  {
-               printk("frpw: port 0x%x, chip %ld, mode %d, test=(%d,%d,%d)\n",
-                      pi->port, (pi->private%2), pi->mode, e[0], e[1], r);
-        }
+       dev_dbg(&pi->dev, "frpw: port 0x%x, chip %ld, mode %d, test=(%d,%d,%d)\n",
+              pi->port, (pi->private%2), pi->mode, e[0], e[1], r);
 
         return (r || (e[0] && e[1]));
 }
@@ -272,9 +268,9 @@ static void frpw_log_adapter(struct pi_adapter *pi)
 {       char    *mode_string[6] = {"4-bit","8-bit","EPP",
                                   "EPP-8","EPP-16","EPP-32"};
 
-       printk("frpw %s, Freecom (%s) adapter at 0x%x, ",
+       dev_info(&pi->dev, "frpw %s, Freecom (%s) adapter at 0x%x, ",
                FRPW_VERSION,((pi->private%2) == 0)?"Xilinx":"ASIC",pi->port);
-        printk("mode %d (%s), delay %d\n",pi->mode,
+       dev_info(&pi->dev, "mode %d (%s), delay %d\n", pi->mode,
                mode_string[pi->mode],pi->delay);
 
 }
index b23632c1da3922aa1a98bea3563bc043dbe58e4f..ea1b931ce20dadab6ade7e407efe779293c81f88 100644 (file)
@@ -234,8 +234,8 @@ static void kbic_log_adapter(struct pi_adapter *pi, char *chip)
 {       char    *mode_string[6] = {"4-bit","5/3","8-bit",
                                   "EPP-8","EPP_16","EPP-32"};
 
-       printk("kbic %s, KingByte %s at 0x%x, ", KBIC_VERSION, chip, pi->port);
-        printk("mode %d (%s), delay %d\n",pi->mode,
+       dev_info(&pi->dev, "kbic %s, KingByte %s at 0x%x, ", KBIC_VERSION, chip, pi->port);
+       dev_info(&pi->dev, "mode %d (%s), delay %d\n", pi->mode,
                mode_string[pi->mode],pi->delay);
 
 }
index 62b0692d1b9469d0caa599c02dcf99f38659cf16..b22c283709dec1b3067727ff894825f592571880 100644 (file)
@@ -93,7 +93,7 @@ static void ktti_disconnect(struct pi_adapter *pi)
 static void ktti_log_adapter(struct pi_adapter *pi)
 
 {
-       printk("ktti %s, KT adapter at 0x%x, delay %d\n",
+       dev_info(&pi->dev, "ktti %s, KT adapter at 0x%x, delay %d\n",
                KTTI_VERSION, pi->port, pi->delay);
 
 }
index a3b0708aed7125ac5b3ce1f61aa9c4188e475f0f..c76d4fb84c6113268f18ef63e52bf761d20bb621 100644 (file)
@@ -115,8 +115,8 @@ static void on20_log_adapter(struct pi_adapter *pi)
 
 {       char    *mode_string[2] = {"4-bit","8-bit"};
 
-       printk("on20 %s, OnSpec 90c20 at 0x%x, ", ON20_VERSION, pi->port);
-        printk("mode %d (%s), delay %d\n",pi->mode,
+       dev_info(&pi->dev, "on20 %s, OnSpec 90c20 at 0x%x, ", ON20_VERSION, pi->port);
+       dev_info(&pi->dev, "mode %d (%s), delay %d\n", pi->mode,
                mode_string[pi->mode],pi->delay);
 
 }
index 8dc8296d50e605be368f65489aefd86cf7918ed4..5406e76ea7277308e14d42fc1904bd1a1bddd704 100644 (file)
@@ -167,7 +167,7 @@ static int on26_test_port(struct pi_adapter *pi)  /* hard reset */
             }
 
            if (i == RESET_WAIT) 
-               printk("on26: Device reset failed (%x,%x)\n",x,y);
+               dev_err(&pi->dev, "on26: Device reset failed (%x,%x)\n", x, y);
 
             w0(4); P1; w0(4); P1;
         }
@@ -280,8 +280,8 @@ static void on26_log_adapter(struct pi_adapter *pi)
 {       char    *mode_string[5] = {"4-bit","8-bit","EPP-8",
                                   "EPP-16","EPP-32"};
 
-       printk("on26 %s, OnSpec 90c26 at 0x%x, ", ON26_VERSION, pi->port);
-        printk("mode %d (%s), delay %d\n",pi->mode,
+       dev_info(&pi->dev, "on26 %s, OnSpec 90c26 at 0x%x, ", ON26_VERSION, pi->port);
+       dev_info(&pi->dev, "mode %d (%s), delay %d\n", pi->mode,
                mode_string[pi->mode],pi->delay);
 
 }