>>> On 16.08.18 at 07:10, <[email protected]> wrote: > No functional change. > > Signed-off-by: Zhenzhong Duan <[email protected]>
Acked-by: Jan Beulich <[email protected]> Albeit I would have wished you would have taken the opportunity to, at the very least, also move cfg's declaration into the for() scope. There's obvious further trivial cleanup potential in the function. Jan > --- a/xen/arch/x86/x86_64/mmconfig-shared.c > +++ b/xen/arch/x86/x86_64/mmconfig-shared.c > @@ -371,8 +371,6 @@ static bool_t __init pci_mmcfg_reject_broken(void) > (pci_mmcfg_config[0].address == 0)) > return 0; > > - cfg = &pci_mmcfg_config[0]; > - > for (i = 0; i < pci_mmcfg_config_num; i++) { > u64 addr, size; > > -- > 1.7.3 _______________________________________________ Xen-devel mailing list [email protected] https://lists.xenproject.org/mailman/listinfo/xen-devel
