ata: libata: drop ata_msg_probe()
authorHannes Reinecke <hare@suse.de>
Tue, 21 Dec 2021 07:21:05 +0000 (08:21 +0100)
committerDamien Le Moal <damien.lemoal@opensource.wdc.com>
Wed, 5 Jan 2022 10:33:02 +0000 (19:33 +0900)
All callsites have been converted to dynamic debugging.

Signed-off-by: Hannes Reinecke <hare@suse.de>
Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
drivers/ata/libata-acpi.c
drivers/ata/libata-core.c
drivers/ata/libata-sff.c
include/linux/libata.h

index 9e1e62b..8cfa8c9 100644 (file)
@@ -402,7 +402,6 @@ EXPORT_SYMBOL_GPL(ata_acpi_stm);
  */
 static int ata_dev_get_GTF(struct ata_device *dev, struct ata_acpi_gtf **gtf)
 {
-       struct ata_port *ap = dev->link->ap;
        acpi_status status;
        struct acpi_buffer output;
        union acpi_object *out_obj;
@@ -418,10 +417,6 @@ static int ata_dev_get_GTF(struct ata_device *dev, struct ata_acpi_gtf **gtf)
        output.length = ACPI_ALLOCATE_BUFFER;
        output.pointer = NULL;  /* ACPI-CA sets this; save/free it later */
 
-       if (ata_msg_probe(ap))
-               ata_dev_dbg(dev, "%s: ENTER: port#: %d\n",
-                           __func__, ap->port_no);
-
        /* _GTF has no input parameters */
        status = acpi_evaluate_object(ata_dev_acpi_handle(dev), "_GTF", NULL,
                                      &output);
@@ -437,11 +432,9 @@ static int ata_dev_get_GTF(struct ata_device *dev, struct ata_acpi_gtf **gtf)
        }
 
        if (!output.length || !output.pointer) {
-               if (ata_msg_probe(ap))
-                       ata_dev_dbg(dev, "%s: Run _GTF: length or ptr is NULL (0x%llx, 0x%p)\n",
-                                   __func__,
-                                   (unsigned long long)output.length,
-                                   output.pointer);
+               ata_dev_dbg(dev, "Run _GTF: length or ptr is NULL (0x%llx, 0x%p)\n",
+                           (unsigned long long)output.length,
+                           output.pointer);
                rc = -EINVAL;
                goto out_free;
        }
@@ -464,9 +457,8 @@ static int ata_dev_get_GTF(struct ata_device *dev, struct ata_acpi_gtf **gtf)
        rc = out_obj->buffer.length / REGS_PER_GTF;
        if (gtf) {
                *gtf = (void *)out_obj->buffer.pointer;
-               if (ata_msg_probe(ap))
-                       ata_dev_dbg(dev, "%s: returning gtf=%p, gtf_count=%d\n",
-                                   __func__, *gtf, rc);
+               ata_dev_dbg(dev, "returning gtf=%p, gtf_count=%d\n",
+                           *gtf, rc);
        }
        return rc;
 
@@ -778,9 +770,8 @@ static int ata_acpi_push_id(struct ata_device *dev)
        struct acpi_object_list input;
        union acpi_object in_params[1];
 
-       if (ata_msg_probe(ap))
-               ata_dev_dbg(dev, "%s: ix = %d, port#: %d\n",
-                           __func__, dev->devno, ap->port_no);
+       ata_dev_dbg(dev, "%s: ix = %d, port#: %d\n",
+                   __func__, dev->devno, ap->port_no);
 
        /* Give the drive Identify data to the drive via the _SDD method */
        /* _SDD: set up input parameters */
index b5334e0..623a6f2 100644 (file)
@@ -2535,9 +2535,6 @@ int ata_dev_configure(struct ata_device *dev)
                return 0;
        }
 
-       if (ata_msg_probe(ap))
-               ata_dev_dbg(dev, "%s: ENTER\n", __func__);
-
        /* set horkage */
        dev->horkage |= ata_dev_blacklisted(dev);
        ata_force_horkage(dev);
@@ -2585,13 +2582,12 @@ int ata_dev_configure(struct ata_device *dev)
                return rc;
 
        /* print device capabilities */
-       if (ata_msg_probe(ap))
-               ata_dev_dbg(dev,
-                           "%s: cfg 49:%04x 82:%04x 83:%04x 84:%04x "
-                           "85:%04x 86:%04x 87:%04x 88:%04x\n",
-                           __func__,
-                           id[49], id[82], id[83], id[84],
-                           id[85], id[86], id[87], id[88]);
+       ata_dev_dbg(dev,
+                   "%s: cfg 49:%04x 82:%04x 83:%04x 84:%04x "
+                   "85:%04x 86:%04x 87:%04x 88:%04x\n",
+                   __func__,
+                   id[49], id[82], id[83], id[84],
+                   id[85], id[86], id[87], id[88]);
 
        /* initialize to-be-configured parameters */
        dev->flags &= ~ATA_DFLAG_CFG_MASK;
@@ -2791,8 +2787,6 @@ int ata_dev_configure(struct ata_device *dev)
        return 0;
 
 err_out_nosup:
-       if (ata_msg_probe(ap))
-               ata_dev_dbg(dev, "%s: EXIT, err\n", __func__);
        return rc;
 }
 
@@ -5339,7 +5333,7 @@ struct ata_port *ata_port_alloc(struct ata_host *host)
 
 #if defined(ATA_VERBOSE_DEBUG)
        /* turn on all debugging levels */
-       ap->msg_enable = 0x0007;
+       ap->msg_enable = 0x0003;
 #elif defined(ATA_DEBUG)
        ap->msg_enable = ATA_MSG_DRV | ATA_MSG_INFO;
 #else
index d5dbeb6..01f1673 100644 (file)
@@ -330,10 +330,6 @@ EXPORT_SYMBOL_GPL(ata_sff_dev_select);
 static void ata_dev_select(struct ata_port *ap, unsigned int device,
                           unsigned int wait, unsigned int can_sleep)
 {
-       if (ata_msg_probe(ap))
-               ata_port_info(ap, "ata_dev_select: ENTER, device %u, wait %u\n",
-                             device, wait);
-
        if (wait)
                ata_wait_idle(ap);
 
index 0e5ed2f..455d7e7 100644 (file)
 enum {
        ATA_MSG_DRV     = 0x0001,
        ATA_MSG_INFO    = 0x0002,
-       ATA_MSG_PROBE   = 0x0004,
 };
 
 #define ata_msg_drv(p)    ((p)->msg_enable & ATA_MSG_DRV)
 #define ata_msg_info(p)   ((p)->msg_enable & ATA_MSG_INFO)
-#define ata_msg_probe(p)  ((p)->msg_enable & ATA_MSG_PROBE)
 
 static inline u32 ata_msg_init(int dval, int default_msg_enable_bits)
 {