From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753151AbcGSHV3 (ORCPT ); Tue, 19 Jul 2016 03:21:29 -0400 Received: from terminus.zytor.com ([198.137.202.10]:45962 "EHLO terminus.zytor.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752553AbcGSHV0 (ORCPT ); Tue, 19 Jul 2016 03:21:26 -0400 Date: Tue, 19 Jul 2016 00:20:43 -0700 From: tip-bot for Richard Cochran Message-ID: Cc: hpa@zytor.com, bigeasy@linutronix.de, linux-kernel@vger.kernel.org, torvalds@linux-foundation.org, tglx@linutronix.de, j.anaszewski@samsung.com, rpurdie@rpsys.net, rcochran@linutronix.de, peterz@infradead.org, anna-maria@linutronix.de, mingo@kernel.org, linus.walleij@linaro.org, paul.gortmaker@windriver.com Reply-To: hpa@zytor.com, bigeasy@linutronix.de, torvalds@linux-foundation.org, tglx@linutronix.de, linux-kernel@vger.kernel.org, mingo@kernel.org, linus.walleij@linaro.org, paul.gortmaker@windriver.com, j.anaszewski@samsung.com, rpurdie@rpsys.net, peterz@infradead.org, anna-maria@linutronix.de, rcochran@linutronix.de In-Reply-To: <20160713153336.465496902@linutronix.de> References: <20160713153336.465496902@linutronix.de> To: linux-tip-commits@vger.kernel.org Subject: [tip:smp/hotplug] leds/trigger/cpu: Convert to hotplug state machine Git-Commit-ID: 911a359de96d5e9d24f50c74a48c6f3cf061539a X-Mailer: tip-git-log-daemon Robot-ID: Robot-Unsubscribe: Contact to get blacklisted from these emails MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset=UTF-8 Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Commit-ID: 911a359de96d5e9d24f50c74a48c6f3cf061539a Gitweb: http://git.kernel.org/tip/911a359de96d5e9d24f50c74a48c6f3cf061539a Author: Richard Cochran AuthorDate: Wed, 13 Jul 2016 17:16:45 +0000 Committer: Ingo Molnar CommitDate: Fri, 15 Jul 2016 10:40:26 +0200 leds/trigger/cpu: Convert to hotplug state machine This is a straightforward conversion. We place this callback last in the list so that the LED illuminates only after a successful bring up sequence. ( NOTE: The patch adds a FIXME question about the callback used, this question should probably be revisited later on.) Signed-off-by: Richard Cochran Signed-off-by: Anna-Maria Gleixner Reviewed-by: Sebastian Andrzej Siewior Acked-by: Jacek Anaszewski Cc: Linus Torvalds Cc: Linus Walleij Cc: Paul Gortmaker Cc: Peter Zijlstra Cc: Richard Purdie Cc: Thomas Gleixner Cc: linux-leds@vger.kernel.org Cc: rt@linutronix.de Link: http://lkml.kernel.org/r/20160713153336.465496902@linutronix.de Signed-off-by: Ingo Molnar --- drivers/leds/trigger/ledtrig-cpu.c | 32 +++++++++++++++----------------- include/linux/cpuhotplug.h | 1 + 2 files changed, 16 insertions(+), 17 deletions(-) diff --git a/drivers/leds/trigger/ledtrig-cpu.c b/drivers/leds/trigger/ledtrig-cpu.c index 938467f..4a6a182 100644 --- a/drivers/leds/trigger/ledtrig-cpu.c +++ b/drivers/leds/trigger/ledtrig-cpu.c @@ -92,25 +92,17 @@ static struct syscore_ops ledtrig_cpu_syscore_ops = { .resume = ledtrig_cpu_syscore_resume, }; -static int ledtrig_cpu_notify(struct notifier_block *self, - unsigned long action, void *hcpu) +static int ledtrig_starting_cpu(unsigned int cpu) { - switch (action & ~CPU_TASKS_FROZEN) { - case CPU_STARTING: - ledtrig_cpu(CPU_LED_START); - break; - case CPU_DYING: - ledtrig_cpu(CPU_LED_STOP); - break; - } - - return NOTIFY_OK; + ledtrig_cpu(CPU_LED_START); + return 0; } - -static struct notifier_block ledtrig_cpu_nb = { - .notifier_call = ledtrig_cpu_notify, -}; +static int ledtrig_dying_cpu(unsigned int cpu) +{ + ledtrig_cpu(CPU_LED_STOP); + return 0; +} static int __init ledtrig_cpu_init(void) { @@ -133,7 +125,13 @@ static int __init ledtrig_cpu_init(void) } register_syscore_ops(&ledtrig_cpu_syscore_ops); - register_cpu_notifier(&ledtrig_cpu_nb); + + /* + * FIXME: Why needs this to happen in the interrupt disabled + * low level bringup phase of a cpu? + */ + cpuhp_setup_state(CPUHP_AP_LEDTRIG_STARTING, "AP_LEDTRIG_STARTING", + ledtrig_starting_cpu, ledtrig_dying_cpu); pr_info("ledtrig-cpu: registered to indicate activity on CPUs\n"); diff --git a/include/linux/cpuhotplug.h b/include/linux/cpuhotplug.h index 456e3d9..24f56e2 100644 --- a/include/linux/cpuhotplug.h +++ b/include/linux/cpuhotplug.h @@ -42,6 +42,7 @@ enum cpuhp_state { CPUHP_AP_QCOM_TIMER_STARTING, CPUHP_AP_MIPS_GIC_TIMER_STARTING, CPUHP_AP_KVM_STARTING, + CPUHP_AP_LEDTRIG_STARTING, CPUHP_AP_NOTIFY_STARTING, CPUHP_AP_ONLINE, CPUHP_TEARDOWN_CPU,