On Friday 05 September 2014 20:20:44 Thomas Petazzoni wrote: > Hum, I think I would actually prefer something like: > > if (DT_FLAGS_TO_TYPE(flags) == DT_TYPE_IO) > rtype = IORESOURCE_IO; > else if (DT_FLAGS_TO_TYPE(flags) == DT_TYPE_MEM32) > rtype = IORESOURCE_MEM; > + else > + continue; > > So that we're explicit with the fact that we only care about I/O and > MEM32 resource types. Agreed, that looks better than my patch as well. Arnd -- 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
