ata: Convert ata_<foo>_printk(KERN_<LEVEL> to ata_<foo>_<level>

Saves text by removing nearly duplicated text format strings by
creating ata_<foo>_printk functions and printf extension %pV.

ata defconfig size shrinks ~5% (~8KB), allyesconfig ~2.5% (~13KB)

Format string duplication comes from:

 #define ata_link_printk(link, lv, fmt, args...) do { \
       if (sata_pmp_attached((link)->ap) || (link)->ap->slave_link)    \
               printk("%sata%u.%02u: "fmt, lv, (link)->ap->print_id,   \
                      (link)->pmp , ##args); \
       else \
               printk("%sata%u: "fmt, lv, (link)->ap->print_id , ##args); \
       } while(0)

Coalesce long formats.

$ size drivers/ata/built-in.*
   text	   data	    bss	    dec	    hex	filename
 544969	  73893	 116584	 735446	  b38d6	drivers/ata/built-in.allyesconfig.ata.o
 558429	  73893	 117864	 750186	  b726a	drivers/ata/built-in.allyesconfig.dev_level.o
 141328	  14689	   4220	 160237	  271ed	drivers/ata/built-in.defconfig.ata.o
 149567	  14689	   4220	 168476	  2921c	drivers/ata/built-in.defconfig.dev_level.o

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Jeff Garzik <jgarzik@pobox.com>

1 files changed, 33 insertions, 36 deletions

diff --git a/drivers/ata/sata_nv.c b/drivers/ata/sata_nv.c
index f215e10c11cf..b33f525df0f5 100644
--- a/drivers/ata/sata_nv.c
+++ b/drivers/ata/sata_nv.c
@@ -620,9 +620,8 @@ static void nv_adma_register_mode(struct ata_port *ap)
                 count++;
         }
         if (count == 20)
-                ata_port_printk(ap, KERN_WARNING,
-                        "timeout waiting for ADMA IDLE, stat=0x%hx\n",
-                        status);
+                ata_port_warn(ap, "timeout waiting for ADMA IDLE, stat=0x%hx\n",
+                              status);
 
         tmp = readw(mmio + NV_ADMA_CTL);
         writew(tmp & ~NV_ADMA_CTL_GO, mmio + NV_ADMA_CTL);
@@ -635,9 +634,9 @@ static void nv_adma_register_mode(struct ata_port *ap)
                 count++;
         }
         if (count == 20)
-                ata_port_printk(ap, KERN_WARNING,
-                         "timeout waiting for ADMA LEGACY, stat=0x%hx\n",
-                         status);
+                ata_port_warn(ap,
+                              "timeout waiting for ADMA LEGACY, stat=0x%hx\n",
+                              status);
 
         pp->flags |= NV_ADMA_PORT_REGISTER_MODE;
 }
@@ -665,7 +664,7 @@ static void nv_adma_mode(struct ata_port *ap)
                 count++;
         }
         if (count == 20)
