PCI: tegra: Use 'seq_puts' instead of 'seq_printf'
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Tue, 4 May 2021 17:17:54 +0000 (19:17 +0200)
committerLorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Thu, 5 Aug 2021 10:42:07 +0000 (11:42 +0100)
As spotted by checkpatch, use 'seq_puts' instead of 'seq_printf' when
possible.
It is slightly more efficient.

Link: https://lore.kernel.org/r/7bdedb342b9221169ab085540cf25d1992e8b97a.1620148539.git.christophe.jaillet@wanadoo.fr
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Reviewed-by: Vidya Sagar <vidyas@nvidia.com>
drivers/pci/controller/pci-tegra.c

index 8cc7197..099a148 100644 (file)
@@ -2546,7 +2546,7 @@ static void *tegra_pcie_ports_seq_start(struct seq_file *s, loff_t *pos)
        if (list_empty(&pcie->ports))
                return NULL;
 
-       seq_printf(s, "Index  Status\n");
+       seq_puts(s, "Index  Status\n");
 
        return seq_list_start(&pcie->ports, *pos);
 }
@@ -2583,16 +2583,16 @@ static int tegra_pcie_ports_seq_show(struct seq_file *s, void *v)
        seq_printf(s, "%2u     ", port->index);
 
        if (up)
-               seq_printf(s, "up");
+               seq_puts(s, "up");
 
        if (active) {
                if (up)
-                       seq_printf(s, ", ");
+                       seq_puts(s, ", ");
 
-               seq_printf(s, "active");
+               seq_puts(s, "active");
        }
 
-       seq_printf(s, "\n");
+       seq_puts(s, "\n");
        return 0;
 }