Thread (10 messages) 10 messages, 4 authors, 2023-03-14

Re: [PATCH v4 1/4] PCI: Introduce pci_dev_for_each_resource()

From: Keith Busch <kbusch@kernel.org>
Date: 2023-03-10 22:15:55
Also in: linux-acpi, linux-alpha, linux-arm-kernel, linux-mips, linux-pci, linux-sh, lkml, sparclinux, xen-devel

On Fri, Mar 10, 2023 at 07:14:13PM +0200, Andy Shevchenko wrote:
+#define __pci_dev_for_each_resource(dev, res, __i, vartype)		\
+	for (vartype __i = 0;						\
+		res = &(dev)->resource[__i], __i < PCI_NUM_RESOURCES;	\
+		__i++)
...
+#define pci_dev_for_each_resource_p(dev, res)				\
+	__pci_dev_for_each_resource(dev, res, i, unsigned int)
It looks dangerous to have a macro declare a variable when starting a new
scope. How do you know the name 'i' won't clash with something defined above?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help