Skip to contentSkip to conversation

pci: added space before the open parenthesis

PATCH In review Ryan Desfosses 2 participants 2 messages Apr 4, 2014 PCI

lore
Share & tools
lore
#0 lore raw

PATCH 2/2 pci: added space before the open parenthesis

change made to resolve checkpatch.pl error

---
 drivers/pci/pci.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/pci/pci.c +1 -1
index fdbc294..56acb63 100644
@@ -114,7 +114,7 @@ unsigned char pci_bus_max_busnr(struct pci_bus* bus)
 	max = bus->busn_res.end;
 	list_for_each(tmp, &bus->children) {
 		n = pci_bus_max_busnr(pci_bus_b(tmp));
-		if(n > max)
+		if (n > max)
 			max = n;
 	}
 	return max;
Signature · 1 line
1.9.1
maintainer
#1 lore raw
Quoted 3 lines · Ryan Desfosses

change made to resolve checkpatch.pl error

Hi Ryan,

Thanks for this and your previous fix.  This "Signed-off-by" looks good. Would you mind reposting the "pci_bus*" fix with a similar "Signed-off-by"? It's a small fix, but strictly speaking, I need the "Signed-off-by" before I can apply a patch.

We normally write the changelog so it can stand alone, i.e., you shouldn't have to read the summary in addition to the changelog for it to make sense. It might be useful to browse other changelogs, e.g., with "git log --no-merges drivers/pci". I do often fix capitalization, spelling, and other errors because I think consistency makes it easier to read the history.

The "[2/2]" notation suggests that there's another patch labeled "[1/2]" (and probably an overall summary labeled "[0/2]"), and that all were posted together.  In this case, I think you intended your "pci_bus*" patch to be [1/2], but since they weren't posted together, it's a bit harder for me to keep track of them as a group.

If you don't mind, I'll collect up this and your previous fix (and any future ones you find) and squash them into a single "whitespace fixes" sort of patch to reduce the number of commits for obviously-correct changes.

Thanks again, and welcome to Linux kernel development!

Oh, and I applied the patch below to my pending/misc branch (and I'll update it with the other one when you post it). I have a few other things I hope to ask Linus to put in v3.15 yet, and I'll include this.

Bjorn

From: Ryan Desfosses <[email protected]>

PCI: Fix whitespace issues

Resolve whitespace issues that checkpatch.pl complained about.

No functional change.

---
 drivers/pci/pci.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/pci/pci.c +1 -1
index 7325d43bf030..1f0bf2eec127 100644
@@ -114,7 +114,7 @@ unsigned char pci_bus_max_busnr(struct pci_bus* bus)
 	max = bus->busn_res.end;
 	list_for_each_entry(tmp, &bus->children, node) {
 		n = pci_bus_max_busnr(tmp);
-		if(n > max)
+		if (n > max)
 			max = n;
 	}
 	return max;