On Mon, May 27, 2013 at 9:50 PM, Yijing Wang <wangyijing@xxxxxxxxxx> wrote: > On 2013/5/28 12:35, Yinghai Lu wrote: >> On Mon, May 27, 2013 at 7:55 PM, Yijing Wang <wangyijing@xxxxxxxxxx> wrote: >>> I separate this patch from patchset "Add hostbridge resource release to >>> support root bus hotplug in IA64", because it's a Separate issue. >>> >>> We should increase info->res_num before we checking pci_use_crs return >>> when pci=nocrs set. >> >> there is no function change or issue here. > > Hi Yinghai, > Thanks for review and comments! > Yes, this patch is no functional change. I just sent this patch for make code flow more clearly. > Because free_pci_root_info_res() will free all info->res and info->res_offset. So I think move info->res_num++ > before pci_use_crs check is better, better readability. sure. -- 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
