On Wed, Jul 18, 2012 at 09:07:46AM +0800, Gavin Shan wrote: > 4. Either "b_res->flags & mask" or "type" to be used for window_alignment(), > I don't think it's big deal because "b_res->flags & mask == type" for > current implementation. However, I'm not sure I still need change it > to "type"? > > min_align = max(min_align, window_alignment(bus, b_res->flags & mask)); Ah. you are right. (b_res->flags & mask) or type, either one is ok. I had a wrong assumption about b_res->flags. I thought it has either IORESOURCE_MEM set or IORESOURCE_PREFETCH set, but not both. Whatever concern I had, i dont have it any more. RP -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html