On Sun, Apr 20, 2014 at 2:02 AM, Borislav Petkov <bp@xxxxxxx> wrote: > On Fri, Apr 18, 2014 at 08:53:31PM -0600, Myron Stowe wrote: >> From: Suravee Suthikulpanit <suravee.suthikulpanit@xxxxxxx> >> >> * Refactoring of the early_fill_mp_bus_info function into multiple helper >> functions since it is getting long, and difficult to follow. > > Much better. > Yes, it had become too big! >> * Merge early_fill_mp_bus_info into amd_postcore_init as there is no need to >> have this as a separate function. >> >> * Use pr_xxx instead of printk > > You can convert the printk(KERN_CONT -> pr_cont( too, while at it. > Although, this would prepend the fmt thing on every line ... > Yeah, I ignored this since the additional pr_fmt() output was too noisy and redundant >> * Prepend "AMD-Bus" for each print. > > That's done with pr_fmt > Done >> * The current code is using "fam10h_mmconf_*" in several place. As this >> file is no longer specific to family10h systems, this patch changes >> such occurrances to "amd_mmconf_*" instead for clarity. > > ... > > -- > Regards/Gruss, > Boris. > > Sent from a fat crate under my desk. Formatting is fine. > -- > -- > 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 -- 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