Message ID | 7bdedb342b9221169ab085540cf25d1992e8b97a.1620148539.git.christophe.jaillet@wanadoo.fr |
---|---|
State | Not Applicable |
Headers | show |
Series | [1/3] PCI: tegra: Fix OF node reference leak | expand |
Reviewed-by: Vidya Sagar <vidyas@nvidia.com> On 5/4/2021 10:47 PM, Christophe JAILLET wrote: > As spotted by checkpatch, use 'seq_puts' instead of 'seq_printf' when > possible. > It is slightly more efficient. > > Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr> > --- > drivers/pci/controller/pci-tegra.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/pci/controller/pci-tegra.c b/drivers/pci/controller/pci-tegra.c > index 006bf0346dec..fe8e21ce3e3d 100644 > --- a/drivers/pci/controller/pci-tegra.c > +++ b/drivers/pci/controller/pci-tegra.c > @@ -2550,7 +2550,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); > } > @@ -2587,16 +2587,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; > } > >
diff --git a/drivers/pci/controller/pci-tegra.c b/drivers/pci/controller/pci-tegra.c index 006bf0346dec..fe8e21ce3e3d 100644 --- a/drivers/pci/controller/pci-tegra.c +++ b/drivers/pci/controller/pci-tegra.c @@ -2550,7 +2550,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); } @@ -2587,16 +2587,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; }
As spotted by checkpatch, use 'seq_puts' instead of 'seq_printf' when possible. It is slightly more efficient. Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr> --- drivers/pci/controller/pci-tegra.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-)