Re: [PATCH 3/5] PCI : Add selected_regions funcs

From: Matthew Wilcox
Date: Thu Nov 23 2006 - 14:39:34 EST


On Thu, Nov 23, 2006 at 08:33:32PM +0100, Ingo Oeser wrote:
> bitfields (and bitmask) should be unsigned and use machine word size,
> which is usually "long". So please pass them in "unsigned long" instead of "int".

Why? We know how many BARs PCI devices have (6. Plus ROM. Plus 4
bridge resources, making 11). We could even use a u16. An int is fine,
there's no need to re-spin this patch.
-
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/