[PATCH v3 10/13] firmware: arm_sdei: Add support for CPU and system power states

Hi Catalin,

On 16/10/17 14:52, Catalin Marinas wrote:

On Fri, Sep 22, 2017 at 07:26:11PM +0100, James Morse wrote:

quoted

diff --git a/include/linux/cpuhotplug.h b/include/linux/cpuhotplug.h
index f24bfb2b9a2d..466b949474df 100644
--- a/include/linux/cpuhotplug.h
+++ b/include/linux/cpuhotplug.h

@@ -88,6 +88,7 @@ enum cpuhp_state {
 	CPUHP_AP_PERF_XTENSA_STARTING,
 	CPUHP_AP_PERF_METAG_STARTING,
 	CPUHP_AP_MIPS_OP_LOONGSON3_STARTING,
+	CPUHP_AP_SDEI_STARTING,

Nitpick: how generic is this as to apply to other architectures?
Probably not, so shall we prefix this with ARM_?

This may get used on 32bit ARM. I blindly copied PSCI, which doesn't have a
prefix. But it makes sense to have one.

Not actually a strong preferences but similar question for the
definitions in the sdei.h files (and maybe the filenames themselves).

Thanks,

James