-                ata_port_printk(ap, KERN_WARNING,
+                ata_port_warn(ap,
                         "timeout waiting for ADMA LEGACY clear and IDLE, stat=0x%hx\n",
                         status);
 
@@ -772,10 +771,10 @@ static int nv_adma_slave_config(struct scsi_device *sdev)
 
         blk_queue_segment_boundary(sdev->request_queue, segment_boundary);
         blk_queue_max_segments(sdev->request_queue, sg_tablesize);
-        ata_port_printk(ap, KERN_INFO,
-                "DMA mask 0x%llX, segment boundary 0x%lX, hw segs %hu\n",
-                (unsigned long long)*ap->host->dev->dma_mask,
-                segment_boundary, sg_tablesize);
+        ata_port_info(ap,
+                      "DMA mask 0x%llX, segment boundary 0x%lX, hw segs %hu\n",
+                      (unsigned long long)*ap->host->dev->dma_mask,
+                      segment_boundary, sg_tablesize);
 
         spin_unlock_irqrestore(ap->lock, flags);
 
@@ -1443,8 +1442,7 @@ static unsigned int nv_adma_qc_issue(struct ata_queued_cmd *qc)
            existing commands. */
         if (unlikely(qc->tf.protocol == ATA_PROT_NCQ &&
                      (qc->flags & ATA_QCFLAG_RESULT_TF))) {
-                ata_dev_printk(qc->dev, KERN_ERR,
-                        "NCQ w/ RESULT_TF not allowed\n");
+                ata_dev_err(qc->dev, "NCQ w/ RESULT_TF not allowed\n");
                 return AC_ERR_SYSTEM;
         }
 
@@ -1581,15 +1579,15 @@ static int nv_hardreset(struct ata_link *link, unsigned int *class,
                 int rc;
 
                 if (!(ehc->i.flags & ATA_EHI_QUIET))
-                        ata_link_printk(link, KERN_INFO, "nv: skipping "
-                                        "hardreset on occupied port\n");
+                        ata_link_info(link,
+                                      "nv: skipping hardreset on occupied port\n");
 
                 /* make sure the link is online */
                 rc = sata_link_resume(link, timing, deadline);
                 /* whine about phy resume failure but proceed */
                 if (rc && rc != -EOPNOTSUPP)
-                        ata_link_printk(link, KERN_WARNING, "failed to resume "
-                                        "link (errno=%d)\n", rc);
+                        ata_link_warn(link, "failed to resume link (errno=%d)\n",
+                                      rc);
         }
 
         /* device signature acquisition is unreliable */
@@ -1686,7 +1684,7 @@ static void nv_adma_error_handler(struct ata_port *ap)
                         u8 cpb_count = readb(mmio + NV_ADMA_CPB_COUNT);
                         u8 next_cpb_idx = readb(mmio + NV_ADMA_NEXT_CPB_IDX);
 
-                        ata_port_printk(ap, KERN_ERR,
+                        ata_port_err(ap,
                                 "EH in ADMA mode, notifier 0x%X "
                                 "notifier_error 0x%X gen_ctl 0x%X status 0x%X "
                                 "next cpb count 0x%X next cpb idx 0x%x\n",
@@ -1697,7 +1695,7 @@ static void nv_adma_error_handler(struct ata_port *ap)
                                 struct nv_adma_cpb *cpb = &pp->cpb[i];
                                 if ((ata_tag_valid(ap->link.active_tag) && i == ap->link.active_tag) ||
                                     ap->link.sactive & (1 << i))
-                                        ata_port_printk(ap, KERN_ERR,
+                                        ata_port_err(ap,
                                                 "CPB %d: ctl_flags 0x%x, resp_flags 0x%x\n",
                                                 i, cpb->ctl_flags, cpb->resp_flags);
                         }
@@ -1799,23 +1797,22 @@ static void nv_swncq_ncq_stop(struct ata_port *ap)
         u32 sactive;
         u32 done_mask;
 
-        ata_port_printk(ap, KERN_ERR,
-                        "EH in SWNCQ mode,QC:qc_active 0x%X sactive 0x%X\n",
-                        ap->qc_active, ap->link.sactive);
-        ata_port_printk(ap, KERN_ERR,
+        ata_port_err(ap, "EH in SWNCQ mode,QC:qc_active 0x%X sactive 0x%X\n",
+                     ap->qc_active, ap->link.sactive);
+        ata_port_err(ap,
                 "SWNCQ:qc_active 0x%X defer_bits 0x%X last_issue_tag 0x%x\n  "
                 "dhfis 0x%X dmafis 0x%X sdbfis 0x%X\n",
                 pp->qc_active, pp->defer_queue.defer_bits, pp->last_issue_tag,
                 pp->dhfis_bits, pp->dmafis_bits, pp->sdbfis_bits);
 
-        ata_port_printk(ap, KERN_ERR, "ATA_REG 0x%X ERR_REG 0x%X\n",
-                        ap->ops->sff_check_status(ap),
-                        ioread8(ap->ioaddr.error_addr));
+        ata_port_err(ap, "ATA_REG 0x%X ERR_REG 0x%X\n",
+                     ap->ops->sff_check_status(ap),
+                     ioread8(ap->ioaddr.error_addr));
 
         sactive = readl(pp->sactive_block);
         done_mask = pp->qc_active ^ sactive;
 
-        ata_port_printk(ap, KERN_ERR, "tag : dhfis dmafis sdbfis sacitve\n");
+        ata_port_err(ap, "tag : dhfis dmafis sdbfis sactive\n");
         for (i = 0; i < ATA_MAX_QUEUE; i++) {
                 u8 err = 0;
                 if (pp->qc_active & (1 << i))
@@ -1825,13 +1822,13 @@ static void nv_swncq_ncq_stop(struct ata_port *ap)
                 else
                         continue;
 
-                ata_port_printk(ap, KERN_ERR,
-                                "tag 0x%x: %01x %01x %01x %01x %s\n", i,
-                                (pp->dhfis_bits >> i) & 0x1,
-                                (pp->dmafis_bits >> i) & 0x1,
-                                (pp->sdbfis_bits >> i) & 0x1,
-                                (sactive >> i) & 0x1,
-                                (err ? "error! tag doesn't exit" : " "));
+                ata_port_err(ap,
+                             "tag 0x%x: %01x %01x %01x %01x %s\n", i,
+                             (pp->dhfis_bits >> i) & 0x1,
+                             (pp->dmafis_bits >> i) & 0x1,
+                             (pp->sdbfis_bits >> i) & 0x1,
+                             (sactive >> i) & 0x1,
+                             (err ? "error! tag doesn't exit" : " "));
         }
 
         nv_swncq_pp_reinit(ap);
@@ -1956,8 +1953,8 @@ static int nv_swncq_slave_config(struct scsi_device *sdev)
 
         if (strncmp(model_num, "Maxtor", 6) == 0) {
                 ata_scsi_change_queue_depth(sdev, 1, SCSI_QDEPTH_DEFAULT);
-                ata_dev_printk(dev, KERN_NOTICE,
-                        "Disabling SWNCQ mode (depth %x)\n", sdev->queue_depth);
+                ata_dev_notice(dev, "Disabling SWNCQ mode (depth %x)\n",
+                               sdev->queue_depth);
         }
 
         return rc;