DECLARE_PCI_UNMAP_{ADDR,LEN} question

From: Marin Mitov
Date: Sun May 25 2008 - 00:48:42 EST


Hi all,

In Documentation/DMA-mapping.txt is written:

NOTE: DO NOT put a semicolon at the end of the
DECLARE_PCI_UNMAP_{ADDR,LEN}() macro.

While the output of
grep DECLARE_PCI_UNMAP_ drivers/net/*
contains lines with and without semicolon:
drivers/net/acenic.h: DECLARE_PCI_UNMAP_ADDR(mapping)
drivers/net/acenic.h: DECLARE_PCI_UNMAP_ADDR(mapping)
drivers/net/acenic.h: DECLARE_PCI_UNMAP_LEN(maplen)
drivers/net/bnx2.h: DECLARE_PCI_UNMAP_ADDR(mapping)
drivers/net/bnx2.h: DECLARE_PCI_UNMAP_ADDR(mapping)
drivers/net/bnx2x.h: DECLARE_PCI_UNMAP_ADDR(mapping)
drivers/net/qla3xxx.h: DECLARE_PCI_UNMAP_ADDR(mapaddr);
drivers/net/qla3xxx.h: DECLARE_PCI_UNMAP_LEN(maplen);
drivers/net/qla3xxx.h: DECLARE_PCI_UNMAP_ADDR(mapaddr);
drivers/net/qla3xxx.h: DECLARE_PCI_UNMAP_LEN(maplen);
drivers/net/skge.h: DECLARE_PCI_UNMAP_ADDR(mapaddr);
drivers/net/skge.h: DECLARE_PCI_UNMAP_LEN(maplen);
drivers/net/sky2.h: DECLARE_PCI_UNMAP_ADDR(mapaddr);
drivers/net/sky2.h: DECLARE_PCI_UNMAP_ADDR(maplen);
drivers/net/sky2.h: DECLARE_PCI_UNMAP_ADDR(data_size);
drivers/net/tg3.h: DECLARE_PCI_UNMAP_ADDR(mapping)
drivers/net/tg3.h: DECLARE_PCI_UNMAP_ADDR(mapping)

My question is:
Why one should NOT put semicolon at the end of the macro?
If it is _really_ important, should it be removed/patched?

Thank you for your answers.

Regards

Marin Mitov

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/