On Thu, Nov 03, 2016 at 03:50:07PM +0100, Sebastian Andrzej Siewior wrote: > @@ -1084,20 +1079,13 @@ static int smp_add_present_cpu(int cpu) > rc = sysfs_create_group(&s->kobj, &cpu_common_attr_group); > if (rc) > goto out_cpu; > - if (cpu_online(cpu)) { > - rc = sysfs_create_group(&s->kobj, &cpu_online_attr_group); > - if (rc) > - goto out_online; > - } > + > rc = topology_cpu_init(c); Please don't add whitespace :) If you fix the minor nits both s390 related patches are Acked-by: Heiko Carstens <heiko.carstens@xxxxxxxxxx> Thanks! -- To unsubscribe from this list: send the line "unsubscribe linux-s390" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html